Development #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.
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-manager-forbid-making-booked-event-recurrent-57609.patch 0001-manager-forbid-making-booked-event-recurrent-57609.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
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.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
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.
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
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.
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Statut changé de Solution proposée à En cours
Mis à jour par Valentin Deniaud il y a plus de 2 ans
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 ?
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
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.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-manager-forbid-making-booked-event-recurrent-57609.patch 0001-manager-forbid-making-booked-event-recurrent-57609.patch ajouté
- Statut changé de En cours à Solution proposée
Voilà j'ai fait comme tu dis.
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
(perso pas fan des textes cachés en title, genre il aurait peut-être pu ici remplacer le help_text)
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Statut changé de Solution validée à 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)
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
manager: forbid making booked event recurrent (#57609)