Fixed issue994: Raphael.parseTransformString() now returns a copy of the array of arr…

[byuksel]

Found&fixed the problem for issue #994. In this pull request, I modified Raphael.parseTransformString() such that it returns the copy of array of arrays from the cache instead of the actual array of arrays.

Here is the bug description:

In Raphael, Raphael.parseTransformString()'s output is cached and reused. In your move() method, you modify the output of Raphael.parseTransformString(), and Raphael tries to use your modified array when you supply it with the same string. This happens when the first drag() event is registered. You ask it to parse the current place, and then update the output array of arrays with the new location. And then, way later, when this.up() is called, you supply the Raphael.parseTransformString() with the same string. Raphael then uses your modified array of arrays. This is the fixed fiddle: http://jsfiddle.net/6ozsfdaf/16/

And here is the code (use a new array of arrays to transform when moved each time):

this.move = function(dx, dy) {
var ts = [];
ts.push(new Array('t'));
ts[0][1] = this.original_x + dx; 
ts[0][2] = this.original_y + dy; 
this.attr({transform: ts.toString()});
};

This can be fixed if Raphael.parseTransformString() always returned a copy of the original result.