From c521111c23847f336b16cc270568438743530c83 Mon Sep 17 00:00:00 2001
From: Jen Jones Arnesen <jennifer@dhis2.org>
Date: Thu, 11 Mar 2021 17:40:18 +0100
Subject: [PATCH] fix: disable View As menu items for maps without a thematic
 layer [DHIS2-10671] (#1632)

If a map doesn't have a thematic layer, then no chart or table can be rendered. Therefore, disable these menu items and add a tooltip informing the user
---
 .../ItemContextMenu/ItemContextMenu.js        |  1 +
 .../ItemContextMenu/ViewAsMenuItems.js        | 75 +++++++++++++---
 .../__tests__/ViewAsMenuItems.spec.js         | 21 +++++
 .../ViewAsMenuItems.spec.js.snap              | 87 +++++++++++++++++--
 .../Visualization/Visualization.js            |  4 +-
 .../__tests__/getVisualizationConfig.spec.js  |  7 +-
 .../getFilteredVisualization.js               |  0
 src/modules/getThematicMapViews.js            |  6 ++
 .../getVisualizationConfig.js                 | 14 ++-
 9 files changed, 180 insertions(+), 35 deletions(-)
 rename src/{components/Item/VisualizationItem/Visualization => modules}/__tests__/getVisualizationConfig.spec.js (96%)
 rename src/{components/Item/VisualizationItem/Visualization => modules}/getFilteredVisualization.js (100%)
 create mode 100644 src/modules/getThematicMapViews.js
 rename src/{components/Item/VisualizationItem/Visualization => modules}/getVisualizationConfig.js (77%)

diff --git a/src/components/Item/VisualizationItem/ItemContextMenu/ItemContextMenu.js b/src/components/Item/VisualizationItem/ItemContextMenu/ItemContextMenu.js
index f0e95c84a..6cc87b2f8 100644
--- a/src/components/Item/VisualizationItem/ItemContextMenu/ItemContextMenu.js
+++ b/src/components/Item/VisualizationItem/ItemContextMenu/ItemContextMenu.js
@@ -115,6 +115,7 @@ const ItemContextMenu = props => {
                                     type={item.type}
                                     activeType={activeType}
                                     onActiveTypeChanged={onActiveTypeChanged}
+                                    visualization={visualization}
                                 />
                                 {(allowVisShowInterpretations ||
                                     (allowVisOpenInApp &&
diff --git a/src/components/Item/VisualizationItem/ItemContextMenu/ViewAsMenuItems.js b/src/components/Item/VisualizationItem/ItemContextMenu/ViewAsMenuItems.js
index 596164ca0..a6f28a183 100644
--- a/src/components/Item/VisualizationItem/ItemContextMenu/ViewAsMenuItems.js
+++ b/src/components/Item/VisualizationItem/ItemContextMenu/ViewAsMenuItems.js
@@ -4,6 +4,7 @@ import i18n from '@dhis2/d2-i18n'
 import TableIcon from '@material-ui/icons/ViewList'
 import ChartIcon from '@material-ui/icons/InsertChart'
 import MapIcon from '@material-ui/icons/Public'
+import getThematicMapViews from '../../../../modules/getThematicMapViews'
 import {
     CHART,
     MAP,
@@ -13,11 +14,16 @@ import {
     isTrackerDomainType,
     hasMapView,
 } from '../../../../modules/itemTypes'
-import { MenuItem, colors } from '@dhis2/ui'
+import { MenuItem, Tooltip, colors } from '@dhis2/ui'
 
 const iconFill = { fill: colors.grey600 }
 
-const ViewAsMenuItems = ({ type, activeType, onActiveTypeChanged }) => {
+const ViewAsMenuItems = ({
+    type,
+    activeType,
+    visualization,
+    onActiveTypeChanged,
+}) => {
     const isTrackerType = isTrackerDomainType(type)
 
     const onViewTable = () =>
@@ -28,23 +34,63 @@ const ViewAsMenuItems = ({ type, activeType, onActiveTypeChanged }) => {
 
     const onViewMap = () => onActiveTypeChanged(MAP)
 
+    const isDisabled = type === MAP && !getThematicMapViews(visualization)
+
+    const ViewAsChartMenuItem = () => {
+        const ChartMenuItem = () => (
+            <MenuItem
+                dense
+                disabled={isDisabled}
+                label={i18n.t('View as Chart')}
+                onClick={onViewChart}
+                icon={<ChartIcon style={iconFill} />}
+            />
+        )
+
+        if (isDisabled) {
+            return (
+                <Tooltip
+                    content={i18n.t("This map can't be displayed as a chart")}
+                >
+                    <ChartMenuItem />
+                </Tooltip>
+            )
+        }
+
+        return <ChartMenuItem />
+    }
+
+    const ViewAsTableMenuItem = () => {
+        const TableMenuItem = () => (
+            <MenuItem
+                dense
+                disabled={isDisabled}
+                label={i18n.t('View as Table')}
+                onClick={onViewTable}
+                icon={<TableIcon style={iconFill} />}
+            />
+        )
+
+        if (isDisabled) {
+            return (
+                <Tooltip
+                    content={i18n.t("This map can't be displayed as a table")}
+                >
+                    <TableMenuItem />
+                </Tooltip>
+            )
+        }
+
+        return <TableMenuItem />
+    }
+
     return (
         <>
             {activeType !== CHART && activeType !== EVENT_CHART && (
-                <MenuItem
-                    dense
-                    label={i18n.t('View as Chart')}
-                    onClick={onViewChart}
-                    icon={<ChartIcon style={iconFill} />}
-                />
+                <ViewAsChartMenuItem />
             )}
             {activeType !== REPORT_TABLE && activeType !== EVENT_REPORT && (
-                <MenuItem
-                    dense
-                    label={i18n.t('View as Table')}
-                    onClick={onViewTable}
-                    icon={<TableIcon style={iconFill} />}
-                />
+                <ViewAsTableMenuItem />
             )}
             {hasMapView(type) && activeType !== MAP && (
                 <MenuItem
@@ -61,6 +107,7 @@ const ViewAsMenuItems = ({ type, activeType, onActiveTypeChanged }) => {
 ViewAsMenuItems.propTypes = {
     activeType: PropTypes.string,
     type: PropTypes.string,
+    visualization: PropTypes.object,
     onActiveTypeChanged: PropTypes.func,
 }
 
diff --git a/src/components/Item/VisualizationItem/ItemContextMenu/__tests__/ViewAsMenuItems.spec.js b/src/components/Item/VisualizationItem/ItemContextMenu/__tests__/ViewAsMenuItems.spec.js
index d8a36b6ac..46b71ae9c 100644
--- a/src/components/Item/VisualizationItem/ItemContextMenu/__tests__/ViewAsMenuItems.spec.js
+++ b/src/components/Item/VisualizationItem/ItemContextMenu/__tests__/ViewAsMenuItems.spec.js
@@ -28,6 +28,23 @@ test('renders menu for active type CHART and type MAP', async () => {
     const props = Object.assign({}, defaultProps, {
         type: MAP,
         activeType: CHART,
+        visualization: {
+            mapViews: [{ layer: 'thematic' }],
+        },
+    })
+
+    const { container } = render(<ViewAsMenuItems {...props} />)
+
+    expect(container).toMatchSnapshot()
+})
+
+test('renders menu for active type MAP and type MAP without Thematic layer', async () => {
+    const props = Object.assign({}, defaultProps, {
+        type: MAP,
+        activeType: MAP,
+        visualization: {
+            mapViews: [{ layer: 'earthEngine' }],
+        },
     })
 
     const { container } = render(<ViewAsMenuItems {...props} />)
@@ -39,6 +56,7 @@ test('renders menu for active type REPORT_TABLE and type CHART', async () => {
     const props = Object.assign({}, defaultProps, {
         type: CHART,
         activeType: REPORT_TABLE,
+        visualization: {},
     })
 
     const { container } = render(<ViewAsMenuItems {...props} />)
@@ -50,6 +68,7 @@ test('renders menu for active type CHART and type REPORT_TABLE', async () => {
     const props = Object.assign({}, defaultProps, {
         type: REPORT_TABLE,
         activeType: CHART,
+        visualization: {},
     })
 
     const { container } = render(<ViewAsMenuItems {...props} />)
@@ -61,6 +80,7 @@ test('renders menu for active type EVENT_REPORT and type EVENT_CHART', async ()
     const props = Object.assign({}, defaultProps, {
         type: EVENT_CHART,
         activeType: EVENT_REPORT,
+        visualization: {},
     })
 
     const { container } = render(<ViewAsMenuItems {...props} />)
@@ -72,6 +92,7 @@ test('renders menu for active type EVENT_CHART and type EVENT_REPORT', async ()
     const props = Object.assign({}, defaultProps, {
         type: EVENT_REPORT,
         activeType: EVENT_CHART,
+        visualization: {},
     })
 
     const { container } = render(<ViewAsMenuItems {...props} />)
diff --git a/src/components/Item/VisualizationItem/ItemContextMenu/__tests__/__snapshots__/ViewAsMenuItems.spec.js.snap b/src/components/Item/VisualizationItem/ItemContextMenu/__tests__/__snapshots__/ViewAsMenuItems.spec.js.snap
index 0a64885f9..8e05fccf8 100644
--- a/src/components/Item/VisualizationItem/ItemContextMenu/__tests__/__snapshots__/ViewAsMenuItems.spec.js.snap
+++ b/src/components/Item/VisualizationItem/ItemContextMenu/__tests__/__snapshots__/ViewAsMenuItems.spec.js.snap
@@ -79,7 +79,7 @@ exports[`renders menu for active type CHART and type REPORT_TABLE 1`] = `
       >
         <svg
           aria-hidden="true"
-          class="MuiSvgIcon-root-28"
+          class="MuiSvgIcon-root-37"
           focusable="false"
           role="presentation"
           style="fill: #6e7a8a;"
@@ -109,7 +109,7 @@ exports[`renders menu for active type CHART and type REPORT_TABLE 1`] = `
       >
         <svg
           aria-hidden="true"
-          class="MuiSvgIcon-root-28"
+          class="MuiSvgIcon-root-37"
           focusable="false"
           role="presentation"
           style="fill: #6e7a8a;"
@@ -144,7 +144,7 @@ exports[`renders menu for active type EVENT_CHART and type EVENT_REPORT 1`] = `
       >
         <svg
           aria-hidden="true"
-          class="MuiSvgIcon-root-46"
+          class="MuiSvgIcon-root-55"
           focusable="false"
           role="presentation"
           style="fill: #6e7a8a;"
@@ -179,7 +179,7 @@ exports[`renders menu for active type EVENT_REPORT and type EVENT_CHART 1`] = `
       >
         <svg
           aria-hidden="true"
-          class="MuiSvgIcon-root-37"
+          class="MuiSvgIcon-root-46"
           focusable="false"
           role="presentation"
           style="fill: #6e7a8a;"
@@ -265,6 +265,81 @@ exports[`renders menu for active type MAP and type CHART 1`] = `
 </div>
 `;
 
+exports[`renders menu for active type MAP and type MAP without Thematic layer 1`] = `
+<div>
+  <span
+    class="jsx-611321200"
+    data-test="dhis2-uicore-tooltip-reference"
+  >
+    <li
+      class="jsx-665727467 disabled dense"
+      data-test="dhis2-uicore-menuitem"
+    >
+      <a
+        class="jsx-665727467"
+      >
+        <span
+          class="jsx-665727467 icon"
+        >
+          <svg
+            aria-hidden="true"
+            class="MuiSvgIcon-root-19"
+            focusable="false"
+            role="presentation"
+            style="fill: #6e7a8a;"
+            viewBox="0 0 24 24"
+          >
+            <path
+              d="M19 3H5c-1.1 0-2 .9-2 2v14c0 1.1.9 2 2 2h14c1.1 0 2-.9 2-2V5c0-1.1-.9-2-2-2zM9 17H7v-7h2v7zm4 0h-2V7h2v10zm4 0h-2v-4h2v4z"
+            />
+          </svg>
+        </span>
+        <span
+          class="jsx-665727467 label"
+        >
+          View as Chart
+        </span>
+      </a>
+    </li>
+  </span>
+  <span
+    class="jsx-611321200"
+    data-test="dhis2-uicore-tooltip-reference"
+  >
+    <li
+      class="jsx-665727467 disabled dense"
+      data-test="dhis2-uicore-menuitem"
+    >
+      <a
+        class="jsx-665727467"
+      >
+        <span
+          class="jsx-665727467 icon"
+        >
+          <svg
+            aria-hidden="true"
+            class="MuiSvgIcon-root-19"
+            focusable="false"
+            role="presentation"
+            style="fill: #6e7a8a;"
+            viewBox="0 0 24 24"
+          >
+            <path
+              d="M4 14h4v-4H4v4zm0 5h4v-4H4v4zM4 9h4V5H4v4zm5 5h12v-4H9v4zm0 5h12v-4H9v4zM9 5v4h12V5H9z"
+            />
+          </svg>
+        </span>
+        <span
+          class="jsx-665727467 label"
+        >
+          View as Table
+        </span>
+      </a>
+    </li>
+  </span>
+</div>
+`;
+
 exports[`renders menu for active type REPORT_TABLE and type CHART 1`] = `
 <div>
   <li
@@ -279,7 +354,7 @@ exports[`renders menu for active type REPORT_TABLE and type CHART 1`] = `
       >
         <svg
           aria-hidden="true"
-          class="MuiSvgIcon-root-19"
+          class="MuiSvgIcon-root-28"
           focusable="false"
           role="presentation"
           style="fill: #6e7a8a;"
@@ -309,7 +384,7 @@ exports[`renders menu for active type REPORT_TABLE and type CHART 1`] = `
       >
         <svg
           aria-hidden="true"
-          class="MuiSvgIcon-root-19"
+          class="MuiSvgIcon-root-28"
           focusable="false"
           role="presentation"
           style="fill: #6e7a8a;"
diff --git a/src/components/Item/VisualizationItem/Visualization/Visualization.js b/src/components/Item/VisualizationItem/Visualization/Visualization.js
index 78184f7ec..af8edda49 100644
--- a/src/components/Item/VisualizationItem/Visualization/Visualization.js
+++ b/src/components/Item/VisualizationItem/Visualization/Visualization.js
@@ -8,8 +8,8 @@ import MapPlugin from './MapPlugin'
 import DataVisualizerPlugin from './DataVisualizerPlugin'
 import NoVisualizationMessage from './NoVisualizationMessage'
 
-import getFilteredVisualization from './getFilteredVisualization'
-import getVisualizationConfig from './getVisualizationConfig'
+import getFilteredVisualization from '../../../../modules/getFilteredVisualization'
+import getVisualizationConfig from '../../../../modules/getVisualizationConfig'
 import {
     VISUALIZATION,
     MAP,
diff --git a/src/components/Item/VisualizationItem/Visualization/__tests__/getVisualizationConfig.spec.js b/src/modules/__tests__/getVisualizationConfig.spec.js
similarity index 96%
rename from src/components/Item/VisualizationItem/Visualization/__tests__/getVisualizationConfig.spec.js
rename to src/modules/__tests__/getVisualizationConfig.spec.js
index d5fb05908..c8493a485 100644
--- a/src/components/Item/VisualizationItem/Visualization/__tests__/getVisualizationConfig.spec.js
+++ b/src/modules/__tests__/getVisualizationConfig.spec.js
@@ -7,10 +7,9 @@ import {
     AXIS_ID_FILTERS,
 } from '@dhis2/analytics'
 
-import getVisualizationConfig, {
-    THEMATIC_LAYER,
-} from '../getVisualizationConfig'
-import { REPORT_TABLE, CHART, MAP } from '../../../../../modules/itemTypes'
+import getVisualizationConfig from '../getVisualizationConfig'
+import { THEMATIC_LAYER } from '../getThematicMapViews'
+import { REPORT_TABLE, CHART, MAP } from '../itemTypes'
 
 describe('getVisualizationConfig', () => {
     let visualization
diff --git a/src/components/Item/VisualizationItem/Visualization/getFilteredVisualization.js b/src/modules/getFilteredVisualization.js
similarity index 100%
rename from src/components/Item/VisualizationItem/Visualization/getFilteredVisualization.js
rename to src/modules/getFilteredVisualization.js
diff --git a/src/modules/getThematicMapViews.js b/src/modules/getThematicMapViews.js
new file mode 100644
index 000000000..27d9f5320
--- /dev/null
+++ b/src/modules/getThematicMapViews.js
@@ -0,0 +1,6 @@
+export const THEMATIC_LAYER = 'thematic'
+
+const getThematicMapViews = map =>
+    map.mapViews && map.mapViews.find(mv => mv.layer.includes(THEMATIC_LAYER))
+
+export default getThematicMapViews
diff --git a/src/components/Item/VisualizationItem/Visualization/getVisualizationConfig.js b/src/modules/getVisualizationConfig.js
similarity index 77%
rename from src/components/Item/VisualizationItem/Visualization/getVisualizationConfig.js
rename to src/modules/getVisualizationConfig.js
index 62c59b511..e83427e61 100644
--- a/src/components/Item/VisualizationItem/Visualization/getVisualizationConfig.js
+++ b/src/modules/getVisualizationConfig.js
@@ -1,14 +1,10 @@
+import getThematicMapViews from './getThematicMapViews'
 import {
     VIS_TYPE_COLUMN,
     VIS_TYPE_PIVOT_TABLE,
     getAdaptedUiLayoutByType,
 } from '@dhis2/analytics'
-import { REPORT_TABLE, CHART, MAP } from '../../../../modules/itemTypes'
-
-export const THEMATIC_LAYER = 'thematic'
-
-const extractMapView = map =>
-    map.mapViews && map.mapViews.find(mv => mv.layer.includes(THEMATIC_LAYER))
+import { REPORT_TABLE, CHART, MAP } from './itemTypes'
 
 const getWithoutId = object => ({
     ...object,
@@ -17,15 +13,15 @@ const getWithoutId = object => ({
 
 const getVisualizationConfig = (visualization, originalType, activeType) => {
     if (originalType === MAP && originalType !== activeType) {
-        const extractedMapView = extractMapView(visualization)
+        const thematicMapViews = getThematicMapViews(visualization)
 
-        if (extractedMapView === undefined) {
+        if (thematicMapViews === undefined) {
             return null
         }
 
         return getWithoutId({
             ...visualization,
-            ...extractedMapView,
+            ...thematicMapViews,
             mapViews: undefined,
             type: activeType === CHART ? VIS_TYPE_COLUMN : VIS_TYPE_PIVOT_TABLE,
         })