feat(contributors): delete contributors from all components

Author: sh-andriy

src/app/features/contributors/contributors.component.ts

@@ -403,8 +403,8 @@ export class ContributorsComponent implements OnInit, OnDestroy {
         header: 'project.contributors.removeDialog.title',
         width: '448px',
         data: {
-          messageKey: 'project.contributors.removeDialog.message',
-          messageParams: { name: contributor.fullName },
+          name: contributor.fullName,
+          hasChildren: !!this.resourceChildren()?.length,
         },
       })
       .onClose.pipe(

src/app/shared/components/contributors/remove-contributor-dialog/remove-contributor-dialog.component.html

@@ -1,13 +1,19 @@
 <div class="flex flex-column">
-  <p [innerHTML]="messageKey ? (messageKey | translate: messageParams) : ''"></p>
+  <p>{{ 'project.contributors.removeDialog.message' | translate: { name: name } }}</p>
 
   <div class="p-field-radiobutton flex align-items-center mt-3 mb-2">
     <p-radioButton name="removeMode" [value]="false" [(ngModel)]="selectedOption" inputId="projectOnly">
     </p-radioButton>
     <label for="projectOnly" class="ml-2">{{ 'project.contributors.removeDialog.thisProjectOnly' | translate }}</label>
   </div>
   <div class="p-field-radiobutton flex align-items-center mb-3">
-    <p-radioButton name="removeMode" [value]="true" [(ngModel)]="selectedOption" inputId="projectAll"> </p-radioButton>
+    <p-radioButton
+      name="removeMode"
+      [value]="true"
+      [(ngModel)]="selectedOption"
+      [disabled]="!hasChildren"
+      inputId="projectAll">
+    </p-radioButton>
     <label for="projectAll" class="ml-2">{{
       'project.contributors.removeDialog.thisProjectAndComponents' | translate
     }}</label>
@@ -17,14 +23,14 @@
     <p-button
       class="w-full"
       styleClass="w-full"
-      (click)="cancel()"
+      (onClick)="cancel()"
       severity="info"
       [label]="'common.buttons.cancel' | translate"
     ></p-button>
     <p-button
       class="w-full"
       styleClass="w-full"
-      (click)="confirm()"
+      (onClick)="confirm()"
       severity="danger"
       [label]="'common.buttons.remove' | translate"
     ></p-button>

src/app/shared/components/contributors/remove-contributor-dialog/remove-contributor-dialog.component.spec.ts

@@ -1,14 +1,23 @@
+import { DynamicDialogConfig, DynamicDialogRef } from 'primeng/dynamicdialog';
+
 import { ComponentFixture, TestBed } from '@angular/core/testing';
 
 import { RemoveContributorDialogComponent } from './remove-contributor-dialog.component';
 
 describe('RemoveContributorDialogComponent', () => {
   let component: RemoveContributorDialogComponent;
   let fixture: ComponentFixture<RemoveContributorDialogComponent>;
+  let dialogRef: DynamicDialogRef;
 
   beforeEach(async () => {
+    dialogRef = { close: jasmine.createSpy('close') } as any;
+
     await TestBed.configureTestingModule({
       imports: [RemoveContributorDialogComponent],
+      providers: [
+        { provide: DynamicDialogRef, useValue: dialogRef },
+        { provide: DynamicDialogConfig, useValue: { data: { name: 'John Doe', hasChildren: true } } },
+      ],
     }).compileComponents();
 
     fixture = TestBed.createComponent(RemoveContributorDialogComponent);
@@ -19,4 +28,19 @@ describe('RemoveContributorDialogComponent', () => {
   it('should create', () => {
     expect(component).toBeTruthy();
   });
+
+  it('should pass name from config', () => {
+    expect(component.name).toBe('John Doe');
+  });
+
+  it('should close dialog with selected option on confirm', () => {
+    component.selectedOption = true;
+    component.confirm();
+    expect(dialogRef.close).toHaveBeenCalledWith(true);
+  });
+
+  it('should close dialog without value on cancel', () => {
+    component.cancel();
+    expect(dialogRef.close).toHaveBeenCalledWith();
+  });
 });

src/app/shared/components/contributors/remove-contributor-dialog/remove-contributor-dialog.component.ts

@@ -19,13 +19,12 @@ export class RemoveContributorDialogComponent {
   readonly config = inject(DynamicDialogConfig);
   selectedOption = false;
 
-  get messageKey(): string | undefined {
-    return this.config?.data?.messageKey as string | undefined;
+  get name(): string | undefined {
+    return this.config?.data?.name;
   }
 
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  get messageParams(): any {
-    return this.config?.data?.messageParams;
+  get hasChildren(): boolean {
+    return this.config?.data?.hasChildren ?? false;
   }
 
   confirm(): void {