BREAKING CHANGE: Tween#chain deprecated. Use Timeline global method instead

Author: dalisoft

package.json

@@ -1,8 +1,8 @@
 {
 	"standard": {
 		"ignore": [
-			"/dist/",
-			"/test/"
+			"dist/",
+			"test/"
 		]
 	},
 	"name": "es6-tween",
@@ -19,8 +19,8 @@
 		"source": "rollup -c --sourcemap",
 		"bundle": "npm run source && npm run build",
 		"dev": "rollup -c -w --sourcemap",
-		"test": "npm run test-unit && npm run lint",
 		"test-unit": "nodeunit test/unit/nodeunitheadless.js",
+		"test": "npm run test-unit && npm run fix && npm run lint",
 		"fix": "standard --fix",
 		"lint": "standard --verbose | snazzy",
 		"semantic-release": "semantic-release pre && npm publish && semantic-release post --owner=sole --user=dalisoft"

src/Tween.js

@@ -6,7 +6,7 @@ import './shim/raf'
 
 import './shim/isArray'
 
-import { getAll, removeAll, remove, add, now, update, autoPlay, on, once, off, emit } from './dist/core'
+import { has, get, getAll, removeAll, remove, add, now, update, autoPlay, on, once, off, emit } from './dist/core'
 
 import Easing from './dist/Easing'
 
@@ -22,4 +22,4 @@ import Plugins from './dist/Plugins'
 
 import { TweenInit } from './dist/Decorators'
 
-export { TweenInit, getAll, removeAll, remove, add, now, update, autoPlay, on, once, off, emit, Tween, Easing, Interpolation, Composite, Timeline, Plugins }
+export { TweenInit, has, get, getAll, removeAll, remove, add, now, update, autoPlay, on, once, off, emit, Tween, Easing, Interpolation, Composite, Timeline, Plugins }

src/dist/Tween.js

@@ -327,7 +327,6 @@ class Tween {
             remove(this);
             this._isPlaying = false;
 
-            this.stopChainedTweens();
             return this.emit('stop', object);
 
         }
@@ -340,17 +339,6 @@ class Tween {
 
             return this.update(_startTime + _duration);
 
-        }
-        stopChainedTweens() {
-
-            let {
-                _chainedTweens = []
-            } = this;
-
-            _chainedTweens.map(item => item.stop());
-
-            return this;
-
         }
         delay(amount) {
 
@@ -401,13 +389,6 @@ class Tween {
 
             return this;
 
-        }
-        chain(...args) {
-
-            this._chainedTweens = args;
-
-            return this;
-
         }
         get(time) {
             this.update(time);
@@ -417,7 +398,6 @@ class Tween {
 
             let {
                 _onStartCallbackFired,
-                _chainedTweens,
                 _easingFunction,
                 _interpolationFunction,
                 _repeat,
@@ -541,10 +521,6 @@ class Tween {
 
                     this.emit('complete', object);
 
-                    if (_chainedTweens) {
-                        _chainedTweens.map(tween => tween.start(_startTime + _duration));
-                    }
-
                     return false;
 
                 }

src/dist/core.js

@@ -8,22 +8,11 @@ let _events = {};
 let root = typeof (window) !== "undefined" ? window : typeof (global) !== "undefined" ? global : {};
 let _nextId = 0;
 
-const nextId = () => {
-	_nextId++;
-	return _nextId;
-}
-
-const getAll = () => {
-	return _tweens;
-}
-
-const autoPlay = (state) => {
-	_autoPlay = state;
-}
-
-const removeAll = () => {
-	_tweens = {}
-}
+Object.defineProperty(_tweens, "length", {
+	enumerable: false,
+	writable: true,
+	value: 0
+});
 
 const emit = function(name, a, b, c, d, e) {
 	let eventFn = _events[name];
@@ -36,6 +25,23 @@ const emit = function(name, a, b, c, d, e) {
 	}
 }
 
+const on = (ev, fn) => {
+	if (_events[ev] === undefined) {
+		_events[ev] = [];
+	}
+	_events[ev].push(fn);
+}
+
+const once = (ev, fn) => {
+	if (_events[ev] === undefined) {
+		_events[ev] = [];
+	}
+	on(ev, (...args) => {
+		fn(...args);
+		off(ev);
+	});
+}
+
 const off = (ev, fn) => {
 	if (ev === undefined || _events[ev] === undefined) {
 		return;
@@ -54,9 +60,10 @@ const off = (ev, fn) => {
 	}
 }
 
-const add = (tween) => {
+const add = tween => {
 	let { id } = tween;
 	_tweens[id] = tween;
+	_tweens.length++;
 
 	if (_autoPlay && !isStarted) {
 		update();
@@ -67,28 +74,57 @@ const add = (tween) => {
 
 }
 
-const on = (ev, fn) => {
-	if (_events[ev] === undefined) {
-		_events[ev] = [];
-	}
-	_events[ev].push(fn);
+const nextId = () => {
+	let id = _nextId;
+	_nextId++;
+	return id;
 }
 
-const once = (ev, fn) => {
-	if (_events[ev] === undefined) {
-		_events[ev] = [];
-	}
-	on(ev, (...args) => {
-		fn(...args);
-		off(ev);
+const getAll = () => {
+	return _tweens;
+}
+
+const autoPlay = (state) => {
+	_autoPlay = state;
+}
+
+const removeAll = () => {
+	_tweens = {}
+
+	Object.defineProperty(_tweens, "length", {
+		enumerable: false,
+		writable: true,
+		value: 0
 	});
 }
 
-const remove = (tween) => {
+const get = tween => {
+
+	for ( let searchTween in _tweens ) {
+
+	if (tween.id === +searchTween) {
+
+		return _tweens[+searchTween];
+
+	}
+
+	}
+
+	return null;
+
+}
+
+const has = tween => {
+
+	return get(tween) !== null;
+
+}
+
+const remove = tween => {
 	for ( let searchTween in _tweens ) {
-		if (tween.id === searchTween.id) {
-			_tweens[searchTween] = null;
-			delete _tweens[searchTween];
+		if (tween.id === +searchTween) {
+			delete _tweens[+searchTween];
+			_tweens.length--;
 		}
 	}
 }
@@ -142,8 +178,8 @@ const update = (time, preserve) => {
 		if (_tweens[i].update(time) || preserve) {
 			i++;
 		} else {
-			_tweens[i] = null;
-			delete _tweens[i];
+			delete _tweens[+i];
+			_tweens.length--;
 		}
 
 	}
@@ -162,11 +198,16 @@ if (root.document) {
 			timePause = now();
 		} else {
 			timeDiff = now() - timePause;
-			_tweens.map(tween => tween._startTime += timeDiff)
+
+			for ( let tween in _tweens ) {
+
+			tween._startTime += timeDiff
+
+			}
 
 		}
 		return true;
 	})
 }
 
-export { nextId, getAll, removeAll, remove, add, now, update, autoPlay, on, once, off, emit };
+export { get, has, nextId, getAll, removeAll, remove, add, now, update, autoPlay, on, once, off, emit };