Skip to content

Commit

Permalink
Revert "Adding in backwards-compatiblity support for jQuery().bind/un…
Browse files Browse the repository at this point in the history
…bind/trigger - and immediately deprecating it. Please explicitly use jQuery(document) in your code."

This reverts commit e9d5947.
  • Loading branch information
jeresig committed Jan 11, 2010
1 parent 5a0ac24 commit 366039a
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 31 deletions.
23 changes: 5 additions & 18 deletions src/event.js
Expand Up @@ -755,7 +755,6 @@ jQuery.each(["bind", "one"], function( i, name ) {
for ( var key in type ) {
this[ name ](key, data, type[key], fn);
}

return this;
}

Expand All @@ -770,17 +769,11 @@ jQuery.each(["bind", "one"], function( i, name ) {
return fn.apply( this, arguments );
}) : fn;

if ( type === "unload" && name !== "one" ) {
this.one( type, data, fn, thisObject );

} else {
// Deprecated: Please don't expect an empty jQuery set to bind to document
(!this.selector && !this.context ? jQuery(document) : this).each(function() {
return type === "unload" && name !== "one" ?
this.one( type, data, fn, thisObject ) :
this.each(function() {
jQuery.event.add( this, type, handler, data );
});
}

return this;
};
});

Expand All @@ -794,20 +787,14 @@ jQuery.fn.extend({
return this;
}

// Deprecated: Please don't expect an empty jQuery set to bind to document
(!this.selector && !this.context ? jQuery(document) : this).each(function() {
return this.each(function() {
jQuery.event.remove( this, type, fn );
});

return this;
},
trigger: function( type, data ) {
// Deprecated: Please don't expect an empty jQuery set to bind to document
(!this.selector && !this.context ? jQuery(document) : this).each(function() {
return this.each(function() {
jQuery.event.trigger( type, data, this );
});

return this;
},

triggerHandler: function( type, data ) {
Expand Down
13 changes: 0 additions & 13 deletions test/unit/event.js
Expand Up @@ -118,19 +118,6 @@ test("bind(), trigger change on select", function() {
}).trigger('change');
});

test("bind/unbind/trigger on empty jQuery set", function() {
expect(1);

jQuery().bind("test", function(){
equals( this, document, "Handler triggered and bound on document." );
});

jQuery().trigger("test");

jQuery().unbind("test");
jQuery().trigger("test");
});

test("bind(), namespaced events, cloned events", function() {
expect(6);

Expand Down

0 comments on commit 366039a

Please sign in to comment.