gian/phpmyadmin
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Scope of JS variables defined in for loop declaration part if not local to the loop. See http://stackoverflow.com/questions/18465211/javascript-loop-variable-scope

Browse Source 
Signed-off-by: Madhura Jayaratne <madhura.cj@gmail.com>
This commit is contained in:
Madhura Jayaratne 2015-02-13 12:42:47 +05:30
parent 31aae4afa4
commit aaf0e8d903
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
js/chart.js
View File

@ -285,8 +285,8 @@ JQPlotLineChart.prototype.populateOptions = function (dataTable, options) {
}
if (optional.axes.xaxis.ticks.length === 0) {
var data = dataTable.getData();
for (var i = 0; i < data.length; i++) {
optional.axes.xaxis.ticks.push(data[i][0].toString());
for (var j = 0; j < data.length; j++) {
optional.axes.xaxis.ticks.push(data[j][0].toString());
}
}
return optional;
@ -574,9 +574,9 @@ JQPlotBarChart.prototype.populateOptions = function (dataTable, options) {
}
}
if (optional.series.length === 0) {
for (var i = 1; i < columns.length; i++) {
for (var j = 1; j < columns.length; j++) {
optional.series.push({
label : columns[i].name.toString()
label : columns[j].name.toString()
});
}
}