[DSC-1940] refactor

Author: FrancescoMolinaro

src/app/audit-page/object-audit-overview/object-audit-overview.component.ts

@@ -85,19 +85,14 @@ export class ObjectAuditOverviewComponent implements OnInit {
       redirectOn4xx(this.router, this.authService)
     ).subscribe((rd) => {
       this.object = rd.payload;
-      this.owningCollection$ = this.collectionDataService.findOwningCollectionFor(this.object).pipe(
+      this.owningCollection$ = this.collectionDataService.findOwningCollectionFor(
+        this.object,
+        true,
+        false,
+        ...COLLECTION_PAGE_LINKS_TO_FOLLOW
+      ).pipe(
         getFirstCompletedRemoteData(),
-        map(data => data.payload),
-        switchMap((collection) => this.collectionDataService.findById(
-            collection.id,
-          false,
-          false,
-          ...COLLECTION_PAGE_LINKS_TO_FOLLOW
-        ).pipe(
-          getFirstCompletedRemoteData(),
-            map(data => data.payload),
-          )
-        )
+        map(data => data?.payload)
       );
       this.setAudits();
     });

src/app/core/data/collection-data.service.ts

@@ -321,9 +321,12 @@ export class CollectionDataService extends ComColDataService<Collection> {
   /**
    * Returns {@link RemoteData} of {@link Collection} that is the owning collection of the given item
    * @param item  Item we want the owning collection of
+   * @param useCachedVersionIfAvailable
+   * @param reRequestOnStale
+   * @param linksToFollow
    */
-  findOwningCollectionFor(item: Item): Observable<RemoteData<Collection>> {
-    return this.findByHref(item._links.owningCollection.href);
+  findOwningCollectionFor(item: Item, useCachedVersionIfAvailable = true, reRequestOnStale = true, ...linksToFollow: FollowLinkConfig<Collection>[]): Observable<RemoteData<Collection>> {
+    return this.findByHref(item._links.owningCollection.href, useCachedVersionIfAvailable, reRequestOnStale, ...linksToFollow);
   }
 
   /**