Issue cocos2d#739: Updating BezierBy, CardinalSplineTo with stackable actions.

Author: rboyd

cocos2d/actions/CCActionCatmullRom.js

@@ -146,6 +146,9 @@ cc.CardinalSplineTo = cc.ActionInterval.extend(/** @lends cc.CardinalSplineTo# *
     startWithTarget:function (target) {
         this._super(target);
         this._deltaT = 1 / this._points.length;
+
+        this._previousPosition = this._target.getPosition();
+        this._accumulatedDiff = cc.p(0, 0);
     },
 
     /**
@@ -169,6 +172,14 @@ cc.CardinalSplineTo = cc.ActionInterval.extend(/** @lends cc.CardinalSplineTo# *
             cc.getControlPointAt( this._points, p + 1),
             cc.getControlPointAt( this._points, p + 2),
             this._tension, lt);
+
+        var node = this._target;
+        var diff = cc.pSub(node.getPosition(), this._previousPosition);
+        if (diff.x != 0 || diff.y != 0) {
+            this._accumulatedDiff = cc.pAdd(this._accumulatedDiff, diff);
+            newPos = cc.pAdd(newPos, this._accumulatedDiff);
+        }
+ 
         this.updatePosition(newPos);
     },
 
@@ -187,6 +198,7 @@ cc.CardinalSplineTo = cc.ActionInterval.extend(/** @lends cc.CardinalSplineTo# *
      */
     updatePosition:function (newPos) {
         this._target.setPosition(newPos);
+        this._previousPosition = newPos;
     },
 
     /**
@@ -295,7 +307,9 @@ cc.CardinalSplineBy = cc.CardinalSplineTo.extend(/** @lends cc.CardinalSplineBy#
      * @param {cc.Point} newPos
      */
     updatePosition:function (newPos) {
-        this._target.setPosition(cc.pAdd(newPos, this._startPosition));
+        var p = cc.pAdd(newPos, this._startPosition);
+        this._target.setPosition(p)
+        this._previousPosition = p;
     }
 });
 

cocos2d/actions/CCActionInterval.js

@@ -1198,7 +1198,7 @@ cc.BezierBy = cc.ActionInterval.extend(/** @lends cc.BezierBy# */{
      */
     startWithTarget:function (target) {
         cc.ActionInterval.prototype.startWithTarget.call(this, target);
-        this._startPosition = target.getPosition();
+        this._previousPosition = this._startPosition = target.getPosition();
     },
 
     /**
@@ -1218,7 +1218,14 @@ cc.BezierBy = cc.ActionInterval.extend(/** @lends cc.BezierBy# */{
 
             var x = cc.bezierat(xa, xb, xc, xd, time);
             var y = cc.bezierat(ya, yb, yc, yd, time);
-            this._target.setPosition(cc.pAdd(this._startPosition, cc.p(x, y)));
+
+            var currentPos = this._target.getPosition();
+            var diff = cc.pSub(currentPos, this._previousPosition);
+            this._startPosition = cc.pAdd(this._startPosition, diff);
+            var newPos = cc.pAdd(this._startPosition, cc.p(x y));
+
+            this._target.setPosition(newPos);
+            this._previousPosition = newPos;
         }
     },