Skip to main content

Bug Tracker

Side navigation

#7817 closed bug (fixed)

Opened December 21, 2010 02:55PM UTC

Closed December 23, 2010 12:33AM UTC

Last modified April 23, 2011 06:09PM UTC

xhr.js breaks if Array.prototype methods are added

Reported by: dmethvin Owned by:
Priority: low Milestone: 1.5.1
Component: ajax Version: 1.4.4
Keywords: Cc:
Blocked by: Blocking:
Description
for (i in prefilters) {

should be

for ( i = 0; i < prefilters.length; i++ ) {
Attachments (0)
Change History (5)

Changed December 21, 2010 03:06PM UTC by dmethvin comment:1

Changed December 21, 2010 10:03PM UTC by jitter comment:2

Can this one be closed? I think on the deferred branch this loop no longer exists.

Changed December 23, 2010 12:33AM UTC by Colin Snover comment:3

resolution: → fixed
status: newclosed

Don't use for-in loops on Arrays. Fixes #7817. Thanks to dmethvin.

Conflicts:

src/manipulation.js

Changeset: 82ac384b494258e17e2856ee2259475ecb0174ca

Changed December 24, 2010 12:05PM UTC by Colin Snover comment:4

Don't use for-in loops on Arrays. Fixes #7817. Thanks to dmethvin.

Conflicts:

src/manipulation.js

Changeset: 82ac384b494258e17e2856ee2259475ecb0174ca

Changed April 23, 2011 06:09PM UTC by john comment:5

component: unfiledajax
milestone: 1.61.5.1
priority: undecidedlow