From 3663a33f14ef4bc3792e0a4089c3edd244f158ff Mon Sep 17 00:00:00 2001
From: Kamil Gabryjelski <kamil.gabryjelski@gmail.com>
Date: Tue, 19 Apr 2022 17:47:35 +0200
Subject: [PATCH] fix(explore): Double divider if no permissions for adding
 reports (#19777)

---
 .../ExploreAdditionalActionsMenu/index.jsx    | 52 ++++++++++---------
 1 file changed, 28 insertions(+), 24 deletions(-)

diff --git a/superset-frontend/src/explore/components/ExploreAdditionalActionsMenu/index.jsx b/superset-frontend/src/explore/components/ExploreAdditionalActionsMenu/index.jsx
index f02ab0162259..fa9b54acf502 100644
--- a/superset-frontend/src/explore/components/ExploreAdditionalActionsMenu/index.jsx
+++ b/superset-frontend/src/explore/components/ExploreAdditionalActionsMenu/index.jsx
@@ -370,31 +370,35 @@ const ExploreAdditionalActionsMenu = ({
               </Menu.Item>
             </Menu.SubMenu>
             <Menu.Divider />
-            {canAddReports &&
-              (report ? (
-                <Menu.SubMenu
-                  title={t('Manage email report')}
-                  key={MENU_KEYS.REPORT_SUBMENU}
-                >
-                  <Menu.Item key={MENU_KEYS.SET_REPORT_ACTIVE}>
-                    <MenuItemWithCheckboxContainer>
-                      <Checkbox checked={isReportActive} onChange={noOp} />
-                      {t('Email reports active')}
-                    </MenuItemWithCheckboxContainer>
-                  </Menu.Item>
-                  <Menu.Item key={MENU_KEYS.EDIT_REPORT}>
-                    {t('Edit email report')}
-                  </Menu.Item>
-                  <Menu.Item key={MENU_KEYS.DELETE_REPORT}>
-                    {t('Delete email report')}
+            {canAddReports && (
+              <>
+                {report ? (
+                  <Menu.SubMenu
+                    title={t('Manage email report')}
+                    key={MENU_KEYS.REPORT_SUBMENU}
+                  >
+                    <Menu.Item key={MENU_KEYS.SET_REPORT_ACTIVE}>
+                      <MenuItemWithCheckboxContainer>
+                        <Checkbox checked={isReportActive} onChange={noOp} />
+                        {t('Email reports active')}
+                      </MenuItemWithCheckboxContainer>
+                    </Menu.Item>
+                    <Menu.Item key={MENU_KEYS.EDIT_REPORT}>
+                      {t('Edit email report')}
+                    </Menu.Item>
+                    <Menu.Item key={MENU_KEYS.DELETE_REPORT}>
+                      {t('Delete email report')}
+                    </Menu.Item>
+                  </Menu.SubMenu>
+                ) : (
+                  <Menu.Item key={MENU_KEYS.SET_UP_REPORT}>
+                    {t('Set up an email report')}
                   </Menu.Item>
-                </Menu.SubMenu>
-              ) : (
-                <Menu.Item key={MENU_KEYS.SET_UP_REPORT}>
-                  {t('Set up an email report')}
-                </Menu.Item>
-              ))}
-            <Menu.Divider />
+                )}
+                <Menu.Divider />
+              </>
+            )}
+
             <Menu.Item key={MENU_KEYS.VIEW_QUERY}>
               <ModalTrigger
                 triggerNode={