Bug Tracker

Opened 6 years ago

Closed 6 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 6 years ago by jitter

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

comment:3 Changed 6 years ago by Colin Snover

  • Resolution set to fixed
  • Status changed from new to closed

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

src/manipulation.js

Changeset: 82ac384b494258e17e2856ee2259475ecb0174ca

comment:4 Changed 6 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 changed from unfiled to ajax
  • Milestone changed from 1.6 to 1.5.1
  • Priority changed from undecided to low
Note: See TracTickets for help on using tickets.