deac/jquery
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Revert "Adding in backwards-compatiblity support for jQuery().bind/unbind/trigger - and immediately deprecating it. Please explicitly use jQuery(document) in your code."

Browse source 
This reverts commit e9d5947b4a.
This commit is contained in:
jeresig 2010-01-11 13:48:40 -05:00
parent 5a0ac24e35
commit 366039a6f0
2 changed files with 5 additions and 31 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
src/event.js
View file

@ -755,7 +755,6 @@ jQuery.each(["bind", "one"], function( i, name ) {
for ( var key in type ) { for ( var key in type ) {
this[ name ](key, data, type[key], fn); this[ name ](key, data, type[key], fn);
} }
return this; return this;
} }
@ -770,17 +769,11 @@ jQuery.each(["bind", "one"], function( i, name ) {
return fn.apply( this, arguments ); return fn.apply( this, arguments );
}) : fn; }) : fn;
if ( type === "unload" && name !== "one" ) { return type === "unload" && name !== "one" ?
this.one( type, data, fn, thisObject ); this.one( type, data, fn, thisObject ) :
this.each(function() {
} else {
// Deprecated: Please don't expect an empty jQuery set to bind to document
(!this.selector && !this.context ? jQuery(document) : this).each(function() {
jQuery.event.add( this, type, handler, data ); jQuery.event.add( this, type, handler, data );
}); });
}
return this;
}; };
}); });
@ -794,20 +787,14 @@ jQuery.fn.extend({
return this; return this;
} }
// Deprecated: Please don't expect an empty jQuery set to bind to document return this.each(function() {
(!this.selector && !this.context ? jQuery(document) : this).each(function() {
jQuery.event.remove( this, type, fn ); jQuery.event.remove( this, type, fn );
}); });
return this;
}, },
trigger: function( type, data ) { trigger: function( type, data ) {
// Deprecated: Please don't expect an empty jQuery set to bind to document return this.each(function() {
(!this.selector && !this.context ? jQuery(document) : this).each(function() {
jQuery.event.trigger( type, data, this ); jQuery.event.trigger( type, data, this );
}); });
return this;
}, },
triggerHandler: function( type, data ) { triggerHandler: function( type, data ) {

13
test/unit/event.js
View file

@ -118,19 +118,6 @@ test("bind(), trigger change on select", function() {
}).trigger('change'); }).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() { test("bind(), namespaced events, cloned events", function() {
expect(6); expect(6);