This is a fix for dojo ticket #18061.

dojo · Sep 4, 2015 · 656dbd1 · 656dbd1
1 parent 828883a
commit 656dbd1
Show file tree

Hide file tree

Showing 7 changed files with 7 additions and 8 deletions.
diff --git a/charting/plot2d/Bars.js b/charting/plot2d/Bars.js
@@ -164,7 +164,7 @@ define(["dojo/_base/lang", "dojo/_base/array", "dojo/_base/declare", "dojo/has",
 			var actualLength = this.series.length;
 			arr.forEach(this.series, function(serie){if(serie.hidden){actualLength--;}});
 			var z = actualLength;
-			for(var i = this.series.length - 1; i >= 0; --i){
+			for(var i = 0; i < this.series.length; i++){
 				var run = this.series[i];
 				if(!this.dirty && !run.dirty){
 					t.skip();

diff --git a/charting/plot2d/Bubble.js b/charting/plot2d/Bubble.js
@@ -72,7 +72,7 @@ define(["dojo/_base/lang", "dojo/_base/declare", "dojo/_base/array", "dojo/has",
 				vt = this._vScaler.scaler.getTransformerFromModel(this._vScaler),
 				events = this.events();
 
-			for(var i = this.series.length - 1; i >= 0; --i){
+			for(var i = 0; i < this.series.length; i++){
 				var run = this.series[i];
 				if(!this.dirty && !run.dirty){
 					t.skip();

diff --git a/charting/plot2d/Columns.js b/charting/plot2d/Columns.js
@@ -99,8 +99,7 @@ define(["dojo/_base/lang", "dojo/_base/array", "dojo/_base/declare", "dojo/has",
 
 			var z = this.series.length;
 			arr.forEach(this.series, function(serie){if(serie.hidden){z--;}});
-
-			for(var i = this.series.length - 1; i >= 0; --i){
+			for(var i = 0; i < this.series.length; i++){
 				var run = this.series[i];
 				if(!this.dirty && !run.dirty){
 					t.skip();

diff --git a/charting/plot2d/Default.js b/charting/plot2d/Default.js
@@ -222,7 +222,7 @@ define(["dojo/_base/lang", "dojo/_base/declare", "dojo/_base/array", "dojo/has",
 			}
 			var t = this.chart.theme, stroke, outline, events = this.events();
 
-			for(var i = this.series.length - 1; i >= 0; --i){
+			for(var i = 0; i < this.series.length; i++){
 				var run = this.series[i];
 				if(!this.dirty && !run.dirty){
 					t.skip();

diff --git a/charting/plot2d/OHLC.js b/charting/plot2d/OHLC.js
@@ -118,7 +118,7 @@ define(["dojo/_base/lang", "dojo/_base/array", "dojo/_base/declare", "dojo/has",
 			f = dc.calculateBarSize(this._hScaler.bounds.scale, this.opt);
 			gap = f.gap;
 			width = f.size;
-			for(var i = this.series.length - 1; i >= 0; --i){
+			for(var i = 0; i < this.series.length; i++){
 				var run = this.series[i];
 				if(!this.dirty && !run.dirty){
 					t.skip();

diff --git a/charting/plot2d/Scatter.js b/charting/plot2d/Scatter.js
@@ -58,7 +58,7 @@ define(["dojo/_base/lang", "dojo/_base/array", "dojo/_base/declare", "dojo/has",
 				df.forEachRev(this.series, function(item){ item.cleanGroup(s); });
 			}
 			var t = this.chart.theme, events = this.events();
-			for(var i = this.series.length - 1; i >= 0; --i){
+			for(var i = 0; i < this.series.length; i++){
 				var run = this.series[i];
 				if(!this.dirty && !run.dirty){
 					t.skip();

diff --git a/charting/plot2d/Spider.js b/charting/plot2d/Spider.js
@@ -194,7 +194,7 @@ define(["dojo/_base/lang", "dojo/_base/declare", "dojo/_base/connect", "dojo/_ba
 				r: r
 			};
 
-			for (i = this.series.length - 1; i >= 0; i--){
+			for(var i = 0; i < this.series.length; i++){
 				serieEntry = this.series[i];
 				if(!this.dirty && !serieEntry.dirty){
 					t.skip();