Opened 9 years ago

Closed 9 years ago

#11675 closed defect (fixed)

Cannot Destroy a dojox.form.Manager ("Too much recursion")

Reported by: oakbehringer Owned by: Eugene Lazutkin
Priority: high Milestone: 1.7
Component: DojoX Form Version: 1.5
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by Eugene Lazutkin)

This...

<div jsId="crappyForm" dojoType="dojox.form.Manager"></div>

...then run this code from the console (or anywhere)...

crappyForm.destroy();

...and you get:

InternalError: too much recursion { message="too much recursion",  more...}

This is a problem when you want to destroy the form programatically. Or consider a ContentPane with a dojox.form.Manager that you want to change the href/content of. When you do, bam, error.

Reproducible in [at least] IE and Firefox, with [at least] Dojo versions 1.5 and 1.4.

Attachments (1)

test.htm (681 bytes) - added by oakbehringer 9 years ago.

Download all attachments as: .zip

Change History (6)

Changed 9 years ago by oakbehringer

Attachment: test.htm added

comment:1 Changed 9 years ago by oakbehringer

For what it's worth, removing this from the dojox.form.Manager declaration "aleviates" the problem:

destroyRendering: function(){

dijit._Templated.prototype.destroyRendering.call(this);

}

However, I'm not sure if that causes other problems.

comment:2 Changed 9 years ago by Eugene Lazutkin

Description: modified (diff)
Milestone: tbd1.5.1
Owner: changed from dante to Eugene Lazutkin
Status: newassigned

comment:3 Changed 9 years ago by Eugene Lazutkin

Milestone: 1.5.11.6

comment:4 Changed 9 years ago by bill

Milestone: 1.61.7

comment:5 Changed 9 years ago by Eugene Lazutkin

Resolution: fixed
Status: assignedclosed

(In [25003]) dojox.form.manager: updated the FM with the latest Dijit guts, thx oakbehringer!, !strict, fixes #11675.

Note: See TracTickets for help on using tickets.