Bug Tracker

Modify

Ticket #7817 (closed bug: fixed)

Opened 3 years ago

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

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

comment:3 Changed 3 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 3 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 3 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

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.