fix(Highcharts plugin): fix callManageTooltipConfig invocation (#504)

gravity-ui · Jul 26, 2024 · 552156f · 552156f
1 parent 7a8b4e4
commit 552156f
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/src/plugins/highcharts/renderer/helpers/config/config.js b/src/plugins/highcharts/renderer/helpers/config/config.js
@@ -546,7 +546,10 @@ function getTooltip(tooltip, options, comments, holidays) {
         };
 
         if (typeof options.manageTooltipConfig === 'function') {
-            json = callManageTooltipConfig(options, json, chart);
+            // manageTooltipConfig could accidently break json variable
+            // and because of it we create new object as an argument
+            const updatedJSON = callManageTooltipConfig(options, {...json}, chart);
+            merge(json, updatedJSON);
         }
 
         const usersPointFormat = get(options.highcharts, 'tooltip.pointFormat');
@@ -855,7 +858,10 @@ function getTooltip(tooltip, options, comments, holidays) {
 
     if (json.lines && json.lines.length > 0) {
         if (typeof options.manageTooltipConfig === 'function') {
-            json = callManageTooltipConfig(options, json, chart);
+            // manageTooltipConfig could accidently break json variable
+            // and because of it we create new object as an argument
+            const updatedJSON = callManageTooltipConfig(options, {...json}, chart);
+            merge(json, updatedJSON);
         }
 
         let hiddenRowsSum = 0;