ChartJsRenderer: Adjust handling of properties

- Update lineTension property when updating (only) data
- Do not overwrite dataset colors, if a function is set

Change-Id: I0ca4822f8ff2f45d83ae7c3aa6724da39fc24a6c
Reviewed-on: https://git.eclipse.org/r/c/scout/org.eclipse.scout.rt/+/180536
Tested-by: Scout Bot <scout-bot@eclipse.org>
Reviewed-by: Fritz Schinkel <fritz.schinkel@bsi-software.com>

1 files changed, 4 insertions, 0 deletions

diff --git a/eclipse-scout-chart/src/chart/ChartJsRenderer.js b/eclipse-scout-chart/src/chart/ChartJsRenderer.js
index a1c919395c..5d07355dd4 100644
--- a/eclipse-scout-chart/src/chart/ChartJsRenderer.js
+++ b/eclipse-scout-chart/src/chart/ChartJsRenderer.js
@@ -334,6 +334,7 @@ export default class ChartJsRenderer extends AbstractChartRenderer {
         transferProperty(sourceDataset, targetDataset, 'hoverBorderColor', true);
         transferProperty(sourceDataset, targetDataset, 'legendColor', true);
         transferProperty(sourceDataset, targetDataset, 'pointHoverBackgroundColor', true);
+        transferProperty(sourceDataset, targetDataset, 'lineTension', true);
       }
       let targetLength = targetData.datasets.length,
         sourceLength = sourceData.datasets.length;
@@ -1163,6 +1164,9 @@ export default class ChartJsRenderer extends AbstractChartRenderer {
         pointHoverBackgroundColor = colors.pointHoverColor;
 
       let setProperty = (identifier, value) => {
+        if (typeof elem[identifier] === 'function') {
+          return;
+        }
         if (value && value.length) {
           elem[identifier] = Array.isArray(value) ? [...value] : value;
         }