Bug Tracker

Opened 7 years ago

Closed 7 years ago

Last modified 6 years ago

#7817 closed bug (fixed)

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++ ) {

Change History (5)

comment:2 Changed 7 years ago by jitter

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

comment:3 Changed 7 years ago by Colin Snover

Resolution: fixed
Status: newclosed

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

src/manipulation.js

Changeset: 82ac384b494258e17e2856ee2259475ecb0174ca

comment:4 Changed 7 years ago by Colin Snover

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

src/manipulation.js

Changeset: 82ac384b494258e17e2856ee2259475ecb0174ca

comment:5 Changed 6 years ago by john

Component: unfiledajax
Milestone: 1.61.5.1
Priority: undecidedlow
Note: See TracTickets for help on using tickets.