Bug Tracker

Ticket #7817 (closed bug: fixed)

Opened 4 years ago

Last modified 4 years ago

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:
Blocking: Blocked by:

Description

for (i in prefilters) {

should be

for ( i = 0; i < prefilters.length; i++ ) {

Change History

comment:2 Changed 4 years ago by jitter

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

comment:3 Changed 4 years ago by Colin Snover

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

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

src/manipulation.js

Changeset: 82ac384b494258e17e2856ee2259475ecb0174ca

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

  • Priority changed from undecided to low
  • Component changed from unfiled to ajax
  • Milestone changed from 1.6 to 1.5.1
Note: See TracTickets for help on using tickets.