Merge pull request #1509 from BlueBrain/studio-enable-search

Studio // Dont override column config when one aleady exists
BlueBrain · Feb 23, 2024 · a97cd4c · a97cd4c
2 parents 833ba78 + fe0d8a2
commit a97cd4c
Show file tree

Hide file tree

Showing 2 changed files with 37 additions and 22 deletions.
diff --git a/src/shared/components/EditTableForm.tsx b/src/shared/components/EditTableForm.tsx
@@ -274,14 +274,18 @@ const EditTableForm: React.FC<{
           return Object.assign(result, current);
         }, {});
 
-        return Object.keys(mergedItem).map(title => ({
-          '@type': '',
-          name: title,
-          format: '',
-          enableSearch: false,
-          enableSort: false,
-          enableFilter: false,
-        }));
+        return Object.keys(mergedItem).map(title => {
+          const currentItemConfig = findCurrentColumnConfig(title);
+          return {
+            '@type': '',
+            name: title,
+            format: '',
+            enableSearch: false,
+            enableSort: false,
+            enableFilter: false,
+            ...currentItemConfig,
+          };
+        });
       }
 
       const result = await querySparql(
@@ -296,14 +300,19 @@ const EditTableForm: React.FC<{
             .sort((a, b) => {
               return a.title > b.title ? 1 : -1;
             })
-            .map(x => ({
-              '@type': 'text',
-              name: x.dataIndex,
-              format: '',
-              enableSearch: false,
-              enableSort: false,
-              enableFilter: false,
-            }));
+            .map(header => {
+              const currentHeaderConfig =
+                findCurrentColumnConfig(header.dataIndex) ?? {};
+              return {
+                '@type': 'text',
+                name: header.dataIndex,
+                format: '',
+                enableSearch: false,
+                enableSort: false,
+                enableFilter: false,
+                ...currentHeaderConfig,
+              };
+            });
         })
         .catch(error => {
           // Sometimes delta's error message can be in `name` or `reason` field.
@@ -323,12 +332,7 @@ const EditTableForm: React.FC<{
     {
       onSuccess: data => {
         updateTableDataError(null);
-        if (
-          isNil(configuration) ||
-          (configuration as TableColumn[]).length === 0
-        ) {
-          setConfiguration(data);
-        }
+        setConfiguration(data);
       },
       onError: (error: Error) => {
         updateTableDataError(error);
@@ -340,6 +344,16 @@ const EditTableForm: React.FC<{
     }
   );
 
+  const findCurrentColumnConfig = (name: string) => {
+    if (Array.isArray(configuration)) {
+      return configuration.find(column => column.name === name);
+    }
+    if (configuration?.name === name) {
+      return { ...configuration };
+    }
+    return undefined;
+  };
+
   const onChangeName = (event: any) => {
     setName(event.target.value);
     setNameError(false);

diff --git a/src/shared/hooks/useAccessDataForTable.tsx b/src/shared/hooks/useAccessDataForTable.tsx
@@ -633,6 +633,7 @@ export const useAccessDataForTable = (
         );
         return result;
       }
+
       return {};
     },
     {
-Original file line number
+Diff line change
@@ Expand Up / @@ -633,6 +633,7 @@ export const useAccessDataForTable = ( @@
             );
             return result;
           }
           return {};
         },
         {
@@ Expand Down @@