Cerdic
commented
1 year ago Ben disons que ça permet de garder une trace du moment où l'utilisateur a dit "je vais payer par chèque" (ce qui n'est parfois pas concomitant de la transaction elle même). C'est pas encore le paiement, certes, mais son intention.
Et ensuite quand l'encaissement est fait pour de vrai la date_paiement est mise à jour à nouveau, pour le vrai paiement cette fois-ci https://github.com/nursit/bank/blob/master/inc/bank.php#L798
Note que c'est pas non plus la date d'encaissement mais la date à laquelle on reporte que le chèque a bien été encaissé.
Donc je suis pas certain qu'il faille changer ce process ni ajouter un autre champ. Si tu te bases uniquement sur les date_paiement pour savoir une transaction a été payée, alors c'est pas bon, il faut se baser sur le statut, et en fonction du statut cette date a donc 2 significations possibles.
Peut-être j'ai pas bien compris le vrai problème ?
Quand on fait un règlement par chèque, le champ date_paiement dans spip_transactions est rempli : https://github.com/nursit/bank/blob/master/inc/bank.php#L731 A ce moment là, on ne sait pas encore si le chèque va être envoyé, ... Pour moi le champ date_paiement correspond à la date d'encaissement. Dans le cas actuel, il me semble qu'il fait doublon avec date_transaction Est il possible de supprimer son initialisation, pour que l'on puisse utiliser le champ date_paiement pour indiquer à posteriori que le chèque a été encaissé ? Sinon, j'ajouterai un champ date_encaissee Merci