Skip to main content

Bug Tracker

Side navigation

#5720 closed bug (duplicate)

Opened December 28, 2009 10:11AM UTC

Closed December 30, 2009 04:45AM UTC

`expando` cleanup in `.html()` and `.clone()` is too greedy

Reported by: maranomynet Owned by:
Priority: major Milestone: 1.4
Component: unfiled Version: 1.3.2
Keywords: Cc:
Blocked by: Blocking:
Description

There's a bug in the rinlinejQuery regexp that makes it a bit too "greedy":

jQuery('<p> jQuery12="0" </p>').html();  // returns: "  "

Proposed fix:

In manipulation.js, line 1 should be changed to something like:

var rinlinejQuery = /(<\\w[^<]+) jQuery\\d+="(?:\\d+|null)"/g,
                     ^^^^^^^^^^

and the replacements in lines 156 and 176 to reinsert the first matching parenthesis:

.replace(rinlinejQuery, "$1")
                        ^^^^

Actually this whole rinlinejQuery business is a bit of an ugly hack. Would it perhaps be feasible to fix the expando value in a way similar to what Andrea Giammarchi suggests in this jquery-dev post?

Attachments (0)
Change History (3)

Changed December 29, 2009 08:31PM UTC by maranomynet comment:1

Ack! this ticket was meant for jQuery 1.4a2

Changed December 29, 2009 08:36PM UTC by maranomynet comment:2

reposted as #5729

Changed December 30, 2009 04:45AM UTC by dmethvin comment:3

resolution: → duplicate
status: newclosed

Replaced by #5729.