Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

reglement cheque et date de paiement #72

Open
tofulm opened this issue Mar 31, 2021 · 1 comment
Open

reglement cheque et date de paiement #72

tofulm opened this issue Mar 31, 2021 · 1 comment

Comments

@tofulm
Copy link

tofulm commented Mar 31, 2021

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

@Cerdic
Copy link
Member

Cerdic commented Dec 28, 2022

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 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants