Opened 12 years ago

Closed 12 years ago

#6537 closed enhancement (fixed)

dijit.form._FormMixin.setValues: only set widget value if it is specified in the input obj

Reported by: liucougar Owned by: liucougar
Priority: high Milestone: 1.2
Component: Dijit Version: 1.1.0
Keywords: Cc:
Blocked By: Blocking:

Description

function dijit.form._FormMixin.setValues: even if a field is not specified in the input obj, setValues will try to set its value to undefined

I think this should be changed so that if a field is not present in the input obj, no attempt should be made to set value on its corresponding widget

attached patch fixes this and it still allows you to explicitly set a widget's value to undefined, as long as you specify it in your input obj, such as:

obj.yourUndefinedField=undefined

Attachments (1)

6537.patch (1.9 KB) - added by liucougar 12 years ago.

Download all attachments as: .zip

Change History (5)

Changed 12 years ago by liucougar

Attachment: 6537.patch added

comment:1 Changed 12 years ago by liucougar

Resolution: fixed
Status: newclosed

(In [13339]) fixes #6537: don't set undefined to widgets in partial setValues added a UT for this

comment:2 Changed 12 years ago by liucougar

Resolution: fixed
Status: closedreopened

comment:3 Changed 12 years ago by liucougar

Owner: set to liucougar
Status: reopenednew

comment:4 Changed 12 years ago by liucougar

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.