From 12895d8bd040afb930c5b8c55261ba15c59386a4 Mon Sep 17 00:00:00 2001
From: Alexey Antonov <alexwizp@gmail.com>
Date: Wed, 23 Jun 2021 18:48:22 +0300
Subject: [PATCH] [TSVB] fix tooltip on annotations with 's are not displayed
 correctly (#102892)

* [TSVB] tooltip on annotations with 's are not displayed correctly

Closes: #102631

* 'handlebars/dist/handlebars' -> 'handlebars'
---
 .../components/lib/{reorder.js => reorder.ts} |  2 +-
 ...lace_vars.test.js => replace_vars.test.ts} |  0
 .../lib/{replace_vars.js => replace_vars.ts}  | 24 +++++++++++++------
 .../components/lib/tick_formatter.js          |  2 +-
 .../components/vis_types/timeseries/vis.js    |  4 +++-
 5 files changed, 22 insertions(+), 10 deletions(-)
 rename src/plugins/vis_type_timeseries/public/application/components/lib/{reorder.js => reorder.ts} (85%)
 rename src/plugins/vis_type_timeseries/public/application/components/lib/{replace_vars.test.js => replace_vars.test.ts} (100%)
 rename src/plugins/vis_type_timeseries/public/application/components/lib/{replace_vars.js => replace_vars.ts} (77%)

diff --git a/src/plugins/vis_type_timeseries/public/application/components/lib/reorder.js b/src/plugins/vis_type_timeseries/public/application/components/lib/reorder.ts
similarity index 85%
rename from src/plugins/vis_type_timeseries/public/application/components/lib/reorder.js
rename to src/plugins/vis_type_timeseries/public/application/components/lib/reorder.ts
index 15c21e19af2a5c..a026b5bb2051e6 100644
--- a/src/plugins/vis_type_timeseries/public/application/components/lib/reorder.js
+++ b/src/plugins/vis_type_timeseries/public/application/components/lib/reorder.ts
@@ -6,7 +6,7 @@
  * Side Public License, v 1.
  */
 
-export const reorder = (list, startIndex, endIndex) => {
+export const reorder = (list: unknown[], startIndex: number, endIndex: number) => {
   const result = Array.from(list);
   const [removed] = result.splice(startIndex, 1);
   result.splice(endIndex, 0, removed);
diff --git a/src/plugins/vis_type_timeseries/public/application/components/lib/replace_vars.test.js b/src/plugins/vis_type_timeseries/public/application/components/lib/replace_vars.test.ts
similarity index 100%
rename from src/plugins/vis_type_timeseries/public/application/components/lib/replace_vars.test.js
rename to src/plugins/vis_type_timeseries/public/application/components/lib/replace_vars.test.ts
diff --git a/src/plugins/vis_type_timeseries/public/application/components/lib/replace_vars.js b/src/plugins/vis_type_timeseries/public/application/components/lib/replace_vars.ts
similarity index 77%
rename from src/plugins/vis_type_timeseries/public/application/components/lib/replace_vars.js
rename to src/plugins/vis_type_timeseries/public/application/components/lib/replace_vars.ts
index 458866f2098a0d..2862fe933bfb75 100644
--- a/src/plugins/vis_type_timeseries/public/application/components/lib/replace_vars.js
+++ b/src/plugins/vis_type_timeseries/public/application/components/lib/replace_vars.ts
@@ -6,20 +6,30 @@
  * Side Public License, v 1.
  */
 
-import _ from 'lodash';
-import handlebars from 'handlebars/dist/handlebars';
-import { emptyLabel } from '../../../../common/empty_label';
+import handlebars from 'handlebars';
 import { i18n } from '@kbn/i18n';
+import { emptyLabel } from '../../../../common/empty_label';
+
+type CompileOptions = Parameters<typeof handlebars.compile>[1];
 
-export function replaceVars(str, args = {}, vars = {}) {
+export function replaceVars(
+  str: string,
+  args: Record<string, unknown> = {},
+  vars: Record<string, unknown> = {},
+  compileOptions: Partial<CompileOptions> = {}
+) {
   try {
-    // we need add '[]' for emptyLabel because this value contains special characters. (https://handlebarsjs.com/guide/expressions.html#literal-segments)
+    /** we need add '[]' for emptyLabel because this value contains special characters.
+     * @see (https://handlebarsjs.com/guide/expressions.html#literal-segments) **/
     const template = handlebars.compile(str.split(emptyLabel).join(`[${emptyLabel}]`), {
       strict: true,
       knownHelpersOnly: true,
+      ...compileOptions,
+    });
+    const string = template({
+      ...vars,
+      args,
     });
-
-    const string = template(_.assign({}, vars, { args }));
 
     return string;
   } catch (e) {
diff --git a/src/plugins/vis_type_timeseries/public/application/components/lib/tick_formatter.js b/src/plugins/vis_type_timeseries/public/application/components/lib/tick_formatter.js
index 70529be78567d0..c1d82a182e509b 100644
--- a/src/plugins/vis_type_timeseries/public/application/components/lib/tick_formatter.js
+++ b/src/plugins/vis_type_timeseries/public/application/components/lib/tick_formatter.js
@@ -6,8 +6,8 @@
  * Side Public License, v 1.
  */
 
-import handlebars from 'handlebars/dist/handlebars';
 import { isNumber } from 'lodash';
+import handlebars from 'handlebars';
 import { isEmptyValue, DISPLAY_EMPTY_VALUE } from '../../../../common/last_value_utils';
 import { inputFormats, outputFormats, isDuration } from '../lib/durations';
 import { getFieldFormats } from '../../../services';
diff --git a/src/plugins/vis_type_timeseries/public/application/components/vis_types/timeseries/vis.js b/src/plugins/vis_type_timeseries/public/application/components/vis_types/timeseries/vis.js
index 8e59e8e1bb628a..097b0a7b5e3327 100644
--- a/src/plugins/vis_type_timeseries/public/application/components/vis_types/timeseries/vis.js
+++ b/src/plugins/vis_type_timeseries/public/application/components/vis_types/timeseries/vis.js
@@ -51,7 +51,9 @@ class TimeseriesVisualization extends Component {
   };
 
   applyDocTo = (template) => (doc) => {
-    const vars = replaceVars(template, null, doc);
+    const vars = replaceVars(template, null, doc, {
+      noEscape: true,
+    });
 
     if (vars instanceof Error) {
       this.showToastNotification = vars.error.caused_by;