Projet

Général

Profil

Bug #34044

interdire les agendas avec des slugs identiques

Ajouté par Frédéric Péters il y a presque 5 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
17 juin 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Rencontré dans #34043 deux agendas se sont trouvés avec le même slug.

MultipleObjectsReturned at /api/agenda/jouons-jouez-jouets-de-plantes-19-juin/datetimes/
get() returned more than one Agenda -- it returned 2!

Fichiers

Révisions associées

Révision fe8f29d4 (diff)
Ajouté par Lauréline Guérin il y a plus de 4 ans

agendas: slug unicity for Agenda, Desk and MeetingType (#34044)

Historique

#2

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

  • Assigné à mis à Lauréline Guérin
#3

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

je vais traiter les slug des models Agenda, MeetingType et Desk

#4

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

Avant de pouvoir ajouter des contraintes d'unicité en DB, il faut traiter les éventuels doublons.
J'ai choisi de ne pas toucher au premier élément (pas ordre de création/pk), et de changer les slugs des doublons suivants.
Mais, ça mériterait peut-être un petit état des lieux en prod et recette avant de passer une telle migration.

#5

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

la même, avec des get_or_create qui vont bien pour ne pas avoir d'IntegrityError lors des imports json

#6

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

Mais, ça mériterait peut-être un petit état des lieux en prod et recette avant de passer une telle migration.

Je viens de vérifier côté SaaS et c'est ok recette et prod.

#7

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

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

Ça marche pour moi; ça marche aussi si tu veux comme dans #37733 réécrire la boucle.

#8

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

J'ai corrigé, le build jenkins est ok; je merge :)

#9

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit fe8f29d4d4814f0285e38aac2ddec2bc56e767d5
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Thu Nov 7 12:27:05 2019 +0100

    agendas: slug unicity for Agenda, Desk and MeetingType (#34044)
#10

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

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

Formats disponibles : Atom PDF