Revert "fix(explore): let admin overwrite slice (apache#16290)" (apac…

…he#16408) This reverts commit d13b081.
QAlexBall · Aug 24, 2021 · b5f2121 · b5f2121
1 parent 564bc07
commit b5f2121
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/superset-frontend/src/explore/components/PropertiesModal/index.tsx b/superset-frontend/src/explore/components/PropertiesModal/index.tsx
@@ -175,7 +175,7 @@ export default function PropertiesModal({
             buttonStyle="primary"
             // @ts-ignore
             onClick={onSubmit}
-            disabled={submitting || !name}
+            disabled={!owners || submitting || !name}
             cta
           >
             {t('Save')}

diff --git a/superset/views/core.py b/superset/views/core.py
@@ -122,7 +122,6 @@
     get_error_msg,
     get_user_roles,
     handle_api_exception,
-    is_user_admin,
     json_error_response,
     json_errors_response,
     json_success,
@@ -788,9 +787,7 @@ def explore(  # pylint: disable=too-many-locals
 
         # slc perms
         slice_add_perm = security_manager.can_access("can_write", "Chart")
-        slice_overwrite_perm = (
-            is_owner(slc, g.user) or is_user_admin() if slc else False
-        )
+        slice_overwrite_perm = is_owner(slc, g.user) if slc else False
         slice_download_perm = security_manager.can_access("can_csv", "Superset")
 
         form_data["datasource"] = str(datasource_id) + "__" + cast(str, datasource_type)