@trackunit/filters-filter-bar 0.0.422 → 0.0.423

package/index.cjs.js

@@ -1006,13 +1006,14 @@ const useFilterBar = ({ name, onValuesChange, filterBarDefinition, initialState,
             };
         });
     }, []);
+    const filterDefinitionsValues = react.useMemo(() => sharedUtils.objectValues(filterBarDefinition), [filterBarDefinition]);
     const [filterBarConfig, setFilterBarConfig] = react.useState(() => {
         let initialFilterBarConfig;
         const storedFilters = localStorage.getItem(`filter-${name}`);
         if (storedFilters && storedFilters !== "undefined") {
             const loadedFilterBarConfig = JSON.parse(storedFilters);
-            if (validateFilter(loadedFilterBarConfig, sharedUtils.objectValues(filterBarDefinition))) {
-                initialFilterBarConfig = loadedFilterBarConfig;
+            if (validateFilter(loadedFilterBarConfig, filterDefinitionsValues)) {
+                initialFilterBarConfig = createInitialState(loadedFilterBarConfig.name, filterDefinitionsValues, initialState || loadedFilterBarConfig.initialState, setValue);
             }
         }
         //WHY WE NEED THIS?
@@ -1020,7 +1021,7 @@ const useFilterBar = ({ name, onValuesChange, filterBarDefinition, initialState,
         //To do this for a changing default value as in customers and sites we would need to recreate the initialFilterBarConfig every time the default value changes.
         //This mean that filterbars that have this functionality wouldn't be able to save the state of the filterbar.
         //Another option would be to create a new filterbar for each customer or site. This also has its drawbacks. Would raise it with the frontend community.
-        const hasNonVisibleDefaultValues = sharedUtils.objectValues(filterBarDefinition).some(value => {
+        const hasNonVisibleDefaultValues = filterDefinitionsValues.some(value => {
             var _a;
             return value.showInStarredMenu &&
                 !value.showInStarredMenu() &&
@@ -1030,7 +1031,7 @@ const useFilterBar = ({ name, onValuesChange, filterBarDefinition, initialState,
                 value.defaultValue.toString().length > 0;
         });
         if (initialFilterBarConfig === undefined || hasNonVisibleDefaultValues) {
-            initialFilterBarConfig = createInitialState(name, sharedUtils.objectValues(filterBarDefinition), initialState || {}, setValue);
+            initialFilterBarConfig = createInitialState(name, filterDefinitionsValues, initialState || {}, setValue);
         }
         // eslint-disable-next-line no-autofix/local-rules/prefer-custom-object-keys
         Object.keys(initialFilterBarConfig.values).forEach(key => {
@@ -1092,7 +1093,7 @@ const useFilterBar = ({ name, onValuesChange, filterBarDefinition, initialState,
     const filterMapActions = react.useMemo(() => {
         // Reset an individual filter to its initial state
         const resetIndividualFilterToInitialState = (key) => {
-            const tmpInitialState = createInitialState(name, sharedUtils.objectValues(filterBarDefinition), initialState || {}, setValue);
+            const tmpInitialState = createInitialState(name, filterDefinitionsValues, initialState || {}, setValue);
             setFilterBarConfig(prevState => {
                 return {
                     ...prevState,
@@ -1294,13 +1295,13 @@ const useFilterBar = ({ name, onValuesChange, filterBarDefinition, initialState,
                 setFilterBarConfig(prevState => {
                     return {
                         ...prevState,
-                        values: createInitialState(name, sharedUtils.objectValues(filterBarDefinition), initialState || {}, setValue).values,
+                        values: createInitialState(name, filterDefinitionsValues, initialState || {}, setValue).values,
                     };
                 });
             },
             resetIndividualFilterToInitialState,
         };
-    }, [filterBarDefinition, initialState, name, setValue]);
+    }, [filterDefinitionsValues, initialState, name, setValue]);
     return react.useMemo(() => {
         return {
             filterBarConfig: { ...filterBarConfig, ...filterMapActions, ...filterMapGetter },

package/index.esm.js

@@ -1004,13 +1004,14 @@ const useFilterBar = ({ name, onValuesChange, filterBarDefinition, initialState,
             };
         });
     }, []);
+    const filterDefinitionsValues = useMemo(() => objectValues(filterBarDefinition), [filterBarDefinition]);
     const [filterBarConfig, setFilterBarConfig] = useState(() => {
         let initialFilterBarConfig;
         const storedFilters = localStorage.getItem(`filter-${name}`);
         if (storedFilters && storedFilters !== "undefined") {
             const loadedFilterBarConfig = JSON.parse(storedFilters);
-            if (validateFilter(loadedFilterBarConfig, objectValues(filterBarDefinition))) {
-                initialFilterBarConfig = loadedFilterBarConfig;
+            if (validateFilter(loadedFilterBarConfig, filterDefinitionsValues)) {
+                initialFilterBarConfig = createInitialState(loadedFilterBarConfig.name, filterDefinitionsValues, initialState || loadedFilterBarConfig.initialState, setValue);
             }
         }
         //WHY WE NEED THIS?
@@ -1018,7 +1019,7 @@ const useFilterBar = ({ name, onValuesChange, filterBarDefinition, initialState,
         //To do this for a changing default value as in customers and sites we would need to recreate the initialFilterBarConfig every time the default value changes.
         //This mean that filterbars that have this functionality wouldn't be able to save the state of the filterbar.
         //Another option would be to create a new filterbar for each customer or site. This also has its drawbacks. Would raise it with the frontend community.
-        const hasNonVisibleDefaultValues = objectValues(filterBarDefinition).some(value => {
+        const hasNonVisibleDefaultValues = filterDefinitionsValues.some(value => {
             var _a;
             return value.showInStarredMenu &&
                 !value.showInStarredMenu() &&
@@ -1028,7 +1029,7 @@ const useFilterBar = ({ name, onValuesChange, filterBarDefinition, initialState,
                 value.defaultValue.toString().length > 0;
         });
         if (initialFilterBarConfig === undefined || hasNonVisibleDefaultValues) {
-            initialFilterBarConfig = createInitialState(name, objectValues(filterBarDefinition), initialState || {}, setValue);
+            initialFilterBarConfig = createInitialState(name, filterDefinitionsValues, initialState || {}, setValue);
         }
         // eslint-disable-next-line no-autofix/local-rules/prefer-custom-object-keys
         Object.keys(initialFilterBarConfig.values).forEach(key => {
@@ -1090,7 +1091,7 @@ const useFilterBar = ({ name, onValuesChange, filterBarDefinition, initialState,
     const filterMapActions = useMemo(() => {
         // Reset an individual filter to its initial state
         const resetIndividualFilterToInitialState = (key) => {
-            const tmpInitialState = createInitialState(name, objectValues(filterBarDefinition), initialState || {}, setValue);
+            const tmpInitialState = createInitialState(name, filterDefinitionsValues, initialState || {}, setValue);
             setFilterBarConfig(prevState => {
                 return {
                     ...prevState,
@@ -1292,13 +1293,13 @@ const useFilterBar = ({ name, onValuesChange, filterBarDefinition, initialState,
                 setFilterBarConfig(prevState => {
                     return {
                         ...prevState,
-                        values: createInitialState(name, objectValues(filterBarDefinition), initialState || {}, setValue).values,
+                        values: createInitialState(name, filterDefinitionsValues, initialState || {}, setValue).values,
                     };
                 });
             },
             resetIndividualFilterToInitialState,
         };
-    }, [filterBarDefinition, initialState, name, setValue]);
+    }, [filterDefinitionsValues, initialState, name, setValue]);
     return useMemo(() => {
         return {
             filterBarConfig: { ...filterBarConfig, ...filterMapActions, ...filterMapGetter },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@trackunit/filters-filter-bar",
-  "version": "0.0.422",
+  "version": "0.0.423",
   "repository": "https://github.com/Trackunit/manager",
   "license": "SEE LICENSE IN LICENSE.txt",
   "engines": {