From 75dc69f42992a9e5f1d4bd9afc63b127665a2edd Mon Sep 17 00:00:00 2001
From: Uladzislau Lasitsa <Uladzislau_Lasitsa@epam.com>
Date: Fri, 24 Apr 2020 12:59:37 +0300
Subject: [PATCH] Some refactoring

---
 .../edit_index_pattern/edit_index_pattern.tsx | 24 +++++++++++--------
 1 file changed, 14 insertions(+), 10 deletions(-)

diff --git a/src/legacy/core_plugins/kibana/public/management/sections/index_patterns/edit_index_pattern/edit_index_pattern.tsx b/src/legacy/core_plugins/kibana/public/management/sections/index_patterns/edit_index_pattern/edit_index_pattern.tsx
index 49fc6308fdadad..e869ac84c2db26 100644
--- a/src/legacy/core_plugins/kibana/public/management/sections/index_patterns/edit_index_pattern/edit_index_pattern.tsx
+++ b/src/legacy/core_plugins/kibana/public/management/sections/index_patterns/edit_index_pattern/edit_index_pattern.tsx
@@ -97,12 +97,22 @@ export const EditIndexPattern = withRouter(
       indexPattern.fields.filter(field => field.type === 'conflict')
     );
     const [defaultIndex, setDefaultIndex] = useState<string>(config.get('defaultIndex'));
+    const [tags, setTags] = useState<any[]>([]);
 
     useEffect(() => {
       setFields(indexPattern.getNonScriptedFields());
       setConflictedFields(indexPattern.fields.filter(field => field.type === 'conflict'));
     }, [indexPattern, indexPattern.fields]);
 
+    useEffect(() => {
+      const indexPatternTags =
+        services.indexPatternManagement.list.getIndexPatternTags(
+          indexPattern,
+          indexPattern.id === defaultIndex
+        ) || [];
+      setTags(indexPatternTags);
+    }, [defaultIndex, indexPattern, services.indexPatternManagement.list]);
+
     const setDefaultPattern = useCallback(() => {
       config.set('defaultIndex', indexPattern.id);
       setDefaultIndex(indexPattern.id || '');
@@ -158,14 +168,10 @@ export const EditIndexPattern = withRouter(
       }
     );
 
-    indexPattern.tags =
-      services.indexPatternManagement.list.getIndexPatternTags(
-        indexPattern,
-        indexPattern.id === defaultIndex
-      ) || [];
-
     services.docTitle.change(indexPattern.title);
 
+    const showTagsSection = Boolean(indexPattern.timeFieldName || (tags && tags.length > 0));
+
     return (
       <EuiFlexGroup direction="column">
         <EuiFlexItem>
@@ -177,16 +183,14 @@ export const EditIndexPattern = withRouter(
             defaultIndex={defaultIndex}
           />
           <EuiSpacer size="s" />
-          {Boolean(
-            indexPattern.timeFieldName || (indexPattern.tags && indexPattern.tags.length > 0)
-          ) && (
+          {showTagsSection && (
             <EuiFlexGroup wrap>
               {Boolean(indexPattern.timeFieldName) && (
                 <EuiFlexItem grow={false}>
                   <EuiBadge color="warning">{timeFilterHeader}</EuiBadge>
                 </EuiFlexItem>
               )}
-              {indexPattern.tags.map((tag: any) => (
+              {tags.map((tag: any) => (
                 <EuiFlexItem grow={false} key={tag.key}>
                   <EuiBadge color="hollow">{tag.name}</EuiBadge>
                 </EuiFlexItem>