SingleCheckboxWidget checked state not saved without value=selected

[thet]

What I did:

I had a field which uses a z3c.form SingleCheckboxWidget. Field definition:

    some_bool = zope.schema.Bool(
        title="some title",
        required=False,
        default=False,
    )

And this was the manually set HTML code for the checkbox:

    <input name="some_bool" type="checkbox" />

What I expect to happen:

Saving the form with a checked checkbox should have resulted in a set boolean for the object.

What actually happened:

The boolean was not set because:

• The default checked value for a input field without a value attribute is on. See: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/checkbox#value

• The extract method of the SequenceWidget did return the default value, because the lookup for self.terms.getTermsByToken(term) failed with term being on:
  

  z3c.form/src/z3c/form/widget.py

  Line 255 in 440eb18

  self.terms.getTermByToken(token)

• The terms for the SingleCheckboxWidget are set here:

  z3c.form/src/z3c/form/browser/checkbox.py

  Line 90 in 440eb18

  vocabulary.SimpleTerm('selected', 'selected',

• Using the following input with a value set to "selected" did solve the issue:

    <input name="some_bool" type="checkbox" value="selected" />

I'd have expected that a <input name="some_bool" type="checkbox"/> as a prototypical representation of booleans in html forms to just work out of the box.

I suggest to also add the term on to

z3c.form/src/z3c/form/browser/checkbox.py

Line 90 in 440eb18

vocabulary.SimpleTerm('selected', 'selected',

Not sure, but I guess that would solve the issue...

What version of Python and Zope/Addons I am using:

• Ubuntu 20.04.1
• Python 3.8.5
• z3c.form-3.7.1-py3.8.egg
• Zope-4.5.5-py3.8.egg
• Products.CMFPlone-5.2.4-py3.8.egg

[d-maurer]

Johannes Raggam wrote at 2021-3-22 17:13 +0000:

... I suggest to also add the term ``on`` to https://github.com/zopefoundation/z3c.form/blob/440eb18c541b88f64e88be9b66fbc27748f75af2/src/z3c/form/browser/checkbox.py#L90 Not sure, but I guess that would solve the issue...

This likely will not work: As the docstring for `SequenceWidget` tells us its purpose is to support the use case "possible input values specified via a sequence of values". You suggest to use a two element sequence as values for a `SingleCheckboxWidget` but such a widget supports only a single value. You might avoid HTML problems by the wrong `SequenceWidget` use through your manually created HTML but you must keep in mind that there will be a value extractor which translates the widget value into the field value and it likely does not know how to handle a new value "on". The `z3c.form` framework is quite complex being composed of a set of **cooperating** components, among them "field", "widget", "input template", "value extractor". Each of them makes assumptions about the cooperating partners. Therefore, it is best for customization purposes to start with the default component and adapt it as necessary instead of developing a component from ground zero. When you look at `checkbox_input.pt` (the input template component for a checkbox), you will see that the widget instance is supposed to provide the value to be used in the HTML (this way, the same widget can be used for checkboxes as well as for radio buttons). I suggest that your customized template does the same.

[icemac]

@thet Did Dieter's explanation solve your problem, so the issue can be closed?