unbind handlers with data + test (#935)

src/event/event.js

@@ -32,8 +32,11 @@ jQuery.event = {
 		}
 
 		// Make sure that the function being executed has a unique ID
-		if ( !handler.guid )
+		if ( !handler.guid ) {
 			handler.guid = this.guid++;
+			// Don't forget to set guid for the original handler function
+			if (fn) fn.guid = handler.guid;
+		}
 
 		// Init the element's event structure
 		if (!element.$events)

src/event/eventTest.js

 module("event");
 
 test("bind()", function() {
-	expect(11);
+	expect(12);
 
 	var handler = function(event) {
 		ok( event.data, "bind() with data, check passed data exists" );
 		ok( event.data.foo == "bar", "bind() with data, Check value of passed data" );
 	};
-	$("#firstp").bind("click", {foo: "bar"}, handler).click();
+	$("#firstp").bind("click", {foo: "bar"}, handler).click().unbind("click", handler);
+	
+	ok( !$("#firstp").get(0).$events, "Event handler unbound when using data." );
 	
 	reset();
 	var handler = function(event, data) {
@@ -16,12 +18,14 @@ test("bind()", function() {
 		ok( data, "Check trigger data" );
 		ok( data.bar == "foo", "Check value of trigger data" );
 	};
-	$("#firstp").bind("click", {foo: "bar"}, handler).trigger("click", [{bar: "foo"}]);
+	$("#firstp").bind("click", {foo: "bar"}, handler).trigger("click", [{bar: "foo"}]).unbind(handler);
 	
+	reset();
 	var handler = function(event) {
 		ok ( !event.data, "Check that no data is added to the event object" );
 	};
-	$("#firstp").unbind().bind("click", handler).trigger("click");
+	$("#firstp").bind("click", handler).trigger("click");
+	
 	
 	// events don't work with iframes, see #939
 	var tmp = document.createElement('iframe');