Projet

Général

Profil

Bug #10981

valeur "raw" d'une checkbox pas cochée

Ajouté par Frédéric Péters il y a presque 8 ans. Mis à jour il y a plus de 7 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
Début:
18 mai 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

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

Révision 50398e12 (diff)
Ajouté par Frédéric Péters il y a presque 8 ans

formdata: set raw value to False in variables of unchecked checkbox (#10981)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 8 ans

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.

#2

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...

#3

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).

#4

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.

#5

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.

#6

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".

#7

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)
#8

Mis à jour par Thomas Noël il y a presque 8 ans

  • Version cible mis à v1.43
#9

Mis à jour par Frédéric Péters il y a plus de 7 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF