Sortable: Pass this to _uiHash() when firing 'remove' events

Fixes #9738: Remove event object doesn't contain sender when moving an item between connected sortables.

Author: NiGhTTraX

tests/unit/sortable/sortable_events.js

@@ -227,11 +227,36 @@ test("#4752: link event firing on sortable with connect list", function () {
 test("receive", function() {
 	ok(false, "missing test - untested code is broken code.");
 });
+*/
 
 test("remove", function() {
-	ok(false, "missing test - untested code is broken code.");
+	expect( 8 );
+
+	var hash,
+		removeCount = 0,
+		item = $( "#sortable" ).find( "li:last" );
+
+	$( ".connectWith" ).sortable({
+		connectWith: ".connectWith",
+		remove: function( e, ui ) {
+			hash = ui;
+			removeCount++;
+		}
+	});
+
+	item.simulate( "drag", {
+		dy: 40
+	});
+
+	ok( hash, "remove event triggered" );
+	ok( !hash.helper, "UI should not include: helper" );
+	ok( hash.placeholder, "UI hash includes: placeholder" );
+	ok( hash.position && ( "top" in hash.position && "left" in hash.position ), "UI hash includes: position" );
+	ok( hash.offset && ( hash.offset.top && hash.offset.left ), "UI hash includes: offset" );
+	equal( hash.item.html(), item.html(), "UI hash includes: item" );
+	equal( hash.sender.html(), $( "#sortable" ).html(), "UI hash includes: sender" );
+	equal( removeCount, 1, "remove fires only once" );
 });
-*/
 
 test( "over", function() {
 	expect( 8 );

ui/jquery.ui.sortable.js

@@ -1181,7 +1181,7 @@ $.widget("ui.sortable", $.ui.mouse, {
 		// events.
 		if (this !== this.currentContainer) {
 			if(!noPropagation) {
-				delayedTriggers.push(function(event) { this._trigger("remove", event, this._uiHash()); });
+				delayedTriggers.push(function(event) { this._trigger("remove", event, this._uiHash(this)); });
 				delayedTriggers.push((function(c) { return function(event) { c._trigger("receive", event, this._uiHash(this)); };  }).call(this, this.currentContainer));
 				delayedTriggers.push((function(c) { return function(event) { c._trigger("update", event, this._uiHash(this));  }; }).call(this, this.currentContainer));
 			}