Bug Tracker

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#11227 closed bug (duplicate)

unwrap method is triggering ready function twice

Reported by: [email protected] Owned by: [email protected]
Priority: undecided Milestone: None
Component: unfiled Version: 1.7.1
Keywords: Cc:
Blocked by: Blocking:

Description

reproducible here

http://jsbin.com/afuzam/2/edit

if you remove the outer div everything works fine.

Change History (4)

comment:1 Changed 10 years ago by sindresorhus

Owner: set to [email protected]
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 10 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 10 years ago by dmethvin

Duplicate of #9134.

comment:4 Changed 10 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.