Projet

Général

Profil

Bug #37987

(recette) les libellés des événements ont disparus au profit d'un identifiant par défaut "none-<id>"

Ajouté par Mikaël Ates il y a plus de 4 ans. Mis à jour il y a plus de 4 ans.

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Constaté en premier sur https://agenda-tsl.test-demarches.sictiam.fr/manage/agendas/2/settings. L'événement qui posait problème a été supprimé. Le nouvel événement créé est bien conforme.
La réservation de places sur cet événement générait une 500 sans que je puisse en savoir davantage : https://demarches-tsl.test-demarches.sictiam.fr/backoffice/management/theatre-concert-salle-prevert/12/inspect

Je constate cependant sur Mauguio la même chose sur les événements existants : https://agenda-mauguio-carnon.test.entrouvert.org/manage/agendas/6/settings
Les événements n'ont plus de Libellé. Ils ont cependant un identifiant initialisé à "none-<id>".
On voit ici l'affichage attendu : https://agendas.demarches.mauguio-carnon.com/manage/agendas/1/settings
J'ai cependant pu ici réserver une place sur l'événement encombrant mauguio du 4 décembre :

Fichiers

Révisions associées

Révision b9c5fe37 (diff)
Ajouté par Frédéric Péters il y a plus de 4 ans

general: make event slugs optional (#37987)

Révision 69067a87 (diff)
Ajouté par Frédéric Péters il y a plus de 4 ans

add missed migration (#37987)

Historique

#1

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

L'événement qui posait problème a été supprimé.

Idéalement ne pas faire ça, pour permettre de creuser.

On voit ici l'affichage attendu : https://agendas.demarches.mauguio-carnon.com/manage/agendas/1/settings

Il n'y a pas de libellés aux événements présentés sur cette page.

~~

Il y a eu ajout d'identifiants aux événements, ils ont été initialisés selon les libellés mais donc, on voit que beaucoup d'événements n'avaient pas de libellés.

Je serais pour 1/ en absence de libellé, initialiser le slug sur base de la date/heure de l'événement, 2/ si c'est possible d'avoir à la fois null=True et unique=True, le permettre, 3/ peut-être même réinitialiser les slugs commençant par "none" (soit à None si on peut, soit sur base de la date/heure), 4/ retirer pour le moment l'identifiant de la liste, il restera accessible en cliquant sur l'événement.

Par rapport à mon interrogation sur null=True/unique=True, je me dis aussi qu'on ne devrait pas avoir de unique=True su le slug, mais un unique_together=(agenda, slug).

#2

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

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

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

  • Fichier 0001-general-make-event-slugs-optional-37987.patch 0001-general-make-event-slugs-optional-37987.patch ajouté
  • Sujet changé de Constaté uniquement sur les recettes, pas en production, les libellés des événements ont disparus au profit d'un identifiant par défaut "none-<id>" (la réservation de certains événements pourrait peut-être échouer à cause de cela) à (recette) les libellés des événements ont disparus au profit d'un identifiant par défaut "none-<id>"
  • Statut changé de En cours à Solution proposée
  • Patch proposed changé de Non à Oui

Les colonnes NULL sont ignorées et je suis donc parti pour simplement permettre ça; par rapport au nettoyage des recettes (suppression des identifiants none-*), je ferai ça en runscript oneshot.

#4

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

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

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit b9c5fe37740a126428075a908f5b606508d33bef
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Nov 27 09:27:44 2019 +0100

    general: make event slugs optional (#37987)
#6

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
#7

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

(et j'avais oublié d'inclure la migration)

#8

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

(et je viens de passer sur les différents serveurs pour

from chrono.agendas.models import Event
Event.objects.filter(slug__startswith='none').update(slug=None)

)

(et noter aussi que le contrainte d'unicité sur le slug n'était pas retirée de la db, parce que j'ai également modifié les migrations en amont, du coup la 0032 zappe ce côté-là, j'ai corrigé en modifiant la migration 0029 pour redéclarer l'unicité puis en retournant à 0031 pour rejouer 0032).

Formats disponibles : Atom PDF