From 907a607a96866810535c337fc7bed59b160b6db1 Mon Sep 17 00:00:00 2001
From: DaryaBalvanovich <darya.balvanovich1@hyland.com>
Date: Thu, 6 Jun 2024 09:50:33 +0200
Subject: [PATCH] ACS-8055 clean up code

---
 .../src/lib/common/services/nodes-api.service.ts       | 10 ++++------
 .../legal-hold/services/legal-holds.service.spec.ts    |  2 ++
 lib/js-api/src/api/gs-core-rest-api/docs/Hold.md       |  9 +++++----
 .../src/api/gs-core-rest-api/docs/LegalHoldApi.md      |  8 ++++----
 4 files changed, 15 insertions(+), 14 deletions(-)

diff --git a/lib/content-services/src/lib/common/services/nodes-api.service.ts b/lib/content-services/src/lib/common/services/nodes-api.service.ts
index c927dcddd2..8d5a6a5b67 100644
--- a/lib/content-services/src/lib/common/services/nodes-api.service.ts
+++ b/lib/content-services/src/lib/common/services/nodes-api.service.ts
@@ -248,16 +248,14 @@ export class NodesApiService {
      */
     getNodeAssignHolds(nodeId: string): Observable<Hold[]> {
         const queryOptions = Object.assign({
-            maxItems: this.preferences.paginationSize,
-            skipCount: 0,
             where: `(assocType='rma:frozenContent')`
         });
 
         return from(this.nodesApi.listParents(nodeId, queryOptions)).pipe(
-            map((holds) =>
-                holds.list?.entries?.map((entity) => ({
-                    id: entity.entry.id,
-                    name: entity.entry.name
+            map(({ list }) =>
+                list?.entries?.map(({ entry }) => ({
+                    id: entry?.id,
+                    name: entry?.name
                 }))
             ),
             catchError((err) => throwError(err))
diff --git a/lib/content-services/src/lib/legal-hold/services/legal-holds.service.spec.ts b/lib/content-services/src/lib/legal-hold/services/legal-holds.service.spec.ts
index e31feac06f..45884a36d8 100644
--- a/lib/content-services/src/lib/legal-hold/services/legal-holds.service.spec.ts
+++ b/lib/content-services/src/lib/legal-hold/services/legal-holds.service.spec.ts
@@ -31,6 +31,7 @@ describe('LegalHoldsService', () => {
             imports: [ContentTestingModule]
         });
         service = TestBed.inject(LegalHoldService);
+
         legalHolds = {
             list: {
                 entries: [
@@ -44,6 +45,7 @@ describe('LegalHoldsService', () => {
                 ]
             }
         } as NodeChildAssociationPaging;
+
         returnedHolds = [
             {
                 id: mockId,
diff --git a/lib/js-api/src/api/gs-core-rest-api/docs/Hold.md b/lib/js-api/src/api/gs-core-rest-api/docs/Hold.md
index a0f1eb9099..486885373d 100644
--- a/lib/js-api/src/api/gs-core-rest-api/docs/Hold.md
+++ b/lib/js-api/src/api/gs-core-rest-api/docs/Hold.md
@@ -13,9 +13,10 @@ export interface Hold {
 ```
 
 ## Properties
+
 Name | Type | Default value | Description
 ------------ | ------------- | ------------- | -------------
-**id** | **string** |  | hold id
-**name** | **string** |  | hold name
-**reason** | **string** |  | hold reason
-**description** | **string** |  | [optional] [additional information for a hold]
+**id** | **string** |  | Hold id
+**name** | **string** |  | Hold name
+**reason** | **string** |  | Hold reason
+**description** | **string** |  | [optional] [Additional information for a hold]
diff --git a/lib/js-api/src/api/gs-core-rest-api/docs/LegalHoldApi.md b/lib/js-api/src/api/gs-core-rest-api/docs/LegalHoldApi.md
index 014ace31a5..b7ab47d6ae 100644
--- a/lib/js-api/src/api/gs-core-rest-api/docs/LegalHoldApi.md
+++ b/lib/js-api/src/api/gs-core-rest-api/docs/LegalHoldApi.md
@@ -39,11 +39,11 @@ legalHoldApi.getHolds('-filePlan-', opts).then((data) => {
 
 ### Parameters
 
-Name | Type | Description  | Notes
+Name | Type | Default value  | Description
 ------------- | ------------- | ------------- | -------------
- **filePlanId** | **string** | The site details |
- **skipCount** | **number**| The number of entities that exist in the collection before those included in this list. | [optional] 
- **maxItems** | **number**| The maximum number of items to return in the list. | [optional] 
+ **filePlanId** | **string** | | The site details 
+ **skipCount** | **number**| `0` | The number of entities that exist in the collection before those included in this list. [optional] 
+ **maxItems** | **number**| `100` | The maximum number of items to return in the list. [optional] 
 
 ### Return type