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

Erreurs dans le calcul de charge #337

Open
M4n0x opened this issue Aug 24, 2021 · 0 comments
Open

Erreurs dans le calcul de charge #337

M4n0x opened this issue Aug 24, 2021 · 0 comments
Labels
bug Something isn't working

Comments

@M4n0x
Copy link
Collaborator

M4n0x commented Aug 24, 2021

Dans le calcul de la charge de la semaine, le lundi n'est pas pris en compte, la fonction isBetween(date1,date2) est utilisée cependant les bornes ne sont pas incluses, il faut rajouter un troisième paramètre "[]" pour définir l'inclusivité des bornes voir documentation de moment.js (concrètement il faut modifier isBetween(start, end) par isBetween(start, end, undefined, "[]"); dans le fichier tasks.ts (utilisez le go to file de github).

Il y a un autre cas d'erreur un peu à la marge, lorsque la tâche la plus éloignée dans le temps (toute tâches confondues) est placée un lundi alors la boucle de calcul des semaines s'arrête une semaine trop tôt et donc la dernière semaine n'est pas prise en compte. C'est un cas très spécifique pour corriger ce problème il faut d'abord corriger le problème précédant (car il s'agit là encore d'un problème de borne exclu/inclus) et ensuite modifier dans le fichier tasks.ts la ligne :
while (start.isBefore(finishLine)) en while (start.isSameOrBefore(finishLine)).

@M4n0x M4n0x added the bug Something isn't working label Aug 24, 2021
@M4n0x M4n0x changed the title Lundi pas pris en compte dans le calcul de charge Erreurs dans le calcul de charge Aug 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant