Project

General

Profile

Development #44512

à la suppression d'un type de rendez-vous, en altérer le slug

Added by Frédéric Péters 10 days ago. Updated 3 days ago.

Status:
Solution déployée
Priority:
Normal
Category:
-
Target version:
-
Start date:
26 Jun 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Pour que le slug puisse être réutilisé par un autre.

0001-manager-allow-reusing-meeting-type-slug-after-deleti.patch View (2.03 KB) Valentin Deniaud, 01 Jul 2020 11:53 AM

0001-manager-allow-reusing-meeting-type-slug-after-deleti.patch View (2.04 KB) Valentin Deniaud, 02 Jul 2020 05:35 PM

Associated revisions

Revision f1d67da6 (diff)
Added by Valentin Deniaud 4 days ago

manager: allow reusing meeting type slug after deletion (#44512)

History

#2 Updated by Valentin Deniaud 5 days ago

  • Assignee set to Valentin Deniaud

#3 Updated by Valentin Deniaud 5 days ago

#4 Updated by Thomas Noël 4 days ago

« '#' + 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__" ?

#5 Updated by Benjamin Dauvergne 4 days ago

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é.

#6 Updated by Thomas Noël 4 days ago

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.

#8 Updated by Thomas Noël 4 days ago

  • Status changed from Solution proposée to Solution validée

#9 Updated by Valentin Deniaud 4 days ago

  • Status changed from Solution validée to 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)

#10 Updated by Frédéric Péters 3 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF