jquery core: Fixed #5202. Fixing selector generation when a manipulat…

…ion function receives a jQuery object.
jquery · Sep 14, 2009 · 0790989 · 0790989
1 parent 66c58e9
commit 0790989
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/src/manipulation.js b/src/manipulation.js
@@ -295,8 +295,7 @@ jQuery.each({
 			jQuery.fn[ original ].apply( jQuery(insert[i]), elems );
 			ret = ret.concat( elems );
 		}
-
-		return this.pushStack( ret, name, selector );
+		return this.pushStack( ret, name, insert.selector );
 	};
 });
 

diff --git a/test/unit/core.js b/test/unit/core.js
@@ -62,7 +62,7 @@ test("jQuery()", function() {
 });
 
 test("selector state", function() {
-	expect(30);
+	expect(31);
 
 	var test;
 
@@ -126,6 +126,13 @@ test("selector state", function() {
 	test = jQuery("#main").eq(0);
 	equals( test.selector, "#main.slice(0,1)", "#main eq Selector" );
 	equals( test.context, document, "#main eq Context" );
+
+	var d = "<div />";
+	equals(
+		jQuery(d).appendTo(jQuery(d)).selector,
+		jQuery(d).appendTo(d).selector,
+		"manipulation methods make same selector for jQuery objects"
+	);
 });
 
 test("browser", function() {