fix: changed filter method to remove content on truthy result content on

amplience · Oct 14, 2024 · 5b2a66f · 5b2a66f
1 parent b66c497
commit 5b2a66f
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/src/lib/ContentClient.spec.ts b/src/lib/ContentClient.spec.ts
@@ -523,7 +523,7 @@ describe('ContentClient', () => {
         rootId: ROOT.content._meta.deliveryId,
       },
       (contentBody) => {
-        return contentBody['propertyName1'] === 'B';
+        return contentBody['propertyName1'] === 'C';
       }
     );
     expect(response).to.deep.eq(expectedContent);
@@ -633,7 +633,7 @@ describe('ContentClient', () => {
         rootId: ROOT.content._meta.deliveryId,
       },
       (contentBody) => {
-        return contentBody['propertyName1'] === 'B';
+        return contentBody['propertyName1'] === 'C';
       },
       (contentBody) => {
         contentBody['mutatedId'] = contentBody._meta.deliveryId;

diff --git a/.../content/coordinators/GetByHierarchy/assemblers/FilteringAndMutatingHierarchyAssembler.ts b/.../content/coordinators/GetByHierarchy/assemblers/FilteringAndMutatingHierarchyAssembler.ts
@@ -21,7 +21,7 @@ export class FilteringAndMutatingHierarchyAssembler<Body extends ContentBody>
         .filter((contentItem) => {
           return (
             isParent(rootItem.content, contentItem.content) &&
-            !this.filterFunction(contentItem.content)
+            this.filterFunction(contentItem.content)
           );
         })
         .map((item) => {

diff --git a/src/lib/content/coordinators/GetByHierarchy/assemblers/FilteringHierachyAssemblerImpl.ts b/src/lib/content/coordinators/GetByHierarchy/assemblers/FilteringHierachyAssemblerImpl.ts
@@ -18,7 +18,7 @@ export class FilteringHierachyAssemblerImpl<Body extends ContentBody>
         .filter((contentItem) => {
           return (
             isParent(rootItem.content, contentItem.content) &&
-            !this.filterFunction(contentItem.content)
+            this.filterFunction(contentItem.content)
           );
         })
         .map((item) => {