Refactoring

apps/client/src/app/components/access-table/access-table.component.html

@@ -17,8 +17,13 @@
     <th *matHeaderCellDef class="px-1" i18n mat-header-cell>Permission</th>
     <td *matCellDef="let element" class="px-1 text-nowrap" mat-cell>
       <div class="align-items-center d-flex">
-        <ion-icon class="mr-1" name="lock-closed-outline" />
+        @if (element.permissions.includes('READ')) {
-        {{ getPermissonDisplayName(element) }}
+          <ion-icon class="mr-1" name="lock-open-outline" />
+          <ng-container i18n>View</ng-container>
+        } @else if (element.permissions.includes('READ_RESTRICTED')) {
+          <ion-icon class="mr-1" name="lock-closed-outline" />
+          <ng-container i18n>Restricted View</ng-container>
+        }
       </div>
     </td>
   </ng-container>

apps/client/src/app/components/access-table/access-table.component.ts

@@ -44,20 +44,6 @@ export class AccessTableComponent implements OnChanges, OnInit {
     }
   }
 
-  public getPermissonDisplayName(access: Access): string {
-    if (access.type === 'PUBLIC') {
-      return $localize`Public`;
-    }
-
-    if (access.permissions.includes('READ')) {
-      return $localize`View`;
-    } else if (access.permissions.includes('READ_RESTRICTED')) {
-      return $localize`Restricted View`;
-    } else {
-      return $localize`Unknown`;
-    }
-  }
-
   public onDeleteAccess(aId: string) {
     const confirmation = confirm(
       $localize`Do you really want to revoke this granted access?`