Changeset 13591

Timestamp:

05/06/08 13:48:53 (7 months ago)

Author:

peller

Message:

keep row ids consistent with dojo.data. Fixes #6670 !strict

Files:

• dojox/trunk/grid/_data/model.js (modified) (5 diffs)

dojox/trunk/grid/_data/model.js

@@ -586 +586 @@
                 // FIXME: where else do we need to keep this in sync?
                 //Handle stores that implement identity and try to handle those that do not.
-                if (this._canIdentify) {
+                if(this._canIdentify){
                         this._rowIdentities[this.store.getIdentity(item)] = {rowId: offset+idx, item: item};
                 }else{
@@ -629 +629 @@
         },
         _storeDatumDelete: function(item){
-                if(dojo.indexOf(this._currentlyProcessing, item) != -1)
+                if(dojo.indexOf(this._currentlyProcessing, item) != -1){
                         return;
+                }
                 // the store has deleted some item under us, need to remove that item from
                 // the view if possible.  It may be the deleted item isn't even in the grid.
@@ -639 +640 @@
         },
         _storeDatumNew: function(item){
-                if(this._disableNew)
+                if(this._disableNew){
                         return;
+                }
                 // the store has added some item under us, need to add it to the view.
                 this._insertItem(item, this.data.length);
@@ -657 +659 @@
                 }
                 var row = this._createRow(storeItem);
+                for(var i in this._rowIdentities){ //increment all the remaining row ids up one
+                        var rowIdentity = this._rowIdentities[i];
+                        if(rowIdentity.rowId >= index){
+                                rowIdentity.rowId++;
+                        }
+                }
                 this._setRowId(storeItem, 0, index);
                 dojox.grid._data.Dynamic.prototype.insert.apply(this, [row, index]);
@@ -738 +746 @@
                 this.pages = [];
                 this.bop = this.eop = -1;
-                this.setData((keepStore?this.store:[]));
+                this.setData(keepStore ? this.store : []);
         },
         processError: function(error, request){