From: Scott González <scott.gonzalez@gmail.com>
Date: Fri, 9 Jan 2009 14:28:39 +0000 (+0000)
Subject: Widget factory: _trigger will now set event.result if the callback returns a value... 
X-Git-Tag: 1.6rc5~50
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=142fddbf85a2903ec745afc4c5d7e85da3c9de7a;p=jquery-ui.git

Widget factory: _trigger will now set event.result if the callback returns a value.  The created event is now always run through $.Event to reset isDefaultPrevent(), isPropagationStopped(), isImmediatePropagationStopped().
---

diff --git a/ui/ui.core.js b/ui/ui.core.js
index d5949c30d..03bb59849 100644
--- a/ui/ui.core.js
+++ b/ui/ui.core.js
@@ -370,13 +370,20 @@ $.widget.prototype = {
 			eventName = (type == this.widgetEventPrefix
 				? type : this.widgetEventPrefix + type);
 
-		event = event ? $.event.fix(event) : $.Event();
+		// event can be null, a hash, a native event, a fixed event
+		event = $.Event(event);
 		event.type = eventName;
 
 		this.element.trigger(event, data);
-
-		return !(callback && callback.call(this.element[0], event, data) === false
-			|| event.isDefaultPrevented());
+		var callbackResult = callback
+			? callback.call(this.element[0], event, data)
+			: undefined;
+		
+		event.result = callbackResult !== undefined
+			? callbackResult
+			: event.result;
+		
+		return event.result !== false;
 	}
 };