GridView 'findHeaderIndex' IE7 bug

[fabrizim]

I added a toolbar to the header panel of a gridview, and after a button is clicked, I was receiving Ext.fly(...) is not an object errors within IE. I tracked it down and found where the problem was.

The GridView class should be changed from (GridView.js Line 401 in 1.0 release):

    findHeaderIndex : function(n){
        var r = Ext.fly(n).findParent("td." + this.hdClass, 6);
        return r ? this.getCellIndex(r) : false;
    },

    findHeaderCell : function(n){
        var r = Ext.fly(n).findParent("td." + this.hdClass, 6);
        return r ? r : false;
    },

to:

    findHeaderIndex : function(n){
        if(!n){
             return false;
        }
        var r = Ext.fly(n).findParent("td." + this.hdClass, 6);
        return r ? this.getCellIndex(r) : false;
    },

    findHeaderCell : function(n){
        if(!n){
             return false;
        }
        var r = Ext.fly(n).findParent("td." + this.hdClass, 6);
        return r ? r : false;
    },

It looks like this check is already in place for findRowIndex, and possibly others. I think the problem is from an event returning a bad target in IE.

Regards-
Mark

[amackay11]

Bump... this still seems to be a problem in 1.1 B1, using ext-base.js

http://extjs.com/forum/showthread.php?p=38777

[evant]

Hey guys,

Bumping this one up, I'm having a similar issue and the proposed code above seems to work. Can you have a(nother?) look into this?

[navdimri]

Hi All,

I am also getting the same javascript error with IE: Ext.fly(...) is null or not an object.
I am using below files in my application:

EditableItem.js
RangeMenu.js
GridFilters.js
Filters.js
StringFilter.js
DateFilter.js
ListFilter.js
NumericFilter.js
columnLock.js
BooleanFilter.js

Also, I have tried both fixes inside my javascript file, but it didn't work.
1.

Ext.override(Ext.EventObjectImpl, {
    getTarget : function(selector, maxDepth, returnEl){
        var targetElement;
        try {
            targetElement = selector ? Ext.fly(this.target).findParent(selector, maxDepth, returnEl) : this.target;
        } catch(e) {
            targetElement = this.target;
        }
        return targetElement;
    }
});

and 2.

Ext.override(Ext.grid.GridView, {
 findHeaderIndex : function(n){
 if(!n){
 return false;
}
var r = Ext.fly(n).findParent("td." + this.hdClass, 6);
 return r ? this.getCellIndex(r) : false;
},
 findHeaderCell : function(n){
 if(!n){
 return false;
}
var r = Ext.fly(n).findParent("td." + this.hdClass, 6);
 return r ? r : false;
 }
}
);

Someone, please suggest me other solution.

Thanks,
navdimri