refactor, improve code according to suggestions

src/app/admin/admin-notifications/admin-notifications-routing.module.ts

@@ -11,7 +11,7 @@ import { AdminQualityAssuranceTopicsPageResolver } from './admin-quality-assuran
 import { AdminQualityAssuranceEventsPageResolver } from './admin-quality-assurance-events-page/admin-quality-assurance-events-page.resolver';
 import { AdminQualityAssuranceSourcePageComponent } from './admin-quality-assurance-source-page-component/admin-quality-assurance-source-page.component';
 import { AdminQualityAssuranceSourcePageResolver } from './admin-quality-assurance-source-page-component/admin-quality-assurance-source-page-resolver.service';
-import { SourceDataResolver } from './admin-quality-assurance-source-page-component/admin-quality-assurance-source-data.reslover';
+import { SourceDataResolver } from './admin-quality-assurance-source-page-component/admin-quality-assurance-source-data.resolver';
 
 @NgModule({
   imports: [

src/app/admin/admin-notifications/admin-notifications.module.ts

@@ -6,7 +6,7 @@ import { AdminNotificationsRoutingModule } from './admin-notifications-routing.m
 import { AdminQualityAssuranceTopicsPageComponent } from './admin-quality-assurance-topics-page/admin-quality-assurance-topics-page.component';
 import { AdminQualityAssuranceEventsPageComponent } from './admin-quality-assurance-events-page/admin-quality-assurance-events-page.component';
 import { AdminQualityAssuranceSourcePageComponent } from './admin-quality-assurance-source-page-component/admin-quality-assurance-source-page.component';
-import {SuggestionNotificationsModule} from '../../suggestion-notifications/suggestion-notifications.module';
+import {NotificationsModule} from '../../notifications/notifications.module';
 
 @NgModule({
   imports: [
@@ -14,7 +14,7 @@ import {SuggestionNotificationsModule} from '../../suggestion-notifications/sugg
     SharedModule,
     CoreModule.forRoot(),
     AdminNotificationsRoutingModule,
-    SuggestionNotificationsModule
+    NotificationsModule
   ],
   declarations: [
     AdminQualityAssuranceTopicsPageComponent,

src/app/admin/admin-notifications/admin-quality-assurance-source-page-component/admin-quality-assurance-source-data.resolver.ts

@@ -3,8 +3,8 @@ import { ActivatedRouteSnapshot, Resolve, RouterStateSnapshot, Router } from '@a
 import { Observable } from 'rxjs';
 import { map } from 'rxjs/operators';
 import { PaginatedList } from '../../../core/data/paginated-list.model';
-import { QualityAssuranceSourceObject } from '../../../core/suggestion-notifications/qa/models/quality-assurance-source.model';
-import { QualityAssuranceSourceService } from '../../../suggestion-notifications/qa/source/quality-assurance-source.service';
+import { QualityAssuranceSourceObject } from '../../../core/notifications/qa/models/quality-assurance-source.model';
+import { QualityAssuranceSourceService } from '../../../notifications/qa/source/quality-assurance-source.service';
 /**
  * This class represents a resolver that retrieve the route data before the route is activated.
  */

src/app/core/core.module.ts

@@ -157,9 +157,9 @@ import { SequenceService } from './shared/sequence.service';
 import { CoreState } from './core-state.model';
 import { GroupDataService } from './eperson/group-data.service';
 import { SubmissionAccessesModel } from './config/models/config-submission-accesses.model';
-import { QualityAssuranceTopicObject } from './suggestion-notifications/qa/models/quality-assurance-topic.model';
-import { QualityAssuranceEventObject } from './suggestion-notifications/qa/models/quality-assurance-event.model';
-import { QualityAssuranceSourceObject } from './suggestion-notifications/qa/models/quality-assurance-source.model';
+import { QualityAssuranceTopicObject } from './notifications/qa/models/quality-assurance-topic.model';
+import { QualityAssuranceEventObject } from './notifications/qa/models/quality-assurance-event.model';
+import { QualityAssuranceSourceObject } from './notifications/qa/models/quality-assurance-source.model';
 import { RatingAdvancedWorkflowInfo } from './tasks/models/rating-advanced-workflow-info.model';
 import { AdvancedWorkflowInfo } from './tasks/models/advanced-workflow-info.model';
 import { SelectReviewerAdvancedWorkflowInfo } from './tasks/models/select-reviewer-advanced-workflow-info.model';

src/app/notifications/notifications-effects.ts

@@ -1,7 +1,7 @@
 import { QualityAssuranceSourceEffects } from './qa/source/quality-assurance-source.effects';
 import { QualityAssuranceTopicsEffects } from './qa/topics/quality-assurance-topics.effects';
 
-export const suggestionNotificationsEffects = [
+export const notificationsEffects = [
   QualityAssuranceTopicsEffects,
   QualityAssuranceSourceEffects
 ];

src/app/notifications/notifications-state.service.spec.ts

@@ -2,8 +2,8 @@ import { TestBed } from '@angular/core/testing';
 import { Store, StoreModule } from '@ngrx/store';
 import { provideMockStore } from '@ngrx/store/testing';
 import { cold } from 'jasmine-marbles';
-import { suggestionNotificationsReducers } from './suggestion-notifications.reducer';
-import { SuggestionNotificationsStateService } from './suggestion-notifications-state.service';
+import { suggestionNotificationsReducers } from './notifications.reducer';
+import { NotificationsStateService } from './notifications-state.service';
 import {
   qualityAssuranceSourceObjectMissingPid,
   qualityAssuranceSourceObjectMoreAbstract,
@@ -16,7 +16,7 @@ import { RetrieveAllTopicsAction } from './qa/topics/quality-assurance-topics.ac
 import { RetrieveAllSourceAction } from './qa/source/quality-assurance-source.actions';
 
 describe('NotificationsStateService', () => {
-  let service: SuggestionNotificationsStateService;
+  let service: NotificationsStateService;
   let serviceAsAny: any;
   let store: any;
   let initialState: any;
@@ -67,14 +67,14 @@ describe('NotificationsStateService', () => {
           ],
           providers: [
             provideMockStore({ initialState }),
-            { provide: SuggestionNotificationsStateService, useValue: service }
+            { provide: NotificationsStateService, useValue: service }
           ]
         }).compileComponents();
       });
 
       beforeEach(() => {
         store = TestBed.get(Store);
-        service = new SuggestionNotificationsStateService(store);
+        service = new NotificationsStateService(store);
         serviceAsAny = service;
         spyOn(store, 'dispatch');
       });
@@ -159,14 +159,14 @@ describe('NotificationsStateService', () => {
           ],
           providers: [
             provideMockStore({ initialState }),
-            { provide: SuggestionNotificationsStateService, useValue: service }
+            { provide: NotificationsStateService, useValue: service }
           ]
         }).compileComponents();
       });
 
       beforeEach(() => {
         store = TestBed.get(Store);
-        service = new SuggestionNotificationsStateService(store);
+        service = new NotificationsStateService(store);
         serviceAsAny = service;
         spyOn(store, 'dispatch');
       });
@@ -255,14 +255,14 @@ describe('NotificationsStateService', () => {
           ],
           providers: [
             provideMockStore({ initialState }),
-            { provide: SuggestionNotificationsStateService, useValue: service }
+            { provide: NotificationsStateService, useValue: service }
           ]
         }).compileComponents();
       });
 
       beforeEach(() => {
         store = TestBed.get(Store);
-        service = new SuggestionNotificationsStateService(store);
+        service = new NotificationsStateService(store);
         serviceAsAny = service;
         spyOn(store, 'dispatch');
       });
@@ -325,14 +325,14 @@ describe('NotificationsStateService', () => {
           ],
           providers: [
             provideMockStore({ initialState }),
-            { provide: SuggestionNotificationsStateService, useValue: service }
+            { provide: NotificationsStateService, useValue: service }
           ]
         }).compileComponents();
       });
 
       beforeEach(() => {
         store = TestBed.get(Store);
-        service = new SuggestionNotificationsStateService(store);
+        service = new NotificationsStateService(store);
         serviceAsAny = service;
         spyOn(store, 'dispatch');
       });
@@ -417,14 +417,14 @@ describe('NotificationsStateService', () => {
           ],
           providers: [
             provideMockStore({ initialState }),
-            { provide: SuggestionNotificationsStateService, useValue: service }
+            { provide: NotificationsStateService, useValue: service }
           ]
         }).compileComponents();
       });
 
       beforeEach(() => {
         store = TestBed.get(Store);
-        service = new SuggestionNotificationsStateService(store);
+        service = new NotificationsStateService(store);
         serviceAsAny = service;
         spyOn(store, 'dispatch');
       });
@@ -513,14 +513,14 @@ describe('NotificationsStateService', () => {
           ],
           providers: [
             provideMockStore({ initialState }),
-            { provide: SuggestionNotificationsStateService, useValue: service }
+            { provide: NotificationsStateService, useValue: service }
           ]
         }).compileComponents();
       });
 
       beforeEach(() => {
         store = TestBed.get(Store);
-        service = new SuggestionNotificationsStateService(store);
+        service = new NotificationsStateService(store);
         serviceAsAny = service;
         spyOn(store, 'dispatch');
       });

src/app/notifications/notifications-state.service.ts

@@ -16,17 +16,17 @@ import {
   getQualityAssuranceSourceCurrentPageSelector,
   getQualityAssuranceSourceTotalsSelector
 } from './selectors';
-import { QualityAssuranceTopicObject } from '../core/suggestion-notifications/qa/models/quality-assurance-topic.model';
-import { SuggestionNotificationsState } from './suggestion-notifications.reducer';
+import { QualityAssuranceTopicObject } from '../core/notifications/qa/models/quality-assurance-topic.model';
+import { SuggestionNotificationsState } from './notifications.reducer';
 import { RetrieveAllTopicsAction } from './qa/topics/quality-assurance-topics.actions';
-import { QualityAssuranceSourceObject } from '../core/suggestion-notifications/qa/models/quality-assurance-source.model';
+import { QualityAssuranceSourceObject } from '../core/notifications/qa/models/quality-assurance-source.model';
 import { RetrieveAllSourceAction } from './qa/source/quality-assurance-source.actions';
 
 /**
  * The service handling the Notifications State.
  */
 @Injectable()
-export class SuggestionNotificationsStateService {
+export class NotificationsStateService {
 
   /**
    * Initialize the service variables.

src/app/notifications/notifications.module.ts

@@ -8,16 +8,16 @@ import { SharedModule } from '../shared/shared.module';
 import { storeModuleConfig } from '../app.reducer';
 import { QualityAssuranceTopicsComponent } from './qa/topics/quality-assurance-topics.component';
 import { QualityAssuranceEventsComponent } from './qa/events/quality-assurance-events.component';
-import { SuggestionNotificationsStateService } from './suggestion-notifications-state.service';
-import { suggestionNotificationsReducers, SuggestionNotificationsState } from './suggestion-notifications.reducer';
-import { suggestionNotificationsEffects } from './suggestion-notifications-effects';
+import { NotificationsStateService } from './notifications-state.service';
+import { suggestionNotificationsReducers, SuggestionNotificationsState } from './notifications.reducer';
+import { notificationsEffects } from './notifications-effects';
 import { QualityAssuranceTopicsService } from './qa/topics/quality-assurance-topics.service';
 import {
   QualityAssuranceTopicDataService
-} from '../core/suggestion-notifications/qa/topics/quality-assurance-topic-data.service';
+} from '../core/notifications/qa/topics/quality-assurance-topic-data.service';
 import {
   QualityAssuranceEventDataService
-} from '../core/suggestion-notifications/qa/events/quality-assurance-event-data.service';
+} from '../core/notifications/qa/events/quality-assurance-event-data.service';
 import { ProjectEntryImportModalComponent } from './qa/project-entry-import-modal/project-entry-import-modal.component';
 import { TranslateModule } from '@ngx-translate/core';
 import { SearchModule } from '../shared/search/search.module';
@@ -25,7 +25,7 @@ import { QualityAssuranceSourceComponent } from './qa/source/quality-assurance-s
 import { QualityAssuranceSourceService } from './qa/source/quality-assurance-source.service';
 import {
   QualityAssuranceSourceDataService
-} from '../core/suggestion-notifications/qa/source/quality-assurance-source-data.service';
+} from '../core/notifications/qa/source/quality-assurance-source-data.service';
 
 const MODULES = [
   CommonModule,
@@ -33,7 +33,7 @@ const MODULES = [
   SearchModule,
   CoreModule.forRoot(),
   StoreModule.forFeature('suggestionNotifications', suggestionNotificationsReducers, storeModuleConfig as StoreConfig<SuggestionNotificationsState, Action>),
-  EffectsModule.forFeature(suggestionNotificationsEffects),
+  EffectsModule.forFeature(notificationsEffects),
   TranslateModule
 ];
 
@@ -50,7 +50,7 @@ const ENTRY_COMPONENTS = [
 ];
 
 const PROVIDERS = [
-  SuggestionNotificationsStateService,
+  NotificationsStateService,
   QualityAssuranceTopicsService,
   QualityAssuranceSourceService,
   QualityAssuranceTopicDataService,
@@ -82,5 +82,5 @@ const PROVIDERS = [
 /**
  * This module handles all components that are necessary for the OpenAIRE components
  */
-export class SuggestionNotificationsModule {
+export class NotificationsModule {
 }

src/app/notifications/qa/events/quality-assurance-events.component.html

@@ -52,13 +52,14 @@
                   </td>
                   <td><a *ngIf="eventElement?.target"
                          target="_blank"
+                         rel="noopener noreferrer"
                          [routerLink]="['/items', eventElement?.target?.id]">{{eventElement.title}}</a>
                     <span *ngIf="!eventElement?.target">{{eventElement.title}}</span>
                   </td>
                   <td *ngIf="showTopic.indexOf('/PID') !== -1">
                     <p><span class="small">{{'quality-assurance.event.table.pidtype' | translate}}</span>&nbsp;<span class="badge badge-info">{{eventElement.event.message.type}}</span></p>
                     <p><span class="small">{{'quality-assurance.event.table.pidvalue' | translate}}</span><br>
-                      <a *ngIf="hasPIDHref(eventElement.event.message); else noPID" href="{{getPIDHref(eventElement.event.message)}}" target="_blank">
+                      <a *ngIf="hasPIDHref(eventElement.event.message); else noPID" href="{{getPIDHref(eventElement.event.message)}}" rel="noopener noreferrer" target="_blank">
                         {{eventElement.event.message.value}}
                       </a>
                       <ng-template #noPID><span class="badge badge-info">{{eventElement.event.message.value}}</span></ng-template>
@@ -84,7 +85,7 @@
                     </p>
                     <p>
                       <span class="small">{{'quality-assurance.event.table.project' | translate}}</span><br>
-                      <a href="https://explore.openaire.eu/search/project?projectId={{ eventElement.event.message.openaireId}}" target="_blank">{{eventElement.event.message.title}}</a>
+                      <a href="{{openAireUrl}}{{ eventElement.event.message.openaireId}}" rel="noopener noreferrer" target="_blank">{{eventElement.event.message.title}}</a>
                     </p>
                     <p>
                       <span *ngIf="eventElement.event.message.acronym"><span class="small">{{'quality-assurance.event.table.acronym' | translate}}</span>&nbsp;<span class="badge badge-info">{{eventElement.event.message.acronym}}</span><br></span>
@@ -96,7 +97,7 @@
                     <hr>
                     <div>
                       {{(eventElement.hasProject ? 'quality-assurance.event.project.found' : 'quality-assurance.event.project.notFound') | translate}}
-                      <a target="_blank" *ngIf="eventElement.hasProject" title="{{eventElement.projectTitle}}" [routerLink]="['/items', eventElement.projectId]">{{eventElement.handle}} </a>
+                      <a target="_blank" rel="noopener noreferrer" *ngIf="eventElement.hasProject" title="{{eventElement.projectTitle}}" [routerLink]="['/items', eventElement.projectId]">{{eventElement.handle}} </a>
                       <div class="btn-group">
                         <button *ngIf="!eventElement.hasProject"
                                 class="btn btn-outline-primary btn-sm"

src/app/notifications/qa/events/quality-assurance-events.component.spec.ts

@@ -7,7 +7,7 @@ import { NgbModal } from '@ng-bootstrap/ng-bootstrap';
 import { of as observableOf } from 'rxjs';
 import {
   QualityAssuranceEventDataService
-} from '../../../core/suggestion-notifications/qa/events/quality-assurance-event-data.service';
+} from '../../../core/notifications/qa/events/quality-assurance-event-data.service';
 import { QualityAssuranceEventsComponent } from './quality-assurance-events.component';
 import {
   getMockQualityAssuranceEventRestService,
@@ -26,7 +26,7 @@ import { ActivatedRouteStub } from '../../../shared/testing/active-router.stub';
 import { PaginationComponentOptions } from '../../../shared/pagination/pagination-component-options.model';
 import {
   QualityAssuranceEventObject
-} from '../../../core/suggestion-notifications/qa/models/quality-assurance-event.model';
+} from '../../../core/notifications/qa/models/quality-assurance-event.model';
 import { QualityAssuranceEventData } from '../project-entry-import-modal/project-entry-import-modal.component';
 import { TestScheduler } from 'rxjs/testing';
 import { cold, getTestScheduler } from 'jasmine-marbles';

src/app/notifications/qa/events/quality-assurance-events.component.ts

@@ -12,10 +12,10 @@ import { RemoteData } from '../../../core/data/remote-data';
 import {
   OpenaireQualityAssuranceEventMessageObject,
   QualityAssuranceEventObject
-} from '../../../core/suggestion-notifications/qa/models/quality-assurance-event.model';
+} from '../../../core/notifications/qa/models/quality-assurance-event.model';
 import {
   QualityAssuranceEventDataService
-} from '../../../core/suggestion-notifications/qa/events/quality-assurance-event-data.service';
+} from '../../../core/notifications/qa/events/quality-assurance-event-data.service';
 import { PaginationComponentOptions } from '../../../shared/pagination/pagination-component-options.model';
 import { Metadata } from '../../../core/shared/metadata.utils';
 import { followLink } from '../../../shared/utils/follow-link-config.model';
@@ -95,6 +95,10 @@ export class QualityAssuranceEventsComponent implements OnInit, OnDestroy {
    * @type {boolean}
    */
   public showMore = false;
+  /**
+   * The Open Aire base url for project search
+   */
+  public openAireUrl = 'https://explore.openaire.eu/search/project?projectId=';
   /**
    * The FindListOptions object
    */

src/app/notifications/qa/project-entry-import-modal/project-entry-import-modal.component.html

@@ -9,7 +9,7 @@
   <small>{{ (labelPrefix + label + '.publication' | translate) }}</small>
   <div class="mb-3">
     <div class="text-truncate">
-      <a target="_blank" href="/items/{{(externalSourceEntry.event.target|async)?.payload?.id}}">
+      <a target="_blank" [routerLink]="'/items/{{(externalSourceEntry.event.target|async)?.payload?.id}}'">
         {{externalSourceEntry.title}}
       </a>
     </div>
@@ -18,7 +18,7 @@
     <small>{{ (labelPrefix + label + '.bountToLocal' |translate) }}</small>
     <div class="mb-3">
       <div class="text-truncate">
-        <a target="_blank" href="/items/{{externalSourceEntry.projectId}}">
+        <a target="_blank" [routerLink]="'/items/{{externalSourceEntry.projectId}}'">
           {{externalSourceEntry.projectTitle}}
         </a>
       </div>

src/app/notifications/qa/project-entry-import-modal/project-entry-import-modal.component.ts

@@ -15,7 +15,7 @@ import { DSpaceObject } from '../../../core/shared/dspace-object.model';
 import {
   OpenaireQualityAssuranceEventMessageObject,
   QualityAssuranceEventObject,
-} from '../../../core/suggestion-notifications/qa/models/quality-assurance-event.model';
+} from '../../../core/notifications/qa/models/quality-assurance-event.model';
 import { hasValue, isNotEmpty } from '../../../shared/empty.util';
 import { Item } from '../../../core/shared/item.model';
 

src/app/notifications/qa/source/quality-assurance-source.actions.ts

@@ -1,7 +1,7 @@
 /* eslint-disable max-classes-per-file */
 import { Action } from '@ngrx/store';
 import { type } from '../../../shared/ngrx/type';
-import { QualityAssuranceSourceObject } from '../../../core/suggestion-notifications/qa/models/quality-assurance-source.model';
+import { QualityAssuranceSourceObject } from '../../../core/notifications/qa/models/quality-assurance-source.model';
 
 /**
  * For each action type in an action group, make a simple
@@ -12,9 +12,9 @@ import { QualityAssuranceSourceObject } from '../../../core/suggestion-notificat
  * action types in the application are unique.
  */
 export const QualityAssuranceSourceActionTypes = {
-  ADD_SOURCE: type('dspace/integration/suggestion-notifications/qa/ADD_SOURCE'),
-  RETRIEVE_ALL_SOURCE: type('dspace/integration/suggestion-notifications/qa/RETRIEVE_ALL_SOURCE'),
-  RETRIEVE_ALL_SOURCE_ERROR: type('dspace/integration/suggestion-notifications/qa/RETRIEVE_ALL_SOURCE_ERROR'),
+  ADD_SOURCE: type('dspace/integration/notifications/qa/ADD_SOURCE'),
+  RETRIEVE_ALL_SOURCE: type('dspace/integration/notifications/qa/RETRIEVE_ALL_SOURCE'),
+  RETRIEVE_ALL_SOURCE_ERROR: type('dspace/integration/notifications/qa/RETRIEVE_ALL_SOURCE_ERROR'),
 };
 
 /**

src/app/notifications/qa/source/quality-assurance-source.component.spec.ts

@@ -11,7 +11,7 @@ import {
   qualityAssuranceSourceObjectMorePid
 } from '../../../shared/mocks/notifications.mock';
 import { QualityAssuranceSourceComponent } from './quality-assurance-source.component';
-import { SuggestionNotificationsStateService } from '../../suggestion-notifications-state.service';
+import { NotificationsStateService } from '../../notifications-state.service';
 import { cold } from 'jasmine-marbles';
 import { PaginationServiceStub } from '../../../shared/testing/pagination-service.stub';
 import { PaginationService } from '../../../core/pagination/pagination.service';
@@ -40,7 +40,7 @@ describe('QualityAssuranceSourceComponent test suite', () => {
         TestComponent,
       ],
       providers: [
-        { provide: SuggestionNotificationsStateService, useValue: mockNotificationsStateService },
+        { provide: NotificationsStateService, useValue: mockNotificationsStateService },
         { provide: ActivatedRoute, useValue: { data: observableOf(activatedRouteParams), params: observableOf({}) } },
         { provide: PaginationService, useValue: paginationService },
         QualityAssuranceSourceComponent

src/app/notifications/qa/source/quality-assurance-source.component.ts

@@ -3,9 +3,9 @@ import { PaginationService } from '../../../core/pagination/pagination.service';
 import { Observable, Subscription } from 'rxjs';
 import { distinctUntilChanged, take } from 'rxjs/operators';
 import { SortOptions } from '../../../core/cache/models/sort-options.model';
-import { QualityAssuranceSourceObject } from '../../../core/suggestion-notifications/qa/models/quality-assurance-source.model';
+import { QualityAssuranceSourceObject } from '../../../core/notifications/qa/models/quality-assurance-source.model';
 import { PaginationComponentOptions } from '../../../shared/pagination/pagination-component-options.model';
-import { SuggestionNotificationsStateService } from '../../suggestion-notifications-state.service';
+import { NotificationsStateService } from '../../notifications-state.service';
 import { AdminQualityAssuranceSourcePageParams } from '../../../admin/admin-notifications/admin-quality-assurance-source-page-component/admin-quality-assurance-source-page-resolver.service';
 import { hasValue } from '../../../shared/empty.util';
 
@@ -50,11 +50,11 @@ export class QualityAssuranceSourceComponent implements OnInit {
   /**
    * Initialize the component variables.
    * @param {PaginationService} paginationService
-   * @param {SuggestionNotificationsStateService} notificationsStateService
+   * @param {NotificationsStateService} notificationsStateService
    */
   constructor(
     private paginationService: PaginationService,
-    private notificationsStateService: SuggestionNotificationsStateService,
+    private notificationsStateService: NotificationsStateService,
   ) { }
 
   /**

src/app/notifications/qa/source/quality-assurance-source.effects.ts

@@ -14,13 +14,13 @@ import {
 } from './quality-assurance-source.actions';
 import {
   QualityAssuranceSourceObject
-} from '../../../core/suggestion-notifications/qa/models/quality-assurance-source.model';
+} from '../../../core/notifications/qa/models/quality-assurance-source.model';
 import { PaginatedList } from '../../../core/data/paginated-list.model';
 import { QualityAssuranceSourceService } from './quality-assurance-source.service';
 import { NotificationsService } from '../../../shared/notifications/notifications.service';
 import {
   QualityAssuranceSourceDataService
-} from '../../../core/suggestion-notifications/qa/source/quality-assurance-source-data.service';
+} from '../../../core/notifications/qa/source/quality-assurance-source-data.service';
 
 /**
  * Provides effect methods for the Quality Assurance source actions.

src/app/notifications/qa/source/quality-assurance-source.reducer.ts

@@ -1,4 +1,4 @@
-import { QualityAssuranceSourceObject } from '../../../core/suggestion-notifications/qa/models/quality-assurance-source.model';
+import { QualityAssuranceSourceObject } from '../../../core/notifications/qa/models/quality-assurance-source.model';
 import { QualityAssuranceSourceActionTypes, QualityAssuranceSourceActions } from './quality-assurance-source.actions';
 
 /**

src/app/notifications/qa/source/quality-assurance-source.service.spec.ts

@@ -13,7 +13,7 @@ import { cold } from 'jasmine-marbles';
 import { buildPaginatedList } from '../../../core/data/paginated-list.model';
 import {
   QualityAssuranceSourceDataService
-} from '../../../core/suggestion-notifications/qa/source/quality-assurance-source-data.service';
+} from '../../../core/notifications/qa/source/quality-assurance-source-data.service';
 import { FindListOptions } from '../../../core/data/find-list-options.model';
 
 describe('QualityAssuranceSourceService', () => {

src/app/notifications/qa/source/quality-assurance-source.service.ts

@@ -5,13 +5,13 @@ import { map } from 'rxjs/operators';
 
 import {
   QualityAssuranceSourceDataService
-} from '../../../core/suggestion-notifications/qa/source/quality-assurance-source-data.service';
+} from '../../../core/notifications/qa/source/quality-assurance-source-data.service';
 import { SortDirection, SortOptions } from '../../../core/cache/models/sort-options.model';
 import { RemoteData } from '../../../core/data/remote-data';
 import { PaginatedList } from '../../../core/data/paginated-list.model';
 import {
   QualityAssuranceSourceObject
-} from '../../../core/suggestion-notifications/qa/models/quality-assurance-source.model';
+} from '../../../core/notifications/qa/models/quality-assurance-source.model';
 import { FindListOptions } from '../../../core/data/find-list-options.model';
 import { getFirstCompletedRemoteData } from '../../../core/shared/operators';
 

src/app/notifications/qa/topics/quality-assurance-topics.actions.ts

@@ -1,7 +1,7 @@
 /* eslint-disable max-classes-per-file */
 import { Action } from '@ngrx/store';
 import { type } from '../../../shared/ngrx/type';
-import { QualityAssuranceTopicObject } from '../../../core/suggestion-notifications/qa/models/quality-assurance-topic.model';
+import { QualityAssuranceTopicObject } from '../../../core/notifications/qa/models/quality-assurance-topic.model';
 
 /**
  * For each action type in an action group, make a simple
@@ -12,9 +12,9 @@ import { QualityAssuranceTopicObject } from '../../../core/suggestion-notificati
  * action types in the application are unique.
  */
 export const QualityAssuranceTopicActionTypes = {
-  ADD_TOPICS: type('dspace/integration/suggestion-notifications/qa/topic/ADD_TOPICS'),
-  RETRIEVE_ALL_TOPICS: type('dspace/integration/suggestion-notifications/qa/topic/RETRIEVE_ALL_TOPICS'),
-  RETRIEVE_ALL_TOPICS_ERROR: type('dspace/integration/suggestion-notifications/qa/topic/RETRIEVE_ALL_TOPICS_ERROR'),
+  ADD_TOPICS: type('dspace/integration/notifications/qa/topic/ADD_TOPICS'),
+  RETRIEVE_ALL_TOPICS: type('dspace/integration/notifications/qa/topic/RETRIEVE_ALL_TOPICS'),
+  RETRIEVE_ALL_TOPICS_ERROR: type('dspace/integration/notifications/qa/topic/RETRIEVE_ALL_TOPICS_ERROR'),
 };
 
 /**

src/app/notifications/qa/topics/quality-assurance-topics.component.spec.ts

@@ -12,7 +12,7 @@ import {
   qualityAssuranceTopicObjectMorePid
 } from '../../../shared/mocks/notifications.mock';
 import { QualityAssuranceTopicsComponent } from './quality-assurance-topics.component';
-import { SuggestionNotificationsStateService } from '../../suggestion-notifications-state.service';
+import { NotificationsStateService } from '../../notifications-state.service';
 import { cold } from 'jasmine-marbles';
 import { PaginationServiceStub } from '../../../shared/testing/pagination-service.stub';
 import { PaginationService } from '../../../core/pagination/pagination.service';
@@ -42,7 +42,7 @@ describe('QualityAssuranceTopicsComponent test suite', () => {
         TestComponent,
       ],
       providers: [
-        { provide: SuggestionNotificationsStateService, useValue: mockNotificationsStateService },
+        { provide: NotificationsStateService, useValue: mockNotificationsStateService },
         { provide: ActivatedRoute, useValue: { data: observableOf(activatedRouteParams), snapshot: {
           paramMap: {
             get: () => 'openaire',

src/app/notifications/qa/topics/quality-assurance-topics.component.ts

@@ -6,10 +6,10 @@ import { distinctUntilChanged, take } from 'rxjs/operators';
 import { SortOptions } from '../../../core/cache/models/sort-options.model';
 import {
   QualityAssuranceTopicObject
-} from '../../../core/suggestion-notifications/qa/models/quality-assurance-topic.model';
+} from '../../../core/notifications/qa/models/quality-assurance-topic.model';
 import { hasValue } from '../../../shared/empty.util';
 import { PaginationComponentOptions } from '../../../shared/pagination/pagination-component-options.model';
-import { SuggestionNotificationsStateService } from '../../suggestion-notifications-state.service';
+import { NotificationsStateService } from '../../notifications-state.service';
 import {
   AdminQualityAssuranceTopicsPageParams
 } from '../../../admin/admin-notifications/admin-quality-assurance-topics-page/admin-quality-assurance-topics-page-resolver.service';
@@ -64,13 +64,13 @@ export class QualityAssuranceTopicsComponent implements OnInit {
    * Initialize the component variables.
    * @param {PaginationService} paginationService
    * @param {ActivatedRoute} activatedRoute
-   * @param {SuggestionNotificationsStateService} notificationsStateService
+   * @param {NotificationsStateService} notificationsStateService
    * @param {QualityAssuranceTopicsService} qualityAssuranceTopicsService
    */
   constructor(
     private paginationService: PaginationService,
     private activatedRoute: ActivatedRoute,
-    private notificationsStateService: SuggestionNotificationsStateService,
+    private notificationsStateService: NotificationsStateService,
     private qualityAssuranceTopicsService: QualityAssuranceTopicsService
   ) {
   }

src/app/notifications/qa/topics/quality-assurance-topics.effects.ts

@@ -14,13 +14,13 @@ import {
 } from './quality-assurance-topics.actions';
 import {
   QualityAssuranceTopicObject
-} from '../../../core/suggestion-notifications/qa/models/quality-assurance-topic.model';
+} from '../../../core/notifications/qa/models/quality-assurance-topic.model';
 import { PaginatedList } from '../../../core/data/paginated-list.model';
 import { QualityAssuranceTopicsService } from './quality-assurance-topics.service';
 import { NotificationsService } from '../../../shared/notifications/notifications.service';
 import {
   QualityAssuranceTopicDataService
-} from '../../../core/suggestion-notifications/qa/topics/quality-assurance-topic-data.service';
+} from '../../../core/notifications/qa/topics/quality-assurance-topic-data.service';
 
 /**
  * Provides effect methods for the Quality Assurance topics actions.

src/app/notifications/qa/topics/quality-assurance-topics.reducer.ts

@@ -1,4 +1,4 @@
-import { QualityAssuranceTopicObject } from '../../../core/suggestion-notifications/qa/models/quality-assurance-topic.model';
+import { QualityAssuranceTopicObject } from '../../../core/notifications/qa/models/quality-assurance-topic.model';
 import { QualityAssuranceTopicActionTypes, QualityAssuranceTopicsActions } from './quality-assurance-topics.actions';
 
 /**

src/app/notifications/qa/topics/quality-assurance-topics.service.spec.ts

@@ -4,7 +4,7 @@ import { QualityAssuranceTopicsService } from './quality-assurance-topics.servic
 import { SortDirection, SortOptions } from '../../../core/cache/models/sort-options.model';
 import {
   QualityAssuranceTopicDataService
-} from '../../../core/suggestion-notifications/qa/topics/quality-assurance-topic-data.service';
+} from '../../../core/notifications/qa/topics/quality-assurance-topic-data.service';
 import { PageInfo } from '../../../core/shared/page-info.model';
 import {
   getMockQualityAssuranceTopicRestService,

src/app/notifications/qa/topics/quality-assurance-topics.service.ts

@@ -3,13 +3,13 @@ import { Observable } from 'rxjs';
 import { map } from 'rxjs/operators';
 import {
   QualityAssuranceTopicDataService
-} from '../../../core/suggestion-notifications/qa/topics/quality-assurance-topic-data.service';
+} from '../../../core/notifications/qa/topics/quality-assurance-topic-data.service';
 import { SortDirection, SortOptions } from '../../../core/cache/models/sort-options.model';
 import { RemoteData } from '../../../core/data/remote-data';
 import { PaginatedList } from '../../../core/data/paginated-list.model';
 import {
   QualityAssuranceTopicObject
-} from '../../../core/suggestion-notifications/qa/models/quality-assurance-topic.model';
+} from '../../../core/notifications/qa/models/quality-assurance-topic.model';
 import { RequestParam } from '../../../core/cache/models/request-param.model';
 import { FindListOptions } from '../../../core/data/find-list-options.model';
 import { getFirstCompletedRemoteData } from '../../../core/shared/operators';

src/app/notifications/selectors.ts

@@ -1,12 +1,12 @@
 import { createFeatureSelector, createSelector, MemoizedSelector } from '@ngrx/store';
 import { subStateSelector } from '../shared/selector.util';
-import { suggestionNotificationsSelector, SuggestionNotificationsState } from './suggestion-notifications.reducer';
-import { QualityAssuranceTopicObject } from '../core/suggestion-notifications/qa/models/quality-assurance-topic.model';
+import { suggestionNotificationsSelector, SuggestionNotificationsState } from './notifications.reducer';
+import { QualityAssuranceTopicObject } from '../core/notifications/qa/models/quality-assurance-topic.model';
 import { QualityAssuranceTopicState } from './qa/topics/quality-assurance-topics.reducer';
 import { QualityAssuranceSourceState } from './qa/source/quality-assurance-source.reducer';
 import {
   QualityAssuranceSourceObject
-} from '../core/suggestion-notifications/qa/models/quality-assurance-source.model';
+} from '../core/notifications/qa/models/quality-assurance-source.model';
 
 /**
  * Returns the Notifications state.

src/app/shared/mocks/notifications.mock.ts

@@ -2,16 +2,16 @@ import { of as observableOf } from 'rxjs';
 import { ResourceType } from '../../core/shared/resource-type';
 import {
   QualityAssuranceTopicObject
-} from '../../core/suggestion-notifications/qa/models/quality-assurance-topic.model';
+} from '../../core/notifications/qa/models/quality-assurance-topic.model';
 import {
   QualityAssuranceEventObject
-} from '../../core/suggestion-notifications/qa/models/quality-assurance-event.model';
+} from '../../core/notifications/qa/models/quality-assurance-event.model';
 import {
   QualityAssuranceTopicDataService
-} from '../../core/suggestion-notifications/qa/topics/quality-assurance-topic-data.service';
+} from '../../core/notifications/qa/topics/quality-assurance-topic-data.service';
 import {
   QualityAssuranceEventDataService
-} from '../../core/suggestion-notifications/qa/events/quality-assurance-event-data.service';
+} from '../../core/notifications/qa/events/quality-assurance-event-data.service';
 import { DSpaceObject } from '../../core/shared/dspace-object.model';
 import { Item } from '../../core/shared/item.model';
 import {
@@ -22,7 +22,7 @@ import {
 import { SearchResult } from '../search/models/search-result.model';
 import {
   QualityAssuranceSourceObject
-} from '../../core/suggestion-notifications/qa/models/quality-assurance-source.model';
+} from '../../core/notifications/qa/models/quality-assurance-source.model';
 
 // REST Mock ---------------------------------------------------------------------
 // -------------------------------------------------------------------------------
@@ -80,7 +80,7 @@ const ItemMockPid1: Item = Object.assign(
       'dc.identifier.uri': [
         {
           language: null,
-          value: 'http://dspace7.4science.it/xmlui/handle/10673/6'
+          value: 'https://demo.dspace.org/handle/10673/6'
         }
       ],
       'dc.description.abstract': [
@@ -199,7 +199,7 @@ const ItemMockPid2: Item = Object.assign(
       'dc.identifier.uri': [
         {
           language: null,
-          value: 'http://dspace7.4science.it/xmlui/handle/10673/6'
+          value: 'https://demo.dspace.org/handle/10673/6'
         }
       ],
       'dc.description.abstract': [
@@ -318,7 +318,7 @@ const ItemMockPid3: Item = Object.assign(
       'dc.identifier.uri': [
         {
           language: null,
-          value: 'http://dspace7.4science.it/xmlui/handle/10673/6'
+          value: 'https://demo.dspace.org/handle/10673/6'
         }
       ],
       'dc.description.abstract': [
@@ -437,7 +437,7 @@ const ItemMockPid4: Item = Object.assign(
       'dc.identifier.uri': [
         {
           language: null,
-          value: 'http://dspace7.4science.it/xmlui/handle/10673/6'
+          value: 'https://demo.dspace.org/handle/10673/6'
         }
       ],
       'dc.description.abstract': [
@@ -556,7 +556,7 @@ const ItemMockPid5: Item = Object.assign(
       'dc.identifier.uri': [
         {
           language: null,
-          value: 'http://dspace7.4science.it/xmlui/handle/10673/6'
+          value: 'https://demo.dspace.org/handle/10673/6'
         }
       ],
       'dc.description.abstract': [
@@ -675,7 +675,7 @@ const ItemMockPid6: Item = Object.assign(
       'dc.identifier.uri': [
         {
           language: null,
-          value: 'http://dspace7.4science.it/xmlui/handle/10673/6'
+          value: 'https://demo.dspace.org/handle/10673/6'
         }
       ],
       'dc.description.abstract': [
@@ -794,7 +794,7 @@ const ItemMockPid7: Item = Object.assign(
       'dc.identifier.uri': [
         {
           language: null,
-          value: 'http://dspace7.4science.it/xmlui/handle/10673/6'
+          value: 'https://demo.dspace.org/handle/10673/6'
         }
       ],
       'dc.description.abstract': [
@@ -913,7 +913,7 @@ export const ItemMockPid8: Item = Object.assign(
       'dc.identifier.uri': [
         {
           language: null,
-          value: 'http://dspace7.4science.it/xmlui/handle/10673/6'
+          value: 'https://demo.dspace.org/handle/10673/6'
         }
       ],
       'dc.description.abstract': [
@@ -1032,7 +1032,7 @@ export const ItemMockPid9: Item = Object.assign(
       'dc.identifier.uri': [
         {
           language: null,
-          value: 'http://dspace7.4science.it/xmlui/handle/10673/6'
+          value: 'https://demo.dspace.org/handle/10673/6'
         }
       ],
       'dc.description.abstract': [
@@ -1151,7 +1151,7 @@ export const ItemMockPid10: Item = Object.assign(
       'dc.identifier.uri': [
         {
           language: null,
-          value: 'http://dspace7.4science.it/xmlui/handle/10673/6'
+          value: 'https://demo.dspace.org/handle/10673/6'
         }
       ],
       'dc.description.abstract': [
@@ -1270,7 +1270,7 @@ export const NotificationsMockDspaceObject: SearchResult<DSpaceObject> = Object.
       'dc.identifier.uri': [
         {
           language: null,
-          value: 'http://dspace7.4science.it/xmlui/handle/10673/6'
+          value: 'https://demo.dspace.org/handle/10673/6'
         }
       ],
       'dc.description.abstract': [
@@ -1799,7 +1799,7 @@ export const qualityAssuranceEventObjectMissingProjectNotFound: QualityAssurance
 // -------------------------------------------------------------------------------
 
 /**
- * Mock for [[SuggestionNotificationsStateService]]
+ * Mock for [[NotificationsStateService]]
  */
 export function getMockNotificationsStateService(): any {
   return jasmine.createSpyObj('NotificationsStateService', {