Fix argument propagation in _signal and weave/unweave

Author: mikaelkaron

src/application/widget.js

@@ -9,22 +9,25 @@ define([ "module", "../component/widget", "when" ], function ApplicationWidgetMo
 	var CHILDREN = "children";
 	var ARRAY_SLICE = Array.prototype.slice;
 
-	function forward(signal) {
+	function forward() {
 		var self = this;
-		var args = arguments;
+		var _signal = this.signal;
 		var children = self[CHILDREN];
 		var length = children ? children.length : 0;
 		var index = 0;
+		var args;
 
 		function next(_args) {
+			// Update args
 			args = _args || args;
 
+			// Return a chained promise of next callback, or a promise resolved with args
 			return length > index
-				? when(children[index++].signal(signal), next)
+				? when(_signal.apply(children[index++], args), next)
 				: when.resolve(args);
 		}
 
-		return next();
+		return next(ARRAY_SLICE.call(arguments));
 	}
 
 	return Widget.extend(function ApplicationWidget($element, name, children) {

src/component/widget.js

@@ -13,6 +13,8 @@ define([ "troopjs-core/component/gadget", "jquery", "troopjs-jquery/weave", "tro
 	var $TRIGGER = $.fn.trigger;
 	var $ON = $.fn.on;
 	var $OFF = $.fn.off;
+	var $WEAVE = $.fn.weave;
+	var $UNWEAVE = $.fn.unweave;
 	var $ELEMENT = "$element";
 	var $HANDLERS = "$handlers";
 	var ATTR_WEAVE = "[data-weave]";
@@ -147,18 +149,18 @@ define([ "troopjs-core/component/gadget", "jquery", "troopjs-jquery/weave", "tro
 
 		/**
 		 * Weaves all children of $element
-		 * @returns self
+		 * @returns {Promise} from $.fn.weave
 		 */
 		"weave" : function weave() {
-			return this[$ELEMENT].find(ATTR_WEAVE).weave();
+			return $WEAVE.apply(this[$ELEMENT].find(ATTR_WEAVE), arguments);
 		},
 
 		/**
 		 * Unweaves all children of $element _and_ self
-		 * @returns self
+		 * @returns {Promise} from $.fn.unweave
 		 */
 		"unweave" : function unweave() {
-			return this[$ELEMENT].find(ATTR_WOVEN).addBack().unweave();
+			return $UNWEAVE.apply(this[$ELEMENT].find(ATTR_WOVEN).addBack(), arguments);
 		},
 
 		/**