Bug Tracker

Opened 13 years ago

Closed 13 years ago

Last modified 12 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 13 years ago by jitter

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

comment:3 Changed 13 years ago by Colin Snover

Resolution: → fixed
Status: new → closed

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

src/manipulation.js

Changeset: 82ac384b494258e17e2856ee2259475ecb0174ca

comment:4 Changed 13 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 12 years ago by john

Component: unfiled → ajax
Milestone: 1.6 → 1.5.1
Priority: undecided → low
Note: See TracTickets for help on using tickets.