Project

General

Profile

Bug #28416

Impossible de tester une valeur accentuée dans un champs liste à choix multiple.

Added by Mikaël Ates 6 months ago. Updated 5 months ago.

Status:
Solution déployée
Priority:
Normal
Start date:
28 Nov 2018
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Mon champs liste contient :

form_var_objet        Adresse postale, Numéro de téléphone, Adresse de courriel
form_var_objet_raw    ['Adresse postale', 'Num\xc3\xa9ro de t\xc3\xa9l\xc3\xa9phone', 'Adresse de courriel']

Une condition d'affichage sur un champs contenant form_var_objet_raw and "Numéro de téléphone" in form_var_objet_raw ou form_var_objet_raw and "Num\xc3\xa9ro de t\xc3\xa9l\xc3\xa9phone" in form_var_objet_raw ne fonctionne pas.

Par exemple https://demarches-validation.test.entrouvert.org/backoffice/forms/139/.

0001-conditions-encode-items-values-for-contains-checks-2.patch View (3.27 KB) Frédéric Péters, 28 Nov 2018 12:09 PM

Associated revisions

Revision d9856313 (diff)
Added by Frédéric Péters 6 months ago

conditions: encode items values for contains checks (#28416)

History

#2 Updated by Mikaël Ates 6 months ago

  • Subject changed from Imposible de tester une valeur accentué dans un champs liste à choix multiple. to Imposible de tester une valeur accentuée dans un champs liste à choix multiple.

#3 Updated by Frédéric Péters 6 months ago

J'aimerais ici prendre le risque de divergence, parce que j'aimerais qu'on considère comme un bug d'utiliser "raw" en Django.

Ainsi, le patch attaché, qui assure que "éléphant" in form_var_savane fonctionne dans les conditions, en comprenant "dynamiquement" que form_var_savane doit être considéré comme une liste d'éléments.

Le test écrit pour l'occasion vérifie que cette forme est ok en condition, en gabarit dans un commentaire et en gabarit dans le workflow (action "afficher un message").

#4 Updated by Benjamin Dauvergne 6 months ago

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

Je n'ai pas bien compris ce qui diverge mais ok.

#5 Updated by Benjamin Dauvergne 6 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit d9856313f37f4bec08f0afe946d94ed4cdd1fbe9 (HEAD -> master)
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Nov 28 11:52:11 2018 +0100

    conditions: encode items values for contains checks (#28416)

#6 Updated by Benjamin Dauvergne 6 months ago

  • Assignee set to Frédéric Péters

#7 Updated by Benjamin Dauvergne 6 months ago

  • Subject changed from Imposible de tester une valeur accentuée dans un champs liste à choix multiple. to Impossible de tester une valeur accentuée dans un champs liste à choix multiple.

#8 Updated by Frédéric Péters 5 months ago

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

Also available in: Atom PDF