Projet

Général

Profil

Bug #65526

Bloc de champs - l'identifiant d'un bloc devrait être unique (quoi qu'on fasse...)

Ajouté par Anaïs Ecuvillon → en congés, retour le 30/04 il y a presque 2 ans. Mis à jour il y a presque 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Frédéric Péters
Version cible:
-
Début:
20 mai 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Je viens de créer un bug sur mon instance à cause de 2 blocs de champs avec le même identifiant.
Voilà ce que j'ai fait :
  1. export d'un bloc de champs avec pour identifiant "identite"
  2. import de ce bloc sur la même instance (avec pour finalité de le dupliquer)
  3. modification de l'identifiant dans les paramètres, avec identifiant "identite" (je n'avais pas tilté que le premier avait le même identifiant...)

Quand je vais une sur démarche en front, j'ai un Internal Server Error

Maintenant je ne peux plus ni modifier, ni supprimer car le bloc est utilisé.

Les 2 blocs en question :

Fichiers

0001-backoffice-check-slug-for-uniqueness-65526.patch (2,85 ko) 0001-backoffice-check-slug-for-uniqueness-65526.patch Frédéric Péters, 23 mai 2022 13:59

Révisions associées

Révision eb3db7b6 (diff)
Ajouté par Frédéric Péters il y a presque 2 ans

backoffice: check slug for uniqueness (#65526)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 2 ans

  • Statut changé de Nouveau à En cours
  • Assigné à mis à Frédéric Péters
#2

Mis à jour par Frédéric Péters il y a presque 2 ans

#3

Mis à jour par Emmanuel Cazenave il y a presque 2 ans

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

existing_slug = ... mais en fait il s'agit d'une instance de BlockDef et du coup ensuite existing_slug.id != self.objectdef.id on comprend pas vraiment au premier abord pourquoi un slug aurait un attribut id, bref un renommage de variable faciliterait la lecture.

#4

Mis à jour par Frédéric Péters il y a presque 2 ans

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

J'ai renommé et ajouté un commentaire,

commit eb3db7b609b4888d1311b77bbe755a33ffda5050
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon May 23 09:48:20 2022 +0200

    backoffice: check slug for uniqueness (#65526)
#5

Mis à jour par Transition automatique il y a presque 2 ans

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

Mis à jour par Anaïs Ecuvillon → en congés, retour le 30/04 il y a presque 2 ans

Frédéric Péters a écrit :

Le patch et aussi, pour rétablir, j'ai supprimé https://demarches-aecuvillon.test.entrouvert.org/backoffice/forms/blocks/14/

merci Fred

#7

Mis à jour par Transition automatique il y a plus d'un an

Automatic expiration

Formats disponibles : Atom PDF