Exemption : autoriser quand même à réserver des créneaux #748
Conversation
| @@ -239,6 +237,7 @@ public function isShiftBookable(Shift $shift, Beneficiary $beneficiary = null) | |||
| return false; | |||
| } | |||
|
|
|||
| // get cycle of shift | |||
| // TODO refactor code to remove shift_cycle | |||
| // canBookDuration method should not use TimeLog but request shifts | |||
| $shift_cycle = 0; | |||
There was a problem hiding this comment.
pas lié à cette PR, mais je ne comprends pas trop la logique de ces dernières lignes 😅
There was a problem hiding this comment.
L’idée est de retrouver ici le cycle sur lequel est le créneau pour en rajouter des contraintes : ex limiter à 3h max de créneau par cycle.
|
Perso, je ne pense pas que cette PR soit une bonne idée. Il faut rester dans une gestion simple. Ouvrir cette possibilité ne va que compliquer la gestion du compteur de temps… Si on demande à être exempté alors on l’est. Je ne vois pas bien ce qui justifierait d’être exempté de créneau mais de quand même en faire ? |
En lien avec ce que je viens de mettre avant, ta PR ne modifie pas la logique de fin de cycle ou le timelog est remis à 0. Ici la personne va accumuler des heures sans prendre des -3h ce qui fait qu’avec l’épargne une personne va mettre ces créneaux de côté. Je ne pense pas que c’est ce qu’on veut. |
Quoi ?
Suite à la PR #659 qui implémente l'exemption de créneaux, on avait interdit la possibilité aux membres exemptés de s'inscrire.
Mais ca ne devrait pas être si restrictif :