Bug #27854
lingo: AddBasketItem ne devrait pas permettre d'ajouter un élément au panier d'une régie remote
0%
Description
Le paiement d'une régie "remote" ne se base pas sur le panier mais uniquement sur le contenu d'un paramètre item contenant les numéros des factures:
362 if regie_id and Regie.objects.get(pk=regie_id).is_remote(): 363 regie = Regie.objects.get(pk=regie_id) 364 # get all items data from regie webservice 365 for item_id in request.POST.getlist('item'): 366 remote_items.append(regie.get_invoice(user, item_id))
Il reste néanmoins possible depuis un formulaire de pousser des éléments dans le panier pour cette régie, amenant à un micmac où on se retrouve avec des transactions vides avec pour montant 0 (qui ne devrait pas être possible non plus) puisque item n'est pas passé, l'absence de notification "paid" au formulaire (puisque pas d'item) et un panier toujours plein.
Dans la mesure où on conserve la vue de paiement actuel qui ne permet pas de payer à la fois le contenu d'un panier et des factures pour une même régie il faudrait donc interdire l'ajout d'un basketitem sur une régie "remote".
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
(Micmac arrivé à Mike aujourd'hui alors qu'il essayait d'utiliser une régie modifiée à l'arrache par le sictiam en ajoutant une URL de source de facture qui n'existait pas)
Mis à jour par Lauréline Guérin il y a plus de 4 ans
est-ce qu'il vaut mieux renvoyer une HttpResponseBadRequest('Unknown regie') ou une erreur spécifique genre HttpResponseBadRequest('Can not add a basket item from a remote regie') ?
Mis à jour par Lauréline Guérin il y a plus de 4 ans
- Fichier 0001-lingo-do-not-add-a-basket-item-from-a-remote-regie-2.patch 0001-lingo-do-not-add-a-basket-item-from-a-remote-regie-2.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
premier patch, en attente de confirmation du message d'erreur
Mis à jour par Frédéric Péters il y a plus de 4 ans
Plutôt un message précis ("Can not add a basket item /to/ a remote regie" est ok).
Mis à jour par Lauréline Guérin il y a plus de 4 ans
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Lauréline Guérin il y a plus de 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit fbfeb64a8322f99413f87c73982002abd872f4e3 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Wed Oct 23 11:41:06 2019 +0200 lingo: do not add a basket item to a remote regie (#27854)
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
lingo: do not add a basket item to a remote regie (#27854)