From f804f9f6d30deb6b6b60d0bb08a0387846f80cd7 Mon Sep 17 00:00:00 2001
From: kurilova <kurilova@google.com>
Date: Tue, 1 Apr 2025 08:54:55 +0000
Subject: [PATCH] Disable copy of copy

---
 .../profile-form/profile-form.component.html  |  2 +-
 .../risk-assessment.component.ts              | 25 +++++++++++++------
 2 files changed, 19 insertions(+), 8 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 20c1f6096..abd348a45 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
@@ -70,7 +70,7 @@
       mat-flat-button
       class="copy-button"
       (click)="onCopyClick()"
-      [disabled]="selectedProfile === null">
+      [disabled]="selectedProfile === null || isCopyProfile">
       Copy
     </button>
     <button
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 a100215ab..1b98d82d6 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
@@ -15,18 +15,26 @@
  */
 import {
   ChangeDetectionStrategy,
+  ChangeDetectorRef,
   Component,
+  ElementRef,
+  inject,
   OnDestroy,
   OnInit,
-  ViewContainerRef,
-  inject,
   viewChild,
-  ChangeDetectorRef,
-  ElementRef,
+  ViewContainerRef,
 } from '@angular/core';
 import { RiskAssessmentStore } from './risk-assessment.store';
 import { SimpleDialogComponent } from '../../components/simple-dialog/simple-dialog.component';
-import { Subject, takeUntil, timer, Observable } from 'rxjs';
+import {
+  combineLatest,
+  Observable,
+  of,
+  skip,
+  Subject,
+  takeUntil,
+  timer,
+} from 'rxjs';
 import { MatDialog } from '@angular/material/dialog';
 import { LiveAnnouncer } from '@angular/cdk/a11y';
 import { Profile, ProfileAction, ProfileStatus } from '../../model/profile';
@@ -51,7 +59,6 @@ import { LayoutType } from '../../model/layout-type';
 import { NoEntitySelectedComponent } from '../../components/no-entity-selected/no-entity-selected.component';
 import { EntityAction, EntityActionResult } from '../../model/entity-action';
 import { CanComponentDeactivate } from '../../guards/can-deactivate.guard';
-import { of, combineLatest, skip } from 'rxjs';
 
 const matFormFieldDefaultOptions: MatFormFieldDefaultOptions = {
   hideRequiredMarker: true,
@@ -280,7 +287,11 @@ export class RiskAssessmentComponent
 
   actions(actions: EntityAction[]) {
     return (profile: Profile) => {
-      if (profile.status === ProfileStatus.EXPIRED) {
+      // expired profiles or unsaved copy of profile can only be removed
+      if (
+        profile.status === ProfileStatus.EXPIRED ||
+        (profile.status === ProfileStatus.DRAFT && !profile.created)
+      ) {
         return [{ action: ProfileAction.Delete, icon: 'delete' }];
       }
       return actions;