Bug Tracker

Opened 10 years ago

Closed 10 years ago

#5720 closed bug (duplicate)

`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?

Change History (3)

comment:1 Changed 10 years ago by maranomynet

Ack! this ticket was meant for jQuery 1.4a2

comment:2 Changed 10 years ago by maranomynet

reposted as #5729

comment:3 Changed 10 years ago by dmethvin

Resolution: duplicate
Status: newclosed

Replaced by #5729.

Note: See TracTickets for help on using tickets.