Développement #57609
Interdire de rendre un évènement récurrent si il a déjà des réservations
0%
Description
Pour ne pas créer un monstre.
Files
Associated revisions
History
Updated by Valentin Deniaud about 3 years ago
- File 0001-manager-forbid-making-booked-event-recurrent-57609.patch 0001-manager-forbid-making-booked-event-recurrent-57609.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Il s'agit juste de désactiver le champ. On pourrait vouloir mettre un help_text pour dire pourquoi, mais comme il va d'un coup s'afficher partout et qu'il n'apportera pas d'info utile dans la majorité des cas, plutôt non.
Updated by Valentin Deniaud about 3 years ago
Lauréline Guerin a écrit :
et si la réservation a été annulée ?
Mini-monstre mais monstre quand même, un évènement récurrent ne doit pas avoir de réservation tout court, je trouve ça plus rassurant.
Updated by Benjamin Dauvergne about 3 years ago
mais comme il va d'un coup s'afficher partout et qu'il n'apportera pas d'info utile dans la majorité des cas, plutôt non.
Je n'ai pas compris ce commentaire, de la même manière que du modifies .disabled
tu pourrais modifier le .help_text.
Updated by Benjamin Dauvergne about 3 years ago
- Status changed from Solution proposée to En cours
Updated by Valentin Deniaud about 3 years ago
Benjamin Dauvergne a écrit :
mais comme il va d'un coup s'afficher partout et qu'il n'apportera pas d'info utile dans la majorité des cas, plutôt non.
Je n'ai pas compris ce commentaire, de la même manière que du modifies
.disabled
tu pourrais modifier le .help_text.
Oui ma remarque n'était pas d'ordre technique. C'est juste que si je modifie le help_text, alors sur les agendas events qui n'utilisent pas les évènements récurrents, donc 99%, quand tu édites un évènement qui a des réservations tu vas avoir un texte qui te dit que attention il ne faut pas que tu puisses rendre cet évènement récurrent, alors que ça ne t'es de toute façon jamais venu à l'idée. D'où « l'info est inutile dans la majorité des cas ».
Mais pour couvrir le cas où quelqu'un se demanderait pourquoi la case est grisée, je peux mettre l'info dans un attribut title peut-être ?
Updated by Benjamin Dauvergne about 3 years ago
Valentin Deniaud a écrit :
Oui ma remarque n'était pas d'ordre technique. C'est juste que si je modifie le help_text, alors sur les agendas events qui n'utilisent pas les évènements récurrents, donc 99%, quand tu édites un évènement qui a des réservations tu vas avoir un texte qui te dit que attention il ne faut pas que tu puisses rendre cet évènement récurrent, alors que ça ne t'es de toute façon jamais venu à l'idée. D'où « l'info est inutile dans la majorité des cas ».
Mais pour couvrir le cas où quelqu'un se demanderait pourquoi la case est grisée, je peux mettre l'info dans un attribut title peut-être ?
Ok je comprends mieux, alors je mettrais plutôt le texte d'aide uniquement quand ça reste modifiable sous forme d'alerte "Attention le choix de la récurrence deviendra définitif/non-modifiable après la première réservation." et sinon title aussi si tu veux, mais ça me parait mieux de prévenir avant qu'après.
Updated by Valentin Deniaud about 3 years ago
- File 0001-manager-forbid-making-booked-event-recurrent-57609.patch 0001-manager-forbid-making-booked-event-recurrent-57609.patch added
- Status changed from En cours to Solution proposée
Voilà j'ai fait comme tu dis.
Updated by Frédéric Péters about 3 years ago
- Status changed from Solution proposée to Solution validée
(perso pas fan des textes cachés en title, genre il aurait peut-être pu ici remplacer le help_text)
Updated by Valentin Deniaud about 3 years ago
- Status changed from Solution validée to Résolu (à déployer)
commit e9a85c134c671edfc18d0e57d681fcc1f4da38f0 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Tue Oct 5 15:45:25 2021 +0200 manager: forbid making booked event recurrent (#57609)
Updated by Frédéric Péters about 3 years ago
- Status changed from Résolu (à déployer) to Solution déployée
manager: forbid making booked event recurrent (#57609)