#10489: Fix 10438: Problems with GeoStory map configurations merge pr…

…ocess (#10516)
geosolutions-it · Sep 5, 2024 · d9f5390 · d9f5390
1 parent 5c52799
commit d9f5390
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 3 deletions.
diff --git a/web/client/components/geostory/common/MapEditor.jsx b/web/client/components/geostory/common/MapEditor.jsx
@@ -31,6 +31,7 @@ import Message from '../../I18N/Message';
 
 import MapConfiguratorTabs from './map/MapConfiguratorTabs';
 import withMapConfiguratorTabs from './enhancers/withMapConfiguratorTabs';
+import set from 'lodash/fp/set';
 
 
 const StepHeader = ({title, description}) => (
@@ -102,7 +103,15 @@ const MapEditor = ({
                         closeNodeEditor={closeNodeEditor}
                         editNode={editNode}
                         map={map}
-                        onChange={onChange}/>
+                        onChange={(path, value) => {
+                            const config = set(path, value, { map });
+                            const { layers, groups } = config?.map || {};
+                            if (path.includes('map.layers[')) {
+                                onChangeMap('layers', layers, "replace");
+                            } else {
+                                onChangeMap('groups', groups, "replace");
+                            }
+                        }}/>
                 ] ||
                 [
                     <MapConfigurator

diff --git a/web/client/components/geostory/common/enhancers/map.jsx b/web/client/components/geostory/common/enhancers/map.jsx
@@ -60,8 +60,8 @@ export const withFocusedContentMap = compose(
  */
 export const handleMapUpdate = withHandlers({
     onChangeMap: ({update, focusedContent = {}}) =>
-        (path, value) => {
-            update(`${focusedContent.path}.map.${path}`, value, "merge");
+        (path, value, mode = "merge") => {
+            update(`${focusedContent.path}.map.${path}`, value, mode);
         },
     onChange: ({update, focusedContent = {}}) =>
         (path, value, mode = 'merge') => {

diff --git a/web/client/utils/__tests__/GeoStoryUtils-test.js b/web/client/utils/__tests__/GeoStoryUtils-test.js
@@ -823,3 +823,4 @@ describe("GeoStory Utils", () => {
         });
     });
 });
+