Changeset 13505

Timestamp:

04/30/08 09:11:37 (7 months ago)

Author:

dylan

Message:

refs #6653, fix a bunch of global reference bugs in dojox.grid, \!strict

Location:

dojox/trunk/grid

Files:

• VirtualGrid.js (modified) (2 diffs)
• _data/model.js (modified) (1 diff)
• _grid/builder.js (modified) (1 diff)
• _grid/lib.js (modified) (1 diff)
• _grid/scroller.js (modified) (1 diff)

dojox/trunk/grid/VirtualGrid.js

@@ -367 +367 @@
                 // private: sets width and position for views and update grid width if necessary
                 var
-                        w = this.autoWidth ? 0 : this.domNode.clientWidth || (this.domNode.offsetWidth - this._getPadBorder().w);
+                        w = this.autoWidth ? 0 : this.domNode.clientWidth || (this.domNode.offsetWidth - this._getPadBorder().w),
                         vw = this.views.arrange(1, w);
                 this.views.onEach("adaptWidth");
@@ -455 +455 @@
                 //              Use after calling beginUpdate to render any changes made to rows.
                 this.updating = false;
-                var i = this.invalidated;
+                var i = this.invalidated, r;
                 if(i.all){
                         this.update();

dojox/trunk/grid/_data/model.js

@@ -496 +496 @@
                 row.__dojo_data_item = item;
                 dojo.forEach(this.fields.values, function(a){
-                        value = this.store.getValue(item, a.name);
+                        var value = this.store.getValue(item, a.name);
                         row[a.name] = (value === undefined || value === null)?"":value;
                 }, this);

dojox/trunk/grid/_grid/builder.js

@@ -322 +322 @@
                         return (e.cellIndex>0) && (e.cellX < this.overResizeWidth) && this.prepareResize(e, -1);
                 }
-                return t = e.cellNode && (e.cellX < this.overResizeWidth);
+                var t = e.cellNode && (e.cellX < this.overResizeWidth);
+                return t;
         },
 

dojox/trunk/grid/_grid/lib.js

@@ -22 +22 @@
         
         getTd: function(rowOwner, rowIndex, cellIndex){
+                // FIXME: what is inTable?
                 return (dojox.grid.getTr(inTable, rowIndex)||0)[cellIndex];
         },

dojox/trunk/grid/_grid/scroller.js

@@ -61 +61 @@
                 this.rowCount = inRowCount;
                 // update page count, adjust document height
-                oldPageCount = this.pageCount;
+                var oldPageCount = this.pageCount;
                 this.pageCount = Math.ceil(this.rowCount / this.rowsPerPage);
                 if(this.pageCount < oldPageCount){