Closed Bug 16441 Opened 25 years ago Closed

[dogfood] output does too much entity conversion

Categories

(Core :: DOM: Serializers, defect, P3)

x86
Windows NT
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: buster, Assigned: akkzilla)

References

()

Details

(Whiteboard: [PDT+])

I think this is a dup but I can't find the original. from relliot's mailnote: Subject: Sorry, one more... Date: Thu, 14 Oct 1999 12:39:58 -0700 From: relliott@netscape.com (Rick Elliott) To: cmanske@netscape.com You're probably getting annoyed by now, so this is the last thing I'll say for a bit. M11 Composer did bad things to comments: it appears to get confused by quotes and angle brackets and converts them to html entities. For example, on the page I sent you the header looked like this in the original: <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2 Final//EN"> but after editing the page and saving it with composer, it looks like this: <!DOCTYPE HTML PUBLIC &quot;-//W3C//DTD HTML 3.2 Final//EN&quot;> It also inserted an extra <style> tag with no closing </style>: orig: <style>A {text-decoration:none;}</style> </head> post 5.0 edit: <style>A {text-decoration:none;} </style> <style> </head> It munged the comments like so: orig: <!-- start left column here --> post 5.0 edit: &lt;!-- start left column here --&gt; All of this made the document unrenderable so I'd say this was DOGFOOD material. Hope this helps! Rick
*** Bug 16467 has been marked as a duplicate of this bug. ***
Whiteboard: [PDT+]
Putting on [PDT+] radar.
Status: NEW → ASSIGNED
Depends on: 8865
Entity conversion is all screwed up. I've been waiting on an entity converter from another group (see bug 8865) for quite a while, but it's looking like I'm going to have to hack in a workaround in the meantime, since output is fairly unusable with things like double quotes getting mapped to entity codes.
Target Milestone: M11
nhotta just checked in the API for 8865; I'll be working on plugging into that shortly, and with any luck it'll cure the entity problems.
Can you please put a sample file which causes this behavior in the URL field to this bug? I've seen things like this, too, in the past, but I can't seem to reproduce any of these problems today with any of the test files I've made.
Marking this fixed; I think recently fixes to the output sinks have fixed it. Reopen if you're still seeing it, with instructions on how to reproduce. Adding relliot to the cc list since I just noticed he wasn't on it even though he was the reporter.
Blocks: 12658
linking to PDT+ tracking bug 12658
Status: RESOLVED → VERIFIED
verified in 11/29 build.
Bulk move of all "Output" component bugs to new "DOM to Test Conversion" component. Output will be deleted as a component.
Component: Output → DOM to Text Conversion
You need to log in before you can comment on or make changes to this bug.