Opened 10 years ago

Closed 10 years ago

Last modified 8 years ago

#9974 closed defect (invalid)

_FormMixin is incorrectly setting/getting radio/checkbox values

Reported by: jake richardson Owned by:
Priority: high Milestone: tbd
Component: Dijit - Form Version: 1.3.2
Keywords: _FormMixin Cc:
Blocked By: Blocking:

Description

_FormMixin should use the "checked" attribute to set/get the values of children that are of type radio or checkbox.

Attachments (1)

_FormMixin-toggle-button-fix (908 bytes) - added by jake richardson 10 years ago.
Patch to fix _FormMixin using the "value" instead of the "checked" attribute for radio/checkbox children

Download all attachments as: .zip

Change History (6)

Changed 10 years ago by jake richardson

Patch to fix _FormMixin using the "value" instead of the "checked" attribute for radio/checkbox children

comment:1 Changed 10 years ago by bill

Component: GeneralDijit
Owner: anonymous deleted

Is something broken?

Although technically checkbox widgets use the checked attribute, they've got sort-of hackish code so that you can also turn them on/off by setting the value attribute.

But if there is a case that's broken we should fix it, but I also want to add a test case for it.

comment:2 Changed 10 years ago by bill

Resolution: invalid
Status: newclosed

Closing for now, will reopen if you supply a test case.

comment:3 Changed 10 years ago by Eugene Lazutkin

Resolution: invalid
Status: closedreopened

comment:4 Changed 10 years ago by Eugene Lazutkin

Resolution: invalid
Status: reopenedclosed

oops, reopened by mistake

comment:5 Changed 8 years ago by bill

Component: DijitDijit - Form
Note: See TracTickets for help on using tickets.