Tuch / angular-dnd

Drag and Drop module for Super heroic 1.x AngularJS. Not jQueryUI. Touch supporting.
67 stars 25 forks source link

array indexes #40

Open khsmaxim opened 8 years ago

khsmaxim commented 8 years ago

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

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++) {} было бы более правильно

Спасибо

Tuch commented 8 years ago

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