From 88e973a2f6f8fa76aae982eaebc56d29b8b78ae1 Mon Sep 17 00:00:00 2001
From: kurilova <kurilova@google.com>
Date: Thu, 20 Feb 2025 10:42:54 +0000
Subject: [PATCH] Disable delete button for unsaved Copy of profile

---
 .../profile-form/profile-form.component.html          |  2 +-
 .../risk-assessment/risk-assessment.component.ts      | 11 +++++++++--
 .../pages/risk-assessment/risk-assessment.store.ts    |  7 +++----
 3 files changed, 13 insertions(+), 7 deletions(-)

diff --git a/modules/ui/src/app/pages/risk-assessment/profile-form/profile-form.component.html b/modules/ui/src/app/pages/risk-assessment/profile-form/profile-form.component.html
index 650326800..005573c97 100644
--- a/modules/ui/src/app/pages/risk-assessment/profile-form/profile-form.component.html
+++ b/modules/ui/src/app/pages/risk-assessment/profile-form/profile-form.component.html
@@ -77,7 +77,7 @@
   </div>
   <button
     mat-flat-button
-    [disabled]="selectedProfile === null"
+    [disabled]="selectedProfile === null || isCopyProfile"
     (click)="onDeleteClick()"
     class="delete-button">
     Delete
diff --git a/modules/ui/src/app/pages/risk-assessment/risk-assessment.component.ts b/modules/ui/src/app/pages/risk-assessment/risk-assessment.component.ts
index ac2eb0f4f..efee6008d 100644
--- a/modules/ui/src/app/pages/risk-assessment/risk-assessment.component.ts
+++ b/modules/ui/src/app/pages/risk-assessment/risk-assessment.component.ts
@@ -24,7 +24,11 @@ import {
   ChangeDetectorRef,
   ElementRef,
 } from '@angular/core';
-import { RiskAssessmentStore } from './risk-assessment.store';
+import {
+  COPY_ACTION,
+  DElETE_ACTION,
+  RiskAssessmentStore,
+} from './risk-assessment.store';
 import { SimpleDialogComponent } from '../../components/simple-dialog/simple-dialog.component';
 import { Subject, takeUntil, timer } from 'rxjs';
 import { MatDialog } from '@angular/material/dialog';
@@ -264,7 +268,10 @@ export class RiskAssessmentComponent
   actions(actions: EntityAction[]) {
     return (profile: Profile) => {
       if (profile.status === ProfileStatus.EXPIRED) {
-        return [{ action: ProfileAction.Delete, icon: 'delete' }];
+        return [DElETE_ACTION];
+      }
+      if (!profile.created) {
+        return [COPY_ACTION];
       }
       return actions;
     };
diff --git a/modules/ui/src/app/pages/risk-assessment/risk-assessment.store.ts b/modules/ui/src/app/pages/risk-assessment/risk-assessment.store.ts
index f55ed231e..bb77ad5ad 100644
--- a/modules/ui/src/app/pages/risk-assessment/risk-assessment.store.ts
+++ b/modules/ui/src/app/pages/risk-assessment/risk-assessment.store.ts
@@ -27,6 +27,8 @@ import { selectRiskProfiles } from '../../store/selectors';
 import { setRiskProfiles } from '../../store/actions';
 import { EntityAction } from '../../model/entity-action';
 
+export const COPY_ACTION = { action: ProfileAction.Copy, icon: 'content_copy' };
+export const DElETE_ACTION = { action: ProfileAction.Delete, icon: 'delete' };
 export interface AppComponentState {
   selectedProfile: Profile | null;
   profiles: Profile[];
@@ -190,10 +192,7 @@ export class RiskAssessmentStore extends ComponentStore<AppComponentState> {
       profiles: [],
       profileFormat: [],
       selectedProfile: null,
-      actions: [
-        { action: ProfileAction.Copy, icon: 'content_copy' },
-        { action: ProfileAction.Delete, icon: 'delete' },
-      ],
+      actions: [COPY_ACTION, DElETE_ACTION],
     });
   }
 }