Bug #47966
Exceptions jours fériés orphelines
0%
Description
Pour ne pas embrouiller #47916, où j'écrivais :
Essonne, chaque exception est en double, une seule source, le doublon a perdu son attribut « source » et son flag « external » : https://agendas.demarches.essonne.fr/manage/agendas/1/settings
Après avoir scruté le code et appuyé sur tous les boutons, je n'ai toujours aucune idée de comment ça a pu arriver. Malheureusement ça s'est produit sur deux autres instances :
https://agendas.demarches.ville-sens.fr/manage/agendas/50/settings
https://agendas.mesdemarches.grand-chatellerault.fr/manage/agendas/1154/settings
Je vais corriger à la main, espérer que ça soit tout de même lié à #47916, et si pas de nouvelle occurrence dans ~1 mois je fermerai.
Files
Associated revisions
agendas: move desk settings exceptions import to view (#47966)
agendas: import/export settings exception source (#47966)
agendas: rename flag on exception sourced from settings (#47966)
agendas: move desk settings exceptions import to view (#47966)
agendas: import/export settings exception source (#47966)
History
Updated by Valentin Deniaud about 4 years ago
- Assignee set to Valentin Deniaud
Le week-end porte conseil, elles apparaissent après import/export...
Updated by Valentin Deniaud about 4 years ago
- File 0001-agendas-rename-flag-on-exception-sourced-from-settin.patch 0001-agendas-rename-flag-on-exception-sourced-from-settin.patch added
- File 0003-agendas-import-export-settings-exception-source-4796.patch 0003-agendas-import-export-settings-exception-source-4796.patch added
- File 0002-agendas-move-desk-settings-exceptions-import-to-view.patch 0002-agendas-move-desk-settings-exceptions-import-to-view.patch added
- Tracker changed from Bug to Développement
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
- exception.external veut dire exactement que exception.source.settings_slug n'est pas nul, donc external ne sert à rien (en plus d'avoir un nom mal choisi). Corrigé dans 0001.
- Faire des trucs cachés dans la méthode save(), c'est pas bien. Corrigé dans 0002.
Reste tout de même qu'il faut importer/exporter la source (pour savoir si elle est activée ou pas), c'est 0003.
Updated by Valentin Deniaud about 4 years ago
- Tracker changed from Développement to Bug
- Status changed from Solution proposée to Nouveau
Updated by Lauréline Guérin about 4 years ago
- Status changed from Solution proposée to Solution validée
Updated by Valentin Deniaud about 4 years ago
- Status changed from Solution validée to Résolu (à déployer)
commit 88ae9d52efd7ff6e502deaf2222d7988331827db
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Mon Oct 26 18:09:53 2020 +0100
agendas: import/export settings exception source (#47966)
commit 0483142578b23f8f50bfebf667f1f0c7634bca7d
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Mon Oct 26 17:26:57 2020 +0100
agendas: move desk settings exceptions import to view (#47966)
commit 06220c72e2ef26574ba01b90a559530ede619917
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Mon Oct 26 16:55:59 2020 +0100
agendas: rename flag on exception sourced from settings (#47966)Updated by Frédéric Péters about 4 years ago
- Status changed from Résolu (à déployer) to Solution déployée
agendas: rename flag on exception sourced from settings (#47966)