diff options
author | Michal Čihař <mcihar@suse.cz> | 2013-04-15 14:05:44 +0400 |
---|---|---|
committer | Michal Čihař <mcihar@suse.cz> | 2013-04-15 14:05:44 +0400 |
commit | 502165de66689e8c5f661d663395137809208a7d (patch) | |
tree | ee07ad9355bda115fbb93cd3adb64e958d4e73fd /js/tbl_chart.js | |
parent | 020a3e1ce401163b6000022a723748721823e67a (diff) |
Explicitly define radix for parseInt
This prevents unexpected behavior when parsing string like 033, which
would be treated as octal.
Diffstat (limited to 'js/tbl_chart.js')
-rw-r--r-- | js/tbl_chart.js | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/js/tbl_chart.js b/js/tbl_chart.js index de1099397c..70a6734999 100644 --- a/js/tbl_chart.js +++ b/js/tbl_chart.js @@ -46,7 +46,7 @@ AJAX.registerOnload('tbl_chart.js', function() { yaxisLabel : $('input[name="yaxis_label"]').val(), title : $('input[name="chartTitle"]').val(), stackSeries : false, - mainAxis : parseInt($('select[name="chartXAxis"]').val()), + mainAxis : parseInt($('select[name="chartXAxis"]').val(), 10), selectedSeries : getSelectedSeries() }; @@ -92,12 +92,12 @@ AJAX.registerOnload('tbl_chart.js', function() { var dateTimeCols = []; var vals = $('input[name="dateTimeCols"]').val().split(' '); $.each(vals, function(i, v) { - dateTimeCols.push(parseInt(v)); + dateTimeCols.push(parseInt(v, 10)); }); // handle changing the x-axis $('select[name="chartXAxis"]').change(function() { - currentSettings.mainAxis = parseInt($(this).val()); + currentSettings.mainAxis = parseInt($(this).val(), 10); if (dateTimeCols.indexOf(currentSettings.mainAxis) != -1) { $('span.span_timeline').show(); } else { @@ -214,7 +214,7 @@ function getSelectedSeries() { var val = $('select[name="chartSeries"]').val() || []; var ret = []; $.each(val, function(i, v) { - ret.push(parseInt(v)); + ret.push(parseInt(v, 10)); }); return ret; } |