Merge pull request #1059 from CarnegieLearningWeb/bugfix/disable-dele…

…tion-of-segments-in-use bug fix to not allow deleting segments which are in use
CarnegieLearningWeb · Oct 12, 2023 · 832d4b7 · 832d4b7
2 parents e22f85f + 28ebae2
commit 832d4b7
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/...pgrade/src/app/features/dashboard/segments/pages/view-segment/view-segment.component.html b/...pgrade/src/app/features/dashboard/segments/pages/view-segment/view-segment.component.html
@@ -29,8 +29,8 @@
             mat-flat-button
             class="ft-14-600 button"
             (click)="deleteSegment()"
-            [ngClass]="{ 'button--disabled': permissions && !permissions.segments.delete }"
-            [disabled]="permissions && !permissions.segments.delete"
+            [ngClass]="{ 'button--disabled': permissions && !permissions.segments.delete || segmentUsed }"
+            [disabled]="permissions && !permissions.segments.delete || segmentUsed"
           >
             {{ 'global.delete.text' | translate }}
           </button>

diff --git a/.../upgrade/src/app/features/dashboard/segments/pages/view-segment/view-segment.component.ts b/.../upgrade/src/app/features/dashboard/segments/pages/view-segment/view-segment.component.ts
@@ -25,6 +25,7 @@ export class ViewSegmentComponent implements OnInit, OnDestroy {
   segment: Segment;
   segmentSub: Subscription;
   members: { type: string; id: string }[] = [];
+  segmentUsed: boolean;
 
   displayedVariationColumns: string[] = ['value', 'name'];
 
@@ -60,6 +61,8 @@ export class ViewSegmentComponent implements OnInit, OnDestroy {
           this.members.push({ type: MemberTypes.SEGMENT, id: subSegment.name });
         });
       });
+
+    this.segmentUsed = this.segment?.status === SEGMENT_STATUS.USED;
   }
 
   openEditSegmentDialog() {