dojox/mvc/StatefulArray - make it work without passing an array to the constructor

[ben hockey]

currently, this fails and it's annoying.

var arr = new StatefulArray();

a minor change will fix it

dojox/mvc/StatefulArray.js

@@ -35 +35 @@
                 //              - Setting an element to this array via set() - Stateful update is done for the new element as well as the new length.
                 //              - Setting a length to this array via set() - Stateful update is done for the removed/added elements as well as the new length.
 
-                var array = lang._toArray(a);
+                var array = lang._toArray(a || []);
                 var ctor = StatefulArray;
                 ctor._meta = {bases: [Stateful]}; // For isInstanceOf()
                 array.constructor = ctor;

[Ed Chatelain]

Thanks Ben, do you want this in 1.8 or just in trunk?

[ben hockey]

the 1.8 branch would be helpful for my immediate project if it's possible. i think it's probably something reasonable to include in that branch.

[Ed Chatelain]

Yes putting it into the 1.8 branch is not a problem.

[Ed Chatelain]

In [30442]:

refs #16612 dojox/mvc/StatefulArray should work without passing an array to the constructor. Thanks neonstalwart. !strict

[Ed Chatelain]

In [30443]:

fixes #16612 dojox/mvc/StatefulArray should work without having to pass an array to the constructor. Thanks neonstalwart. !strict