Anomalie #486
closedLes contributions ne s'enregistrent pas correctement
100%
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
Files
Updated by Johan Cwiklinski over 11 years ago
- Assignee set to Johan Cwiklinski
- Status changed from Nouveau to In Progress
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;
Updated by Johan Cwiklinski over 11 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 over 11 years ago
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;
Updated by Raphaël Hertzog over 11 years ago
J'ai appliqué le patch, maintenant faut que je trouve un nouveau cobaye. :)
Updated by Raphaël Hertzog over 11 years ago
- File galette.png galette.png added
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 over 11 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 over 11 years ago
- 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">
Updated by Johan Cwiklinski over 11 years ago
- % Done changed from 0 to 100
- Status changed from In Progress to Résolu
Appliqué par commit commit:a519e2ed3a42a8df404a6cd0d78c4ed1133fbf00.
Updated by Johan Cwiklinski over 11 years ago
Appliqué par commit 8a7d83132646def5d75ce1e1ab62065ca5c5264c.
Updated by Johan Cwiklinski over 11 years ago
- Status changed from Résolu to Fermé
Les différents problèmes sont corrigés avec Galette 0.7.3.2 et le plugin Paypal 1.2.1 :)