Projet

Général

Profil

Bug #47966

Exceptions jours fériés orphelines

Ajouté par Valentin Deniaud il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
22 octobre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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.


Fichiers

Révisions associées

Révision 06220c72 (diff)
Ajouté par Valentin Deniaud il y a plus de 3 ans

agendas: rename flag on exception sourced from settings (#47966)

Révision 04831425 (diff)
Ajouté par Valentin Deniaud il y a plus de 3 ans

agendas: move desk settings exceptions import to view (#47966)

Révision 88ae9d52 (diff)
Ajouté par Valentin Deniaud il y a plus de 3 ans

agendas: import/export settings exception source (#47966)

Révision a3085e5c (diff)
Ajouté par Valentin Deniaud il y a plus de 3 ans

agendas: rename flag on exception sourced from settings (#47966)

Révision 48700e3d (diff)
Ajouté par Valentin Deniaud il y a plus de 3 ans

agendas: move desk settings exceptions import to view (#47966)

Révision 0f6e6ff3 (diff)
Ajouté par Valentin Deniaud il y a plus de 3 ans

agendas: import/export settings exception source (#47966)

Historique

#3

Mis à jour par Valentin Deniaud il y a plus de 3 ans

  • Assigné à mis à Valentin Deniaud

Le week-end porte conseil, elles apparaissent après import/export...

#4

Mis à jour par Valentin Deniaud il y a plus de 3 ans

Deux erreurs de conception, qu'on pourrait dire à l'origine du bug :
  • 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.

#5

Mis à jour par Valentin Deniaud il y a plus de 3 ans

  • Tracker changé de Development à Bug
  • Statut changé de Solution proposée à Nouveau
#7

Mis à jour par Valentin Deniaud il y a plus de 3 ans

  • Statut changé de Nouveau à Solution proposée
#8

Mis à jour par Lauréline Guérin il y a plus de 3 ans

  • Statut changé de Solution proposée à Solution validée
#9

Mis à jour par Valentin Deniaud il y a plus de 3 ans

  • Statut changé de Solution validée à 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)
#10

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF