isFunction fails in certain cases

[dmitriid]

I'm not sure how relevant it is, but:

var a, b, c;

a = /^\dfunction\d/;
b = new String('Wow, J(ava)Script is soooo functional...');
c = ['method', 'function', 'object'];
c.constructor = 'Lego';

alert([$.isFunction(a), $.isFunction(b), $.isFunction(c)]); // -> true, true, true!!!

If you write a = /^\dunction\d/; then isFunction(a) returns false (as it probably should).

Marked the bug as minor as it doesn't seem to affect how jQuery behaves overall.

[flesler]

(a) and (c) are REALLY irrelevant, specially (c). You can always find a way around a hack, that doesn't mean you need to expect it.

The 2nd case with the string can actually happen, though it's pretty specific too.

The regex can be replaced to:

/^ ?function/

IE seems to add a whitespace at position 0, the insensitive case is really neccesary ?

[flesler]

Fixed at [5601], none of these 3 cases should yield true now.