Bug Tracker

Opened 12 years ago

Closed 12 years ago

Last modified 11 years ago

#9072 closed bug (fixed)

jQuery 1.6 Crashes IE 6

Reported by: john Owned by:
Priority: blocker Milestone: 1.6.1
Component: manipulation Version: 1.6
Keywords: Cc:
Blocked by: Blocking:

Description (last modified by john)

Simple reproduce case:

  <html>
  <head>
  <script src="http://code.jquery.com/jquery-1.6.min.js"></script>
  </head>
  <body>
  <script>
  $('body').append(
    '<span></span>',
    '<span></span>',
    '<span></span>'
  )
  </script>
  </body>
  </html>

The bug is in this block: https://github.com/jquery/jquery/blob/1.6/src/manipulation.js#L652

Note the inner loop is clobbering the outer loop's "i" variable.

From here: http://news.ycombinator.com/item?id=2510844

Change History (6)

comment:1 Changed 12 years ago by john

Description: modified (diff)

comment:2 Changed 12 years ago by john

Component: unfiledmanipulation
Milestone: 1.next1.6.1
Priority: undecidedblocker

comment:3 Changed 12 years ago by John Resig

Resolution: fixed
Status: newclosed

Fix problem with appending multiple string arguments in IE 6. Fixes #9072.

Changeset: bfad45fe4510db78a38170752ccc9c9efe5f38b6

comment:4 Changed 12 years ago by Timmy Willison

#9107 is a duplicate of this ticket.

comment:5 Changed 12 years ago by Timmy Willison

#9110 is a duplicate of this ticket.

comment:6 Changed 12 years ago by anonymous

A heads up: I see this was fixed in jQuery 1.6.1 but http://ajax.googleapis.com/ajax/libs/jquery/1/jquery.js, which I thought was the latest jQuery, has the bug in it. I wasted hours debugging today because of this.

Note: See TracTickets for help on using tickets.