Project

General

Profile

Bug #27854

lingo: AddBasketItem ne devrait pas permettre d'ajouter un élément au panier d'une régie remote

Added by Benjamin Dauvergne 5 months ago. Updated 5 months ago.

Status:
Nouveau
Priority:
Normal
Assignee:
-
Start date:
08 Nov 2018
Due date:
% Done:

0%

Patch proposed:
No
Planning:
No

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".

History

#2 Updated by Benjamin Dauvergne 5 months ago

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

Also available in: Atom PDF