#1 Correction du décompte des abonnements xDSL

Merged
guillaume merged 1 commits from chl/master into franciliens.net/master 6 years ago
chl commented 6 years ago
There is no content yet.
pitchum commented 6 years ago
Owner

Merci chl. Perso je validerais bien cette PR mais je laisse Guillaume donner son avis.

J'avoue que personnellement par précaution j'aurais conservé les critères de date dans la requête :

d.date_ouverture IS NOT null
AND d.date_cloture

Mais ça c'est parce que je suis peureux par nature et parce que je ne connais pas aussi bien que chl le fonctionnement de notre dolibarr :)

Merci chl. Perso je validerais bien cette PR mais je laisse Guillaume donner son avis. J'avoue que personnellement par précaution j'aurais conservé les critères de date dans la requête : d.date_ouverture IS NOT null AND d.date_cloture Mais ça c'est parce que je suis peureux par nature et parce que je ne connais pas aussi bien que chl le fonctionnement de notre dolibarr :)
chl commented 6 years ago
Collaborator

Pas de souci, n'hésite pas à retoucher. Je me disais qu'il y avait eu un doute et comme j'avais déjà farfouillé dans le code de Dolibarr, je suis allé reprendre la valeur. Avec les automatismes de Dolibarr, ça doit être plus ou moins équivalent, il faudrait juste éviter de compter quand la date d'ouverture est dans le futur mais pas bien grave avec notre utilisation.

Pas de souci, n'hésite pas à retoucher. Je me disais qu'il y avait eu un doute et comme j'avais déjà farfouillé dans le code de Dolibarr, je suis allé reprendre la valeur. Avec les automatismes de Dolibarr, ça doit être plus ou moins équivalent, il faudrait juste éviter de compter quand la date d'ouverture est dans le futur mais pas bien grave avec notre utilisation.
This pull request has been merged successfully!
Sign in to join this conversation.
No Label
No Milestone
No assignee
2 Participants
Loading...
Cancel
Save
There is no content yet.