не могу найти источник ошибки...

[hedgehog]

Здравствуйте. Есть очень-очень старый сайт: mackevision.de

При клике на кнопку Gallery (справа под банером) в консоли ФФ появляется сообщение: "Error: useless setTimeout call (missing quotes around argument?)", в ИЕ8-9, при включенном показе ошибок, вываливается модальное окно с ошибкой "Invalid argument", вот скрин дебагера в ИЕ: http://i.imgur.com/74gEf.png

Т.е. понятно, почему возникает ошибка. Но не понятно, где происходит проблемный вызов setTimeout. Никак не могу придумать, как ее найти. Текстовый поиск по setTimeout и ссылкам на него не помог.

[nerv]

ну так первый аргумент таймеров ссылка на функцию (предпочтительно) или строка (не рекомендуется).

файл embed_flashFade.js

Embed.prototype.hideCnt=function(fake){
	if(this.newLayer){
		replaceClass(this.oldLayer,"show","hide");
		swfobject.removeSWF(this.oldFlID);
		var timeout=isMoz?250:1;
		window.setTimeout('replaceClass(embed.newLayer,"hide","show");',timeout);
	}
}

[hedgehog]

ну так первый аргумент таймеров ссылка на функцию (предпочтительно) или строка (не рекомендуется).

Это я знаю. Строка там стоит потому, что писался код очень давно.

файл embed_flashFade.js

Embed.prototype.hideCnt=function(fake){
	if(this.newLayer){
		replaceClass(this.oldLayer,"show","hide");
		swfobject.removeSWF(this.oldFlID);
		var timeout=isMoz?250:1;
		window.setTimeout('replaceClass(embed.newLayer,"hide","show");',timeout);
	}
}

Я тоже первым делом подумал на эту функцию, но дело, скорее всего, не в ней. Я закомментировал в ней вызов window.setTimeout, но ошибка осталась. Кеш очищал, в дебагере ИЕ виден комментарий на этой строке.

К самому сайту у меня нет доступа и не будет, я выкачал его к себе в статичном виде вместе со всеми (я надеюсь) скриптами и воюю уже локально.

[nerv]

вериться с трудом. А так?

window.setTimeout(
	function () { replaceClass(embed.newLayer,"hide","show"); },
	timeout
);

Если ошибка не исчезнет, сайт в файле (архиве)

Edited October 2, 2012 by nerv

[hedgehog]

вериться с трудом. А так?

window.setTimeout(
	function () { replaceClass(embed.newLayer,"hide","show"); },
	timeout
);

И так тоже. Я в прошлый раз просто комментировал вызов этого конкретного setTimeout, но ошибка все равно возникала. Источником ошибки эта строка не является, инфа 100%. Как видно по логу, в вызов setTimeout передается задержка 10мс, а в случае с функцией, на которую вы грешите - 1 или 250.

Если обратить внимание на скриншот окна дебага, то можно увидеть что по таймауту пытаются вызвать не replaceClass, а embed.hideCnt. Я пробовал перехватывать setTimeout

(function(){
  var _setTimeout = window.setTimeout;

  window.setTimeout = function (handler, timeout) {
    console.log(handler timeout);
    return _setTimeout(handler, timeout);
  }
}());

При возникновении ошибки запись в консоли выглядит как "LOG: undefined, 10". Откуда оно вызывается - я так и не смог найти, временно поставил костыль в виде:

      if (typeof(handler) != 'undefined')
        return _setTimeout(handler, timeout);

И пропатчил свою карму от понижения за такой ужас.

Edited October 5, 2012 by hedgehog

[nerv]

hedgehog, твоя правда, не доглядел. Есть там "замечательный" файл swfaddress.js, в котором не менее "чудесная" строка var _st = setTimeout; Соответственно, если забить в поиск по этому файлу _st, то можно наткнутся на такое _st(_track, 10); что уже ближе к правде А вообще, код мама дорогая...

Edited October 7, 2012 by nerv

[hedgehog]

hedgehog, твоя правда, не доглядел. Есть там "замечательный" файл swfaddress.js, в котором не менее "чудесная" строка var _st = setTimeout; Соответственно, если забить в поиск по этому файлу _st, то можно наткнутся на такое _st(_track, 10); что уже ближе к правде А вообще, код мама дорогая...

Хм.. На _st я даже не смотрел, потому что первым делом закомментировал эту самую строку, где присваивается setTimeout, но ошибка осталась. По идее _track - вполне себе правильный синтаксис. Спасибо, попробую сегодня еще раз.