Project

General

Profile

Bug #10981

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

Added by Frédéric Péters (de retour le 10/10) over 6 years ago. Updated over 6 years ago.

Status:
Fermé
Priority:
Normal
Target version:
Start date:
18 May 2016
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
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.


Files

Associated revisions

Revision 50398e12 (diff)
Added by Frédéric Péters (de retour le 10/10) over 6 years ago

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

History

#1

Updated by Frédéric Péters (de retour le 10/10) over 6 years ago

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

Updated by Thomas Noël over 6 years ago

là, je n'ai pas compris comment le None est devenu un False...

#3

Updated by Frédéric Péters (de retour le 10/10) over 6 years ago

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

Updated by Thomas Noël over 6 years ago

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

Updated by Frédéric Péters (de retour le 10/10) over 6 years ago

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

Updated by Thomas Noël over 6 years ago

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

Updated by Frédéric Péters (de retour le 10/10) over 6 years ago

  • Status changed from En cours to 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

Updated by Thomas Noël over 6 years ago

  • Target version set to v1.43
#9

Updated by Frédéric Péters (de retour le 10/10) over 6 years ago

  • Status changed from Résolu (à déployer) to Fermé

Also available in: Atom PDF