Bug #10981
valeur "raw" d'une checkbox pas cochée
0%
Description
Pour le moment, cochée, form_var_checkbox 'True', form_var_checkbox isTrue; et pas cochée, form_var_checbox 'False', form_var_checkbox is None.
J'aimerais changer cette dernière valeur en False.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 8 ans
- Fichier 0001-formdata-set-raw-value-to-False-in-variables-of-unch.patch 0001-formdata-set-raw-value-to-False-in-variables-of-unch.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
J'ai hésité entre "is not None" ou "raw_value or raw_value is False" (pour être sûr de n'attraper que cette situation) mais au final, je trouve plus clean la première.
Mis à jour par Thomas Noël il y a presque 8 ans
là, je n'ai pas compris comment le None est devenu un False...
Mis à jour par Frédéric Péters il y a presque 8 ans
C'est mon explication initiale qui n'est pas correcte, avant le patch il n'y a pas de form_var_checkbox_raw, c'est juste qu'un assert variables.get('form_var_xxx_raw') is None
validait (mais c'est parce que la clé n'existait pas).
Mis à jour par Thomas Noël il y a presque 8 ans
Ack.
Pour être encore plus propre, on pourrait peut-être initialiser raw_value avec une valeur sentinelle (et ainsi permettre que raw soit None) ? C'est toujours très désagréable d'avoir une variable absente.
Mis à jour par Frédéric Péters il y a presque 8 ans
J'ai l'impression que poser ça de manière nette risque d'amener des problèmes et du coup je suis frileux et je ne m'y aventurerais pas.
Mis à jour par Thomas Noël il y a presque 8 ans
Frédéric Péters a écrit :
je suis frileux et je ne m'y aventurerais pas.
J'ai du mal à voir de soucis, mais je vais pas me plaindre d'un peu de frilosité : Ack pour ce patch "is not None".
Mis à jour par Frédéric Péters il y a presque 8 ans
- Statut changé de En cours à Résolu (à déployer)
commit 50398e12fb540d99f347ab1c706126f8dfe04a08 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Wed May 18 16:07:06 2016 +0200 formdata: set raw value to False in variables of unchecked checkbox (#10981)
formdata: set raw value to False in variables of unchecked checkbox (#10981)