Bug Tracker

Modify

Ticket #12178 (closed bug: duplicate)

Opened 10 months ago

Last modified 9 months ago

Crash on IE8 in data.js' removeData function

Reported by: dwilks Owned by: dwilks
Priority: low Milestone: None
Component: effects Version: 1.7.2
Keywords: Cc:
Blocking: Blocked by:

Description

Browser: IE8, IE9 in IE8 mode Version: 1.7.1, 1.7.2 and jquery-git (2012-08-01)

Here's a full description of the issue:  http://pdfcast.org/download/jquery-calendar-problem.pdf

The short version:

  • document.body has a jquery change event on it
  • when a jquery ui calendar's month select list is used to change the month
  • jquery crashes in (data.js) removeData as jqui is trying to replace the calendar

because the select list's removeAttribute property's type is unknown and just testing for causes an error.

} else if ( elem.removeAttribute ) {

  • the select list's removeAttribute starts out ok when the date picker is being created and jquery.append initially creates the fragment but becomes 'unknown' when it's actually appended to the document after

this.appendChild( elem );

This is the same issue as #9476. Of course a simple jsfiddle works fine. When the calendar is added to the document its month select list's removeAttribute remains typeof === object. I am going to try to keep whittling down my fail case and get it into a jsfiddle and in the mean time add a typeof test to our local version of jquery.

What I do know...

  • Fails in IE8 regardless of doctype
  • Works in IE9 with html5 doctype or with IE9 meta tag
  • Fails in IE9 with IE8 meta tag

Change History

comment:1 Changed 10 months ago by timmywil

  • Owner set to dwilks
  • Status changed from new to pending

Thank you for taking the time to contribute to the jQuery project!

We will need a test case that does not include jQuery UI. If this is a bug in jQuery UI, please file a bug report at  http://bugs.jqueryui.com.

comment:2 Changed 10 months ago by dwilks

  • Status changed from pending to new

Here's the simple fail case:  http://jsfiddle.net/ZpgGT/

The key conditions:

  • body must have a change event
  • the dynamically appended element must have an associated style with a behavior:url(#default#savehistory)

comment:3 follow-up: ↓ 6 Changed 10 months ago by timmywil

  • Priority changed from undecided to low
  • Resolution set to duplicate
  • Status changed from new to closed
  • Component changed from unfiled to effects

Duplicate of #10394.

comment:4 Changed 10 months ago by dwilks

How long before this fix appears in the jquery-git build? I updated my jsfiddle to jQuery (edge) and it's still crashing in exactly the same place. The edge url is:  http://jsfiddle.net/ZpgGT/1/

comment:5 Changed 9 months ago by dwilks

This is similar, but not identical to #10394. Please re-open this ticket and test it against jQuery (edge) at  http://jsfiddle.net/ZpgGT/1/. Thanks.

comment:6 in reply to: ↑ 3 Changed 9 months ago by dwilks

Replying to timmywil:

Duplicate of #10394.

Ahh, maybe replying is what's needed to get your attention. :)

This is similar, but not identical to #10394. It has nothing to do with the effects module. It's a similar issue in data.js. Please re-open this ticket and test it against jQuery (edge) at  http://jsfiddle.net/ZpgGT/1/. Thanks.

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.