Invalid call to cell.restore() in dojox/grid/_EditManager.js

[nobbynobbs77]

The patch illustrates the invalid call:

Index: dojox/grid/_EditManager.js
===================================================================
--- dojox/grid/_EditManager.js	(Revision 24064)
+++ dojox/grid/_EditManager.js	(Arbeitskopie)
@@ -233,7 +233,7 @@
 		//		Grid view
 		var c = this.info.cell;
 		if(this.isEditRow(inRowIndex) && c.view == inView && c.editable){
-			c.restore(c, this.info.rowIndex);
+			c.restore(this.info.rowIndex);
 		}
 	}
 });
Index: dojox/grid/cells/_base.js
===================================================================
--- dojox/grid/cells/_base.js	(Revision 24064)
+++ dojox/grid/cells/_base.js	(Arbeitskopie)
@@ -238,7 +238,9 @@
 			//	restore editor state
 			// inRowIndex: int
 			// grid row index
-			this.setValue(inRowIndex, this.value);
+		  if (this.value) {
+		    this.setValue(inRowIndex, this.value);
+		  }
 			//console.log("restore", this.value, inCell.index, inRowIndex);
 		},
 		//protected

[bill]

DojoX Grid and EnhancedGrid are deprecated in favor of ​dgrid and ​gridx.

You should upgrade your code to use one of those two grids.

We will consider patches to the old DojoX Grid code though.