[Workspace] Fix non-workspace admin update defaultIndex

changelogs/fragments/8675.yml

@@ -0,0 +1,2 @@
+fix:
+- [Workspace] Fix non-workspace admin update defaultIndex ([#8675](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/8675))

src/core/public/workspace/types.ts

@@ -5,7 +5,7 @@
 
 import { WorkspaceAttribute } from '../../types';
 
-export type WorkspaceObject = WorkspaceAttribute & { readonly?: boolean };
+export type WorkspaceObject = WorkspaceAttribute & { readonly?: boolean; owner?: boolean };
 
 export type IWorkspaceResponse<T> =
   | {

src/plugins/data/common/index_patterns/index_patterns/ensure_default_index_pattern.ts

@@ -36,13 +36,18 @@
 
 export const createEnsureDefaultIndexPattern = (
   uiSettings: UiSettingsCommon,
-  onRedirectNoIndexPattern: () => Promise<unknown> | void
+  onRedirectNoIndexPattern: () => Promise<unknown> | void,
+  isCurrentWorkspaceOwner?: boolean
 ) => {
   /**
    * Checks whether a default index pattern is set and exists and defines
    * one otherwise.
    */
   return async function ensureDefaultIndexPattern(this: IndexPatternsContract) {
+    // If workspace is enabled, only workspace owner can update ui setting.
+    if (isCurrentWorkspaceOwner === false) {
+      return;
+    }
     const patterns = await this.getIds();
     let defaultId = await uiSettings.get('defaultIndex');
     let defined = !!defaultId;

src/plugins/data/common/index_patterns/index_patterns/index_patterns.ts

@@ -74,6 +74,7 @@ interface IndexPatternsServiceDeps {
   onError: OnError;
   onRedirectNoIndexPattern?: () => void;
   onUnsupportedTimePattern: OnUnsupportedTimePattern;
+  isCurrentWorkspaceOwner?: boolean;
 }
 
 export class IndexPatternsService {
@@ -96,6 +97,7 @@ export class IndexPatternsService {
     onError,
     onUnsupportedTimePattern,
     onRedirectNoIndexPattern = () => {},
+    isCurrentWorkspaceOwner,
   }: IndexPatternsServiceDeps) {
     this.apiClient = apiClient;
     this.config = uiSettings;
@@ -106,7 +108,8 @@ export class IndexPatternsService {
     this.onUnsupportedTimePattern = onUnsupportedTimePattern;
     this.ensureDefaultIndexPattern = createEnsureDefaultIndexPattern(
       uiSettings,
-      onRedirectNoIndexPattern
+      onRedirectNoIndexPattern,
+      isCurrentWorkspaceOwner
     );
   }
 

src/plugins/data/public/plugin.ts

@@ -193,7 +193,15 @@ export class DataPublicPlugin
   }
 
   public start(core: CoreStart, { uiActions }: DataStartDependencies): DataPublicPluginStart {
-    const { uiSettings, http, notifications, savedObjects, overlays, application } = core;
+    const {
+      uiSettings,
+      http,
+      notifications,
+      savedObjects,
+      overlays,
+      application,
+      workspaces,
+    } = core;
     setNotifications(notifications);
     setOverlays(overlays);
     setUiSettings(uiSettings);
@@ -220,6 +228,9 @@ export class DataPublicPlugin
         notifications.toasts,
         application.navigateToApp
       ),
+      isCurrentWorkspaceOwner:
+        workspaces?.currentWorkspace$.getValue()?.owner ||
+        application.capabilities?.dashboards?.isDashboardAdmin !== false,
     });
     setIndexPatterns(indexPatterns);
 

src/plugins/workspace/public/workspace_client.test.ts

@@ -178,6 +178,7 @@ describe('#WorkspaceClient', () => {
     expect(workspaceMock.workspaceList$.getValue()).toEqual([
       {
         id: 'foo',
+        owner: true,
         readonly: false,
       },
     ]);

src/plugins/workspace/public/workspace_client.ts

@@ -117,13 +117,21 @@ export class WorkspaceClient implements IWorkspaceClient {
         perPage: 999,
         permissionModes: [WorkspacePermissionMode.LibraryWrite],
       });
-      if (resultWithWritePermission?.success) {
+      const resultWithOwnerPermission = await this.list({
+        perPage: 999,
+        permissionModes: [WorkspacePermissionMode.Write],
+      });
+      if (resultWithWritePermission?.success && resultWithOwnerPermission?.success) {
         const workspaceIdsWithWritePermission = resultWithWritePermission.result.workspaces.map(
           (workspace: WorkspaceAttribute) => workspace.id
         );
+        const workspaceIdsWithOwnerPermission = resultWithOwnerPermission.result.workspaces.map(
+          (workspace: WorkspaceAttribute) => workspace.id
+        );
         const workspaces = result.result.workspaces.map((workspace: WorkspaceAttribute) => ({
           ...workspace,
           readonly: !workspaceIdsWithWritePermission.includes(workspace.id),
+          owner: workspaceIdsWithOwnerPermission.includes(workspace.id),
         }));
         this.workspaces.workspaceList$.next(workspaces);
       }