Bug #16885
recalcul des frais additionnels à l'envoi vers le prestataire de paiement
0%
Description
Pour le moment ils ne sont pas recalculés si jamais il n'y a pas de régie passée dans le POST.
Comme c'est exactement la situation que #16824, j'ai pris l'option de refactoriser la méthode pour mieux séparer les deux cas : éléments de panier vs régie distante.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 7 ans
- Fichier 0001-lingo-refactor-payment-view-to-make-sure-extra-fees-.patch 0001-lingo-refactor-payment-view-to-make-sure-extra-fees-.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Le patch n'est pas nécessairement super lisible vu des changements d'indentation et déplacements de blocs, mais appliqué, la méthode est plus lisible je trouve. (et corrige le bug).
Mis à jour par Thomas Noël il y a presque 7 ans
Je sais que j'ai mes marottes, mais le « assert not regie.is_remote() » je pense que c'est à éviter quand même, parce qu'en (vraie) prod (python -O) les assert ne sont pas interprétés par Python. Plutôt if/raise.
Ligne 310 : le filtre "extra_fee=False" normalement il n'est pas nécessaire, si ? (les extra_fee doivent appartenir à la même régie que les paiements...)
En dehors de ça, tout me parait ok, et c'est effectivement nettement plus clair.
Mis à jour par Frédéric Péters il y a presque 7 ans
- Statut changé de En cours à Résolu (à déployer)
Voilà, j'ai retiré le extra_fee=False, qui n'était effectivement pas nécessaire. Pour l'assert, je l'ai simplement retiré.
commit a5b4dd37f10d7927dff32faa8c0e849720aae8fa
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Wed Jun 14 15:46:10 2017 +0200
lingo: refactor payment view to make sure extra fees are computed (#16885)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
lingo: refactor payment view to make sure extra fees are computed (#16885)