Development #46504
lingo : dans le cas du payment d'un item unique, transmettre display_name à eopayment
0%
Description
Pour le support du backend SAGA/PayFiP il est obligatoire de passer un libellé décrivant ce qui est payé au backend.
Le contenu de la requête créant l'item est déjà sauvegardé dans item.request_data il suffit de prévoir qu'une clé nommé description peut-être transmise et dans PayMixin.handle_payment si un unique item est présent cette donnée est extraite est passé à l'appel (order_id, kind, data) = payment.request(total_amount, **kwargs) dans kwargs.
Fichiers
Révisions associées
lingo: assert preconditions in handle_payment() (#46504)
It checks preconditions on item and remote_items, with specific
preconditions for regie flagged "can_pay_only_one_basket_item".
lingo: coding style (#46504)
lingo: remove non tracing check Regie.can_pay_only_one_basket_item (#46504)
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
Il y a déjà display_name qui est obligatoire, on va prendre ça, il ne reste plus qu'à passer cela à eopayment.
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Sujet changé de lingo : dans le cas du payment d'un item unique, pouvoir passer un libellé à lingo : dans le cas du payment d'un item unique, transmettre display_name à eopayment
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Fichier 0003-lingo-coding-style-46504.patch 0003-lingo-coding-style-46504.patch ajouté
- Fichier 0002-lingo-assert-preconditions-in-handle_payment-46504.patch 0002-lingo-assert-preconditions-in-handle_payment-46504.patch ajouté
- Fichier 0001-lingo-pass-item-subject-to-eopayment-46504.patch 0001-lingo-pass-item-subject-to-eopayment-46504.patch ajouté
- Tracker changé de Support à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Vraiment pas fan des asserts dans la vue introduits par 0002. D'ailleurs il y a déjà dans PayView
473 if regie.can_pay_only_one_basket_item and len(items) > 1:
474 messages.error(request, _('Grouping basket items is not allowed.'))
475 return HttpResponseRedirect(next_url)
Donc pourquoi pas déplacer ça dans handle_payment en adaptant avec les préconditions que tu introduis.
Le reste OK.
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
Valentin Deniaud a écrit :
Vraiment pas fan des asserts dans la vue introduits par 0002. D'ailleurs il y a déjà dans PayView
[...]Donc pourquoi pas déplacer ça dans handle_payment en adaptant avec les préconditions que tu introduis.
Le reste OK.
Ce qui m'embête avec des messages plutôt que des assert c'est que ça ne doit pas arriver, si ça arrive c'est qu'on a un bug et je préférerai qu'on le sache, parce que les gens ne pourront rien payer si c'est systématique (quand t'as plusieurs items dans ton panier, et que c'est pas le bon bouton qui est actif t'es foutu).
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Statut changé de Solution validée à Solution proposée
Voilà avec le check actuel retiré pour ne pas cacher les problèmes.
Mis à jour par Valentin Deniaud il y a plus de 3 ans
Benjamin Dauvergne a écrit :
je préférerai qu'on le sache
Pourquoi pas ajouter logger.error, ça envoie un mail non ?
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Statut changé de Solution proposée à En cours
Valentin Deniaud a écrit :
Benjamin Dauvergne a écrit :
je préférerai qu'on le sache
Pourquoi pas ajouter logger.error, ça envoie un mail non ?
Allez ok.
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Fichier 0005-lingo-remove-non-tracing-check-Regie.can_pay_only_on.patch 0005-lingo-remove-non-tracing-check-Regie.can_pay_only_on.patch ajouté
- Fichier 0003-lingo-assert-preconditions-in-handle_payment-46504.patch 0003-lingo-assert-preconditions-in-handle_payment-46504.patch ajouté
- Fichier 0004-lingo-coding-style-46504.patch 0004-lingo-coding-style-46504.patch ajouté
- Fichier 0002-lingo-pass-item-subject-to-eopayment-46504.patch 0002-lingo-pass-item-subject-to-eopayment-46504.patch ajouté
- Fichier 0001-tox-limit-mock-version-for-compatibility-with-python.patch 0001-tox-limit-mock-version-for-compatibility-with-python.patch ajouté
- Statut changé de En cours à Solution proposée
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 6c53fdd37f9e82fa44884864a7ff41ade7ed9d74
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Fri Oct 2 18:07:24 2020 +0200
lingo: remove non tracing check Regie.can_pay_only_one_basket_item (#46504)
commit d38d1a4347f36e1ea6714b946fb66b49885580bb
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Thu Oct 1 15:13:36 2020 +0200
lingo: coding style (#46504)
commit d93ecda18f9a90493e54d601619646aa939921d1
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Thu Oct 1 15:14:04 2020 +0200
lingo: assert preconditions in handle_payment() (#46504)
It checks preconditions on item and remote_items, with specific
preconditions for regie flagged "can_pay_only_one_basket_item".
commit 16d9a5cbfc10790dc0f54691c05a535845bbd7af
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Wed Sep 30 09:43:43 2020 +0200
lingo: pass item subject to eopayment (#46504)Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
lingo: pass item subject to eopayment (#46504)