fix: enable admin to edit dataset in explore (#20613)

* add conditional

* add test

* fix lint

superset-frontend/src/explore/components/controls/DatasourceControl/DatasourceControl.test.tsx

@@ -139,6 +139,27 @@ test('Click on Edit dataset', async () => {
   ).toBeInTheDocument();
 });
 
+test('Edit dataset should be disabled when user is not admin', async () => {
+  const props = createProps();
+  // @ts-expect-error
+  props.user.roles = {};
+  props.datasource.owners = [];
+  SupersetClientGet.mockImplementation(
+    async () => ({ json: { result: [] } } as any),
+  );
+
+  render(<DatasourceControl {...props} />, {
+    useRedux: true,
+  });
+
+  userEvent.click(screen.getByTestId('datasource-menu-trigger'));
+
+  expect(screen.getByTestId('edit-dataset')).toHaveAttribute(
+    'aria-disabled',
+    'true',
+  );
+});
+
 test('Click on View in SQL Lab', async () => {
   const props = createProps();
   const postFormSpy = jest.spyOn(SupersetClient, 'postForm');

superset-frontend/src/explore/components/controls/DatasourceControl/index.jsx

@@ -236,10 +236,9 @@ class DatasourceControl extends React.PureComponent {
 
     const isSqlSupported = datasource.type === 'table';
     const { user } = this.props;
-    const allowEdit = datasource.owners
-      ?.map(o => o.id || o.value)
-      .includes(user.userId);
-    isUserAdmin(user);
+    const allowEdit =
+      datasource.owners?.map(o => o.id || o.value).includes(user.userId) ||
+      isUserAdmin(user);
 
     const editText = t('Edit dataset');