Skip to content

Commit bf592e6

Browse files
authored
Merge pull request #846 from Codeinwp/bugfix/845
Fix chart series indexing issue in table chart
2 parents 13408e7 + 8e44e61 commit bf592e6

File tree

1 file changed

+13
-6
lines changed

1 file changed

+13
-6
lines changed

js/render-google.js

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,12 @@ var __visualizer_chart_images = [];
2020
// remember, some charts do not support annotations so they should not be included in this.
2121
var no_annotation_charts = ['tabular', 'timeline', 'gauge', 'geo', 'bubble', 'candlestick'];
2222
if ( undefined !== chart.settings && undefined !== chart.settings.series && undefined === chart.settings.series.length ) {
23-
chart.settings.series = Object.values( chart.settings.series );
23+
var chartSeries = [];
24+
var chartSeriesValue = Object.values( chart.settings.series );
25+
$.each( Object.keys( chart.settings.series ), function( index, element ) {
26+
chartSeries[element] = chartSeriesValue[index];
27+
} );
28+
chart.settings.series = chartSeries;
2429
}
2530
if(id !== 'canvas' && typeof chart.series !== 'undefined' && typeof chart.settings.series !== 'undefined' && ! no_annotation_charts.includes(chart.type) ) {
2631
hasAnnotation = chart.series.length - chart.settings.series.length > 1;
@@ -310,11 +315,13 @@ var __visualizer_chart_images = [];
310315
var arr = id.split('-');
311316
__visualizer_chart_images[ arr[0] + '-' + arr[1] ] = '';
312317
try{
313-
var img = render.getImageURI();
314-
__visualizer_chart_images[ arr[0] + '-' + arr[1] ] = img;
315-
$('body').trigger('visualizer:render:chart', {id: arr[1], image: img});
316-
if ( $( '#chart-img' ).length ) {
317-
$( '#chart-img' ).val( img );
318+
if ( typeof render.getImageURI !== 'undefined' ) {
319+
var img = render.getImageURI();
320+
__visualizer_chart_images[ arr[0] + '-' + arr[1] ] = img;
321+
$('body').trigger('visualizer:render:chart', {id: arr[1], image: img});
322+
if ( $( '#chart-img' ).length ) {
323+
$( '#chart-img' ).val( img );
324+
}
318325
}
319326
}catch(error){
320327
var canvas = document.getElementById( 'canvas' );

0 commit comments

Comments
 (0)