angular-dnd icon indicating copy to clipboard operation
angular-dnd copied to clipboard

array indexes

Open khsmaxim opened this issue 9 years ago • 1 comments

Отличная директива, единственное если вдруг у тебя есть в коде раширения для масива вот токого типа:

Array.prototype.getObjByValue = function(key, value) {
  if (!this) return null;
  if (!value) return null;
  for (var ii in this) {
    if (this[ii][key] == value) {
      return this[ii];
    }
  }
  return null;
};

то, возникет конфликт в позициях где масив обрабатывается как объект for(var key in regions) { ... в какой-то момент regions[key] возвращает функцию getObjByValue и выдает ошибку. с for (var key=0; key<regions.length; key++) {} было бы более правильно

Спасибо

khsmaxim avatar Apr 26 '16 12:04 khsmaxim

@khsmaxim, Да, там нужно hasOwnProperty по хорошему поставить, вы праввы

Tuch avatar Apr 26 '16 12:04 Tuch