fix description, glossary and relatedTerm escape char issue

open-metadata · Sep 30, 2024 · 9734649 · 9734649
1 parent 2162217
commit 9734649
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 13 deletions.
diff --git a/openmetadata-ui/src/main/resources/ui/src/utils/CSV/CSV.utils.tsx b/openmetadata-ui/src/main/resources/ui/src/utils/CSV/CSV.utils.tsx
@@ -144,8 +144,6 @@ export const getCSVStringFromColumnsAndDataSource = (
           value.includes(',') ||
           value.includes('\n') ||
           colName.includes('tags') ||
-          colName.includes('glossaryTerms') ||
-          colName.includes('relatedTerms') ||
           colName.includes('domain')
         ) {
           return isEmpty(value) ? '' : `"${value}"`;

diff --git a/openmetadata-ui/src/main/resources/ui/src/utils/CSV/CSVUtilsClassBase.tsx b/openmetadata-ui/src/main/resources/ui/src/utils/CSV/CSVUtilsClassBase.tsx
@@ -37,7 +37,14 @@ class CSVUtilsClassBase {
   }
 
   public columnsWithMultipleValuesEscapeNeeded() {
-    return ['parent', 'extension', 'synonyms'];
+    return [
+      'parent',
+      'extension',
+      'synonyms',
+      'description',
+      'glossaryTerms',
+      'relatedTerms',
+    ];
   }
 
   public getEditor(
@@ -151,16 +158,10 @@ class CSVUtilsClassBase {
           ) => {
             if (Array.isArray(option)) {
               props.onChange(
-                option
-                  .map((tag) =>
-                    toString(tag.value)?.replace(new RegExp('"', 'g'), '""')
-                  )
-                  .join(';')
+                option.map((tag) => toString(tag.value)).join(';')
               );
             } else {
-              props.onChange(
-                toString(option.value)?.replace(new RegExp('"', 'g'), '""')
-              );
+              props.onChange(toString(option.value));
             }
           };
 
@@ -280,8 +281,9 @@ class CSVUtilsClassBase {
               popoverProps={{
                 open: true,
               }}
-              onUpdate={handleChange}
-            />
+              onUpdate={handleChange}>
+              {' '}
+            </UserTeamSelectableList>
           );
         };
       case 'extension':