Gros bug si premier jour de la semaine = dimanche

  • Yan
  • Developpeur GRR
  • Developpeur GRR
More
1 year 4 months ago - 1 year 4 months ago #4596 by Yan
Bonjour,
il semble que seul la page week soit affectée.
Dans l'urgence, je propose ce patch github.com/JeromeDevome/GRR/commit/ca9ef...12d49b777840512aa218 qui devrait corriger le problème pour le dimanche, mais dysfonctionne en l'état pour le mardi ;-)
À suivre
Cordialement,
YN
Last edit: 1 year 4 months ago by Yan.

Please Log in or Create an account to join the conversation.

  • Yan
  • Developpeur GRR
  • Developpeur GRR
More
1 year 4 months ago #4597 by Yan
Finalement c'était apparemment plus simple que je ne le craignais.
Ce correctif semble résoudre le problème d'affichage dans la page week lorsque le premier jour n'est pas un lundi : github.com/JeromeDevome/GRR/commit/6da32...8d3dc860cd2057c41f61
C'était une variable non initialisée...
Merci de confirmer.
Cordialement,
YN

Please Log in or Create an account to join the conversation.

  • mlucas
  • Topic Author
  • New Member
  • New Member
More
1 year 4 months ago #4599 by mlucas
Bonjour,
négatif, le problème est identique lors de l'affichage de de la ressource pour une semaine si le premier jour est le dimanche.
week.php?year=2022&month=12&day=18&area=1&room=23
Cordialement.

Please Log in or Create an account to join the conversation.

  • Yan
  • Developpeur GRR
  • Developpeur GRR
More
1 year 4 months ago - 1 year 4 months ago #4603 by Yan
Bonjour,
désolé, j'ai fait une erreur de fichier lors du commit.
Celui-ci semble fonctionner :  github.com/JeromeDevome/GRR/commit/51780...1aeb4b5572d96c1411d9
Vous pouvez le confirmer ?
Cordialement,
YN
Last edit: 1 year 4 months ago by Yan.

Please Log in or Create an account to join the conversation.

  • mlucas
  • Topic Author
  • New Member
  • New Member
More
1 year 3 months ago #4623 by mlucas
Bonjour,
Tous mes meilleurs vœux pour cette nouvelle année.
Je viens de mettre à jour, ça a l'air de bien fonctionner. J'attends d'éventuels retours utilisateurs.
Merci !
Cordialement.
Matthieu.

Please Log in or Create an account to join the conversation.

Moderators: Yan