Development #44512
à la suppression d'un type de rendez-vous, en altérer le slug
0%
Description
Pour que le slug puisse être réutilisé par un autre.
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a presque 4 ans
- Fichier 0001-manager-allow-reusing-meeting-type-slug-after-deleti.patch 0001-manager-allow-reusing-meeting-type-slug-after-deleti.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a presque 4 ans
« '#' + str(uuid.uuid4())[8:] » y'a un petit risque de collision.
Y'a la place pour mettre un uuid4 complet, et ajouter "#deleted-" devant, n'hésite pas. Et dans le test, tu pourras explicitement vérifier que le slug commence par "#deleted"
Pour ma part je suis un peu étonné que slugField permette un # : "containing only letters, numbers, underscores or hyphens. " dans la doc. Peut-être plutôt commencer par "__delete__" ?
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
Thomas Noël a écrit :
Pour ma part je suis un peu étonné que slugField permette un # : "containing only letters, numbers, underscores or hyphens. " dans la doc. Peut-être plutôt commencer par "__delete__" ?
La validation ça ne concerne que les formulaires/API DRF, validators=[] est ignoré quand on manipule directement un modèle, et en dessous de SlugField c'est un simple varchar[n] qui est utilisé.
Mis à jour par Thomas Noël il y a presque 4 ans
Benjamin Dauvergne a écrit :
Thomas Noël a écrit :
Pour ma part je suis un peu étonné que slugField permette un # : "containing only letters, numbers, underscores or hyphens. " dans la doc. Peut-être plutôt commencer par "__delete__" ?
La validation ça ne concerne que les formulaires/API DRF, validators=[] est ignoré quand on manipule directement un modèle, et en dessous de SlugField c'est un simple
varchar[n]qui est utilisé.
Ouaip j'ai vu ça aussi... mais un jour ça pourrait changer, ou nous gêner, et donc je me suis dit autant ne pas utiliser # qui n'est pas décrit comme valide dans la doc de SlugField.
Mis à jour par Valentin Deniaud il y a presque 4 ans
- Fichier 0001-manager-allow-reusing-meeting-type-slug-after-deleti.patch 0001-manager-allow-reusing-meeting-type-slug-after-deleti.patch ajouté
Yep, hop.
Mis à jour par Thomas Noël il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Valentin Deniaud il y a presque 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit f1d67da603394de4651caa2f4f5aec9d516185c5
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Wed Jul 1 11:51:50 2020 +0200
manager: allow reusing meeting type slug after deletion (#44512)Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
manager: allow reusing meeting type slug after deletion (#44512)