Refactor effects to use inject() (#4080)

Alfresco · Oct 15, 2024 · 70ae4ed · 70ae4ed
1 parent 8f54629
commit 70ae4ed
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 14 deletions.
diff --git a/projects/aca-content/src/lib/store/effects/library.effects.ts b/projects/aca-content/src/lib/store/effects/library.effects.ts
@@ -26,15 +26,15 @@ import {
   AppStore,
   CreateLibraryAction,
   DeleteLibraryAction,
+  getAppSelection,
   LeaveLibraryAction,
   LibraryActionTypes,
   NavigateLibraryAction,
   NavigateRouteAction,
-  UpdateLibraryAction,
-  getAppSelection
+  UpdateLibraryAction
 } from '@alfresco/aca-shared/store';
 import { inject, Injectable } from '@angular/core';
-import { Actions, ofType, createEffect } from '@ngrx/effects';
+import { Actions, createEffect, ofType } from '@ngrx/effects';
 import { Store } from '@ngrx/store';
 import { map, mergeMap, take } from 'rxjs/operators';
 import { ContentApiService } from '@alfresco/aca-shared';
@@ -44,13 +44,10 @@ import { NotificationService } from '@alfresco/adf-core';
 @Injectable()
 export class LibraryEffects {
   private notificationService = inject(NotificationService);
-
-  constructor(
-    private store: Store<AppStore>,
-    private actions$: Actions,
-    private content: ContentManagementService,
-    private contentApi: ContentApiService
-  ) {}
+  private store = inject(Store<AppStore>);
+  private actions$ = inject(Actions);
+  private content = inject(ContentManagementService);
+  private contentApi = inject(ContentApiService);
 
   deleteLibrary$ = createEffect(
     () =>

diff --git a/projects/aca-shared/store/src/effects/router.effects.ts b/projects/aca-shared/store/src/effects/router.effects.ts
@@ -24,19 +24,20 @@
 
 import { inject, Injectable } from '@angular/core';
 import { Router } from '@angular/router';
-import { Actions, ofType, createEffect } from '@ngrx/effects';
+import { Actions, createEffect, ofType } from '@ngrx/effects';
 import { Node, PathInfo } from '@alfresco/js-api';
 import { map } from 'rxjs/operators';
 import { Location } from '@angular/common';
-import { NavigateUrlAction, NavigateRouteAction, NavigateToFolder, NavigateToParentFolder, NavigateToPreviousPage } from '../actions/router.actions';
+import { NavigateRouteAction, NavigateToFolder, NavigateToParentFolder, NavigateToPreviousPage, NavigateUrlAction } from '../actions/router.actions';
 import { RouterActionTypes } from '../actions/router-action-types';
 import { NotificationService } from '@alfresco/adf-core';
 
 @Injectable()
 export class RouterEffects {
   private notificationService = inject(NotificationService);
-
-  constructor(private actions$: Actions, private router: Router, private location: Location) {}
+  private actions$ = inject(Actions);
+  private router = inject(Router);
+  private location = inject(Location);
 
   navigateUrl$ = createEffect(
     () =>