#1 Correction du décompte des abonnements xDSL

Fusionné
guillaume a fusionné 1 commits à partir de chl/master vers franciliens.net/master il y a 6 ans
chl a commenté il y a 6 ans
Il n'existe pas encore de contenu.
pitchum a commenté il y a 6 ans
Propriétaire

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 a commenté il y a 6 ans
Collaborateur

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.
Cette Pull Request a été fusionnée avec succès !
Connectez-vous pour rejoindre cette conversation.
Pas d'étiquette
Aucun jalon
Pas d'assignataire
2 Participants
Chargement…
Annuler
Enregistrer
Il n'existe pas encore de contenu.