Patch attached. I guess I should have checked whether I have check-in privileges by default, but I assume not.

I can tell you in advance, what a core developer will say: this function is internal, is the most used function all over the framework, so if this behavior (iterating functions) is not required by the core, then it won't be changed.

Also, this patch doesn't support any array-like (arguments, nodelist, jQuery object, etc).

>I guess I should have checked whether I have check-in privileges by default, but I assume not.

There's no permissions system integrated into the SVN. But only those allowed can "legally" commit.

Internal? Really? http://docs.jquery.com/Core/each#callback

I mean, if it's in the docs, it seems pretty public to me.

Also, legit criticism for array-like objects; I guess the only technically correct solution would be to special-case for Functions.

You're confusing $.each and $().each.

$.each is a generic iterator, it receives 2 arguments: collection and callback

$().each ($.fn.each) is a special use of $.each, it iterates over the elements of a jQuery collection. It only receives the callback, and the calling collection is used.

The function mentioned in the docs, refers to the callback. Functions aren't supported.

Reopening this, as it still exists and IMO is indeed a valid bug.

In http://dev.jquery.com/browser/tags/1.3.2/src/core.js#L662 $.each checks for length === undefined, which breaks in this use case:

var someFunction = function(){};
$.each(someFunction, function(key){
    alert(key);
});

The docs say "Other objects are iterated via their named properties.", so you'd expect that code to iterate over the built-in properties and methods of that instance of Function, but instead it tries to loop through properties named [0...arity], which don't exist.

The fix is pretty painless. Change the two offending lines from:

if ( length === undefined ) {

to:

if ( length === undefined || object.constructor == Function ) {

...et voila, $.each should hit the for...in condition that works on functions.

I only added the specific exception for Function because I can't think of anything else off the top of my head that allows for...in but doesn't currently work with $.each.

Fixed at [6410].

#5002 is a duplicate of this ticket.