dojo.forEach() doesn't match native behaviour with empty first argument

[Thomas Bachem]

Doing

dojo.forEach(null, function() { });

works without throwing an error, while

Array.prototype.forEach.call(null, function() { })

does not (at least not in Google Chrome 13).

I realized this when using the WebKit? Mobile build in place of the normal build, which uses the native Array.forEach implementation like shown above.

Changing dojo/_base/array.js around line 123 from

// match the behavior of the built-in forEach WRT empty arrs
if(!arr || !arr.length){ return; }

to

// match the behavior of the built-in forEach WRT empty arrs
if(!arr.length){ return; }

would align the behaviours.

[Thomas Bachem]

Regarding #13468, it may perhaps be better to deviate from the native behaviour of Array.forEach() and make dojo.forEach() always ignore an empty first argument, even in WebKit? Mobile builds.

[bill]

As mentioned in #13468, [24328] removed the webkit special path from that file. And since there's code depending on the current behavior of those array functions we don't want to change their API (especially not before 2.0). Thus, marking this as wontfix.

[Thomas Bachem]

A note in the documentation/docblock of dojo.forEach() would be nice nonetheless, as it claims to match the native behaviour.