Development #33557
Utilisation réelle de token_check : AttributeError: 'LazyFieldVar' object has no attribute 'strip'
0%
Description
Avec une expression du genre :
form_var_secret_genere|token_check:saisie_secret_var_secret_saisi
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Fichier 0001-cast-argument-of-token_check-to-unicode-33557.patch 0001-cast-argument-of-token_check-to-unicode-33557.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a presque 5 ans
En préfixant le commit de "templatetags: ".
Aussi, et plus important, ça a été testé en vrai en tapant un caractère non-ascii ?
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
Frédéric Péters a écrit :
En préfixant le commit de "templatetags: ".
Ok.
Aussi, et plus important, ça a été testé en vrai en tapant un caractère non-ascii ?
Non mais comme j'ai vu des unicode partout dans qommon.py je me suis dis que c'était la procédure.
Mis à jour par Frédéric Péters il y a presque 5 ans
Je parlais d'un vrai test "réel", avec un bout tapé par l'usager, genre le vrai truc où ça va être utilisé. (pas un test artificiel avec deux caractères nécessairement ok parce que des chaines directement tirées des templates).
Mis à jour par Frédéric Péters il y a presque 5 ans
(et ce test "complet", "réel", il n'y a vraiment pas besoin pour moi que ça soit un test unitaire, juste que "clic clic tap tap dans mon navigateur, ça passe"). (le truc qui aurait évité ce ticket)
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Fichier 0001-templatetags-cast-argument-of-token_check-to-unicode.patch 0001-templatetags-cast-argument-of-token_check-to-unicode.patch ajouté
Et hop avec un test et des accents dedans.
Mis à jour par Frédéric Péters il y a presque 5 ans
(et ce test "complet", "réel", il n'y a vraiment pas besoin pour moi que ça soit un test unitaire, juste que "clic clic tap tap dans mon navigateur, ça passe"). (le truc qui aurait évité ce ticket)
Bref, j'ai essayé de faire ça, mais obligé d'imaginer en local ce qui pouvait échouer "en utilisation réelle", j'ai tapé un formulaire, avec un champ et des conditions de sortie de page, où j'ai fait form_var_blah|token_check:"whatéver" et "whatéver"|token_check:form_var_blah, et puis avec un champ supplémentaire, form_var_blah|token_check:form_var_blah2. Puis j'ai essayé avec une condition sur un champ. Et rien de tout ça ne m'a donné la trace mentionnée dans ce ticket. Du coup bien mal en peine de vérifier que le changement ici est ok "en condition réelle".
C'est arrivé où/comment ?
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
Frédéric Péters a écrit :
(et ce test "complet", "réel", il n'y a vraiment pas besoin pour moi que ça soit un test unitaire, juste que "clic clic tap tap dans mon navigateur, ça passe"). (le truc qui aurait évité ce ticket)
Bref, j'ai essayé de faire ça, mais obligé d'imaginer en local ce qui pouvait échouer "en utilisation réelle", j'ai tapé un formulaire, avec un champ et des conditions de sortie de page, où j'ai fait form_var_blah|token_check:"whatéver" et "whatéver"|token_check:form_var_blah, et puis avec un champ supplémentaire, form_var_blah|token_check:form_var_blah2. Puis j'ai essayé avec une condition sur un champ. Et rien de tout ça ne m'a donné la trace mentionnée dans ce ticket. Du coup bien mal en peine de vérifier que le changement ici est ok "en condition réelle".
C'est arrivé où/comment ?
Je ne trouve pas de trace dans les mails donc je suppose que Mike a du tester ça dans l'inspecteur. Et donc oui c'est ça: aller sur https://demarches-departement06.test.entrouvert.org/backoffice/management/appairage-new/15/inspect et taper la condition "form_var_secret_genere|token_check:saisie_secret_var_secret_saisi". Ça donne cela :
Message d'erreur : AttributeError: 'LazyFieldVar' object has no attribute 'strip'
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Solution proposée à Solution validée
Ça roule, ok (testé en local y compris avec des accents).
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 247ad96d6a839bf6cb2860859bac76570b85c79e
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Wed May 29 16:26:27 2019 +0200
templatetags: cast argument of token_check to unicode (#33557)Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
templatetags: cast argument of token_check to unicode (#33557)