fix: avoid 3 call to getById on aip detail

src/app/features/preservation-space/contributor/components/routables/contributor-list/contributor-list.routable.ts

@@ -21,9 +21,9 @@ import {RoutesEnum} from "@shared/enums/routes.enum";
 import {RouterExtService} from "@shared/services/router-ext.service";
 import {SecurityService} from "@shared/services/security.service";
 import {
+  MARK_AS_TRANSLATABLE,
   OrderEnum,
   Override,
-  MARK_AS_TRANSLATABLE,
 } from "solidify-frontend";
 
 @Component({

src/app/features/preservation/aip/components/routables/aip-detail-edit/aip-detail-edit.routable.ts

@@ -78,6 +78,9 @@ export class AipDetailEditRoutable extends SharedAbstractDetailEditCommonRoutabl
   isLoadingHistoryObs: Observable<boolean> = MemoizedUtil.isLoading(this._store, PreservationAipStatusHistoryState);
   queryParametersObs: Observable<QueryParameters> = MemoizedUtil.select(this._store, PreservationAipStatusHistoryState, state => state.queryParameters);
 
+  @OverrideProperty()
+  getByIdIfAlreadyInState: boolean = false;
+
   readonly KEY_PARAM_NAME: keyof Aip & string = "archiveId";
 
   tabSelected: AipTabEnum;
@@ -172,12 +175,11 @@ export class AipDetailEditRoutable extends SharedAbstractDetailEditCommonRoutabl
   }
 
   ngOnInit(): void {
-    super.ngOnInit();
     this.mode = AipHelper.determineMode(this._store);
     if (this.mode === AipMode.AIP) {
       this.storagion_number = +this._route.snapshot.parent.paramMap.get(PreservationPlanningRoutesEnum.storagionNumberWithoutPrefixParam);
     }
-    this.retrieveCurrentModelWithUrl();
+    super.ngOnInit();
 
     this._computeCurrentTab();
 

src/app/shared/components/routables/shared-abstract-detail-edit/shared-abstract-detail-edit.routable.ts

@@ -72,6 +72,7 @@ export abstract class SharedAbstractDetailEditRoutable<TResourceModel extends Ba
   isEditObs: Observable<boolean>;
   urlQueryParameters: MappingObject<string | undefined>;
 
+  getByIdIfAlreadyInState: boolean = true;
   readonly editAvailable: boolean = true;
   readonly deleteAvailable: boolean = true;
 
@@ -118,7 +119,9 @@ export abstract class SharedAbstractDetailEditRoutable<TResourceModel extends Ba
 
   protected retrieveCurrentModelWithUrl(): void {
     this.retrieveResIdFromUrl();
-    this.getById(this._resId);
+    if (this.getByIdIfAlreadyInState || isNullOrUndefined(this.current) || this.current?.resId !== this._resId) {
+      this.getById(this._resId);
+    }
   }
 
   protected getById(id: string): void {