_each method and context

[Yaffle]

Hello,

Why with JavaScript <= 1.5, Array.prototype._each method doesn't use
a context object

like

Array.prototype.forEach(callback[, thisObject]) in JavaScript 1.6

or

Enumerable.prototype.each(iterator[, context]) ?

thank you.

[kangax]

Because `Array.prototype._each` (defined by Prototype.js) is bound to
custom `each` implementation when native `Array.prototype.forEach` is
unavailable (e.g. in older versions of Firefox). That custom
implementation currently looks like:

function each(iterator) {
for (var i = 0, length = this.length; i < length; i++)
iterator(this[i]);
}

while, to be consistent with `forEach`, it would need to look like:

function each(iterator, context) {
for (var i = 0, length = this.length; i < length; i++)
iterator.call(context, this[i], i, this);
}

--
kangax

[Tobie Langel]

Can you open a ticket for this?

Thanks,

Tobie

[kangax]

On Feb 16, 6:09 pm, Tobie Langel <tobie.lan...@gmail.com> wrote:
> Can you open  a ticket for this?

Sure, patch and test attached.
http://prototype.lighthouseapp.com/projects/8886-prototype/tickets/554-_each-to-match-native-foreach-behavior

--
kangax