Les contributions ne s'enregistrent pas correctement (Anomalie #486)


Added by Raphaël Hertzog almost 2 years ago. Updated almost 2 years ago.


Status:Fermé Start date:01/16/2013
Priority:Normal Due date:
Assignee:Johan Cwiklinski % Done:

100%

Category:-
Target version:-

Description

J'ai galette-plugin-paypal-1.1.0.1.tar.bz2 avec galette 0.7.2.2 et lorsqu'un utilisateur renouvelle sa cotisation par Paypal, le paiement est bien enregistré dans l'historique Paypal que propose le plugin, par contre aucune contribution n'est enregistrée et la période d'adhésion n'est pas prolongée.

J'ai cette erreur dans les logs:

2013-01-14 22:23:46 - ERROR --> Something went wrong :'( | SQLSTATE23502: Not null violation: 7 ERROR: null value in column "type_paiement_cotis" violates not-null constraint
#0 /srv/d_france/www/hosts/france.debian.net/galette/includes/Zend-1.11.11/Zend/Db/Statement.php(300): Zend_Db_Statement_Pdo->_execute(Array)
#1 /srv/d_france/www/hosts/france.debian.net/galette/includes/Zend-1.11.11/Zend/Db/Adapter/Abstract.php(479): Zend_Db_Statement->execute(Array)
#2 /srv/d_france/www/hosts/france.debian.net/galette/includes/Zend-1.11.11/Zend/Db/Adapter/Pdo/Abstract.php(238): Zend_Db_Adapter_Abstract->query('INSERT INTO "ga...', Array)
#3 /srv/d_france/www/hosts/france.debian.net/galette/includes/Zend-1.11.11/Zend/Db/Adapter/Abstract.php(575): Zend_Db_Adapter_Pdo_Abstract->query('INSERT INTO "ga...', Array)
#4 /srv/d_france/www/hosts/france.debian.net/galette/lib/Galette/Entity/Contribution.php(539): Zend_Db_Adapter_Abstract->insert('galette_cotisat...', Array)
#5 /srv/d_france/www/hosts/france.debian.net/galette/plugins/galette-plugin-paypal/paypal_notify.php(118): Galette\Entity\Contribution->store()
#6 {main}
2013-01-14 22:23:46 - ERROR --> An error occured while storing a new contribution from Paypal payment


galette.png - Écran de retour sans les informations de paiement (49.6 kB) Raphaël Hertzog, 01/19/2013 03:13 pm


Associated revisions

Revision a519e2ed
Added by Johan Cwiklinski almost 2 years ago

Set payment type in constructor; partially fixes #486

Revision 8a7d8313
Added by Johan Cwiklinski almost 2 years ago

Add payment type on paypal notification; fixes #486

Revision bf8fb88b
Added by Johan Cwiklinski almost 2 years ago

Success URL was not showing infos anymore; refs #486, fix encoding, fix wrong translation

History

Updated by Johan Cwiklinski almost 2 years ago

En effet, il semble que j'ai oublié d'envoyer une information (je ne suis toujours pas en mesure de tester virtuellement les paiements paypal :/).

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

$contrib->payment_type = Contribution::PAYMENT_PAYPAL;

  • Assignee set to Johan Cwiklinski
  • Status changed from Nouveau to In Progress

Updated by Johan Cwiklinski almost 2 years ago

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.

Updated by Johan Cwiklinski almost 2 years ago

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

 1diff --git a/galette/lib/Galette/Entity/Contribution.php b/galette/lib/Galette/Entity/Contribution.php
 2index c45dba1..9ede27f 100644
 3--- a/galette/lib/Galette/Entity/Contribution.php
 4+++ b/galette/lib/Galette/Entity/Contribution.php
 5@@ -184,6 +184,9 @@ class Contribution
 6                 }
 7                 $this->_amount = $this->_transaction->getMissingAmount();
 8             }
 9+            if ( isset($args['payment_type']) ) {
10+                $this->_payment_type = $args['payment_type'];
11+            }
12         }
13             $this->_loadFromRS($args);
14         }

Et côté plugin-paypal :

 1diff --git a/paypal_notify.php b/paypal_notify.php
 2index c7cd5b8..e2234f4 100644
 3--- a/paypal_notify.php
 4+++ b/paypal_notify.php
 5@@ -86,8 +86,9 @@ if ( isset($_POST) && isset($_POST['mc_gross'])
 6              * - item_number: contribution type id
 7              */
 8             $args = array(
 9-                    'type'  => $_POST['item_number'],
10-                    'adh'   => $_POST['custom']
11+                    'type'          => $_POST['item_number'],
12+                    'adh'           => $_POST['custom'],
13+                    'payment_type'  => Contribution::PAYMENT_PAYPAL
14             );
15             if ( $preferences->pref_membership_ext != '' ) {
16                 $args['ext'] = $preferences->pref_membership_ext;

Updated by Raphaël Hertzog almost 2 years ago

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

Updated by Raphaël Hertzog almost 2 years ago

Finalement j'ai re-cotisé avec un peu d'avance... la contribution a bien été enregistrée. Par contre la page sur laquelle je reviens ne semble pas être correcte (cf image jointe). Aucune info concernant le paiement juste effectué n'apparaît.

Updated by Johan Cwiklinski almost 2 years ago

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 :)

Updated by Johan Cwiklinski almost 2 years ago

Alors, le correctif ; et 2 bonus :
  • correction du problème d'encodage foiré sur le site paypal,
  • il manquait un remplacement de variable
 1diff --git a/templates/default/paypal_form.tpl b/templates/default/paypal_form.tpl
 2index 222a6b9..7b2f83a 100644
 3--- a/templates/default/paypal_form.tpl
 4+++ b/templates/default/paypal_form.tpl
 5@@ -27,11 +27,12 @@
 6     {*<input type="hidden" name="bn" value="PP-BuyNowBF:btn_buynowCC_LG.gif:NonHostedGuest"/><!-- notfound :( -->*}
 7     <!-- Paypal dialogs -->
 8     <input type="hidden" name="return" value="{$plugin_url}paypal_success.php"/>
 9-    <input type="hidden" name="rm" value="POST"/>{*Send POST values back to Galette after payment. Will be sent to return url above*}
10+    <input type="hidden" name="rm" value="2"/>{*Send POST values back to Galette after payment. Will be sent to return url above*}
11+    <input type="hidden" name="charset" value="UTF-8"/>
12     <input type="hidden" name="image_url" value="{$galette_url}picture.php?logo=true"/>
13     <input type="hidden" name="cancel_return" value="{$plugin_url}paypal_form.php?cancelled=true"/>
14     <input type="hidden" name="notify_url" value="{$plugin_url}paypal_notify.php"/>
15-    <input type="hidden" name="cbt" value="{_T string="Go back to %s Website to complete your inscription."}"/>
16+    <input type="hidden" name="cbt" value="{_T string="Go back to %s's website to complete your inscription." pattern="/%s/" replace=$preferences->pref_nom}"/>
17
18     <fieldset id="paypal_form">
19         <legend class="ui-state-active ui-corner-top">

Updated by Johan Cwiklinski almost 2 years ago

Appliqué par commit commit:a519e2ed3a42a8df404a6cd0d78c4ed1133fbf00.

  • % Done changed from 0 to 100
  • Status changed from In Progress to Résolu

Updated by Johan Cwiklinski almost 2 years ago

Les différents problèmes sont corrigés avec Galette 0.7.3.2 et le plugin Paypal 1.2.1 :)

  • Status changed from Résolu to Fermé

Also available in: Atom PDF