Galette » Plugins » Paypal

Johan Cwiklinski a écrit :

Pour corriger ça de façon temporaire ; dans le fichier paypal_notify.php à la ligne ~97, il devrait suffire d'ajouter :
[...]

Bon ben non, ça ne marchera pas en fait ; il manque aussi quelque chose côté Galette... J'ai 2 ou 3 petits bogues qui sont corrigés ; je vais sortir une version de Galette qui corrige tout ça assez rapidement, ainsi qu'une nouvelle version du plugin paypal dans la foulée.

Pour corriger (en attendant...), côté Galette :

diff --git a/galette/lib/Galette/Entity/Contribution.php b/galette/lib/Galette/Entity/Contribution.php
index c45dba1..9ede27f 100644
--- a/galette/lib/Galette/Entity/Contribution.php
+++ b/galette/lib/Galette/Entity/Contribution.php
@@ -184,6 +184,9 @@ class Contribution
                 }
                 $this->_amount = $this->_transaction->getMissingAmount();
             }
+            if ( isset($args['payment_type']) ) {
+                $this->_payment_type = $args['payment_type'];
+            }
         } elseif ( is_object($args) ) {
             $this->_loadFromRS($args);
         }

Et côté plugin-paypal :

diff --git a/paypal_notify.php b/paypal_notify.php
index c7cd5b8..e2234f4 100644
--- a/paypal_notify.php
+++ b/paypal_notify.php
@@ -86,8 +86,9 @@ if ( isset($_POST) && isset($_POST['mc_gross'])
              * - item_number: contribution type id
              */
             $args = array(
-                    'type'  => $_POST['item_number'],
-                    'adh'   => $_POST['custom']
+                    'type'          => $_POST['item_number'],
+                    'adh'           => $_POST['custom'],
+                    'payment_type'  => Contribution::PAYMENT_PAYPAL
             );
             if ( $preferences->pref_membership_ext != '' ) {
                 $args['ext'] = $preferences->pref_membership_ext;

J'ai appliqué le patch, maintenant faut que je trouve un nouveau cobaye. :)

Attention, jouer les cobayes, ça peut être dangereux :D

Cool que le problème soit résolu. Je vais regarder dans la foulée pour les infos de retour, ça ne devrait pas être grand chose :)

• correction du problème d'encodage foiré sur le site paypal,
• il manquait un remplacement de variable

diff --git a/templates/default/paypal_form.tpl b/templates/default/paypal_form.tpl
index 222a6b9..7b2f83a 100644
--- a/templates/default/paypal_form.tpl
+++ b/templates/default/paypal_form.tpl
@@ -27,11 +27,12 @@
     {*<input type="hidden" name="bn" value="PP-BuyNowBF:btn_buynowCC_LG.gif:NonHostedGuest"/><!-- notfound :( -->*}
     <!-- Paypal dialogs -->
     <input type="hidden" name="return" value="{$plugin_url}paypal_success.php"/>
-    <input type="hidden" name="rm" value="POST"/>{*Send POST values back to Galette after payment. Will be sent to return url above*}
+    <input type="hidden" name="rm" value="2"/>{*Send POST values back to Galette after payment. Will be sent to return url above*}
+    <input type="hidden" name="charset" value="UTF-8"/>
     <input type="hidden" name="image_url" value="{$galette_url}picture.php?logo=true"/>
     <input type="hidden" name="cancel_return" value="{$plugin_url}paypal_form.php?cancelled=true"/>
     <input type="hidden" name="notify_url" value="{$plugin_url}paypal_notify.php"/>
-    <input type="hidden" name="cbt" value="{_T string="Go back to %s Website to complete your inscription."}"/>
+    <input type="hidden" name="cbt" value="{_T string="Go back to %s's website to complete your inscription." pattern="/%s/" replace=$preferences->pref_nom}"/>

     <fieldset id="paypal_form">
         <legend class="ui-state-active ui-corner-top">

Appliqué par commit 8a7d83132646def5d75ce1e1ab62065ca5c5264c.