Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#13942 closed defect (fixed)

dojox.mobile.CheckBox "value" attribute not set

Reported by: Philippe May Owned by: ykami
Priority: high Milestone: 1.7
Component: DojoX Mobile Version: 1.7.0b1
Keywords: 1.7-mobile Cc: Douglas Hays, Atsushi Ono
Blocked By: Blocking:

Description

For example: in dojox/mobile/tests/test_FormControls.html: dijit.byId('cbox').get('value') returns always "on".

Attachments (1)

CheckBox.js.diff​ (347 bytes) - added by Philippe May 8 years ago.
Proposed patch

Download all attachments as: .zip

Change History (4)

Changed 8 years ago by Philippe May

Attachment: CheckBox.js.diff​ added

Proposed patch

comment:1 Changed 8 years ago by ykami

Cc: Douglas Hays Atsushi Ono added
Keywords: 1.7-mobile added
Status: new → assigned

comment:2 Changed 8 years ago by ykami

Resolution: → fixed
Status: assigned → closed

In [26605]:

Fixes #13942 !strict Added _getValueAttr, which was taken from dijit/form/CheckBox.js. Thanks onoat for your investigation.

comment:3 Changed 8 years ago by Atsushi Ono

Thanks for reporting this and proposing a patch, aprilmay. dijit.form.CheckBox already has _getValueAttr function, which returns the checkbox value ("on" if the value is not specified) if checked, and false if not checked, so the fix above added the same function to dojox.mobile.CheckBox.

Note: See TracTickets for help on using tickets.