Bug #6846
les < et > passent dans les slug, et « et » aussi (par transliteration)
0%
Description
(remontée par l'AUF)
Un formulaire nommé "ceci est un <<test>>" ou "ceci est un «test»" a une URL "ceci-est-un-<<test>>'
Même si elle n'empeche pas le fonctionnement, la présente des < ou > n'est sans doute pas souhaitable dans une URL ?
Fichiers
Révisions associées
misc: rewrite qommon.misc.simplify() to use unicodedata module (#6846)
Historique
Mis à jour par Thomas Noël il y a environ 9 ans
- Fichier 0001-rewrite-qommon.misc.simplify-6846.patch 0001-rewrite-qommon.misc.simplify-6846.patch ajouté
En regardant le slugify de Django, je me suis dit qu'on pouvait s'en inspirer pour "moderniser" notre fonction simplify.
Mis à jour par Frédéric Péters il y a environ 9 ans
J'ai fait un tour (mais peut-être pas tout à fait exhaustif) des endroits où la fonction est utilisée et son résultat non stocké, pour voir les risques de perturbation en cas de résultat différent entre la version actuelle et la version proposée; le seul endroit que j'ai noté c'est dans la production de nom de classe CSS pour les catégories et pour les liens utiles (côté auquo).
Mis à jour par Thomas Noël il y a environ 9 ans
- Assigné à mis à Thomas Noël
J'ai vu la même chose. Idéalement il faudrait écrire quelques tests maintenant puis appliquer le patch et voir que rien n'a bougé. Je regarde ça, ça me fera un petit entrainement sans trop d'enjeu pour mettre les pieds dans les tests.
Mis à jour par Thomas Noël il y a environ 9 ans
- Fichier 0001-tests-add-tests-for-qommon.misc.simplify-6846.patch 0001-tests-add-tests-for-qommon.misc.simplify-6846.patch ajouté
Par exemple ces tests très simples, à appliquer maintenant (avec le simplify actuel).
Mis à jour par Frédéric Péters il y a environ 9 ans
- Statut changé de Nouveau à Résolu (à déployer)
J'ai copié les tests dans test_misc.py et j'ai ajouté un cleanup() pour être sûr de ne pas avoir un get_publisher() restant d'autres tests et qui aurait interféré avec le 'iso-8859-1' spécifié explicitement dans un des tests.
commit 8d61f8a015c163afaa4eafb57ec3cdbee67544bc
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Fri Mar 27 19:10:58 2015 +0100
misc: rewrite qommon.misc.simplify() to use unicodedata module (#6846)
commit 4230ff7d9d371f8b2dfab1d22c1040c496217337
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Sun Mar 29 14:21:50 2015 +0200
tests: add tests for qommon.misc.simplify (#6846)
commit b22b3b126fb2f0a9704cbe8ce29bca23ed07a2c0
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sun Apr 12 11:03:16 2015 +0200
tests: add a workaround for backoffice/api circular dependency
tests: add tests for qommon.misc.simplify (#6846)