Development #43621
Supprimer un type de rendez-vous flingue toutes les réservations de ce type
0%
Description
elif data['kind'] == 'meetings':
if overwrite:
MeetingType.objects.filter(agenda=agenda).delete()
Desk.objects.filter(agenda=agenda).delete()
et les on_delete=models.CASCADE se propagent jusqu'au Booking, merci au revoir, là où l'import passe l'herbe ne repousse pas.
Fichiers
Demandes liées
Révisions associées
manager: don't allow deletion of busy meeting type (#43621)
Historique
Mis à jour par Emmanuel Cazenave il y a presque 4 ans
- Sujet changé de Importer un agenda déjà existant flingue toutes les réservations de cet agenda à Supprimer un type de rendez-vous flingue toutes les réservations de ce type
Ça a lieu aussi dans via l'UI en supprimant un type de rendez-vous, c'est d'la bombe bébé.
Mis à jour par Emmanuel Cazenave il y a presque 4 ans
- Statut changé de Nouveau à En cours
- Assigné à mis à Emmanuel Cazenave
Mis à jour par Emmanuel Cazenave il y a presque 4 ans
- Fichier 0001-manager-do-not-overwrite-when-importing-43621.patch 0001-manager-do-not-overwrite-when-importing-43621.patch ajouté
- Fichier 0002-manager-don-t-allow-deletion-of-busy-meeting-type-43.patch 0002-manager-don-t-allow-deletion-of-busy-meeting-type-43.patch ajouté
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a presque 4 ans
Sur 0002, comme ailleurs va venir derrière le besoin quand même de ne plus proposer à réservation un type de rendez-vous; plutôt simplement poser un flag deleted sur le modèle, qui ferait qu'il ne serait plus affiché, etc. ?
Mis à jour par Emmanuel Cazenave il y a presque 4 ans
Je pourrais regarder ça mais je n'ai fait que reprendre la technique déjà utilisée sur suppression de guichet et de type de rendez-vous.
Et donc comme le problème que tu soulèves se pose aussi pour la suppression de ces objets, on gérerais pas ça dans un ticket à part "permettre de désactiver un agenda/type de rdv/guichet/évènement" ?
Mis à jour par Emmanuel Cazenave il y a presque 4 ans
Emmanuel Cazenave a écrit :
Je pourrais regarder ça mais je n'ai fait que reprendre la technique déjà utilisée sur suppression de guichet et
de type de rendez-vous.
d'agenda
Mis à jour par Frédéric Péters il y a presque 4 ans
Mais sur un guichet formellement on peut retirer les horaires d'ouverture et le voilà désactivé. (et les agendas, je ne pense pas qu'on ait de démarches qui laisse en choix libre à l'usager tous les agendas, même si /api/agenda/ le permettrait).
Je pense vraiment en ce sens le cas "type de rendez-vous" différent.
Mais si tu préfères d'abord ce ticket avant l'autre, ok.
Mis à jour par Emmanuel Cazenave il y a presque 4 ans
Ok convaincu, j'allais poursuivre dans ce ticket, et là vient #44125, ça commence à devenir pénible.
Du coup je serai pour passer mes deux patchs en hotfix, et à poursuivre calmement sur tes remarques dans un nouveau ticket.
Mis à jour par Emmanuel Cazenave il y a presque 4 ans
- Lié à Development #44132: Permettre de désactiver un type de rendez-vous ajouté
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Emmanuel Cazenave il y a presque 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 5616ee60a0ccd61a2456e7329576df95e6e648d9
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date: Mon Jun 8 16:04:54 2020 +0200
manager: don't allow deletion of busy meeting type (#43621)
commit 27985b4c53088bb48ba15c5c8e3158a51b0dc3bd
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date: Mon Jun 8 15:21:48 2020 +0200
manager: do not overwrite when importing (#43621)
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
Mis à jour par Emmanuel Cazenave il y a plus de 3 ans
- Lié à Bug #48095: Import agenda (update) : duplication des périodes horaires et des exceptions ajouté
manager: do not overwrite when importing (#43621)