Merged in task/dspace-cris-2023_02_x/DSC-2028 (pull request DSpace#2545)

Task/dspace cris 2023 02 x/DSC-2028

Approved-by: Giuseppe Digilio

Author: FrancescoMolinaro

src/app/collection-page/collection-page.component.spec.ts

@@ -14,14 +14,14 @@ import { RouterStub } from '../shared/testing/router.stub';
 import { environment } from 'src/environments/environment.test';
 import { createSuccessfulRemoteDataObject$ } from '../shared/remote-data.utils';
 import { Collection } from '../core/shared/collection.model';
-import { SearchService } from '../core/shared/search/search.service';
 import { By } from '@angular/platform-browser';
 import { FormsModule } from '@angular/forms';
 import { RouterTestingModule } from '@angular/router/testing';
 import { TranslateModule } from '@ngx-translate/core';
 import { VarDirective } from '../shared/utils/var.directive';
 import { BrowserAnimationsModule } from '@angular/platform-browser/animations';
 import { Bitstream } from '../core/shared/bitstream.model';
+import { SearchManager } from '../core/browse/search-manager';
 
 describe('CollectionPageComponent', () => {
   let component: CollectionPageComponent;
@@ -33,7 +33,7 @@ describe('CollectionPageComponent', () => {
   let paginationServiceSpy: jasmine.SpyObj<PaginationService>;
   let authorizationDataServiceSpy: jasmine.SpyObj<AuthorizationDataService>;
   let dsoNameServiceSpy: jasmine.SpyObj<DSONameService>;
-  let searchServiceSpy: jasmine.SpyObj<SearchService>;
+  let searchServiceSpy: jasmine.SpyObj<SearchManager>;
   let aroute = new ActivatedRouteStub();
   let router = new RouterStub();
 
@@ -44,7 +44,7 @@ describe('CollectionPageComponent', () => {
     paginationServiceSpy = jasmine.createSpyObj('PaginationService', ['getCurrentPagination', 'getCurrentSort', 'clearPagination']);
     authorizationDataServiceSpy = jasmine.createSpyObj('AuthorizationDataService', ['isAuthorized']);
     collectionDataServiceSpy = jasmine.createSpyObj('CollectionDataService', ['findById', 'getAuthorizedCollection']);
-    searchServiceSpy = jasmine.createSpyObj('SearchService', ['search']);
+    searchServiceSpy = jasmine.createSpyObj('SearchManager', ['search']);
     dsoNameServiceSpy = jasmine.createSpyObj('DSONameService', ['getName']);
 
     await TestBed.configureTestingModule({
@@ -58,7 +58,7 @@ describe('CollectionPageComponent', () => {
         { provide: PaginationService, useValue: paginationServiceSpy },
         { provide: AuthorizationDataService, useValue: authorizationDataServiceSpy },
         { provide: DSONameService, useValue: dsoNameServiceSpy },
-        { provide: SearchService, useValue: searchServiceSpy },
+        { provide: SearchManager, useValue: searchServiceSpy },
         { provide: APP_CONFIG, useValue: environment },
         { provide: PLATFORM_ID, useValue: 'browser' },
       ]

src/app/collection-page/collection-page.component.ts

@@ -4,9 +4,7 @@ import { ActivatedRoute, Router } from '@angular/router';
 import { BehaviorSubject, combineLatest as observableCombineLatest, Observable, Subject } from 'rxjs';
 import { filter, map, mergeMap, startWith, switchMap, take } from 'rxjs/operators';
 import { PaginatedSearchOptions } from '../shared/search/models/paginated-search-options.model';
-import { SearchService } from '../core/shared/search/search.service';
 import { SortDirection, SortOptions } from '../core/cache/models/sort-options.model';
-import { CollectionDataService } from '../core/data/collection-data.service';
 import { PaginatedList } from '../core/data/paginated-list.model';
 import { RemoteData } from '../core/data/remote-data';
 import { Bitstream } from '../core/shared/bitstream.model';
@@ -30,6 +28,7 @@ import { redirectOn4xx } from '../core/shared/authorized.operators';
 import { BROWSE_LINKS_TO_FOLLOW } from '../core/browse/browse.service';
 import { DSONameService } from '../core/breadcrumbs/dso-name.service';
 import { APP_CONFIG, AppConfig } from '../../../src/config/app-config.interface';
+import { SearchManager } from '../core/browse/search-manager';
 
 @Component({
   selector: 'ds-collection-page',
@@ -64,8 +63,7 @@ export class CollectionPageComponent implements OnInit {
 
   constructor(
     @Inject(PLATFORM_ID) private platformId: Object,
-    private collectionDataService: CollectionDataService,
-    private searchService: SearchService,
+    private searchManager: SearchManager,
     private route: ActivatedRoute,
     private router: Router,
     private authService: AuthService,
@@ -113,14 +111,13 @@ export class CollectionPageComponent implements OnInit {
         getFirstSucceededRemoteData(),
         map((rd) => rd.payload.id),
         switchMap((id: string) => {
-          return this.searchService.search<Item>(
+          return this.searchManager.search<Item>(
             new PaginatedSearchOptions({
               scope: id,
               pagination: currentPagination,
               sort: currentSort,
               dsoTypes: [DSpaceObjectType.ITEM],
               forcedEmbeddedKeys: ['metrics'],
-              projection: 'preventMetadataSecurity'
             }), null, true, true, ...BROWSE_LINKS_TO_FOLLOW)
             .pipe(toDSpaceObjectListRD()) as Observable<RemoteData<PaginatedList<Item>>>;
         }),

src/app/core/browse/search-manager.ts

@@ -45,7 +45,7 @@ export class SearchManager {
    * @returns {Observable<RemoteData<PaginatedList<Item>>>}
    */
   getBrowseItemsFor(filterValue: string, filterAuthority: string, options: BrowseEntrySearchOptions, ...linksToFollow: FollowLinkConfig<any>[]): Observable<RemoteData<PaginatedList<Item>>> {
-    const browseOptions = Object.assign({}, options, { projection: 'preventMetadataSecurity' });
+    const browseOptions = Object.assign({}, options, { projection: options.projection ?? 'preventMetadataSecurity' });
     return this.browseService.getBrowseItemsFor(filterValue, filterAuthority, browseOptions, ...linksToFollow)
       .pipe(this.completeWithExtraData());
   }
@@ -67,7 +67,8 @@ export class SearchManager {
     useCachedVersionIfAvailable = true,
     reRequestOnStale = true,
     ...linksToFollow: FollowLinkConfig<T>[]): Observable<RemoteData<SearchObjects<T>>> {
-    return this.searchService.search(searchOptions, responseMsToLive, useCachedVersionIfAvailable, reRequestOnStale, ...linksToFollow)
+    const optionsWithDefaultProjection = Object.assign(new PaginatedSearchOptions({}), searchOptions, { projection: searchOptions.projection ?? 'preventMetadataSecurity' });
+    return this.searchService.search(optionsWithDefaultProjection, responseMsToLive, useCachedVersionIfAvailable, reRequestOnStale, ...linksToFollow)
       .pipe(this.completeSearchObjectsWithExtraData());
   }
 

src/app/cris-layout/cris-layout-matrix/cris-layout-box-container/boxes/metadata/rendering-types/crisref/crisref.component.spec.ts

@@ -20,7 +20,7 @@ describe('CrisrefComponent', () => {
   let fixture: ComponentFixture<CrisrefComponent>;
 
   const itemService = jasmine.createSpyObj('ItemDataService', {
-    findById: jasmine.createSpy('findById')
+    findByIdWithProjections: jasmine.createSpy('findByIdWithProjections')
   });
   const metadataValue = Object.assign(new MetadataValue(), {
     'value': 'test item title',
@@ -106,7 +106,7 @@ describe('CrisrefComponent', () => {
   beforeEach(() => {
     fixture = TestBed.createComponent(CrisrefComponent);
     component = fixture.componentInstance;
-    itemService.findById.and.returnValue(createSuccessfulRemoteDataObject$(testPerson));
+    itemService.findByIdWithProjections.and.returnValue(createSuccessfulRemoteDataObject$(testPerson));
     fixture.detectChanges();
   });
 

src/app/my-dspace-page/my-dspace-page.component.html

@@ -9,7 +9,6 @@
   [configurationList]="(configurationList$ | async)"
   [context]="context"
   [viewModeList]="viewModeList"
-  [projection]="projection"
   [showThumbnails]="false"
   [selectable]="(currentConfiguration$ | async) === workflowType"
   [selectionConfig]="{ repeatable: true, listId: listId }"

src/app/my-dspace-page/my-dspace-page.component.ts

@@ -60,10 +60,6 @@ export class MyDSpacePageComponent implements OnInit {
    */
   roleTypeEnum = RoleType;
 
-  /**
-   * Projection to use during the search
-   */
-  projection = 'preventMetadataSecurity';
 
   /**
    * List of available view mode

src/app/shared/browse-most-elements/abstract-browse-elements.component.ts

@@ -3,7 +3,6 @@ import { CollectionElementLinkType } from '../object-collection/collection-eleme
 import { Component, Input, OnChanges, OnInit, PLATFORM_ID, inject } from '@angular/core';
 import { isPlatformServer } from '@angular/common';
 
-import { SearchService } from '../../core/shared/search/search.service';
 import { PaginatedSearchOptions } from '../search/models/paginated-search-options.model';
 import { DSpaceObject } from '../../core/shared/dspace-object.model';
 import { SearchResult } from '../search/models/search-result.model';
@@ -21,6 +20,7 @@ import { BehaviorSubject, Observable, mergeMap } from 'rxjs';
 import { Item } from '../../core/shared/item.model';
 import { getItemPageRoute } from '../../item-page/item-page-routing-paths';
 import { TopSection } from '../../core/layout/models/section.model';
+import { SearchManager } from '../../core/browse/search-manager';
 
 @Component({
   template: ''
@@ -29,7 +29,7 @@ export abstract class AbstractBrowseElementsComponent implements OnInit, OnChang
 
   protected readonly appConfig = inject(APP_CONFIG);
   protected readonly platformId = inject(PLATFORM_ID);
-  protected readonly searchService = inject(SearchService);
+  protected readonly searchManager = inject(SearchManager);
 
   protected abstract followMetricsLink: boolean; // to be overridden
   protected abstract followThumbnailLink: boolean; // to be overridden
@@ -47,7 +47,7 @@ export abstract class AbstractBrowseElementsComponent implements OnInit, OnChang
   /**
    * Optional projection to use during the search
    */
-  @Input() projection = 'preventMetadataSecurity';
+  @Input() projection;
 
   /**
    * Whether to show the badge label or not
@@ -96,10 +96,12 @@ export abstract class AbstractBrowseElementsComponent implements OnInit, OnChang
     this.paginatedSearchOptions = Object.assign(new PaginatedSearchOptions({}), this.paginatedSearchOptions, {
       projection: this.projection
     });
+
     this.paginatedSearchOptions$ = new BehaviorSubject<PaginatedSearchOptions>(this.paginatedSearchOptions);
+
     this.searchResults$ = this.paginatedSearchOptions$.asObservable().pipe(
       mergeMap((paginatedSearchOptions) =>
-        this.searchService.search(paginatedSearchOptions, null, true, true, ...followLinks),
+        this.searchManager.search(paginatedSearchOptions, null, true, true, ...followLinks),
       ),
       getAllCompletedRemoteData(),
     );

src/app/shared/browse-most-elements/browse-most-elements.component.ts

@@ -25,7 +25,7 @@ export class BrowseMostElementsComponent implements OnInit, OnChanges {
   /**
    * Optional projection to use during the search
    */
-  @Input() projection = 'preventMetadataSecurity';
+  @Input() projection;
 
   /**
    * Whether to show the badge label or not

src/app/shared/browse-most-elements/themed-browse-most-elements.component.ts

@@ -19,7 +19,7 @@ export class ThemedBrowseMostElementsComponent extends ThemedComponent<BrowseMos
 
   @Input() paginatedSearchOptions: PaginatedSearchOptions;
 
-  @Input() projection = 'preventMetadataSecurity';
+  @Input() projection: string;
 
   @Input() showLabel: boolean;
 

src/app/shared/metadata-link-view/metadata-link-view.component.spec.ts

@@ -90,7 +90,7 @@ describe('MetadataLinkViewComponent', () => {
   });
 
   itemService = jasmine.createSpyObj('ItemDataService', {
-    findById: jasmine.createSpy('findById')
+    findByIdWithProjections: jasmine.createSpy('findByIdWithProjections')
   });
 
   beforeEach(waitForAsync(() => {
@@ -110,7 +110,7 @@ describe('MetadataLinkViewComponent', () => {
   describe('Check metadata without authority', () => {
     beforeEach(() => {
       fixture = TestBed.createComponent(MetadataLinkViewComponent);
-      itemService.findById.and.returnValue(createSuccessfulRemoteDataObject$(testOrgunit));
+      itemService.findByIdWithProjections.and.returnValue(createSuccessfulRemoteDataObject$(testOrgunit));
       component = fixture.componentInstance;
       component.item = testPerson;
       component.metadata = testMetadataValueWithoutAuthority;
@@ -135,7 +135,7 @@ describe('MetadataLinkViewComponent', () => {
     describe('when item is found with orcid', () => {
       beforeEach(() => {
         fixture = TestBed.createComponent(MetadataLinkViewComponent);
-        itemService.findById.and.returnValue(createSuccessfulRemoteDataObject$(testPerson));
+        itemService.findByIdWithProjections.and.returnValue(createSuccessfulRemoteDataObject$(testPerson));
         component = fixture.componentInstance;
         component.item = testPerson;
         component.metadata = testMetadataValueWithAuthority;
@@ -162,7 +162,7 @@ describe('MetadataLinkViewComponent', () => {
     describe('when item is found without orcid', () => {
       beforeEach(() => {
         fixture = TestBed.createComponent(MetadataLinkViewComponent);
-        itemService.findById.and.returnValue(createSuccessfulRemoteDataObject$(testOrgunit));
+        itemService.findByIdWithProjections.and.returnValue(createSuccessfulRemoteDataObject$(testOrgunit));
         component = fixture.componentInstance;
         component.item = testPerson;
         component.metadata = testMetadataValueWithAuthority;
@@ -189,7 +189,7 @@ describe('MetadataLinkViewComponent', () => {
     describe('when item is not found', () => {
       beforeEach(() => {
         fixture = TestBed.createComponent(MetadataLinkViewComponent);
-        itemService.findById.and.returnValue(createFailedRemoteDataObject$());
+        itemService.findByIdWithProjections.and.returnValue(createFailedRemoteDataObject$());
         component = fixture.componentInstance;
         component.item = testPerson;
         component.metadata = testMetadataValueWithAuthority;