Development #42629
Sur un guichet permettre d'ajouter une exception avant une période horaire
0%
Description
Sur un agenda très sollicité auquel on ajouterait un guichet, on ne veux pas qu'une réservation puisse être faite sur une plage horaire que l'on fermera quelques secondes plus tard au moyen d'une exception.
Fichiers
Révisions associées
Historique
Mis à jour par Emmanuel Cazenave il y a presque 4 ans
Pour modérer l'intérêt de ce ticket, il y a un contournement assez simple.
Renseigner une période horaire fictive très courte par exemple le lundi de 10h00 à 10h01, comme comme c'est plus court que la durée de n'importe quel type de rendez-vous (dans la vie la vraie), aucune réservation ne peut-être faite. Puis renseigner les exception, et revenir sur les périodes horaires pour y mettre les vraies.
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
Ne serait-ce pas plus clair d'avoir un champ "enabled" sur un guichet, qu'il soit désactivé au départ le temps de le configurer ?
Mis à jour par Emmanuel Cazenave il y a presque 4 ans
Oui et ça pourrait avoir des cas d'usages plus large du coup, genre préparer un tas de guichets en plus et les 'mettre en service' ultérieurement.
Mais méfiance sur les cas d'usage théoriques.
Mis à jour par Frédéric Péters il y a presque 4 ans
- Lié à Bug #42783: ne plus empêcher de poser des exceptions quand des rendez-vous existent déjà ajouté
Mis à jour par Frédéric Péters il y a presque 4 ans
- Lié à Bug #42783: ne plus empêcher de poser des exceptions quand des rendez-vous existent déjà supprimé
Mis à jour par Lauréline Guérin il y a presque 4 ans
La duplication des guichets ajouté récemment répond déjà partiellement au cas d'usage décrit.
Du coup je propose de juste permettre l'ajout d'exceptions même s'il n'y a pas de plages horaires, sans mécanisme d'activation d'un guichet.
Mis à jour par Lauréline Guérin il y a presque 4 ans
- Fichier 0001-agendas-manager-can-add-exception-even-if-not-timepe.patch 0001-agendas-manager-can-add-exception-even-if-not-timepe.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Emmanuel Cazenave il y a presque 4 ans
Je comprends pas pourquoi il y a autant de code de tests supprimé dans ton patch.
Mis à jour par Lauréline Guérin il y a presque 4 ans
Comme il n'y a plus besoin de créer une TimePeriod pour poser une TimePeriodException, j'ai enlevé les clicks pour créer une TimePeriod dans tous les tests qui ne font que créer des Exceptions.
Dans test_meetings_agenda_add_time_period_exception
j'ai remplacé la partie "création de data via l'interface" par des créations d'objet. Le test est plus rapide à tourner.
Sachant que toutes ces étapes sont répétées quand quasiment tous les tests, donc elles sont déjà couvertes par le reste.
Mis à jour par Emmanuel Cazenave il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
Lauréline Guerin a écrit :
Comme il n'y a plus besoin de créer une TimePeriod pour poser une TimePeriodException, j'ai enlevé les clicks pour créer une TimePeriod dans tous les tests qui ne font que créer des Exceptions.
Cool, j'aime quand on fait le ménage.
Mis à jour par Lauréline Guérin il y a presque 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit a65930a150ee23c8992f87dbe66e5fd26fe1c6a5 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon Jun 1 10:11:42 2020 +0200 agendas: manager can add exception even if not timeperiod defined (#42629)
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
agendas: manager can add exception even if not timeperiod defined (#42629)