[TSVB] Visualization crashes when it is opened from Metrics Ui (#98120)…

… (#98279) * [TSVB] Metrics UI crashes when it is opened from Metrics Ui * Fix bug on initialization Co-authored-by: Kibana Machine <[email protected]> Co-authored-by: Kibana Machine <[email protected]>
elastic · Apr 26, 2021 · 49ed9f7 · 49ed9f7
1 parent b4db266
commit 49ed9f7
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/src/plugins/vis_type_timeseries/public/application/components/vis_types/timeseries/config.js b/src/plugins/vis_type_timeseries/public/application/components/vis_types/timeseries/config.js
@@ -329,12 +329,17 @@ export const TimeseriesConfig = injectI18n(function (props) {
     ? props.model.series_index_pattern
     : props.indexPatternForQuery;
 
-  const initialPalette = {
-    ...model.palette,
+  const initialPalette = model.palette ?? {
+    type: 'palette',
+    name: 'default',
+  };
+
+  const palette = {
+    ...initialPalette,
     name:
       model.split_color_mode === 'kibana'
         ? 'kibana_palette'
-        : model.split_color_mode || model.palette.name,
+        : model.split_color_mode || initialPalette.name,
   };
 
   return (
@@ -427,7 +432,7 @@ export const TimeseriesConfig = injectI18n(function (props) {
             >
               <PalettePicker
                 palettes={palettesRegistry}
-                activePalette={initialPalette}
+                activePalette={palette}
                 setPalette={handlePaletteChange}
                 color={model.color}
               />

diff --git a/src/plugins/vis_type_timeseries/public/application/lib/get_split_by_terms_color.ts b/src/plugins/vis_type_timeseries/public/application/lib/get_split_by_terms_color.ts
@@ -57,7 +57,7 @@ export const getSplitByTermsColor = ({
         }
       : seriesPalette.params;
 
-  const outputColor = palettesRegistry?.get(paletteName).getColor(
+  const outputColor = palettesRegistry?.get(paletteName || 'default').getColor(
     [
       {
         name: seriesName || emptyLabel,