[TYPO3-dev] phpMyAdmin 4.1.1+ breaks relogin

Steffen Kamper info at sk-typo3.de
Sat Jan 17 10:12:42 CET 2009


Hi,

Dmitry Dulepov schrieb:
> Hi!
> 
> Christoph Koehler wrote:
>> Found some stuff with the help of JSLint and line numbers, and it seems
>> to work in IE now.
>> Attached is a patch, please review. I still get some JS errors about
>> "Object required" on line 3777, but that may be unrelated.
> 
> When variables are defined in javascript, they must use "var" keyword. Otherwise these variables become global (== properties of "window" object) and it is a memory leak in MSIE.
> 

there seems to be one exception. I checked BE-scripts with JSlint 
yesterday and there where many things not correct, and with this 
statemaent var is not valid

var anything = AJAX.Request(...);
must be
anything = AJAX.Request(...);

some other errors i found are:

var a = array() //php-syntax
var a = []; // js syntax

var T3AJAX = new Object(); // php-syntax
var T3AJAX = {}; // js-syntax

if (a !=b) return;
must be
if (a !=b) {
	return
};

new used as statement

many linebreak errors

I will provide a patch soon for these errors.

vg Steffen




More information about the TYPO3-dev mailing list