Development #50672
Exception read_only ?
0%
Description
En lisant les patchs de #50561 je m'interroge sur la property read_only
d'une exception.
Actuellement elle est vraie si l'exception provient d'une source settings ou d'un calendrier d'indisponibilité.
Pour les autres types de source (file, url), une exception peut être supprimée, mais comme on lance la commande de synchro toutes les heures, qui supprime toutes les exceptions puis les recrée, alors autant déclarer une exception provenant d'une source quelle quelle soit comme read_only, non ?
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a environ 3 ans
Ouep je pense qu'il faudrait faire comme ça, le flag read_only a été inventé dans l'idée de ne pas toucher l'existant sans trop se poser de questions.
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Fichier 0001-agendas-read_only-property-for-exception-source-5067.patch 0001-agendas-read_only-property-for-exception-source-5067.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Valentin Deniaud il y a environ 3 ans
Vu qu'on ne regarde plus self.source.settings_slug
je pense qu'on peut virer les deux endroits où on fait un select_related('source')
.
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Fichier 0001-agendas-read_only-property-for-exception-source-5067.patch 0001-agendas-read_only-property-for-exception-source-5067.patch ajouté
yep, en effet
Mis à jour par Valentin Deniaud il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 61c88e2246bd4bca5c84f3861e295d32cb0ffa29 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Jan 29 10:46:10 2021 +0100 agendas: read_only property for exception & source (#50672)
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
agendas: read_only property for exception & source (#50672)