Bug #25815
base: virer la déclaration doublon du champ "slug"
Début:
24 août 2018
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
De #13053:
title = models.CharField(max_length=50, verbose_name=_('Title'))
- slug = models.SlugField(verbose_name=_('Identifier'))
+ slug = models.SlugField(verbose_name=_('Identifier'), unique=True)
description = models.TextField(verbose_name=_('Description'))
+ slug = models.SlugField(unique=True)
Fichiers
Révisions associées
Historique
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Fichier 0001-base-remove-duplicate-slug-field-declaration-25815.patch 0001-base-remove-duplicate-slug-field-declaration-25815.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus de 5 ans
Faut conserver celui qui est nommé (cf migrations)
Mis à jour par Serghei Mihai il y a plus de 5 ans
Si je garde celui qui est nommé make_migrations crée une nouvelle migration pour rajouter le verbose_name.
Mis à jour par Thomas Noël il y a plus de 5 ans
Parce que ça a été flingué avant, mais il y avait bien un verbose_name, faut le laisser (et oui, c'est over relou)
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
Il faut modifier la migration aussi, c'est tout simple.
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Fichier 0001-base-remove-duplicate-slug-field-declaration-25815.patch 0001-base-remove-duplicate-slug-field-declaration-25815.patch ajouté
Plutôt "les migrations" étant donné que BaseResource est abstraite.
Donc je modifie toutes les migrations ou slug n'a pas de verbose_name. Cela donne des migrations inutiles, que je trouve utile de garder pour faire les migrations backward proprement.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
- Assigné à mis à Serghei Mihai
commit bfa6827b87d729d28187d574de41b5eb0c48c1c5 (origin/master, origin/HEAD)
Author: Serghei Mihai <smihai@entrouvert.com>
Date: Fri Aug 24 13:59:49 2018 +0200
base: remove duplicate slug field declaration (#25815)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de Solution déployée à Fermé
base: remove duplicate slug field declaration (#25815)