@eeacms/volto-eea-map 3.0.3 → 3.2.0

package/CHANGELOG.md

@@ -4,6 +4,28 @@ All notable changes to this project will be documented in this file. Dates are d
 
 Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 
+### [3.2.0](https://github.com/eea/volto-eea-map/compare/3.1.0...3.2.0) - 13 December 2023
+
+#### :rocket: New Features
+
+- feat: use data_query from connected_data_parameters defined in volto-datablocks [Miu Razvan - [`e755c7d`](https://github.com/eea/volto-eea-map/commit/e755c7d3f3076d751cbace7ac69c19d7fcf37ba9)]
+
+#### :bug: Bug Fixes
+
+- fix: Invalid DOM property class. Did you mean className [Alin Voinea - [`2c02e99`](https://github.com/eea/volto-eea-map/commit/2c02e99a2d9bb4bf456710d7045af0a8842de952)]
+
+#### :hammer_and_wrench: Others
+
+- merge develop [andreiggr - [`eba7e68`](https://github.com/eea/volto-eea-map/commit/eba7e6883481bd863d659f7904fb1bc8023689f8)]
+- updates on data_query_parameters [andreiggr - [`06744e4`](https://github.com/eea/volto-eea-map/commit/06744e460398b72a9a6abd065d5ba76bf729588d)]
+- Release 3.2.0 [Alin Voinea - [`deae811`](https://github.com/eea/volto-eea-map/commit/deae811cf402d59ec8d7408c6b9278bc480b07df)]
+- clean up [Miu Razvan - [`21edc9e`](https://github.com/eea/volto-eea-map/commit/21edc9eb5c38d063420743be1e50e376f81cb620)]
+- get unsaved queries from connected data params [andreiggr - [`6833dcd`](https://github.com/eea/volto-eea-map/commit/6833dcdb78404704776aff17439d9058e6d4fd39)]
+### [3.1.0](https://github.com/eea/volto-eea-map/compare/3.0.3...3.1.0) - 8 December 2023
+
+#### :hammer_and_wrench: Others
+
+- Release 3.1.0 [Alin Voinea - [`2fad87e`](https://github.com/eea/volto-eea-map/commit/2fad87e5a8013b80183797df45ba6494e7ca396d)]
 ### [3.0.3](https://github.com/eea/volto-eea-map/compare/3.0.2...3.0.3) - 29 November 2023
 
 #### :hammer_and_wrench: Others
@@ -31,8 +53,6 @@ Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 - fix prettier [Dobricean Ioan Dorian - [`78fc2ba`](https://github.com/eea/volto-eea-map/commit/78fc2baa2f5264ca1bba61b70bc0bdc5b2788e67)]
 - Fix Sonorqube [dobri1408 - [`e04011a`](https://github.com/eea/volto-eea-map/commit/e04011aa811422f865d0bbb1e66af558a4310e77)]
 - adapt embedeeamap to use serialized data [Dobricean Ioan Dorian - [`005965c`](https://github.com/eea/volto-eea-map/commit/005965c6793a9a9977fef52c08ba9a3173eec348)]
-- test: [JENKINS] Use java17 for sonarqube scanner [valentinab25 - [`de2f4f6`](https://github.com/eea/volto-eea-map/commit/de2f4f6f516c2c9f7c2b4ea84373f4b1a21cd0af)]
-- test: [JENKINS] Run cypress in started frontend container [valentinab25 - [`4722984`](https://github.com/eea/volto-eea-map/commit/47229842132b406101c340ab200c193925b784ad)]
 ### [2.0.7](https://github.com/eea/volto-eea-map/compare/2.0.6...2.0.7) - 10 November 2023
 
 #### :rocket: New Features
@@ -49,7 +69,6 @@ Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 
 #### :house: Internal changes
 
-- chore: [JENKINS] Refactor automated testing [valentinab25 - [`7afd81a`](https://github.com/eea/volto-eea-map/commit/7afd81a76bf407b07a94228daf3ab0415fbe4621)]
 - chore: husky, lint-staged use fixed versions [valentinab25 - [`0ee38a7`](https://github.com/eea/volto-eea-map/commit/0ee38a7bee84694a5e0230e2d17083fbc071f0c8)]
 - chore:volto 16 in tests, update docs, fix stylelint overrides [valentinab25 - [`8efeb44`](https://github.com/eea/volto-eea-map/commit/8efeb4423da70e421a88cabdeb559eb3613fb96e)]
 - chore: display legend for content type item, fix case when no note is set [dana-cfc4 - [`ae630a8`](https://github.com/eea/volto-eea-map/commit/ae630a88cffa2a8ef60e0bbfead392508b015497)]
@@ -59,10 +78,7 @@ Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 
 #### :hammer_and_wrench: Others
 
-- test: [JENKINS] Add cpu limit on cypress docker [valentinab25 - [`335133d`](https://github.com/eea/volto-eea-map/commit/335133dc5186a53c1077bbead36100101bd9af45)]
 - Refs #260203: make background_image required [laszlocseh - [`baa5c4f`](https://github.com/eea/volto-eea-map/commit/baa5c4fa2fe618af51df12d48b4512ecafc357a8)]
-- test: [JENKINS] Increase shm-size to cypress docker [valentinab25 - [`4874649`](https://github.com/eea/volto-eea-map/commit/48746492a2b32a82d33d1c6415eab8a6f6b9f5be)]
-- test: [JENKINS] Improve cypress time [valentinab25 - [`7fa0b83`](https://github.com/eea/volto-eea-map/commit/7fa0b837b24f41a01285fd1b17daeefde99a9c76)]
 - use toolbar from volto-embed [Miu Razvan - [`29b73fb`](https://github.com/eea/volto-eea-map/commit/29b73fb8378ca56b3634bf1f7d57ce69a242b18e)]
 ### [2.0.6](https://github.com/eea/volto-eea-map/compare/2.0.5...2.0.6) - 29 September 2023
 
@@ -79,7 +95,6 @@ Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 
 - test: EN locales, pre-commit fix, feature PRs checks Refs #257193 [valentinab25 - [`4c6d141`](https://github.com/eea/volto-eea-map/commit/4c6d1413b67ffc3fe27a7d09dcc99ef98867cec9)]
 - test: Fix package.json scripts to use makefile [Alin Voinea - [`1cd6751`](https://github.com/eea/volto-eea-map/commit/1cd675118165d8631c41d35c00a8c902c7edd803)]
-- test: Fix eslint and yarn i18n [Alin Voinea - [`0184641`](https://github.com/eea/volto-eea-map/commit/018464116ba5b86b2c1e0a4ebc9b2fd95b772b66)]
 - i18n: Add en [Alin Voinea - [`1d29304`](https://github.com/eea/volto-eea-map/commit/1d293049a21e138fa933cd5ed9a04cceac1a5cff)]
 ### [2.0.5](https://github.com/eea/volto-eea-map/compare/2.0.4...2.0.5) - 23 August 2023
 
@@ -136,14 +151,11 @@ Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 #### :hammer_and_wrench: Others
 
 - Release 2.0.0 [Alin Voinea - [`7be280d`](https://github.com/eea/volto-eea-map/commit/7be280d4809186e663f422687aca3c33dd732904)]
-- Add Sonarqube tag using eea-website-frontend addons list [EEA Jenkins - [`ba6ed22`](https://github.com/eea/volto-eea-map/commit/ba6ed22118406db979457384d4739cced2096137)]
 ### [1.0.2](https://github.com/eea/volto-eea-map/compare/1.0.1...1.0.2) - 21 July 2023
 
 #### :hammer_and_wrench: Others
 
 - Allow blocks in content type [Tiberiu Ichim - [`2ac09a6`](https://github.com/eea/volto-eea-map/commit/2ac09a613aba07f6cbaf42c386660f2d7d09bd13)]
-- Add Sonarqube tag using eea-website-frontend addons list [EEA Jenkins - [`403b4c0`](https://github.com/eea/volto-eea-map/commit/403b4c009e5aaded58203bc292ba3baa4631488b)]
-- Add Sonarqube tag using eea-website-frontend addons list [EEA Jenkins - [`aedb654`](https://github.com/eea/volto-eea-map/commit/aedb6544fd45b147a3385425aba46ffba108f5ca)]
 ### [1.0.1](https://github.com/eea/volto-eea-map/compare/1.0.0...1.0.1) - 22 June 2023
 
 #### :rocket: New Features
@@ -204,7 +216,6 @@ Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 
 - chore: clear unused #254131 [andreiggr - [`ccec2ea`](https://github.com/eea/volto-eea-map/commit/ccec2eaab173c905cd4ea0190a9c6cfd630250f9)]
 - chore: clear block style #254131 [andreiggr - [`af500bc`](https://github.com/eea/volto-eea-map/commit/af500bc10a5f280eff0335f45bae1596d66a7e6b)]
-- chore: [JENKINS] Remove alpha testing version [valentinab25 - [`76b29e5`](https://github.com/eea/volto-eea-map/commit/76b29e510f5156590723831a524bc39dfba6deba)]
 
 #### :hammer_and_wrench: Others
 
@@ -214,7 +225,6 @@ Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 - * versions [andreiggr - [`6d9ff2b`](https://github.com/eea/volto-eea-map/commit/6d9ff2b26a236b6445678e3d84698d1c3c0826f4)]
 - add razzle-plugin-scss dep [andreiggr - [`d74cee5`](https://github.com/eea/volto-eea-map/commit/d74cee51d30c866afe55730e5b25e0bf11d2caea)]
 - volto-object-widget [andreiggr - [`18d0ed7`](https://github.com/eea/volto-eea-map/commit/18d0ed72c1ba7b0f7159a0eefc09ccea9acee5cb)]
-- Add Sonarqube tag using bise-frontend addons list [EEA Jenkins - [`5911d01`](https://github.com/eea/volto-eea-map/commit/5911d01f7f3bc79a2c834fab75a9a4ac05d2d042)]
 ### [0.1.32](https://github.com/eea/volto-eea-map/compare/0.1.31...0.1.32) - 20 February 2023
 
 #### :hammer_and_wrench: Others
@@ -222,7 +232,6 @@ Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 - remove console.log [tedw87 - [`3aeb3e8`](https://github.com/eea/volto-eea-map/commit/3aeb3e8ec5474387f3cd22eb9b387c6681e17120)]
 - refactor code [tedw87 - [`6d7830e`](https://github.com/eea/volto-eea-map/commit/6d7830eb934f56c255eea5642bdefba30d892946)]
 - add Blossom basemap layer [tedw87 - [`5226ee2`](https://github.com/eea/volto-eea-map/commit/5226ee218df3401cf467afbc7a4d720dc30719e1)]
-- Add Sonarqube tag using cca-frontend addons list [EEA Jenkins - [`1dad5c6`](https://github.com/eea/volto-eea-map/commit/1dad5c6617096ff85223165a6dd2b1d5d7deea6a)]
 ### [0.1.31](https://github.com/eea/volto-eea-map/compare/0.1.30...0.1.31) - 11 November 2022
 
 #### :hammer_and_wrench: Others

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eeacms/volto-eea-map",
-  "version": "3.0.3",
+  "version": "3.2.0",
   "description": "@eeacms/volto-eea-map: Volto add-on",
   "main": "src/index.js",
   "author": "European Environment Agency: IDM2 A-Team",

package/src/components/Blocks/EmbedEEAMap/Edit.jsx

@@ -12,13 +12,24 @@ import ExtraViews from '@eeacms/volto-eea-map/components/ExtraViews';
 
 import { Schema } from './Schema';
 import { applyQueriesToMapLayers } from '@eeacms/volto-eea-map/utils';
-
-import { getMapVisualizationData } from './helpers';
+import { deepUpdateDataQueryParams, getMapVisualizationData } from './helpers';
+import { isEqual } from 'lodash';
 
 const Edit = (props) => {
-  const { id, block, onChangeBlock, selected, data, getContent } = props;
   const {
-    data_query_params,
+    id,
+    block,
+    onChangeBlock,
+    selected,
+    data,
+    getContent,
+    connected_data_parameters, // page level queries live from widget
+    data_query, // page level queries
+    mapContent,
+  } = props;
+
+  const {
+    data_query_params, //block level queries
     enable_queries,
     show_legend = true,
     show_note = true,
@@ -28,14 +39,30 @@ const Edit = (props) => {
     dataprotection = { enabled: true },
     height = '',
   } = data;
+
   const schema = Schema(props);
   const [mapData, setMapData] = React.useState('');
 
   const vis_url = useMemo(() => flattenToAppURL(data.vis_url), [data.vis_url]);
 
-  const map_visualization_data = useMemo(() => getMapVisualizationData(props), [
-    props,
-  ]);
+  const map_visualization_data = useMemo(
+    () => getMapVisualizationData({ mapContent, data }),
+    [mapContent, data],
+  );
+
+  const effectiveQueryParams =
+    connected_data_parameters && connected_data_parameters.length > 0
+      ? connected_data_parameters
+      : data_query;
+
+  React.useEffect(() => {
+    deepUpdateDataQueryParams(
+      block,
+      props.data,
+      effectiveQueryParams,
+      onChangeBlock,
+    );
+  }, [block, props.data, effectiveQueryParams, onChangeBlock]);
 
   useEffect(() => {
     const mapVisId = flattenToAppURL(map_visualization_data['@id'] || '');
@@ -47,21 +74,42 @@ const Edit = (props) => {
     }
   }, [id, getContent, vis_url, map_visualization_data]);
 
-  React.useEffect(() => {
+  useEffect(() => {
+    const mergedQueries =
+      connected_data_parameters &&
+      connected_data_parameters.length > 0 &&
+      connected_data_parameters.map((unsavedQuery, index) => {
+        const correspondingQuery =
+          data_query_params && data_query_params[index];
+        return { ...unsavedQuery, alias: correspondingQuery?.alias };
+      });
+    const queriesToUse =
+      mergedQueries && mergedQueries.length > 0
+        ? mergedQueries
+        : data_query_params;
+
     const updatedMapData = applyQueriesToMapLayers(
       map_visualization_data,
-      data_query_params,
+      queriesToUse,
       enable_queries,
     );
-    setMapData(updatedMapData);
-  }, [map_visualization_data, data_query_params, enable_queries]);
+
+    if (!isEqual(mapData, updatedMapData)) {
+      setMapData(updatedMapData);
+    }
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [
+    map_visualization_data,
+    data_query_params,
+    enable_queries,
+    connected_data_parameters,
+  ]);
 
   return (
     <>
       {!vis_url && (
         <Message>Please select a visualization from block editor.</Message>
       )}
-
       {!!vis_url && mapData && (
         <div>
           <Webmap data={mapData} height={height} isEdit={true} />
@@ -99,12 +147,16 @@ const Edit = (props) => {
 
 export default compose(
   connect(
-    (state, props) => ({
-      mapContent: state.content.subrequests?.[props.id]?.data,
-      data_query: state.content.data.data_query,
-    }),
-    {
-      getContent,
+    (state, props) => {
+      const pathname = flattenToAppURL(state.content.data['@id']);
+      return {
+        mapContent: state.content.subrequests?.[props.id]?.data,
+        data_query: state.content.data.data_query,
+        connected_data_parameters:
+          state?.connected_data_parameters?.byContextPath &&
+          state.connected_data_parameters?.byContextPath[pathname],
+      };
     },
+    { getContent },
   ),
 )(Edit);

package/src/components/Blocks/EmbedEEAMap/Schema.js

@@ -1,6 +1,3 @@
-import React from 'react';
-import { deepUpdateDataQueryParams } from './helpers';
-
 const ProtectionSchema = () => ({
   title: 'Data Protection',
 
@@ -68,12 +65,6 @@ const ProtectionSchema = () => ({
 });
 
 export const Schema = (props) => {
-  const { block, onChangeBlock } = props;
-
-  const dataQuery = React.useMemo(() => props.data_query, [props.data_query]);
-  const data = React.useMemo(() => props.data, [props.data]);
-
-  deepUpdateDataQueryParams(block, data, dataQuery, onChangeBlock);
   return {
     title: 'Embed Map layers (ArcGis)',
     fieldsets: [
@@ -107,7 +98,7 @@ export const Schema = (props) => {
     ],
     properties: {
       vis_url: {
-        widget: 'object_by_path',
+        widget: 'internal_url',
         title: 'Visualization',
       },
       height: {

package/src/components/Blocks/EmbedEEAMap/helpers.js

@@ -1,37 +1,41 @@
 import { pickMetadata } from '@eeacms/volto-embed/helpers';
-import { updateBlockQueryFromPageQuery } from '@eeacms/volto-eea-map/utils';
+// import { updateBlockQueryFromPageQuery } from '@eeacms/volto-eea-map/utils';
 
-const deepUpdateDataQueryParams = (block, data, data_query, onChangeBlock) => {
-  // If block data_query_params do not exist, init them
-  if (!data?.data_query_params && data_query) {
+const deepUpdateDataQueryParams = (
+  block,
+  data,
+  effectiveQueryParams,
+  onChangeBlock,
+) => {
+  const updatedQueryParams =
+    effectiveQueryParams &&
+    effectiveQueryParams.length > 0 &&
+    effectiveQueryParams.map((param) => {
+      // Find the matching query in the block's current data_query_params
+      const existingParam =
+        data?.data_query_params &&
+        data.data_query_params.find((p) => p.i === param.i);
+
+      // If found, merge it with the effective query parameter, preserving the alias
+      return existingParam ? { ...param, alias: existingParam.alias } : param;
+    });
+
+  // Update the block data if there are changes
+  if (
+    JSON.stringify(data.data_query_params) !==
+    JSON.stringify(updatedQueryParams)
+  ) {
     onChangeBlock(block, {
       ...data,
-      data_query_params: [...data_query],
+      data_query_params: updatedQueryParams,
     });
   }
-
-  // If block data_query_params exist, deep check them then change them in block data
-  if (data_query && data?.data_query_params) {
-    const newDataQuery = updateBlockQueryFromPageQuery(
-      data_query,
-      data?.data_query_params,
-    );
-
-    if (
-      JSON.stringify(newDataQuery) !== JSON.stringify(data.data_query_params)
-    ) {
-      onChangeBlock(block, {
-        ...data,
-        data_query_params: [...newDataQuery],
-      });
-    }
-  }
 };
 
-function getMapVisualizationData(props) {
-  const content = props.mapContent || {};
+function getMapVisualizationData({ mapContent, data }) {
+  const content = mapContent || {};
   const map_visualization_data =
-    content.map_visualization_data || props.data?.map_visualization_data || {};
+    content.map_visualization_data || data?.map_visualization_data || {};
   return {
     ...pickMetadata(content),
     ...map_visualization_data,

package/src/components/MoreInfoLink.jsx

@@ -13,7 +13,7 @@ const MoreInfoLink = ({ contentTypeLink }) => {
   return (
     <Link href={contentTypeLink}>
       <button className={cx('eea-map-more-info-button')}>
-        More info <i class="ri-external-link-line"></i>
+        More info <i className="ri-external-link-line"></i>
       </button>
     </Link>
   );

package/src/components/Share.jsx

@@ -34,7 +34,7 @@ const Share = ({ contentTypeLink = '' }) => {
         <div className="eea-map-share-container">
           <button className={cx('eea-map-share-button', { expanded })}>
             <span>Share</span>
-            <i class="ri-share-fill"></i>
+            <i className="ri-share-fill"></i>
           </button>
         </div>
       }

package/src/components/widgets/DataQueryWidget.jsx

@@ -26,7 +26,8 @@ const DataQueryWidget = (props) => {
               <Accordion.Content active={true}>
                 <Segment>
                   <p className="data-param-title">
-                    <strong>{param.i}:</strong> {param.v.join(', ')}
+                    <strong>{param?.i}:</strong>{' '}
+                    {param?.v && param.v.join(', ')}
                   </p>
                   <Field
                     id={i}

package/src/components/widgets/LayerSelectWidget.jsx

@@ -4,6 +4,7 @@ import { Input, Select, Button, Grid, Checkbox } from 'semantic-ui-react';
 import { QueryBuilder } from 'react-querybuilder';
 import 'react-querybuilder/dist/query-builder.css';
 
+import { flattenToAppURL } from '@plone/volto/helpers';
 import RichTextWidget from '@plone/volto-slate/widgets/RichTextWidget';
 
 import { connect } from 'react-redux';
@@ -442,10 +443,13 @@ const LayerSelectWidget = (props) => {
 
 export default compose(
   connect(
-    (state, props) => ({
-      content: state.content.data,
-      data_query: state.content.data.data_query,
-    }),
+    (state) => {
+      const pathname = flattenToAppURL(state.content.data['@id']);
+      return {
+        content: state.content.data,
+        data_query: state.connected_data_parameters.byContextPath[pathname],
+      };
+    },
     {
       getContent,
     },