From f3a7310344b760bc8b4246bf69f47912068d6d90 Mon Sep 17 00:00:00 2001
From: Tiberiu Ichim <tiberiu.ichim@gmail.com>
Date: Wed, 13 Sep 2023 12:37:23 +0300
Subject: [PATCH 1/4] Fix crash in dataset view

---
 .../ItemView/Datasets/DatasetEsri.jsx         |  30 ++---
 .../ItemView/Datasets/DatasetHttpLinks.jsx    | 116 +++++++++---------
 .../Datasets/DatasetItemDownloadList.jsx      |  13 +-
 3 files changed, 80 insertions(+), 79 deletions(-)

diff --git a/src/components/ItemView/Datasets/DatasetEsri.jsx b/src/components/ItemView/Datasets/DatasetEsri.jsx
index b527699..3ea862c 100644
--- a/src/components/ItemView/Datasets/DatasetEsri.jsx
+++ b/src/components/ItemView/Datasets/DatasetEsri.jsx
@@ -18,25 +18,25 @@ const ItemLink = ({ item }) => (
         >
           {item.protocol} {item.name || item.description}
         </a>
-        {isInternalURL(item.url) && <SVGIcon name={lockSVG} size="20px" />}
+        {isInternalURL(item.url) ? (
+          <SVGIcon name={lockSVG} size="20px" />
+        ) : null}
       </div>
     </List.Content>
   </List.Item>
 );
 
 export default function DatasetEsri({ dataset }) {
-  return (
-    (dataset['ESRI:REST'] || dataset['OGC:WMS']) && (
-      <>
-        <h5>Services:</h5>
-        <List divided relaxed>
-          {[...(dataset['ESRI:REST'] || []), ...(dataset['OGC:WMS'] || [])].map(
-            (item, i) => (
-              <ItemLink item={item} key={item.id} />
-            ),
-          )}
-        </List>
-      </>
-    )
-  );
+  return dataset['ESRI:REST'] || dataset['OGC:WMS'] ? (
+    <>
+      <h5>Services:</h5>
+      <List divided relaxed>
+        {[...(dataset['ESRI:REST'] || []), ...(dataset['OGC:WMS'] || [])].map(
+          (item, i) => (
+            <ItemLink item={item} key={item.id} />
+          ),
+        )}
+      </List>
+    </>
+  ) : null;
 }
diff --git a/src/components/ItemView/Datasets/DatasetHttpLinks.jsx b/src/components/ItemView/Datasets/DatasetHttpLinks.jsx
index c8ea8aa..e56f861 100644
--- a/src/components/ItemView/Datasets/DatasetHttpLinks.jsx
+++ b/src/components/ItemView/Datasets/DatasetHttpLinks.jsx
@@ -5,63 +5,61 @@ import { SVGIcon } from '../utils.js';
 import lockSVG from 'remixicon/icons/System/lock-line.svg';
 
 export default function DatasetHttpLinks({ dataset }) {
-  return (
-    (dataset['WWW:LINK-1.0-http--link'] || dataset['DOI']) && (
-      <>
-        <h5>Links:</h5>
-        <List divided relaxed>
-          {(dataset['WWW:LINK-1.0-http--link'] || []).map((item, i) => {
-            return (
-              <List.Item key={i}>
-                <List.Content>
-                  <div className="dataset-item">
-                    <Icon className="linkify" />
-                    <a
-                      href={item.url}
-                      className="item-link"
-                      target="_blank"
-                      rel="noreferrer"
-                    >
-                      <span>{item.name || item.url}</span>
-                    </a>
-                    {isInternalURL(item.url) && (
-                      <SVGIcon name={lockSVG} size="20px" />
-                    )}
-                  </div>
-                </List.Content>
-                {item.description && (
-                  <span className="item-description">{item.description}</span>
-                )}
-              </List.Item>
-            );
-          })}
-          {(dataset['DOI'] || []).map((item, i) => {
-            return (
-              <List.Item key={i}>
-                <List.Content>
-                  <div className="dataset-item">
-                    <Icon className="info circle" />
-                    <a
-                      href={item.url}
-                      className="item-link"
-                      target="_blank"
-                      rel="noreferrer"
-                    >
-                      <span>{item.name || item.url}</span>
-                    </a>
-                    {isInternalURL(item.url) && (
-                      <SVGIcon name={lockSVG} size="20px" />
-                    )}
-                  </div>
-                </List.Content>
-                {item.description && (
-                  <span className="item-description">{item.description}</span>
-                )}
-              </List.Item>
-            );
-          })}
-        </List>
-      </>
-    )
-  );
+  return dataset['WWW:LINK-1.0-http--link'] || dataset['DOI'] ? (
+    <>
+      <h5>Links:</h5>
+      <List divided relaxed>
+        {(dataset['WWW:LINK-1.0-http--link'] || []).map((item, i) => {
+          return (
+            <List.Item key={i}>
+              <List.Content>
+                <div className="dataset-item">
+                  <Icon className="linkify" />
+                  <a
+                    href={item.url}
+                    className="item-link"
+                    target="_blank"
+                    rel="noreferrer"
+                  >
+                    <span>{item.name || item.url}</span>
+                  </a>
+                  {isInternalURL(item.url) ? (
+                    <SVGIcon name={lockSVG} size="20px" />
+                  ) : null}
+                </div>
+              </List.Content>
+              {item.description ? (
+                <span className="item-description">{item.description}</span>
+              ) : null}
+            </List.Item>
+          );
+        })}
+        {(dataset['DOI'] || []).map((item, i) => {
+          return (
+            <List.Item key={i}>
+              <List.Content>
+                <div className="dataset-item">
+                  <Icon className="info circle" />
+                  <a
+                    href={item.url}
+                    className="item-link"
+                    target="_blank"
+                    rel="noreferrer"
+                  >
+                    <span>{item.name || item.url}</span>
+                  </a>
+                  {isInternalURL(item.url) ? (
+                    <SVGIcon name={lockSVG} size="20px" />
+                  ) : null}
+                </div>
+              </List.Content>
+              {item.description ? (
+                <span className="item-description">{item.description}</span>
+              ) : null}
+            </List.Item>
+          );
+        })}
+      </List>
+    </>
+  ) : null;
 }
diff --git a/src/components/ItemView/Datasets/DatasetItemDownloadList.jsx b/src/components/ItemView/Datasets/DatasetItemDownloadList.jsx
index 44f942d..e3db3f7 100644
--- a/src/components/ItemView/Datasets/DatasetItemDownloadList.jsx
+++ b/src/components/ItemView/Datasets/DatasetItemDownloadList.jsx
@@ -10,13 +10,16 @@ import DatasetHttpLinks from './DatasetHttpLinks';
 const DatasetItemDownloadList = (props) => {
   const { link } = props;
   const dataset = groupBy(link, 'protocol');
+  const showDownload = !!(
+    dataset['WWW:URL'] ||
+    dataset['WWW:LINK'] ||
+    dataset['EEA:FILEPATH'] ||
+    dataset['EEA:FOLDERPATH']
+  );
 
   return (
     <div>
-      {(dataset['WWW:URL'] ||
-        dataset['WWW:LINK'] ||
-        dataset['EEA:FILEPATH'] ||
-        dataset['EEA:FOLDERPATH']) && (
+      {showDownload ? (
         <>
           <h5>Download:</h5>
           <List divided relaxed>
@@ -24,7 +27,7 @@ const DatasetItemDownloadList = (props) => {
             <DatasetLinks dataset={dataset} />
           </List>
         </>
-      )}
+      ) : null}
 
       <DatasetEsri dataset={dataset} />
       <DatasetHttpLinks dataset={dataset} />

From 568338374247690706c4b5b1b766e1663602ace9 Mon Sep 17 00:00:00 2001
From: kreafox <krisztina.n.elekes@gmail.com>
Date: Wed, 13 Sep 2023 13:31:50 +0300
Subject: [PATCH 2/4] Add test

---
 .../ItemView/Datasets/DatasetUrls.test.jsx    | 25 +++++++++++
 .../__snapshots__/DatasetUrls.test.jsx.snap   | 41 +++++++++++++++++++
 2 files changed, 66 insertions(+)
 create mode 100644 src/components/ItemView/Datasets/DatasetUrls.test.jsx
 create mode 100644 src/components/ItemView/Datasets/__snapshots__/DatasetUrls.test.jsx.snap

diff --git a/src/components/ItemView/Datasets/DatasetUrls.test.jsx b/src/components/ItemView/Datasets/DatasetUrls.test.jsx
new file mode 100644
index 0000000..080517e
--- /dev/null
+++ b/src/components/ItemView/Datasets/DatasetUrls.test.jsx
@@ -0,0 +1,25 @@
+import React from 'react';
+import renderer from 'react-test-renderer';
+
+import DatasetUrls from './DatasetUrls';
+
+jest.mock('@eeacms/search', () => ({
+  runRequest: jest.fn(),
+}));
+
+const dataset = {
+  'WWW:URL': [
+    {
+      url: 'https://example.org/1',
+      name: 'Direct download',
+      description: 'Download link description',
+    },
+  ],
+};
+
+describe('DatasetUrl', () => {
+  it('renders correctly', () => {
+    const tree = renderer.create(<DatasetUrls dataset={dataset} />).toJSON();
+    expect(tree).toMatchSnapshot();
+  });
+});
diff --git a/src/components/ItemView/Datasets/__snapshots__/DatasetUrls.test.jsx.snap b/src/components/ItemView/Datasets/__snapshots__/DatasetUrls.test.jsx.snap
new file mode 100644
index 0000000..68e0cc8
--- /dev/null
+++ b/src/components/ItemView/Datasets/__snapshots__/DatasetUrls.test.jsx.snap
@@ -0,0 +1,41 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`DatasetEsri renders correctly 1`] = `null`;
+
+exports[`DatasetUrl renders correctly 1`] = `
+<div
+  className="item"
+  onClick={[Function]}
+  role="listitem"
+>
+  <div
+    className="content"
+  >
+    <div
+      className="dataset-item"
+    >
+      <i
+        aria-hidden="true"
+        className="icon download"
+        onClick={[Function]}
+      />
+      <a
+        className="item-link"
+        href="https://example.org/1"
+        rel="noreferrer"
+        target="_blank"
+      >
+        <span>
+          Direct download
+        </span>
+      </a>
+    </div>
+  </div>
+  <span
+    className="item-description"
+    title="Download link description"
+  >
+    Download link description
+  </span>
+</div>
+`;

From 3e070c5973e314935d674c5d41a361882317937b Mon Sep 17 00:00:00 2001
From: kreafox <krisztina.n.elekes@gmail.com>
Date: Wed, 13 Sep 2023 13:41:18 +0300
Subject: [PATCH 3/4] Fix snapshot

---
 .../ItemView/Datasets/__snapshots__/DatasetUrls.test.jsx.snap   | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/components/ItemView/Datasets/__snapshots__/DatasetUrls.test.jsx.snap b/src/components/ItemView/Datasets/__snapshots__/DatasetUrls.test.jsx.snap
index 68e0cc8..305ad8c 100644
--- a/src/components/ItemView/Datasets/__snapshots__/DatasetUrls.test.jsx.snap
+++ b/src/components/ItemView/Datasets/__snapshots__/DatasetUrls.test.jsx.snap
@@ -1,7 +1,5 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`DatasetEsri renders correctly 1`] = `null`;
-
 exports[`DatasetUrl renders correctly 1`] = `
 <div
   className="item"

From 86e3d252323224862a95591df58402060d305019 Mon Sep 17 00:00:00 2001
From: EEA Jenkins <@users.noreply.github.com>
Date: Wed, 13 Sep 2023 10:53:26 +0000
Subject: [PATCH 4/4] Automated release 1.0.29

---
 CHANGELOG.md | 7 +++++++
 package.json | 2 +-
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 7a93140..383f5eb 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -4,6 +4,13 @@ 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).
 
+### [1.0.29](https://github.com/eea/volto-datahub/compare/1.0.28...1.0.29) - 13 September 2023
+
+#### :hammer_and_wrench: Others
+
+- Fix snapshot [kreafox - [`3e070c5`](https://github.com/eea/volto-datahub/commit/3e070c5973e314935d674c5d41a361882317937b)]
+- Add test [kreafox - [`5683383`](https://github.com/eea/volto-datahub/commit/568338374247690706c4b5b1b766e1663602ace9)]
+- Fix crash in dataset view [Tiberiu Ichim - [`f3a7310`](https://github.com/eea/volto-datahub/commit/f3a7310344b760bc8b4246bf69f47912068d6d90)]
 ### [1.0.28](https://github.com/eea/volto-datahub/compare/1.0.27...1.0.28) - 8 September 2023
 
 #### :hammer_and_wrench: Others
diff --git a/package.json b/package.json
index 955dcab..e785113 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@eeacms/volto-datahub",
-  "version": "1.0.28",
+  "version": "1.0.29",
   "description": "@eeacms/volto-datahub: Volto add-on",
   "main": "src/index.js",
   "author": "European Environment Agency: IDM2 A-Team",