Feature #37180
closed
Extjs shall not overwrite $GLOBALS['error']
Added by Franz Holzinger over 12 years ago.
Updated about 6 years ago.
Description
The file 't3lib/extjs/class.t3lib_extjs_extdirectrouter.php' overwrites the global error variable.
public function route($ajaxParams, TYPO3AJAX $ajaxObj) {
$GLOBALS['error'] = t3lib_div::makeInstance('t3lib_extjs_ExtDirectDebug');
This leads to a collision with other debug extensions which use the $GLOBALS['error']. It should better check if this variable has already been set by another extension and not overwrite it.
Another solution would be to change the variable $GLOBALS['error'] into an array of objects. Then all the debug functions of all those objects should be called in config_default.php.
But changing it into an array would break existing extensions, wouldn't it?
Yes, a change into an error would break current extensions. However there are only very few debug extension which use the global ERROR object. So this won't hurt. The extension which use debugging just have lines containing 'debug' and nothing else.
I think that currently there are only 3 debug extensions. Their code can be adapted for a newer version of TYPO3.
- Target version set to 7.1 (Cleanup)
should be namespaced to TYPO3.ERROR or something alike.
- Target version changed from 7.1 (Cleanup) to 7.4 (Backend)
- Target version changed from 7.4 (Backend) to 7.5
- Target version deleted (
7.5)
- Related to Epic #68075: Remove ExtJS from backend added
- Status changed from New to Under Review
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
- Status changed from Resolved to Closed
Also available in: Atom
PDF