chore(headless, atomic, quantic)!: remove deprecated functions/properties

packages/atomic/src/components/search/facets/atomic-category-facet/atomic-category-facet.tsx

@@ -305,7 +305,7 @@ export class AtomicCategoryFacet implements InitializableComponent {
     this.bindings.store.registerFacet('categoryFacets', facetInfo);
     initializePopover(this.host, {
       ...facetInfo,
-      hasValues: () => !!this.facet.state.values.length,
+      hasValues: () => !!this.facet.state.valuesAsTrees.length,
       numberOfActiveValues: () => (this.facetState.hasActiveValues ? 1 : 0),
     });
     this.initializeDependenciesManager();
@@ -344,7 +344,8 @@ export class AtomicCategoryFacet implements InitializableComponent {
     return (
       this.searchStatusState.hasError ||
       !this.facet.state.enabled ||
-      (!this.facet.state.values.length && !this.facet.state.parents.length)
+      (!this.facet.state.selectedValueAncestry.length &&
+        !this.facet.state.valuesAsTrees.length)
     );
   }
 
@@ -381,7 +382,7 @@ export class AtomicCategoryFacet implements InitializableComponent {
   }
 
   private get hasParents() {
-    return !!this.facetState.parents.length;
+    return !!this.facetState.selectedValueAncestry.length;
   }
 
   private initializeDependenciesManager() {
@@ -447,7 +448,10 @@ export class AtomicCategoryFacet implements InitializableComponent {
     );
   }
 
-  private renderValuesTree(parents: CategoryFacetValue[], isRoot: boolean) {
+  private renderValuesTree(
+    valuesAsTrees: CategoryFacetValue[],
+    isRoot: boolean
+  ) {
     if (!this.hasParents) {
       return this.renderChildren();
     }
@@ -463,14 +467,14 @@ export class AtomicCategoryFacet implements InitializableComponent {
             }}
           />
           <CategoryFacetParentAsTreeContainer isTopLevel={false}>
-            {this.renderValuesTree(parents, false)}
+            {this.renderValuesTree(valuesAsTrees, false)}
           </CategoryFacetParentAsTreeContainer>
         </CategoryFacetTreeValueContainer>
       );
     }
 
-    if (parents.length > 1) {
-      const parentValue = parents[0];
+    if (valuesAsTrees.length > 1) {
+      const parentValue = valuesAsTrees[0];
 
       return (
         <CategoryFacetTreeValueContainer>
@@ -484,13 +488,13 @@ export class AtomicCategoryFacet implements InitializableComponent {
             }}
           />
           <CategoryFacetParentAsTreeContainer isTopLevel={false}>
-            {this.renderValuesTree(parents.slice(1), false)}
+            {this.renderValuesTree(valuesAsTrees.slice(1), false)}
           </CategoryFacetParentAsTreeContainer>
         </CategoryFacetTreeValueContainer>
       );
     }
 
-    const activeParent = parents[0];
+    const activeParent = valuesAsTrees[0];
     const activeParentDisplayValue = getFieldValueCaption(
       this.field,
       activeParent.value,
@@ -551,11 +555,22 @@ export class AtomicCategoryFacet implements InitializableComponent {
   }
 
   private renderChildren() {
-    if (!this.facetState.values.length) {
+    if (!this.facetState.valuesAsTrees.length) {
       return;
     }
+    if (this.facetState.selectedValueAncestry.length > 0) {
+      return this.facetState.selectedValueAncestry
+        .find((value) => value.state === 'selected')
+        ?.children.map((value, i) =>
+          this.renderChild(
+            value,
+            i === 0,
+            i === this.resultIndexToFocusOnShowMore
+          )
+        );
+    }
 
-    return this.facetState.values.map((value, i) =>
+    return this.facetState.valuesAsTrees.map((value, i) =>
       this.renderChild(value, i === 0, i === this.resultIndexToFocusOnShowMore)
     );
   }
@@ -598,7 +613,8 @@ export class AtomicCategoryFacet implements InitializableComponent {
           label={this.label}
           i18n={this.bindings.i18n}
           onShowMore={() => {
-            this.resultIndexToFocusOnShowMore = this.facetState.values.length;
+            this.resultIndexToFocusOnShowMore =
+              this.facetState.valuesAsTrees[0].children.length;
             this.focusTargets.showMoreFocus.focusAfterSearch();
             this.facet.showMoreValues();
           }}
@@ -627,7 +643,7 @@ export class AtomicCategoryFacet implements InitializableComponent {
     const {
       bindings: {i18n},
       label,
-      facetState: {facetSearch, enabled, valuesAsTrees, parents},
+      facetState: {facetSearch, enabled, valuesAsTrees, selectedValueAncestry},
       searchStatusState: {hasError, firstSearchExecuted},
     } = this;
 
@@ -666,7 +682,7 @@ export class AtomicCategoryFacet implements InitializableComponent {
                         isTopLevel={true}
                         className="mt-3"
                       >
-                        {this.renderValuesTree(parents, true)}
+                        {this.renderValuesTree(selectedValueAncestry, true)}
                       </CategoryFacetParentAsTreeContainer>
                     ) : (
                       <CategoryFacetChildrenAsTreeContainer className="mt-3">

packages/headless/src/app/logger.ts

@@ -1,4 +1,4 @@
-import pino, {LevelWithSilent, LogEvent} from 'pino';
+import pino, {LevelWithSilent} from 'pino';
 
 export type LogLevel = LevelWithSilent;
 
@@ -12,12 +12,6 @@ export interface LoggerOptions {
    * All arguments passed to the log method, except the message, will be pass to this function. By default it does not change the shape of the log object.
    */
   logFormatter?: (object: {}) => {};
-  /**
-   * Function which will be called after writing the log message in the browser.
-   *
-   * @deprecated This option is deprecated and will be removed in a future version.
-   */
-  browserPostLogHook?: (level: LogLevel, logEvent: LogEvent) => void;
 }
 
 export function buildLogger(options: LoggerOptions | undefined) {

packages/headless/src/controllers/core/facets/category-facet/headless-core-category-facet.test.ts

@@ -152,16 +152,10 @@ describe('category facet', () => {
   });
 
   describe('when the search response is empty', () => {
-    it('#state.values is an empty array', () => {
-      expect(state.search.response.facets).toEqual([]);
-      expect(categoryFacet.state.values).toEqual([]);
+    it('#state.selectedValueAncestry is an empty array', () => {
+      expect(categoryFacet.state.selectedValueAncestry).toEqual([]);
     });
-
-    it('#state.parents is an empty array', () => {
-      expect(categoryFacet.state.parents).toEqual([]);
-    });
-
-    it('#state.valuesAsTrees', () => {
+    it('#state.valuesAsTrees is an empty array', () => {
       expect(categoryFacet.state.valuesAsTrees).toEqual([]);
     });
   });
@@ -174,10 +168,6 @@ describe('category facet', () => {
       state.search.response.facets = [response];
     });
 
-    it('#state.values contains the same values', () => {
-      expect(categoryFacet.state.values).toBe(values);
-    });
-
     it('#state.valuesAsTrees contains the same values', () => {
       expect(categoryFacet.state.valuesAsTrees).toBe(values);
     });
@@ -212,18 +202,6 @@ describe('category facet', () => {
       state.search.response.facets = [response];
     });
 
-    it('#state.parents contains the outer and middle values', () => {
-      expect(categoryFacet.state.parents).toEqual([outerValue, middleValue]);
-    });
-
-    it('#state.values contains the innermost values', () => {
-      expect(categoryFacet.state.values).toBe(innerValues);
-    });
-
-    it('#state.parents contains the outer and middle values', () => {
-      expect(categoryFacet.state.parents).toEqual([outerValue, middleValue]);
-    });
-
     it('#state.valueAsTree contains the outer value', () => {
       expect(categoryFacet.state.valuesAsTrees).toEqual([outerValue]);
     });
@@ -283,18 +261,14 @@ describe('category facet', () => {
       state.search.response.facets = [response];
     });
 
-    it('#state.parents contains the selected leaf value', () => {
-      expect(categoryFacet.state.parents).toEqual([selectedValue]);
-    });
-
     it('#state.selectedValueAncestry contains the selected leaf value', () => {
       expect(categoryFacet.state.selectedValueAncestry).toEqual([
         selectedValue,
       ]);
     });
 
-    it('#state.values is an empty array', () => {
-      expect(categoryFacet.state.values).toEqual([]);
+    it('#state.activeValue.children an empty array', () => {
+      expect(categoryFacet.state.activeValue?.children).toEqual([]);
     });
 
     it('#state.activeValue is the selected leaf value', () => {

packages/headless/src/controllers/core/facets/category-facet/headless-core-category-facet.ts

@@ -164,19 +164,6 @@ export interface CoreCategoryFacetState {
    */
   isHierarchical: boolean;
 
-  /**
-   * The facet's parent values.
-   * @deprecated uses `valuesAsTrees` instead.
-   *
-   */
-  parents: CategoryFacetValue[];
-
-  /**
-   * The facet's values.
-   * @deprecated use `selectedValueAncestry` instead.
-   */
-  values: CategoryFacetValue[];
-
   /**
    * The selected facet values ancestry.
    * The first element is the "root" of the selected value ancestry tree.

packages/headless/src/controllers/facets/category-facet/headless-category-facet.test.ts

@@ -119,13 +119,10 @@ describe('category facet', () => {
   });
 
   describe('when the search response is empty', () => {
-    it('#state.values is an empty array', () => {
+    it('#state.valuesAsTrees and #state.selectedValueAncestry are empty arrays', () => {
       expect(state.search.response.facets).toEqual([]);
-      expect(categoryFacet.state.values).toEqual([]);
-    });
-
-    it('#state.parents is an empty array', () => {
-      expect(categoryFacet.state.parents).toEqual([]);
+      expect(categoryFacet.state.valuesAsTrees).toEqual([]);
+      expect(categoryFacet.state.selectedValueAncestry).toEqual([]);
     });
   });
 
@@ -135,7 +132,7 @@ describe('category facet', () => {
     const response = buildMockCategoryFacetResponse({facetId, values});
 
     state.search.response.facets = [response];
-    expect(categoryFacet.state.values).toBe(values);
+    expect(categoryFacet.state.valuesAsTrees).toBe(values);
   });
 
   describe('when the search response has a category facet with nested values', () => {
@@ -160,16 +157,12 @@ describe('category facet', () => {
       state.search.response.facets = [response];
     });
 
-    it('#state.parents contains the outer and middle values', () => {
-      expect(categoryFacet.state.parents).toEqual([outerValue, middleValue]);
-    });
-
-    it('#state.values contains the innermost values', () => {
-      expect(categoryFacet.state.values).toBe(innerValues);
+    it('#state.valueAsTree contains the outer value', () => {
+      expect(categoryFacet.state.valuesAsTrees).toEqual([outerValue]);
     });
 
-    it('#state.parents contains the outer and middle values', () => {
-      expect(categoryFacet.state.parents).toEqual([outerValue, middleValue]);
+    it('#state.isHierarchical should be true', () => {
+      expect(categoryFacet.state.isHierarchical).toBe(true);
     });
   });
 
@@ -188,12 +181,14 @@ describe('category facet', () => {
       state.search.response.facets = [response];
     });
 
-    it('#state.parents contains the selected leaf value', () => {
-      expect(categoryFacet.state.parents).toEqual([selectedValue]);
+    it('#state.valuesAsTrees contains the selected leaf value', () => {
+      expect(categoryFacet.state.valuesAsTrees).toEqual([selectedValue]);
     });
 
-    it('#state.values is an empty array', () => {
-      expect(categoryFacet.state.values).toEqual([]);
+    it('#state.selectedValueAncestry contains the selected leaf value', () => {
+      expect(categoryFacet.state.selectedValueAncestry).toEqual([
+        selectedValue,
+      ]);
     });
   });
 

packages/headless/src/controllers/insight/quickview/headless-insight-quickview.ts

@@ -28,12 +28,6 @@ export interface QuickviewOptions {
    * The maximum preview size to retrieve, in bytes. By default, the full preview is retrieved.
    */
   maximumPreviewSize?: number;
-  /**
-   * Whether to only update the `contentURL` attribute when using `fetchResultContent` rather than updating `content`.
-   * Use this if you are using an iframe with `state.contentURL` as the source url.
-   * @deprecated This option is always set to `true` ad the Insight Quickview only supports `contentURL` mode.
-   */
-  onlyContentURL?: boolean;
 }
 
 export interface Quickview extends Controller {
@@ -49,14 +43,6 @@ export interface Quickview extends Controller {
 }
 
 export interface QuickviewState {
-  /**
-   * The result preview HTML content.
-   *
-   * @default ""
-   * @deprecated This value will always be empty as the InsightQuickview only supports usage of the `contentURL`.
-   */
-  content: string;
-
   /**
    * `true` if the configured result has a preview, and `false` otherwise.
    */

packages/headless/src/controllers/product-listing/category-facet/headless-product-listing-category-facet.test.ts

@@ -50,13 +50,13 @@ describe('category facet', () => {
   });
 
   describe('when the product listing response is empty', () => {
-    it('#state.values is an empty array', () => {
+    it('#state.valuesAsTrees is an empty array', () => {
       expect(state.productListing.facets.results).toEqual([]);
-      expect(categoryFacet.state.values).toEqual([]);
+      expect(categoryFacet.state.valuesAsTrees).toEqual([]);
     });
 
-    it('#state.parents is an empty array', () => {
-      expect(categoryFacet.state.parents).toEqual([]);
+    it('#state.selectedValueAncestry is an empty array', () => {
+      expect(categoryFacet.state.selectedValueAncestry).toEqual([]);
     });
   });
 
@@ -82,16 +82,12 @@ describe('category facet', () => {
       state.productListing.facets.results = [response];
     });
 
-    it('#state.parents contains the outer and middle values', () => {
-      expect(categoryFacet.state.parents).toEqual([outerValue, middleValue]);
+    it('#state.valueAsTree contains the outer value', () => {
+      expect(categoryFacet.state.valuesAsTrees).toEqual([outerValue]);
     });
 
-    it('#state.values contains the innermost values', () => {
-      expect(categoryFacet.state.values).toBe(innerValues);
-    });
-
-    it('#state.parents contains the outer and middle values', () => {
-      expect(categoryFacet.state.parents).toEqual([outerValue, middleValue]);
+    it('#state.isHierarchical should be true', () => {
+      expect(categoryFacet.state.isHierarchical).toBe(true);
     });
   });
 
@@ -110,12 +106,14 @@ describe('category facet', () => {
       state.productListing.facets.results = [response];
     });
 
-    it('#state.parents contains the selected leaf value', () => {
-      expect(categoryFacet.state.parents).toEqual([selectedValue]);
+    it('#state.valuesAsTrees contains the selected leaf value', () => {
+      expect(categoryFacet.state.valuesAsTrees).toEqual([selectedValue]);
     });
 
-    it('#state.values is an empty array', () => {
-      expect(categoryFacet.state.values).toEqual([]);
+    it('#state.selectedValueAncestry contains the selected leaf value', () => {
+      expect(categoryFacet.state.selectedValueAncestry).toEqual([
+        selectedValue,
+      ]);
     });
   });