Modify ↓
Ticket #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: | ||
| Blocking: | Blocked by: |
Description
for (i in prefilters) {
should be
for ( i = 0; i < prefilters.length; i++ ) {
Change History
comment:2 Changed 2 years ago by jitter
Can this one be closed? I think on the deferred branch this loop no longer exists.
comment:3 Changed 2 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 2 years ago by Colin Snover
Don't use for-in loops on Arrays. Fixes #7817. Thanks to dmethvin. Conflicts:
src/manipulation.js
Changeset: 82ac384b494258e17e2856ee2259475ecb0174ca
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.
Note: See
TracTickets for help on using
tickets.
Pull: https://github.com/jquery/jquery/pull/142