#16612 closed defect (fixed)
dojox/mvc/StatefulArray - make it work without passing an array to the constructor
Reported by: | ben hockey | Owned by: | Ed Chatelain |
---|---|---|---|
Priority: | undecided | Milestone: | 1.8.4 |
Component: | DojoX MVC | Version: | 1.8.3 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
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. 36 36 // - Setting a length to this array via set() - Stateful update is done for the removed/added elements as well as the new length. 37 37 38 var array = lang._toArray(a );38 var array = lang._toArray(a || []); 39 39 var ctor = StatefulArray; 40 40 ctor._meta = {bases: [Stateful]}; // For isInstanceOf() 41 41 array.constructor = ctor;
Change History (6)
comment:1 Changed 8 years ago by
Status: | new → assigned |
---|
comment:2 Changed 8 years ago by
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.
comment:3 Changed 8 years ago by
Type: | enhancement → defect |
---|
Yes putting it into the 1.8 branch is not a problem.
comment:6 Changed 8 years ago by
Milestone: | tbd → 1.8.4 |
---|
Note: See
TracTickets for help on using
tickets.
Thanks Ben, do you want this in 1.8 or just in trunk?