dojox.grid throws null pointer exception when calling _delayedCellFocus

[kludgeless]

There are a number of places in dojox/grid/_FocusManager.js where we check if the column head node is null before performing blurHeader, and only one where we don't which throws exceptions under some circumstances:

	_delayedCellFocus: function(){
		if(this.isNavHeader()||!this.grid._focused){
				return;
		}
		var n = this.cell && this.cell.getNode(this.rowIndex);
		if(n){ 
			try{
				if(!this.grid.edit.isEditing()){
					dojo.toggleClass(n, this.focusClass, true);
					this.blurHeader();
					dojox.grid.util.fire(n, "focus");
				}
			} 
			catch(e){}
		}
	},

We should add a check in the code as follows before calling blurHeader:

Index: _FocusManager.js
===================================================================
--- _FocusManager.js	(revision 1571)
+++ _FocusManager.js	(working copy)
@@ -104,7 +104,9 @@
 			try{
 				if(!this.grid.edit.isEditing()){
 					dojo.toggleClass(n, this.focusClass, true);
-					this.blurHeader();
+					if (this._colHeadNode){
+						this.blurHeader();
+					}
 					dojox.grid.util.fire(n, "focus");
 				}
 			} 

[Karl Tiedt]

Any reason to prolong committing this? Ran across this bug in 1.6 still (obviously not closed) I came to the same conclusion for fixing it, just looked for dupe bugs before opening a new one and found this....

[Karl Tiedt]

claiming to get this fixed... just using my patch since I actually found an additional spot where blurHeaders was being called w/o checking

[Karl Tiedt]

(In [25140]) fixes #11188 - should these _colHeadNode checks be replaced with isNavHeader() checks instead?

[Adam Peller]

Karl, are the ticket refs backwards on those two checkins?

[Karl Tiedt]

Replying to peller:

Karl, are the ticket refs backwards on those two checkins?

Crap, yes they are :/

[Adam Peller]

see [25141]

[evan]

(In [25157]) Refs #11188 backport to 1.6 branch

[evan]

(In [25158]) Refs #11188 backport to 1.6 branch