IE version of forEach uses undeclared var. about store.js HOT 7 CLOSED

That's what I get for merging a pr without thorough review and testing! Will fix.

-- while mobile

On Wed, Oct 2, 2013 at 9:04 AM, Fredrik Blomqvist
[email protected] wrote:

The IE version of the store.forEach function uses an undeclared "ret" variable when calling the callback (at about line 146).
Regards

/ Fredrik Blomqvist

Reply to this email directly or view it on GitHub:
#79

from store.js.

This was fixed today btw! New version has been tagged, built and released to npm.

-- while mobile

On Wed, Oct 2, 2013 at 9:04 AM, Fredrik Blomqvist
[email protected] wrote:

The IE version of the store.forEach function uses an undeclared "ret" variable when calling the callback (at about line 146).
Regards

/ Fredrik Blomqvist

Reply to this email directly or view it on GitHub:
#79

from store.js.

umm, now the variable "key" (attr.name) is undefined instead.. ;)

from store.js.

Yeah, that's embarrassing. My bad.

I'm having issues testing. I do hope it now is fixed, but I haven't verified on old IE yet. Hope to shortly though.

from store.js.

nm, still broken. Argh. Fixing with actual IE access now..

from store.js.

Alright. This time may actually be legit.

Tested on http://marcuswestin.github.io/store.js/test.html with success. Would love your verification.

from store.js.

Closing as fixed.

from store.js.