Booking : fix ne pas permettre de réserver des shifts déjà pris #717
Conversation
| @@ -193,42 +193,42 @@ public function getBeneficiariesWhoCanBookForCycle(Membership $member, $cycle = | |||
|
|
|||
| public function isShiftBookable(Shift $shift, Beneficiary $beneficiary = null) | |||
| { | |||
| // Do not book old or locked shifts | |||
| if ($shift->getIsPast() || $shift->isLocked()) { | |||
| if (!$beneficiary) { | |||
There was a problem hiding this comment.
pour le if !beneficiary, dans quel cas ca pourrait se produire en fait ?
There was a problem hiding this comment.
Si un salarié utilise le lien direct pour réserver un créneau (oui, c'est un peu tordu car il n'a pas le lien dans sa vue)
raphodn
force-pushed
the
raphodn/book-shift-rules
branch
from
fb91f92 to
9ab8ddb
Compare
| <div style="height:2px; width: 100%;position: absolute;top:-2px;"> | ||
| {% if nbBookableShifts < nbShifts %} | ||
| {% for shifter in 1..(nbShifts - nbBookableShifts) %} | ||
| {% if nbBookedShifts %} |
There was a problem hiding this comment.
Un peu ambigue en condition ?
There was a problem hiding this comment.
yes t'as raison j'ai rajouté un commit : e7cde22
| } | ||
|
|
||
| // Do not book old or locked or booked shifts | ||
| if ($shift->getIsPast() || $shift->isLocked() || $shift->getShifter()) { |
There was a problem hiding this comment.
Attention, tu es sur pour le $shift->getShifter() ?
Je me demande si on n'appelle pas cette methode avec un shift mais ca pourrait aussi concerner un bucket ?
There was a problem hiding this comment.
je viens de re-vérifier, c'est toujours un $shift qui est le premier paramètre lorsque la fonction isShiftBookable() est appelée. mais ca me parait bizarre en effet que ca n'ait pas été fix plus tôt
…an-grenoble#717) * Add rule in isShiftBookable (shift must not be already booked) * Cleanup templates, clarify
Quoi ?
Dans la fonction
ShiftService.isShiftBookable()il n'y avait pas le cas où leshiftavait déjà un "shifter", et donc n'est pas bookable.En local j'avais des incohérences dû à cela
J'ai aussi fait quelques modifs de "cleanup" dans les templates
availableShiftsenbookableShifts