Bug Tracker

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#11227 closed bug (duplicate)

unwrap method is triggering ready function twice

Reported by: mgambill@… Owned by: mgambill@…
Priority: undecided Milestone: None
Component: unfiled Version: 1.7.1
Keywords: Cc:
Blocked by: Blocking:


reproducible here


if you remove the outer div everything works fine.

Change History (4)

comment:1 Changed 9 years ago by sindresorhus

Owner: set to mgambill@…
Status: newpending

Thanks for taking the time to contribute to the jQuery project! Please provide a complete reduced test case on jsFiddle to help us assess your ticket.

Additionally, be sure to test against the jQuery Edge version to ensure the issue still exists. To get you started, use this boilerplate: http://jsfiddle.net/FrKyN/ Open the link and click to "Fork" (in the top menu) to get started.

comment:2 Changed 9 years ago by dmethvin

Resolution: duplicate
Status: pendingclosed

The div you're unwrapping contains the Javascript with ready handler. When it is unwrapped, the script is executed a second time. Since the document is ready, the referenced code runs a second time.

Bug #9134 is essentially the same and has been open a while, but it is hard to fix and also encouraging bad practices. Don't sprinkle Javascript throughout markup and then try to manipulate it.

comment:3 Changed 9 years ago by dmethvin

Duplicate of #9134.

comment:4 Changed 9 years ago by anonymous

I'm working in .Net project so some of final html created its necessary to have inline script. With that being said I would prefer you guys not fix it and just add a comment about it in the API documentation.

Note: See TracTickets for help on using tickets.