fix #18563 - can't access property toString, data[j][0] is null (#18736)

talktogauravsingh · MauricioFauth · web-flow · commit e4642a7dfbee · 2023-11-15T19:45:36.000-03:00
* fix #18563 - can't access property toString, data[j][0] is null Signed-off-by: Gaurav Singh <talktoogaurav@gmail.com> * Update js/src/chart.js --------- Signed-off-by: Gaurav Singh <talktoogaurav@gmail.com> Co-authored-by: Maurício Meneghini Fauth <mauricio@fauth.dev>
diff --git a/js/src/chart.js b/js/src/chart.js
@@ -287,7 +287,7 @@ JQPlotLineChart.prototype.populateOptions = function (dataTable, options) {
     if (optional.axes.xaxis.ticks.length === 0) {
         var data = dataTable.getData();
         for (var j = 0; j < data.length; j++) {
-            optional.axes.xaxis.ticks.push(data[j][0].toString());
+            optional.axes.xaxis.ticks.push(data[j][0] !== null ? data[j][0].toString() : null);
         }
     }
     return optional;

Original file line number	Diff line number	Diff line change
`@@ -287,7 +287,7 @@ JQPlotLineChart.prototype.populateOptions = function (dataTable, options) {`
`287`	`287`	`if (optional.axes.xaxis.ticks.length === 0) {`
`288`	`288`	`var data = dataTable.getData();`
`289`	`289`	`for (var j = 0; j < data.length; j++) {`
`290`		`- optional.axes.xaxis.ticks.push(data[j][0].toString());`
	`290`	`+ optional.axes.xaxis.ticks.push(data[j][0] !== null ? data[j][0].toString() : null);`
`291`	`291`	`}`
`292`	`292`	`}`
`293`	`293`	`return optional;`