fix: use take(1) inside handleStateChange instead of unsubscribe()

mselerin · Feb 6, 2021 · 704679f · 704679f
1 parent 4b813e4
commit 704679f
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/projects/ngx-security/src/lib/directives/ngx-security.directive.ts b/projects/ngx-security/src/lib/directives/ngx-security.directive.ts
@@ -40,6 +40,7 @@ export class BaseSecurityDirective implements OnInit, OnDestroy {
 
   protected handleStateChange(): void {
     this.isAuthorized().pipe(
+      take(1),
       map(hasPerm => {
         if (this.lastValue !== hasPerm) {
           this.lastValue = hasPerm;
@@ -52,7 +53,7 @@ export class BaseSecurityDirective implements OnInit, OnDestroy {
             this.viewContainer.createEmbeddedView(this.elseTemplateRef);
         }
       })
-    ).subscribe().unsubscribe();
+    ).subscribe();
   }
 
   /* istanbul ignore next */