Changeset 12830

Timestamp:

03/03/08 08:28:12 (10 months ago)

Author:

sorvell

Message:

!strict fixes #5032 To correct this problem, support was added to dojox.grid.data.Objects to allow for data structure changes. The issue here was specifically with automatic assignment of model fields. Fields are now auto-assigned when data is set and grid is notified that fields may have changed.

Location:

dojox/trunk/grid

Files:

• tests/support/test_data_objects.js (added)
• tests/test_grid_object_model_change.html (added)
• _data/model.js (modified) (1 diff)

dojox/trunk/grid/_data/model.js

@@ -257 +257 @@
                 }
         },
+        allChange: function(){
+                this.notify("FieldsChange");
+                this.inherited(arguments);
+        },
         autoAssignFields: function(){
-                var d = this.data[0], i = 0;
+                var d = this.data[0], i = 0, field;
                 for(var f in d){
-                        this.fields.get(i++).key = f;
-                }
+                        field = this.fields.get(i++);
+                        if (!dojo.isString(field.key)){
+                                field.key = f;
+                        }
+                }
+        },
+        setData: function(inData){
+                this.data = (inData || []);
+                this.autoAssignFields();
+                this.allChange();
         },
         getDatum: function(inRowIndex, inColIndex){