Bug #47512
subslug, bad character in group name
Début:
09 octobre 2020
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
File "/usr/lib/python3/dist-packages/combo/public/views.py" in page
495. extra = extract_context_from_sub_slug(page.sub_slug, parts[i+1])
File "/usr/lib/python3/dist-packages/combo/data/models.py" in extract_context_from_sub_slug
126. match = re.match('^' + sub_slug + '$', sub_url)
File "/usr/lib/python3.7/re.py" in match
173. return _compile(pattern, flags).match(string)
File "/usr/lib/python3.7/re.py" in _compile
286. p = sre_compile.compile(pattern, flags)
File "/usr/lib/python3.7/sre_compile.py" in compile
764. p = sre_parse.parse(p, flags)
File "/usr/lib/python3.7/sre_parse.py" in parse
930. p = _parse_sub(source, pattern, flags & SRE_FLAG_VERBOSE, 0)
File "/usr/lib/python3.7/sre_parse.py" in _parse_sub
426. not nested and not items))
File "/usr/lib/python3.7/sre_parse.py" in _parse
685. raise source.error(msg, len(name) + 1)
Exception Type: error at /services/ma-vie-de-quartier/niveau-de-pollution/qualite-de-l-air-localisee/4.865372,45.770818/
Exception Value: bad character in group name 'qP<g1' at position 21
Sur un subslug (?P<q_lon>[0-9.]+),(?P<q_lat>[0-9.]+), pour une page appelée .../qualite-de-l-air-localisee/4.865372,45.770818/
Fichiers
Révisions associées
misc: handle multi-part subslugs (#47512)
Historique
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Fichier 0001-misc-handle-multi-part-subslugs-47512.patch 0001-misc-handle-multi-part-subslugs-47512.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Le bug est qu'on remplace dans la chaine selon les références données par la regex, qui ne sont plus correctes après le premier remplacement d'un groupe par un groupe au nom plus court, où on arrive à ici '(?P<g0>[0-9.]+),(?P<qP<g1>9.]+)'
Patch qui est un peu con à plutôt remplacer <ancien nom> par <nouveau nom>, en imaginant bien qu'on pourrait risquer la présence de <ancien nom> ailleurs dans l'expression mais que c'est quand même bien peu probable.
Mis à jour par Lauréline Guérin il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
ha ben oui, bien vu
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 20f05567925c5dcccb3f1f52bf81e8587cda02ff
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Fri Oct 9 12:16:20 2020 +0200
misc: handle multi-part subslugs (#47512)
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: handle multi-part subslugs (#47512)