Projet

Général

Profil

Bug #27854

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

Ajouté par Benjamin Dauvergne il y a plus de 5 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
08 novembre 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Révision fbfeb64a (diff)
Ajouté par Lauréline Guérin il y a plus de 4 ans

lingo: do not add a basket item to a remote regie (#27854)

Historique

#2

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)

#3

Mis à jour par Lauréline Guérin il y a plus de 4 ans

  • Assigné à mis à Lauréline Guérin
#4

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') ?

#5

Mis à jour par Lauréline Guérin il y a plus de 4 ans

premier patch, en attente de confirmation du message d'erreur

#6

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

#8

Mis à jour par Frédéric Péters il y a plus de 4 ans

  • Statut changé de Solution proposée à Solution validée
#9

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)
#10

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

Formats disponibles : Atom PDF