From 2f00e568e4bfd45cf1f0f735d01ad7a9ae8d21d6 Mon Sep 17 00:00:00 2001
From: Timo Bejan <timo@devplant.ro>
Date: Wed, 4 Nov 2020 23:45:36 +0200
Subject: [PATCH] added conditionals for actions

---
 .../file-preview-screen.component.html                |  5 +++--
 .../file-preview-screen.component.ts                  |  4 ++++
 .../screens/file/pdf-viewer/pdf-viewer.component.ts   |  6 +++---
 apps/red-ui/src/app/state/app-state.service.ts        | 11 +++++++++++
 4 files changed, 21 insertions(+), 5 deletions(-)

diff --git a/apps/red-ui/src/app/screens/file/file-preview-screen/file-preview-screen.component.html b/apps/red-ui/src/app/screens/file/file-preview-screen/file-preview-screen.component.html
index f5fb56949..142c28620 100644
--- a/apps/red-ui/src/app/screens/file/file-preview-screen/file-preview-screen.component.html
+++ b/apps/red-ui/src/app/screens/file/file-preview-screen/file-preview-screen.component.html
@@ -99,7 +99,7 @@
     <div class="flex red-content-inner">
         <div class="left-container">
             <redaction-pdf-viewer
-                [isReviewer]="appStateService.isActiveFileDocumentReviewer"
+                [canPerformActions]="canPerformAnnotationActions"
                 (annotationSelected)="handleAnnotationSelected($event)"
                 (keyUp)="handleKeyEvent($event)"
                 (manualAnnotationRequested)="openManualRedactionDialog($event)"
@@ -189,6 +189,7 @@
 
                             <div
                                 [class.visible]="isAnnotationMenuOpen(annotation)"
+                                *ngIf="canPerformAnnotationActions"
                                 class="annotation-actions"
                             >
                                 <button
@@ -243,7 +244,7 @@
                                     "
                                     mat-icon-button
                                 >
-                                    <mat-icon svgIcon="red:trash"></mat-icon>
+                                    <mat-icon svgIcon="red:close"></mat-icon>
                                 </button>
                                 <button
                                     (click)="suggestRemoveAnnotation($event, annotation)"
diff --git a/apps/red-ui/src/app/screens/file/file-preview-screen/file-preview-screen.component.ts b/apps/red-ui/src/app/screens/file/file-preview-screen/file-preview-screen.component.ts
index 4576838e1..502cb5c09 100644
--- a/apps/red-ui/src/app/screens/file/file-preview-screen/file-preview-screen.component.ts
+++ b/apps/red-ui/src/app/screens/file/file-preview-screen/file-preview-screen.component.ts
@@ -95,6 +95,10 @@ export class FilePreviewScreenComponent implements OnInit {
         return this.instance?.docViewer?.getCurrentPage();
     }
 
+    get canPerformAnnotationActions() {
+        return this.appStateService.canPerformAnnotationActionsOnCurrentFile();
+    }
+
     private projectId: string;
     private _activeViewer: 'ANNOTATED' | 'REDACTED' = 'ANNOTATED';
     private instance: WebViewerInstance;
diff --git a/apps/red-ui/src/app/screens/file/pdf-viewer/pdf-viewer.component.ts b/apps/red-ui/src/app/screens/file/pdf-viewer/pdf-viewer.component.ts
index d757bb155..0f65cf2a5 100644
--- a/apps/red-ui/src/app/screens/file/pdf-viewer/pdf-viewer.component.ts
+++ b/apps/red-ui/src/app/screens/file/pdf-viewer/pdf-viewer.component.ts
@@ -42,7 +42,7 @@ export class PdfViewerComponent implements OnInit, AfterViewInit, OnChanges {
 
     @Input() fileData: Blob;
     @Input() fileStatus: FileStatusWrapper;
-    @Input() isReviewer = false;
+    @Input() canPerformActions = false;
 
     @Output() fileReady = new EventEmitter();
     @Output() annotationSelected = new EventEmitter<string>();
@@ -72,7 +72,7 @@ export class PdfViewerComponent implements OnInit, AfterViewInit, OnChanges {
         if (changes.fileData && !changes.fileData.firstChange) {
             this._changeDocument();
         }
-        if (changes.isReviewer) {
+        if (changes.canPerformActions) {
             this._handleCustomActions();
         }
     }
@@ -208,7 +208,7 @@ export class PdfViewerComponent implements OnInit, AfterViewInit, OnChanges {
 
     private _handleCustomActions() {
         if (this.instance) {
-            if (this.isReviewer) {
+            if (this.canPerformActions) {
                 this.instance.enableElements(['add-redaction', 'add-dictionary']);
             } else {
                 this.instance.disableElements(['add-redaction', 'add-dictionary']);
diff --git a/apps/red-ui/src/app/state/app-state.service.ts b/apps/red-ui/src/app/state/app-state.service.ts
index 410dda863..41ab2be1f 100644
--- a/apps/red-ui/src/app/state/app-state.service.ts
+++ b/apps/red-ui/src/app/state/app-state.service.ts
@@ -507,4 +507,15 @@ export class AppStateService {
             fileStatus.dictionaryVersion !== this.dictionaryVersion
         );
     }
+
+    canPerformAnnotationActionsOnCurrentFile() {
+        const status = this.activeFile.status;
+        if (status === 'UNDER_REVIEW') {
+            return this.isActiveProjectOwnerAndManager || this.isActiveFileDocumentReviewer;
+        }
+        if (status === 'UNDER_APPROVAL') {
+            return this.isActiveProjectOwnerAndManager;
+        }
+        return false;
+    }
 }