Opened 7 years ago

Closed 5 years ago

#17026 closed defect (wontfix)

parser.js fails to parse XHTML <foo/> tags for IE6-7 code path

Reported by: pags Owned by: bill
Priority: undecided Milestone: tbd
Component: Parser Version: 1.8.3
Keywords: Cc:
Blocked By: Blocking:

Description

"construct" function in parser.js fails to pull out element attributes correctly for the block commented as "Special path for IE6-7". Trailing forward slash used to close element is not removed properly by the existing regexp. Attaching a file with the code fix.

Attachments (1)

fix.js​ (11.9 KB) - added by pags 7 years ago.
Patch for "construct" method.

Download all attachments as: .zip

Change History (3)

Changed 7 years ago by pags

Attachment: fix.js​ added

Patch for "construct" method.

comment:1 Changed 7 years ago by bill

Summary: parser.js fails to parse element attributes for IE6-7 code path → parser.js fails to parse XHTML <foo/> tags for IE6-7 code path

Hi pags. It's much better to post a ​patch file, rather than posting full js files. The other issues are that we no longer officially support IE6-7, and I'm not sure we really support XHTML either.

I'll leave this ticket here for now, but may ending up closing as wontfix, especially if we end up removing the IE6-7 code path altogether. If you can post an actual patch including the update to the code plus an update to the automated test I'd check it in.

comment:2 Changed 5 years ago by bill

Resolution: → wontfix
Status: new → closed

Closing since IE6-7 are desupported.

Note: See TracTickets for help on using tickets.