Development #47530
lingo/SelfInvoiceView : ne pas permettre le paiement d'une facture déjà payée
0%
Description
Actuellement sur SelfInvoiceView on ne fait qu'une seul vérification :
if invoice.total_amount != invoice_amount: continue
Il faudrait aussi vérifier que la date de paiement n'est pas dépassée (invoice.pay_limit_date) ou que la facture n'est pas déjà payée (invoice.paid)
Demandes liées
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
Ça me semble tout à fait sensé, mais j'ai une petite question tout à fait non critique : un rapide git grep pay_limit_date
dans combo me dit qu'il n'y aucun contrôle fait dessus nul part; j'en déduis que c'est le boulot du web-service de listing de ne pas renvoyer ces factures pour qu'on ne puisse pas les payer ou est-ce que le contrôle manque sur la cellule de paiement classique (enfin sur PayView qui est utilisé). Dans le doute je proposerai bien de faire le contrôle partout (PayView et SelfInvoice.., BasketItemPayView comme son nom l'indique ne permet pas de payer une facture).
Même chose pour paid
(on ne voit qu'un usage dans item.html utilisé par ItemView).
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
En fait je me demande si on avait pas statué que c'était au web-service invoices/ de déterminer si on pouvait ou pas payer une facture (via online_payment
).
- paid : ok c'est déjà payé
- pay_limit_date : date de paiement limite 'théorique'
- online_payment : c'est encore payable en ligne ou pas
- no_online_payment_reason : la raison de pourquoi on peut plus
Je pense que si on doit bloquer par rapport à pay_limit_date c'est à chaque connecteur de positionner online_payment à False et no_online_payment_reason
à « date limite de paiement dépassé (le 08/10/2020) », wadoyousink ?
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
Et c'est même 'no_online_payment_reason': 'past-due-date'
qu'il faut mettre.
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Lié à Bug #13687: lingo : ne pas permettre le paiement de factures/items déjà payées ajouté
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Statut changé de Nouveau à Rejeté
C'est la même chose que #13687.