Почему дублирует события при state.reload?

Question

[byms]

export class AppController {
    constructor(Guard, $state, $rootScope, $injector) {
        'ngInject';

        this.guard = Guard;
        this.state = $state;
        this.rootScope = $rootScope;
        this.init();
    }

    init() {
        this.pageTitle = this.state.current.ncyBreadcrumb.label;
        this.help(this.state.current.url);

        this.rootScope.$on('$locationChangeStart', (event, toState, toParams, fromState, fromParams) => {
            this.guard.loadUser();
        });

        this.rootScope.$on('$stateChangeSuccess', (event, toState, toParams, fromState, fromParams) => {
            console.log(1);
        });
    }
}

есть такие события, при вызове this.state.go(state, {}, { reload: true }) события начинают дублироваться т.е this.guard.loadUser() и console.log(1) отработают 2 раза.
"angular": "^1.5.8",
"angular-ui-router": "0.4.2",

Answer 1

[Николай]

Я подозреваю, что при каждой инициализации вешается новый хэндлер, а старый не удаляется. За событиями нужно следить.Почему это вообще в контроллере?

Comments

[byms]

а где это должно быть?

[Владимир]

Вообще вешать в this контроллера все что не используется в темплейте не следует, засирается this и падает производительность.
Можно использовать WeakMap чтобы имитировать приватные свойства.

[Николай]

byms: ну вариантов много:
в сервисе, в state.resolve, в ран блоке. Всё зависит от того, какова ваша цель.

[Николай]

Владимир: а есть бенчмарк?

[byms]

Николай: цель, чтобы this.guard.loadUser(); отрабатывал во всех компонентах

[Николай]

byms: тогда в run блоке модуля можно вешать эти хэндлеры.

[byms]

<button ng-click="$ctrl.login(shop.user.id)" uib-tooltip="Login">
      <span class="fa fa-sign-in"></span>
</button>

[Stepanya]

сделайте так

this.$scope.$on('$stateChangeSuccess', (event, toState, toParams, fromState, fromParams) => {
            console.log(1);
        });

[byms]

scope помог

Answer 2

[ozknemoy]

лучше переместить this.rootScope.$on в config