DojoX resizeHandle fails on destroy(). (this._connects is filled with incorrect entries)

[guest]

When trying to destroy() a resizeHandle it fails and complains that dojo.disconnect must be passed an array. After tracing through the code a bit I found that after you disconnect the listeners in resizeHandles's _endSizing function that the _connects array is not changed to reflect that the events have been disconnected. I added a this._connects = []; whicih is shown in bold below, but you may have a better solution.

_endSizing: function(/*Event*/ e){

summary: disconnect listenrs and cleanup sizing

dojo.forEach(this._connects,function(c){

dojo.disconnect(c);

});

this._connects = [];

if(!this.activeResize){

dojo.fadeOut({ node:this._activeResizeNode, duration:250,

onEnd: dojo.hitch(this,function(){

this._activeResizeNode.style.display="none";

})

}).play(); this._changeSizing(e);

} this._isSizing = false;

}

[Adam Peller]

If the solution is this trivial, it might be hard to argue against inclusion in 1.0.3, though we typically would put non-critical dojox fixes in 1.1.

[Adam Peller]

also, it might be good to use dijit._Widget.connect instead of dojo.connect

[dante]

i'm against dojox fixes in branch unless deemed absolutely critical.

[dante]

(In [11902]) fixes #5442 - adds tests for .destroy() a resize handle, privitized persistant connect, and other cleanups. !strict