Commit e2ec5da2 authored by Dave Methvin's avatar Dave Methvin

Core: Remove deprecated context and selector properties

Fixes gh-1908
Closes gh-2000
(cherry picked from commit 0ea8c32863af31fb5cfc184e8d513bbae35583e8)
parent 26150f09
...@@ -39,9 +39,6 @@ jQuery.fn = jQuery.prototype = { ...@@ -39,9 +39,6 @@ jQuery.fn = jQuery.prototype = {
constructor: jQuery, constructor: jQuery,
// Start with an empty selector
selector: "",
// The default length of a jQuery object is 0 // The default length of a jQuery object is 0
length: 0, length: 0,
...@@ -70,7 +67,6 @@ jQuery.fn = jQuery.prototype = { ...@@ -70,7 +67,6 @@ jQuery.fn = jQuery.prototype = {
// Add the old object onto the stack (as a reference) // Add the old object onto the stack (as a reference)
ret.prevObject = this; ret.prevObject = this;
ret.context = this.context;
// Return the newly-formed element set // Return the newly-formed element set
return ret; return ret;
......
...@@ -75,12 +75,9 @@ var rootjQuery, ...@@ -75,12 +75,9 @@ var rootjQuery,
if ( elem ) { if ( elem ) {
// Inject the element directly into the jQuery object // Inject the element directly into the jQuery object
this.length = 1;
this[0] = elem; this[0] = elem;
this.length = 1;
} }
this.context = document;
this.selector = selector;
return this; return this;
} }
...@@ -96,7 +93,7 @@ var rootjQuery, ...@@ -96,7 +93,7 @@ var rootjQuery,
// HANDLE: $(DOMElement) // HANDLE: $(DOMElement)
} else if ( selector.nodeType ) { } else if ( selector.nodeType ) {
this.context = this[0] = selector; this[0] = selector;
this.length = 1; this.length = 1;
return this; return this;
...@@ -109,11 +106,6 @@ var rootjQuery, ...@@ -109,11 +106,6 @@ var rootjQuery,
selector( jQuery ); selector( jQuery );
} }
if ( selector.selector !== undefined ) {
this.selector = selector.selector;
this.context = selector.context;
}
return jQuery.makeArray( selector, this ); return jQuery.makeArray( selector, this );
}; };
......
...@@ -72,10 +72,7 @@ jQuery.fn.extend({ ...@@ -72,10 +72,7 @@ jQuery.fn.extend({
jQuery.find( selector, self[ i ], ret ); jQuery.find( selector, self[ i ], ret );
} }
// Needed because $( selector, context ) becomes $( context ).find( selector ) return this.pushStack( len > 1 ? jQuery.unique( ret ) : ret );
ret = this.pushStack( len > 1 ? jQuery.unique( ret ) : ret );
ret.selector = this.selector ? this.selector + " " + selector : selector;
return ret;
}, },
filter: function( selector ) { filter: function( selector ) {
return this.pushStack( winnow(this, selector || [], false) ); return this.pushStack( winnow(this, selector || [], false) );
......
...@@ -57,7 +57,7 @@ test("jQuery()", function() { ...@@ -57,7 +57,7 @@ test("jQuery()", function() {
equal( jQuery(undefined).length, 0, "jQuery(undefined) === jQuery([])" ); equal( jQuery(undefined).length, 0, "jQuery(undefined) === jQuery([])" );
equal( jQuery(null).length, 0, "jQuery(null) === jQuery([])" ); equal( jQuery(null).length, 0, "jQuery(null) === jQuery([])" );
equal( jQuery("").length, 0, "jQuery('') === jQuery([])" ); equal( jQuery("").length, 0, "jQuery('') === jQuery([])" );
equal( jQuery(obj).selector, "div", "jQuery(jQueryObj) == jQueryObj" ); deepEqual( jQuery(obj).get(), obj.get(), "jQuery(jQueryObj) == jQueryObj" );
// Invalid #id goes to Sizzle which will throw an error (gh-1682) // Invalid #id goes to Sizzle which will throw an error (gh-1682)
try { try {
...@@ -156,49 +156,6 @@ test("jQuery(selector, context)", function() { ...@@ -156,49 +156,6 @@ test("jQuery(selector, context)", function() {
deepEqual( jQuery("div p", jQuery("#qunit-fixture")).get(), q("sndp", "en", "sap"), "Basic selector with jQuery object as context" ); deepEqual( jQuery("div p", jQuery("#qunit-fixture")).get(), q("sndp", "en", "sap"), "Basic selector with jQuery object as context" );
}); });
test( "selector state", function() {
expect( 18 );
var test;
test = jQuery( undefined );
equal( test.selector, "", "Empty jQuery Selector" );
equal( test.context, undefined, "Empty jQuery Context" );
test = jQuery( document );
equal( test.selector, "", "Document Selector" );
equal( test.context, document, "Document Context" );
test = jQuery( document.body );
equal( test.selector, "", "Body Selector" );
equal( test.context, document.body, "Body Context" );
test = jQuery("#qunit-fixture");
equal( test.selector, "#qunit-fixture", "#qunit-fixture Selector" );
equal( test.context, document, "#qunit-fixture Context" );
test = jQuery("#notfoundnono");
equal( test.selector, "#notfoundnono", "#notfoundnono Selector" );
equal( test.context, document, "#notfoundnono Context" );
test = jQuery( "#qunit-fixture", document );
equal( test.selector, "#qunit-fixture", "#qunit-fixture Selector" );
equal( test.context, document, "#qunit-fixture Context" );
test = jQuery( "#qunit-fixture", document.body );
equal( test.selector, "#qunit-fixture", "#qunit-fixture Selector" );
equal( test.context, document.body, "#qunit-fixture Context" );
// Test cloning
test = jQuery( test );
equal( test.selector, "#qunit-fixture", "#qunit-fixture Selector" );
equal( test.context, document.body, "#qunit-fixture Context" );
test = jQuery( document.body ).find("#qunit-fixture");
equal( test.selector, "#qunit-fixture", "#qunit-fixture find Selector" );
equal( test.context, document.body, "#qunit-fixture find Context" );
});
test( "globalEval", function() { test( "globalEval", function() {
expect( 3 ); expect( 3 );
Globals.register("globalEvalTest"); Globals.register("globalEvalTest");
......
...@@ -478,9 +478,9 @@ testIframe("offset/body", "body", function( $ ) { ...@@ -478,9 +478,9 @@ testIframe("offset/body", "body", function( $ ) {
test("chaining", function() { test("chaining", function() {
expect(3); expect(3);
var coords = { "top": 1, "left": 1 }; var coords = { "top": 1, "left": 1 };
equal( jQuery("#absolute-1").offset(coords).selector, "#absolute-1", "offset(coords) returns jQuery object" ); equal( jQuery("#absolute-1").offset(coords).jquery, jQuery.fn.jquery, "offset(coords) returns jQuery object" );
equal( jQuery("#non-existent").offset(coords).selector, "#non-existent", "offset(coords) with empty jQuery set returns jQuery object" ); equal( jQuery("#non-existent").offset(coords).jquery, jQuery.fn.jquery, "offset(coords) with empty jQuery set returns jQuery object" );
equal( jQuery("#absolute-1").offset(undefined).selector, "#absolute-1", "offset(undefined) returns jQuery object (#5571)" ); equal( jQuery("#absolute-1").offset(undefined).jquery, jQuery.fn.jquery, "offset(undefined) returns jQuery object (#5571)" );
}); });
test("offsetParent", function(){ test("offsetParent", function(){
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment