Как поправить ошибку Uncaught TypeError: this.switchers[i].hideConfirm is not a function?

Question

[Сергей Арсентьев]

Есть сайт https://klen-house.ru/, в нем модуль определения города по ip и странный баг: на главной все ок, работает, города выбираются и т.п. - на внутренних нет, в консоли ошибка Uncaught TypeError: this.switchers[i].hideConfirm is not a function при клике на выбор города (вверху в шапке)

js файл модуля грузится через header, то есть по идее по загрузке все там одинаково.
но почему-то на внутренних выдает ошибку.
не могу сообразить почему.
может есть какие-то мнения куда можно копать, почему именно на внутренних страницах ошибка?

Answer 1

[Вадим]

У Вас на сайте расширен прототип Array методом remove, который сделан перечисляемым. (Расширение прототипов давно признано плохой практикой)

Когда в цикле делаете

for (i in this.switchers) {
   this.switchers[i].hideConfirm();
}

Сначала перебирает нормально, а в конце i = "remove".
Cоответственно this.switchers['remove'].hideConfirm // undefined.

Хотфикс - читать перед тем, как пользоваться.
for...in цикл не для массивов.
Используйте forEach вместо него.
Либо добавьте проверку на .hasOwnProperty

P.s. а если бы Вы умели в отладку кода, то решили бы эту проблему минут за 15.
https://learn.javascript.ru/debugging-chrome

Comments

[Сергей Арсентьев]

спасибо, а можно ли как-то решить проблему не изменяя код этого модуля?
просто автор вряд ли будет его менять, а обновление затрёт изменения.
на демо сайте - все работает (тот же код) citymanagerpro-demo.progroman.ru/index.php?route=p...
на главной странице https://klen-house.ru/ тоже
там почему-то i=remove не происходит...

получается что проблема может не в самом коде, а в том, что что-то влияет на него на внутренних страницах? можно ли как-то найти то, что на него влияет? есть ли какой-то алгоритм?

[Сергей Арсентьев]

Нашел один момент в коде - на внутренних страницах есть вот такая штука:

var findtext = [];
  Array.prototype.remove = function() {
    var what, a = arguments, L = a.length, ax;
    while (L && this.length) {
        what = a[--L];
        while ((ax = this.indexOf(what)) !== -1) {
            this.splice(ax, 1);
        }
    }
    return this;
  };

если ее убрать - то работает и на внутренних.
но этот код, насколько я понял, от модуля меню, который отвечает за расхлопывание подменю (если его убрать, то уже не расхлопывается). Как он работает не понимаю, да и менять его нельзя, ведь модуль тоже обновляется. А можно ли как-то убрать его влияние (параметра remove) на остальные скрипты? чтобы он никак не влиял на скрипт модуля определения города? Может есть какой-то механизм?

[Вадим]

Сергей Арсентьев, Это именно тот код, о котором я говорил - расширяющий прототип. Что является плохой практикой.
Вы хотите оставить сомнительный код в проекте? Вопрос риторический. Дело ваше.

Можно заменить этот код на следующий:

Object.defineProperty(Array.prototype, 'remove', {
  value: function() {
    var what, a = arguments, L = a.length, ax;
    while (L && this.length) {
      what = a[--L];
      while ((ax = this.indexOf(what)) !== -1) {
        this.splice(ax, 1);
       }
    }
    return this;
  },
  writable: false,
  enumerable: false,
  configurable: false
})

Этот код так же расширяет прототип массива тем же методом, но устанавливает ему дескрипторы, запрещающие перечисление и изменение. Таким образом, код из первого сообщения отработает так, что i не примет значение remove
Код, который я выделил в первом сообщении:

for (i in this.switchers) {
   this.switchers[i].hideConfirm();
}

Тоже не корректный. Мало вероятно что написан он в каком то модуле, но если это так, то Вы хотите 2 модуля с кривым кодом оставить в проекте? Вопрос опять же риторический.
В нормальных модулях - нормальный код.

[Сергей Арсентьев]

Вадим, спасибо, вы правы, поменял код в модуле меню на ваш, всё стало хорошо - и меню работает, и город выбирается ) а обновление модификатором в опенкарт можно сделать будет, заодно автору модуля написал, может поправит у себя на будущее.