Bug #13554
closedJavascript errors in grid view in IE7
0%
Description
Hi there
Version: TYPO3 4.5.2
I just spent a lot of time fixing an issue in workspaces and would like to share my findings:
When clicking on any change in the workspace module in IE, you get an error saying "fly(...) is not an object".
This results from the function 'getTarget' of Ext.EventObject.
It does not check whether Ext.grid.GridView.fly() returns an object or not.
The reason behind this is, that the HTML that is generated for the GroupingView is not valid XHTML and therefore not parsed!
IE8, Firefox etc. ignore this and are able to work properly anyway.
There is a quick fix:
ext-all-debug.js at line 5055
getTarget : function(selector, maxDepth, returnEl){
/* MODIFIED nb@cabag.ch start /
if (selector) {
var trg = Ext.fly(this.target);
if (trg) {
return trg.findParent(selector, maxDepth, returnEl);
} else {
return this.target
}
} else {
return (returnEl ? Ext.get(this.target) : this.target);
}
//return selector ? Ext.fly(this.target).findParent(selector, maxDepth, returnEl) : (returnEl ? Ext.get(this.target) : this.target);
/ MODIFIED nb@cabag.ch end */
},
I also changed two JavaScripts in workspaces, however I am not sure if they actually contribute something to the solution.
They were a first attempt at fixing the issue while rendering, but didn't solve the problem.
I will find out tomorrow if they actually contribute to the solution or not.
To sum it up:
The rendered HTML must become valid XHTML.
In the mean time a modified Ext-js hides the underlying problem.
More tomorrow
Regards Nils
Updated by Stefan Neufeind almost 13 years ago
There is a fix proposed in #26618 which is almost identical to what you proposed here.
If you can, please try if that patch fixes it for you.
Updated by Tolleiv Nietsch over 12 years ago
- Status changed from New to Closed
- TYPO3 Version set to 4.5
Closed due to missing feedback and due to the fact that this seems to be a duplicate anyways.
Updated by Michael Stucki over 10 years ago
- Category changed from Bugs to Workspaces
Updated by Michael Stucki over 10 years ago
- Project changed from 624 to TYPO3 Core
- Category changed from Workspaces to Workspaces