Как внедрить сервис с promise в scope и не нарваться на Infinite $digest Loop?

Question

[Devate]

Здравствуйте!

Никак не могу решить одну проблему :( Стоит задача: сделать ACL. Список разрешений загружается с сервера. Далее я проверяю - есть ли доступ к такому-то ресурсу у пользователя, просматривая полученный список. Для удобства работы внедряю созданный сервис в scope в виде вызова динамической функции, проверяющей имеет ли юзер право доступа к ресурсу. Так как без промиса я здесь работать никак не могу (всё-таки данные идут с сервера) - при его возврате и работе со scope вполне заслуженно получаю ошибку Infinite $digest Loop. Но никак не могу понять - как её обойти?

Даже если упростить пример до простейшего уровня:

module.factory('ACL', function ($q) {
  return {
    access: function () {
      return $q.when(true);
    }
  };
});

$rootScope.access = ACL.access;

и во вьюхе прописать:

<div ng-if="access()"></div>

Нарываюсь на ошибку :( Как быть?

Answer 1

[Devate]

Реализовать задачу можно примерно так:

module.service('ACL', function () {
  var self = this;

  // Разрешения.
  self.permissions;

  // Инициализация списка разрешений.
  promise.then(function (data) {
    self.permissions = ...;
  });

  // Проверка доступа.
  self.access = function (permission) {
    if (angular.isDefined(self.permissions)) {
      return permission in self.permissions;
    }
  };
});

Использование:

<div ng-if="::access('permission')"></div>

Николай: Спасибо, за помощь в поиске данного решения!

Answer 2

[Николай]

Вы понимаете, что даже если бы ангуляр не уходил в бесконечный digest вы значение true не получили бы?
Сделайте в котроллере вашей вьюшки какое-то свойство, которое будет как-то так инициализироваться:

access().then(function(data) {
    if (data.foo) {
         vm.hasAccess = true;
    }
});

Если запрос выполняется при переходе в этот route/state - можно перенести всё в resolve.

Comments

[Devate]

"Вы понимаете, что даже если бы ангуляр не уходил в бесконечный digest вы значение true не получили бы?"

Тоже обратил внимание, но просто где-то видел информацию, что angular сам получит из промиса значение и возьмёт его...

"Сделайте в котроллере вашей вьюшки какое-то свойство, которое будет как-то так инициализироваться"

Так и думал, что мне предложат данное решение. Понимаете, разрешений в ACL очень много... Например: пользователь может добавить что-то, может редактировать, может удалять... И на каждое такое разрешение мне по свойству создавать? :) А разрешения у меня не зависят от роли пользователя, где все разрешения изначально известны - права задаются динамически для каждого менеджера.

[Николай]

ну сохраните их все в один параметр объектного типа. По ключу будете получать разрешение на конкретное действие.

[Devate]

Николай: То есть, ничего интереснее здесь не придумать? Хочется красивое решение, так сказать...

[Николай]

ну можно эту логику в сервис убрать. Будет немного меньше кода в каждом контроллере.

[Devate]

Николай: описал своё решение ниже :) Как считаете - оно оптимальное?

[Николай]

Если есть возможность загрузить права при загрузке страницы - то можно использовать везде одноразовые биндинги.

[Devate]

Николай: возможность-то есть, но тут же параллельно загрузится вьюшка, где идёт проверка прав :( А разрешения ещё не загрузились-то...

[Николай]

Devate: если возвращать undefined пока права не загрузились - одноразовый биндинг тоже должен сработать, т.к.он ожидает первого не undefined значения

[Devate]

Николай: точно, огромное спасибо!!! Даже не знал о такой фиче!