From 67913b878372640b9dc02f2b27a3b46b6ef9f5a2 Mon Sep 17 00:00:00 2001
From: Sofia Papacharalampous <spapacharalampous@cite.gr>
Date: Wed, 26 Jun 2024 13:15:53 +0300
Subject: [PATCH] added description role filter

---
 .../src/app/core/common/enum/dmp-user-role.ts |  2 +-
 .../src/app/core/query/description.lookup.ts  |  4 +--
 .../src/app/core/query/dmp-user.lookup.ts     | 26 ++++++++++++++++
 dmp-frontend/src/app/core/query/dmp.lookup.ts | 12 ++++----
 .../description-listing.component.html        |  2 +-
 .../listing/description-listing.component.ts  | 30 +++++++++++++++----
 .../description-filter.component.html         | 10 +++----
 .../filtering/description-filter.component.ts |  2 ++
 8 files changed, 68 insertions(+), 20 deletions(-)
 create mode 100644 dmp-frontend/src/app/core/query/dmp-user.lookup.ts

diff --git a/dmp-frontend/src/app/core/common/enum/dmp-user-role.ts b/dmp-frontend/src/app/core/common/enum/dmp-user-role.ts
index 7d8c28d58..b9d0d6eef 100644
--- a/dmp-frontend/src/app/core/common/enum/dmp-user-role.ts
+++ b/dmp-frontend/src/app/core/common/enum/dmp-user-role.ts
@@ -2,5 +2,5 @@ export enum DmpUserRole {
 	Owner = 0,
 	Viewer = 1,
 	DescriptionContributor = 2,
-	Reviewer= 3
+	Reviewer = 3
 }
\ No newline at end of file
diff --git a/dmp-frontend/src/app/core/query/description.lookup.ts b/dmp-frontend/src/app/core/query/description.lookup.ts
index 9220144de..c494bdbc4 100644
--- a/dmp-frontend/src/app/core/query/description.lookup.ts
+++ b/dmp-frontend/src/app/core/query/description.lookup.ts
@@ -25,8 +25,7 @@ export class DescriptionLookup extends Lookup implements DescriptionFilter {
 	descriptionTagSubQuery: DescriptionTagLookup;
 	descriptionReferenceSubQuery: DescriptionReferenceLookup;
 	
-	roleInDmp: DmpUserRole[]; //TODO
-	showAllVersions: boolean; //TODO
+	showAllVersions: boolean; //TODO ??
 	
 	constructor() {
 		super();
@@ -44,7 +43,6 @@ export interface DescriptionFilter {
 	isActive: IsActive[];
 	statuses: DescriptionStatus[];
 	showAllVersions: boolean;
-	roleInDmp: DmpUserRole[];
 
 	dmpSubQuery: DmpLookup;
 	descriptionTemplateSubQuery: DescriptionTemplateLookup;
diff --git a/dmp-frontend/src/app/core/query/dmp-user.lookup.ts b/dmp-frontend/src/app/core/query/dmp-user.lookup.ts
new file mode 100644
index 000000000..9cd94dd3b
--- /dev/null
+++ b/dmp-frontend/src/app/core/query/dmp-user.lookup.ts
@@ -0,0 +1,26 @@
+import { Guid } from "@common/types/guid";
+import { IsActive } from "../common/enum/is-active.enum";
+import { Lookup } from "@common/model/lookup";
+
+export class DmpUserLookup extends Lookup implements DmpUserFilter {
+  ids: Guid[];
+  isActive: IsActive[];
+  dmpIds: Guid[];
+  userIds: Guid[];
+  sectionIds: Guid[];
+  userRoles: Guid[];
+
+  constructor() {
+    super();
+  }
+}
+
+export interface DmpUserFilter {
+	ids: Guid[];
+	isActive: IsActive[];
+  
+	dmpIds: Guid[];
+	userIds: Guid[];
+	sectionIds: Guid[];
+	userRoles: Guid[];
+}
\ No newline at end of file
diff --git a/dmp-frontend/src/app/core/query/dmp.lookup.ts b/dmp-frontend/src/app/core/query/dmp.lookup.ts
index b2e95387f..86451b3d0 100644
--- a/dmp-frontend/src/app/core/query/dmp.lookup.ts
+++ b/dmp-frontend/src/app/core/query/dmp.lookup.ts
@@ -5,8 +5,8 @@ import { DmpStatus } from '../common/enum/dmp-status';
 import { DmpVersionStatus } from '../common/enum/dmp-version-status';
 import { IsActive } from '../common/enum/is-active.enum';
 import { DmpDescriptionTemplateLookup } from './dmp-description-template.lookup';
-import { DmpUserRole } from '../common/enum/dmp-user-role';
 import { UntypedFormBuilder, UntypedFormGroup } from '@angular/forms';
+import { DmpUserLookup } from './dmp-user.lookup';
 
 export class DmpLookup extends Lookup implements DmpFilter {
 	ids: Guid[];
@@ -20,9 +20,10 @@ export class DmpLookup extends Lookup implements DmpFilter {
 	dmpDescriptionTemplateSubQuery: DmpDescriptionTemplateLookup;
 	groupIds: Guid[];
 
+	dmpUserSubQuery: DmpUserLookup; 
+
 	dmpBlueprintIds: Guid[]; //TODO
 	descriptionTemplateIds: Guid[]; //TODO
-	roleInDmp: DmpUserRole[]; //TODO
 
 	constructor() {
 		super();
@@ -30,7 +31,7 @@ export class DmpLookup extends Lookup implements DmpFilter {
 
 	from(formGroup: UntypedFormGroup): void {
 		this.statuses = formGroup.get('status')?.value ? [formGroup.get('status')?.value] : null;
-		this.roleInDmp = formGroup.get('role')?.value ? [formGroup.get('role')?.value] : null;
+		// this.roleInDmp = formGroup.get('role')?.value ? [formGroup.get('role')?.value] : null;
 		this.dmpBlueprintIds = formGroup.get('dmpBlueprintIds')?.value ? formGroup.get('dmpBlueprintIds')?.value : null;
 		this.descriptionTemplateIds = formGroup.get('descriptionTemplateIds')?.value ? formGroup.get('descriptionTemplateIds')?.value : null;
 	}
@@ -38,7 +39,7 @@ export class DmpLookup extends Lookup implements DmpFilter {
 	buildForm(): UntypedFormGroup {
 		return (new UntypedFormBuilder()).group({
 			status: [this.statuses ? this.statuses[0] : null],
-			role: [this.roleInDmp ? this.roleInDmp[0] : null],
+			// role: [this.roleInDmp ? this.roleInDmp[0] : null],
 			dmpBlueprintIds: [this.dmpBlueprintIds],
 			descriptionTemplateIds: [this.descriptionTemplateIds],
 		});
@@ -56,7 +57,8 @@ export interface DmpFilter {
 	versions: Number[];
 	groupIds: Guid[];
 
+	dmpUserSubQuery: DmpUserLookup; 
+
 	dmpBlueprintIds: Guid[]; //TODO
 	descriptionTemplateIds: Guid[]; //TODO
-	roleInDmp: DmpUserRole[]; //TODO
 }
diff --git a/dmp-frontend/src/app/ui/description/listing/description-listing.component.html b/dmp-frontend/src/app/ui/description/listing/description-listing.component.html
index cc1ae9b06..a9c3dee06 100644
--- a/dmp-frontend/src/app/ui/description/listing/description-listing.component.html
+++ b/dmp-frontend/src/app/ui/description/listing/description-listing.component.html
@@ -29,7 +29,7 @@
 					</button>
 				</div>
 			</div>
-			<div *ngIf="listingItems && listingItems.length > 0 || lookup.like || lookup.descriptionTemplateSubQuery || lookup.dmpSubQuery || lookup.roleInDmp || lookup.descriptionTagSubQuery || lookup.descriptionReferenceSubQuery" 
+			<div *ngIf="listingItems && listingItems.length > 0 || lookup.like || lookup.descriptionTemplateSubQuery || lookup.dmpSubQuery || lookup.dmpUserRoles || lookup.descriptionTagSubQuery || lookup.descriptionReferenceSubQuery" 
 				class="filter-btn" [style.right]="dialog.getDialogById('filters') ? '446px' : '0px'" [style.width]="listingItems.length > 2 ? '57px' : '37px'" (click)="openFiltersDialog()">
 				<button mat-raised-button class="p-0">
 					<mat-icon class="mr-4 filter-icon">filter_alt</mat-icon>
diff --git a/dmp-frontend/src/app/ui/description/listing/description-listing.component.ts b/dmp-frontend/src/app/ui/description/listing/description-listing.component.ts
index fb97e4ea0..ec6b688ac 100644
--- a/dmp-frontend/src/app/ui/description/listing/description-listing.component.ts
+++ b/dmp-frontend/src/app/ui/description/listing/description-listing.component.ts
@@ -42,6 +42,7 @@ import { Tag } from '@app/core/model/tag/tag';
 import { D } from '@angular/cdk/keycodes';
 import { DmpLookup } from '@app/core/query/dmp.lookup';
 import { DescriptionTemplateLookup } from '@app/core/query/description-template.lookup';
+import { DmpUserLookup } from '@app/core/query/dmp-user.lookup';
 
 @Component({
 	selector: 'app-description-listing-component',
@@ -232,6 +233,19 @@ export class DescriptionListingComponent extends BaseListingComponent<BaseDescri
 
 		return lookup;
 	}
+	
+	protected initializeDmpUserLookup(): DmpUserLookup {
+		const lookup = new DmpUserLookup();
+		lookup.metadata = { countAll: true };
+		lookup.isActive = [IsActive.Active];
+		lookup.project = {
+			fields: [
+				[nameof<Dmp>(x => x.dmpUsers), nameof<DmpUser>(x => x.role)].join('.'),
+			]
+		};
+
+		return lookup;
+	}
 
 	protected initializeDescriptionTemplateLookup(): DescriptionTemplateLookup {
 		const lookup = new DescriptionTemplateLookup();
@@ -422,7 +436,6 @@ export class DescriptionListingComponent extends BaseListingComponent<BaseDescri
 
 		lookup.statuses = formGroup.get("status")?.value !== null ? [formGroup.get("status")?.value] : null;
 		lookup.showAllVersions = formGroup.get("showAllVersions")?.value ?? false;
-		lookup.roleInDmp = formGroup.get("role")?.value !== null ? [formGroup.get("role")?.value] : null;
 		
 		// Description Templates
 		let descriptionTemplates = formGroup.get("descriptionTemplates")?.value ?? null;
@@ -432,10 +445,17 @@ export class DescriptionListingComponent extends BaseListingComponent<BaseDescri
 		} else lookup.descriptionTemplateSubQuery = null;
 
 		// Dmps
-		let dmps = formGroup.get("associatedDmpIds")?.value ?? null;
-		if (dmps && dmps?.length > 0) {
+		let dmps = formGroup.get("associatedDmpIds")?.value ?? null; let addDmps = dmps && dmps?.length > 0;
+		let roles = formGroup.get("role")?.value !== null ? [formGroup.get("role")?.value] : null; let addRoles = roles && roles?.length > 0;
+		if (addDmps || addRoles) {
 			lookup.dmpSubQuery = this.initializeDmpLookup();
-			lookup.dmpSubQuery.ids = dmps;
+			
+			if (addDmps) lookup.dmpSubQuery.ids = dmps?.length > 0 ? dmps : null;
+
+			if (addRoles) {
+				lookup.dmpSubQuery.dmpUserSubQuery = this.initializeDmpUserLookup();
+				lookup.dmpSubQuery.dmpUserSubQuery.userRoles = roles;
+			}
 		} else lookup.dmpSubQuery = null;
 
 		// Tags
@@ -476,7 +496,7 @@ export class DescriptionListingComponent extends BaseListingComponent<BaseDescri
 
 		return (new UntypedFormBuilder()).group({
 			status: [lookup.statuses?.length > 0 ? lookup.statuses[0] : null],
-			role: [lookup.roleInDmp?.length > 0 ? lookup.roleInDmp[0] : null],
+			role:  lookup.dmpSubQuery?.dmpUserSubQuery?.userRoles ? lookup.dmpSubQuery?.dmpUserSubQuery?.userRoles[0] : [],
 			showAllVersions: [lookup.showAllVersions ?? false],
 			descriptionTemplates: lookup.descriptionTemplateSubQuery?.ids ? [lookup.descriptionTemplateSubQuery?.ids] : [],
 			associatedDmpIds: lookup.dmpSubQuery?.ids ? [lookup.dmpSubQuery?.ids] : [],
diff --git a/dmp-frontend/src/app/ui/description/listing/filtering/description-filter.component.html b/dmp-frontend/src/app/ui/description/listing/filtering/description-filter.component.html
index 33b3c3298..e77c9a903 100644
--- a/dmp-frontend/src/app/ui/description/listing/filtering/description-filter.component.html
+++ b/dmp-frontend/src/app/ui/description/listing/filtering/description-filter.component.html
@@ -56,11 +56,11 @@
 			<div class="col-10" *ngIf="isAuthenticated()">
 				<h6 class="category-title">{{'DESCRIPTION-LISTING.FILTERS.ROLE.NAME' | translate }}</h6>
 				<mat-radio-group aria-label="Select an option" [formControl]="formGroup.get('role')" class="row">
-					<mat-radio-button value="null" [checked]="formGroup.get('role').value == null" class="col-12">{{ 'DESCRIPTION-LISTING.FILTERS.ROLE.TYPES.ANY' | translate }}</mat-radio-button>
-					<mat-radio-button value="0" class="col-12">{{ 'DESCRIPTION-LISTING.FILTERS.ROLE.TYPES.OWNER' | translate }}</mat-radio-button>
-					<mat-radio-button value="1" class="col-12">{{ 'DESCRIPTION-LISTING.FILTERS.ROLE.TYPES.VIEWER' | translate }}</mat-radio-button>
-					<mat-radio-button value="2" class="col-12">{{ 'DESCRIPTION-LISTING.FILTERS.ROLE.TYPES.DESCRIPTION-CONTRIBUTOR' | translate }}</mat-radio-button>
-					<mat-radio-button value="3" class="col-12">{{ 'DESCRIPTION-LISTING.FILTERS.ROLE.TYPES.REVIEWER' | translate }}</mat-radio-button>
+					<mat-radio-button [value]="null" [checked]="formGroup.get('role').value == null" class="col-12">{{ 'DESCRIPTION-LISTING.FILTERS.ROLE.TYPES.ANY' | translate }}</mat-radio-button>
+					<mat-radio-button [value]="dmpRole.Owner" class="col-12">{{ 'DESCRIPTION-LISTING.FILTERS.ROLE.TYPES.OWNER' | translate }}</mat-radio-button>
+					<mat-radio-button [value]="dmpRole.Viewer" class="col-12">{{ 'DESCRIPTION-LISTING.FILTERS.ROLE.TYPES.VIEWER' | translate }}</mat-radio-button>
+					<mat-radio-button [value]="dmpRole.DescriptionContributor" class="col-12">{{ 'DESCRIPTION-LISTING.FILTERS.ROLE.TYPES.DESCRIPTION-CONTRIBUTOR' | translate }}</mat-radio-button>
+					<mat-radio-button [value]="dmpRole.Reviewer" class="col-12">{{ 'DESCRIPTION-LISTING.FILTERS.ROLE.TYPES.REVIEWER' | translate }}</mat-radio-button>
 				</mat-radio-group>
 				<hr>
 			</div>
diff --git a/dmp-frontend/src/app/ui/description/listing/filtering/description-filter.component.ts b/dmp-frontend/src/app/ui/description/listing/filtering/description-filter.component.ts
index 63c403b61..8dd87c697 100644
--- a/dmp-frontend/src/app/ui/description/listing/filtering/description-filter.component.ts
+++ b/dmp-frontend/src/app/ui/description/listing/filtering/description-filter.component.ts
@@ -2,6 +2,7 @@
 import { Component, EventEmitter, Input, OnChanges, OnInit, Output, SimpleChanges } from '@angular/core';
 import { FormBuilder, UntypedFormArray, UntypedFormGroup } from '@angular/forms';
 import { DescriptionStatus } from '@app/core/common/enum/description-status';
+import { DmpUserRole } from '@app/core/common/enum/dmp-user-role';
 import { ReferenceType } from '@app/core/model/reference-type/reference-type';
 import { Reference } from '@app/core/model/reference/reference';
 import { AuthService } from '@app/core/services/auth/auth.service';
@@ -34,6 +35,7 @@ export class DescriptionFilterComponent extends BaseCriteriaComponent implements
 	public filteringTagsAsync = false;
 
 	statuses = DescriptionStatus;
+	dmpRole = DmpUserRole;
 	options: UntypedFormGroup;
 	selectedReferenceTypes: Guid[];