fixed play in yoyo mode, bug found by @Ben-Millions phaserjs#3837

khaleb85 · khaleb85 · commit a1bb80997e8c · 2018-09-01T10:48:23.000-03:00
diff --git a/src/animations/Animation.js b/src/animations/Animation.js
@@ -583,6 +583,12 @@ var Animation = new Class({
             //  Yoyo? (happens before repeat)
             if (component._yoyo)
             {
+                if (component._reverse !== false)
+                {
+                    this.completeAnimation(component);
+                    return;
+                }
+                
                 component.forward = false;
                 this._updateAndGetNextTick(component, frame.prevFrame);
             }
@@ -643,6 +649,12 @@ var Animation = new Class({
 
             if (component._yoyo)
             {
+                if (component._reverse !== true)
+                {
+                    this.completeAnimation(component);
+                    return;
+                }
+                
                 component.forward = true;
                 this._updateAndGetNextTick(component, frame.nextFrame);
             }

Original file line number	Diff line number	Diff line change
`@@ -583,6 +583,12 @@ var Animation = new Class({`
`583`	`583`	`// Yoyo? (happens before repeat)`
`584`	`584`	`if (component._yoyo)`
`585`	`585`	`{`
	`586`	`+ if (component._reverse !== false)`
	`587`	`+ {`
	`588`	`+ this.completeAnimation(component);`
	`589`	`+ return;`
	`590`	`+ }`
	`591`	`+`
`586`	`592`	`component.forward = false;`
`587`	`593`	`this._updateAndGetNextTick(component, frame.prevFrame);`
`588`	`594`	`}`
`@@ -643,6 +649,12 @@ var Animation = new Class({`
`643`	`649`
`644`	`650`	`if (component._yoyo)`
`645`	`651`	`{`
	`652`	`+ if (component._reverse !== true)`
	`653`	`+ {`
	`654`	`+ this.completeAnimation(component);`
	`655`	`+ return;`
	`656`	`+ }`
	`657`	`+`
`646`	`658`	`component.forward = true;`
`647`	`659`	`this._updateAndGetNextTick(component, frame.nextFrame);`
`648`	`660`	`}`