Project

General

Profile

Développement #57609

Interdire de rendre un évènement récurrent si il a déjà des réservations

Added by Valentin Deniaud about 3 years ago. Updated about 3 years ago.

Status:
Fermé
Priority:
Normal
Category:
-
Target version:
-
Start date:
05 October 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Pour ne pas créer un monstre.


Files

Associated revisions

Revision e9a85c13 (diff)
Added by Valentin Deniaud about 3 years ago

manager: forbid making booked event recurrent (#57609)

History

#1

Updated by Valentin Deniaud about 3 years ago

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.

#2

Updated by Lauréline Guérin about 3 years ago

et si la réservation a été annulée ?

#3

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.

#4

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.

#5

Updated by Benjamin Dauvergne about 3 years ago

  • Status changed from Solution proposée to En cours
#6

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 ?

#7

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.

#8

Updated by Valentin Deniaud about 3 years ago

Voilà j'ai fait comme tu dis.

#9

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)

#10

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)
#11

Updated by Frédéric Péters about 3 years ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF