From d76219514163367d57dbc4727e15215e4f0385b0 Mon Sep 17 00:00:00 2001
From: sulemanof <daniil_suleiman@epam.com>
Date: Tue, 24 Nov 2020 13:47:16 +0300
Subject: [PATCH 1/2] Remove string escaping

---
 src/plugins/vis_type_timelion/public/to_ast.ts | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/src/plugins/vis_type_timelion/public/to_ast.ts b/src/plugins/vis_type_timelion/public/to_ast.ts
index 7044bbf4e58318..535e8e8fe0f77e 100644
--- a/src/plugins/vis_type_timelion/public/to_ast.ts
+++ b/src/plugins/vis_type_timelion/public/to_ast.ts
@@ -21,14 +21,12 @@ import { buildExpression, buildExpressionFunction } from '../../expressions/publ
 import { Vis } from '../../visualizations/public';
 import { TimelionExpressionFunctionDefinition, TimelionVisParams } from './timelion_vis_fn';
 
-const escapeString = (data: string): string => {
-  return data.replace(/\\/g, `\\\\`).replace(/'/g, `\\'`);
-};
-
 export const toExpressionAst = (vis: Vis<TimelionVisParams>) => {
+  const { expression, interval } = vis.params;
+
   const timelion = buildExpressionFunction<TimelionExpressionFunctionDefinition>('timelion_vis', {
-    expression: escapeString(vis.params.expression),
-    interval: escapeString(vis.params.interval),
+    expression,
+    interval,
   });
 
   const ast = buildExpression([timelion]);

From 525bbfd7772d85bd7f1e4cac676fd3bbf630b779 Mon Sep 17 00:00:00 2001
From: sulemanof <daniil_suleiman@epam.com>
Date: Tue, 24 Nov 2020 14:07:26 +0300
Subject: [PATCH 2/2] Add unit test

---
 .../public/__snapshots__/to_ast.test.ts.snap  | 20 +++++++++++++++++++
 .../vis_type_timelion/public/to_ast.test.ts   |  6 ++++++
 2 files changed, 26 insertions(+)

diff --git a/src/plugins/vis_type_timelion/public/__snapshots__/to_ast.test.ts.snap b/src/plugins/vis_type_timelion/public/__snapshots__/to_ast.test.ts.snap
index 9e32a6c4ae17cb..7635e5214795ad 100644
--- a/src/plugins/vis_type_timelion/public/__snapshots__/to_ast.test.ts.snap
+++ b/src/plugins/vis_type_timelion/public/__snapshots__/to_ast.test.ts.snap
@@ -19,3 +19,23 @@ Object {
   "type": "expression",
 }
 `;
+
+exports[`timelion vis toExpressionAst function should not escape single quotes 1`] = `
+Object {
+  "chain": Array [
+    Object {
+      "arguments": Object {
+        "expression": Array [
+          ".es(index=my*,timefield=\\"date\\",split='test field:3',metric='avg:value')",
+        ],
+        "interval": Array [
+          "auto",
+        ],
+      },
+      "function": "timelion_vis",
+      "type": "function",
+    },
+  ],
+  "type": "expression",
+}
+`;
diff --git a/src/plugins/vis_type_timelion/public/to_ast.test.ts b/src/plugins/vis_type_timelion/public/to_ast.test.ts
index 8a9d4b83f94d20..f2030e4b83c197 100644
--- a/src/plugins/vis_type_timelion/public/to_ast.test.ts
+++ b/src/plugins/vis_type_timelion/public/to_ast.test.ts
@@ -37,4 +37,10 @@ describe('timelion vis toExpressionAst function', () => {
     const actual = toExpressionAst(vis);
     expect(actual).toMatchSnapshot();
   });
+
+  it('should not escape single quotes', () => {
+    vis.params.expression = `.es(index=my*,timefield="date",split='test field:3',metric='avg:value')`;
+    const actual = toExpressionAst(vis);
+    expect(actual).toMatchSnapshot();
+  });
 });