@bytebrand/fe-ui-core 4.8.112 → 4.8.113

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bytebrand/fe-ui-core",
-  "version": "4.8.112",
+  "version": "4.8.113",
   "description": "UI components for the auto.de project",
   "main": "index.ts",
   "module": "dist/common.js",

package/source/components/VehicleDetailedSidebar/VehicleDetailedSidebar.tsx

@@ -1,4 +1,4 @@
-import React, { useState, useEffect, useRef, FunctionComponent } from 'react';
+import React, { useState, useEffect, FunctionComponent } from 'react';
 import _get from 'lodash/get';
 import { Hidden } from 'react-grid-system';
 import PriceContent from './partials/PriceContent';
@@ -104,31 +104,22 @@ const VehicleDetailedSidebar: FunctionComponent<IVehicleDetailedSidebarProps> =
     }
   ];
 
-  const initialDefaultAppliedForPath = useRef<string | null>(null);
-
+  // ADV-116: the default CDP tab is driven solely by `activeTabCDP` (set to Financing by the
+  // originating card, or to the exact tab when a payment type is clicked directly). No
+  // leasing-based default override.
   useEffect(() => {
     const getlocalStorageActiveTabCDP = localStorage.getItem('activeTabCDP');
-    if (getlocalStorageActiveTabCDP) {
-      const activeTab = tabsTitles.findIndex(({ title }: { title: string }) => {
-        return title === getlocalStorageActiveTabCDP;
-      });
-      if (activeTab === -1) {
-        setActiveTab(null, 0);
-      } else {
-        setActiveTab(null, activeTab);
-      }
-      localStorage.removeItem('activeTabCDP');
-      initialDefaultAppliedForPath.current = pathname;
-      return;
-    }
-    if (
-      initialDefaultAppliedForPath.current !== pathname
-      && _get(price, 'leasing.isActive', false)
-    ) {
-      setActiveTab(null, 1);
-      initialDefaultAppliedForPath.current = pathname;
+    if (!getlocalStorageActiveTabCDP) return;
+    const activeTab = tabsTitles.findIndex(({ title }: { title: string }) => {
+      return title === getlocalStorageActiveTabCDP;
+    });
+    if (activeTab === -1) {
+      setActiveTab(null, 0);
+    } else {
+      setActiveTab(null, activeTab);
     }
-  }, [pathname, activeTab, price]); // tslint:disable-line:align
+    localStorage.removeItem('activeTabCDP');
+  }, [pathname, activeTab]); // tslint:disable-line:align
 
   const priceProps = {
     tabsTitles,

package/source/components/VehicleSmallCard/VehicleData/VehicleInfo/VehicleInfo.tsx

@@ -1,7 +1,6 @@
 /* tslint:disable */
 import * as React from 'react';
 import classnames from 'classnames';
-import _get from 'lodash/get';
 import styles from './VehicleInfo.styl';
 import PriceRating from '../../../PriceRating/PriceRating';
 import { Visible, Hidden } from 'react-grid-system';
@@ -45,7 +44,6 @@ const VehicleInfo: React.FC<IVehicleInfoProps> = (props) => {
     location,
     offer,
     engineData,
-    leasing,
     suppressConsumption
   } = props as (typeof props & { suppressConsumption?: boolean });
   const firstRegistrationDate = regDate && regDate !== 0 ? regDate : 'N/A';
@@ -168,12 +166,8 @@ const VehicleInfo: React.FC<IVehicleInfoProps> = (props) => {
       ),
         ...routeObj,
         className: wrapClasses,
-        onClick: () => localStorage.setItem(
-          'activeTabCDP',
-          _get(leasing, 'isActive', false)
-            ? t('vehicleProps:title.leasing')
-            : t('vehicleProps:title.financing')
-        ),
+        // ADV-116: opening a CDP from a card defaults to the Financing tab.
+        onClick: () => localStorage.setItem('activeTabCDP', t('vehicleProps:title.financing')),
         href: url,
         target,
         rel

package/source/components/VehicleSmallCard/VehicleSmallCard.tsx

@@ -318,12 +318,8 @@ const VehicleSmallCard: FunctionComponent<IVehicleSmallCardProps> = (props) => {
           target,
           rel,
           className: `${styles.redirectClassLink} ${styles.gridAreaTitle}`,
-          onClick: () => localStorage.setItem(
-            'activeTabCDP',
-            _get(price, 'leasing.isActive', false)
-              ? t('vehicleProps:title.leasing')
-              : t('vehicleProps:title.financing')
-          ),
+          // ADV-116: opening a CDP from a card defaults to the Financing tab.
+          onClick: () => localStorage.setItem('activeTabCDP', t('vehicleProps:title.financing')),
           href: url
         })}
 
@@ -376,12 +372,8 @@ const VehicleSmallCard: FunctionComponent<IVehicleSmallCardProps> = (props) => {
           target,
           rel,
           className: `${styles.redirectClassLink} ${styles.gridAreaImage}`,
-          onClick: () => localStorage.setItem(
-            'activeTabCDP',
-            _get(price, 'leasing.isActive', false)
-              ? t('vehicleProps:title.leasing')
-              : t('vehicleProps:title.financing')
-          ),
+          // ADV-116: opening a CDP from a card defaults to the Financing tab.
+          onClick: () => localStorage.setItem('activeTabCDP', t('vehicleProps:title.financing')),
           href: url
         })}
         <VehicleInfo {...vehicleInfoProps} />