From 4b09f02e096fb39fdb0ca8e752ba25209a035c09 Mon Sep 17 00:00:00 2001
From: TheDiaval <thediavall@gmail.com>
Date: Fri, 9 Aug 2019 10:59:15 +0300
Subject: [PATCH] perf(permission-management): handle attr.disabled while
 rendering to DOM

---
 .../src/lib/components/permission-management.component.html | 2 +-
 .../src/lib/components/permission-management.component.ts   | 6 +-----
 2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/npm/ng-packs/packages/permission-management/src/lib/components/permission-management.component.html b/npm/ng-packs/packages/permission-management/src/lib/components/permission-management.component.html
index 75118201fe..bc93f0df19 100644
--- a/npm/ng-packs/packages/permission-management/src/lib/components/permission-management.component.html
+++ b/npm/ng-packs/packages/permission-management/src/lib/components/permission-management.component.html
@@ -65,7 +65,7 @@
                   [value]="getChecked(permission.name)"
                   [attr.id]="permission.name"
                   class="custom-control-input"
-                  [disabled]="isDisabled(permission)"
+                  [disabled]="permission.isGranted && permission.grantedProviders.length > 0"
                 />
                 <label
                   class="custom-control-label"
diff --git a/npm/ng-packs/packages/permission-management/src/lib/components/permission-management.component.ts b/npm/ng-packs/packages/permission-management/src/lib/components/permission-management.component.ts
index 069825760b..a6e4ace105 100644
--- a/npm/ng-packs/packages/permission-management/src/lib/components/permission-management.component.ts
+++ b/npm/ng-packs/packages/permission-management/src/lib/components/permission-management.component.ts
@@ -95,12 +95,8 @@ export class PermissionManagementComponent implements OnInit, OnChanges {
     return (this.permissions.find(per => per.name === name) || { isGranted: false }).isGranted;
   }
 
-  isDisabled({ isGranted, grantedProviders }: PermissionManagement.Permission): boolean {
-    return isGranted && grantedProviders.length > 0;
-  }
-
   onClickCheckbox(clickedPermission: PermissionManagement.Permission, value) {
-    if (this.isDisabled(clickedPermission)) return;
+    if (clickedPermission.isGranted && clickedPermission.grantedProviders.length > 0) return;
 
     setTimeout(() => {
       this.permissions = this.permissions.map(per => {