fix: if permission should use else block for undefined permissions (o…

…necx#589)
markuczy · Nov 12, 2024 · 137a2a1 · 137a2a1
1 parent 2940931
commit 137a2a1
Showing 1 changed file with 15 additions and 13 deletions.
diff --git a/libs/angular-accelerator/src/lib/directives/if-permission.directive.ts b/libs/angular-accelerator/src/lib/directives/if-permission.directive.ts
@@ -71,22 +71,24 @@ export class IfPermissionDirective implements OnInit {
   }
 
   ngOnInit() {
-    if (this.permission) {
-      if (this.negate === this.hasPermission(Array.isArray(this.permission) ? this.permission : [this.permission])) {
-        if (this.ocxIfPermissionElseTemplate) {
-          this.viewContainer.createEmbeddedView(this.ocxIfPermissionElseTemplate)
-        } else {
-          if (this.onMissingPermission === 'disable') {
-            this.renderer.setAttribute(this.el.nativeElement, 'disabled', 'disabled')
-          } else {
-            this.viewContainer.clear()
-          }
-        }
+    if (
+      (this.permission &&
+        this.negate === this.hasPermission(Array.isArray(this.permission) ? this.permission : [this.permission])) ||
+      !this.permission
+    ) {
+      if (this.ocxIfPermissionElseTemplate) {
+        this.viewContainer.createEmbeddedView(this.ocxIfPermissionElseTemplate)
       } else {
-        if (this.templateRef) {
-          this.viewContainer.createEmbeddedView(this.templateRef)
+        if (this.onMissingPermission === 'disable') {
+          this.renderer.setAttribute(this.el.nativeElement, 'disabled', 'disabled')
+        } else {
+          this.viewContainer.clear()
         }
       }
+    } else {
+      if (this.templateRef) {
+        this.viewContainer.createEmbeddedView(this.templateRef)
+      }
     }
   }