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