Development #42336
Permettre de forcer la pose d'une exception même si des réservations sont présentes sur la tranche temporelle de l'exception
0%
Description
C'est pour pouvoir bloquer la prise de rendez-vous sur un moment donné même si il y a déjà eu des réservation de posées, parce que ... shit happens : #42296.
A vérifier l'impact sur les vues journalières/mensuelles, il faudrait qu'on puisse encore y voir les réservations déjà posées.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 4 ans
- Sujet changé de Permettre de forcer la pause d'une exception même si des réservations sont présentes sur la tranche temporelle de l'exception à Permettre de forcer la pose d'une exception même si des réservations sont présentes sur la tranche temporelle de l'exception
Mis à jour par Frédéric Péters il y a presque 4 ans
- Dupliqué par Bug #42783: ne plus empêcher de poser des exceptions quand des rendez-vous existent déjà ajouté
Mis à jour par Thomas Noël il y a presque 4 ans
Pour ma part, je pense qu'il n'y a même pas besoin de demander à "forcer". On se fiche que des rendez-vous existent déjà, il faut juste éventuellement en informer la personne qui pose l'exception.
Mis à jour par Lauréline Guérin il y a presque 4 ans
- Fichier 0001-agenda-dont-check-for-bookings-on-exception-creation.patch 0001-agenda-dont-check-for-bookings-on-exception-creation.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
les rendez-vous s'affichent bien sur les vues journalières et mensuelles, même s'il y a une exception
Mis à jour par Thomas Noël il y a presque 4 ans
En fait je me disais que patch serait plutôt de remplacer le ValidationError par un « messages.warning(self.request, _('One or several bookings exists within this new exception.')) » ... est-ce que ça marcherait ?
Mis à jour par Lauréline Guérin il y a presque 4 ans
- Fichier 0001-agenda-dont-check-for-bookings-on-exception-creation.patch 0001-agenda-dont-check-for-bookings-on-exception-creation.patch ajouté
Un truc comme ça ?
Je n'ai fait le check que sur la création d'exception, parce que ça ne me paraît pas pertinent sur l'update.
On alors il faudrait d'abord vérifier si les dates ont changé, et dans ce cas seulement faire le check. Mais, je l'avoue, j'ai eu la flemme :)
Mis à jour par Thomas Noël il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
Lauréline Guerin a écrit :
Un truc comme ça ?
Je n'ai fait le check que sur la création d'exception, parce que ça ne me paraît pas pertinent sur l'update.
On alors il faudrait d'abord vérifier si les dates ont changé, et dans ce cas seulement faire le check. Mais, je l'avoue, j'ai eu la flemme :)
Ca me semble bon.
Si tu as quand même un peu de force, peut-être préciser dans le message.info que l'exception a bien été ajoutée, parce que ce n'est jamais bien clair pour l'utilisateur qu'un message d'information n'est pas une erreur :
- message = _('One or several bookings exists within this time slot.')
+ message = _('Exception added. Note: one or several bookings exists within this time slot.')
Mais je valide le patch ainsi, tu peux pousser avec ou sans ce détail là.v
Mis à jour par Lauréline Guérin il y a presque 4 ans
j'ai changé le wording; j'attends jenkins et je merge :)
Mis à jour par Lauréline Guérin il y a presque 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 4e681509380a34c2f8117b3ae5c52ec9ca3b0a03
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Fri May 22 14:18:41 2020 +0200
agenda: dont check for bookings on exception creation (#42336)
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
agenda: dont check for bookings on exception creation (#42336)