Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#13919 closed defect (fixed)

dojox.form.manager unregisterWidgetDescendants

Reported by: vtsuper Owned by: Eugene Lazutkin
Priority: high Milestone: 1.8
Component: DojoX Form Version: 1.6.1
Keywords: dojox.form.manager unregisterWidgetDescendants Cc:
Blocked By: Blocking:

Description

in unregsiterWidgetDescendants, I found that it never register the dijit that I expecpt, and finally I take a look on the source code and I do believe that it should be 'unregisterWidget' rather than 'unregisterNode'

unregisterWidgetDescendants: function(widget){
...
// unregister widgets by names
dojo.forEach(
    dojo.map(
        widget.getDescendants(),
        function(w){
            return w instanceof dijit.form._FormWidget && w.get("name") || null;
        }
    ),
    function(name){
        if(name){
            //this.unregisterNode(name);
            this.unregisterWidget(name);
        }
    },this
); 
...
}, 

Change History (3)

comment:1 Changed 8 years ago by bill

Owner: changed from dante to Eugene Lazutkin

comment:2 Changed 8 years ago by Eugene Lazutkin

Resolution: fixed
Status: newclosed

In [28523]:

Form Manager: corrected a method name, fixes #13919, !strict, thx vtsuper!

comment:3 Changed 8 years ago by bill

Milestone: tbd1.8
Note: See TracTickets for help on using tickets.