Closed Bug 270125 Opened 20 years ago Closed 20 years ago

RSS feeds don't render html code for quotes as quotes, instead just shows raw HTML

Categories

(MailNews Core :: Feed Reader, defect)

Product:
MailNews Core
Component:
Feed Reader
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: mail, Assigned: mscott)

References

(
URL
)

Details

Attachments

(2 files)

Screencap showing problem
194.00 KB, application/pdf
Details
the fix
1.46 KB, patch
jshin1987
: review+
Bienvenu
: superreview+
Details | Diff | Splinter Review 
User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X; en-us) AppleWebKit/125.5.5 (KHTML, like Gecko) Safari/125.11
Build Identifier: version 0.9 (20041103)

When I get new headlines from the E! Online RSS feeds, the quotes around movie or TV show titles don't 
render as quote marks, but as the HTML for quote marks, i.e. <pre>&quot;</pre>.

Reproducible: Always
Steps to Reproduce:
1. Add http://www.eonline.com/About/Rss/news.latest.xml to News & Blogs
Screencap showing problem
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Hardware: Macintosh → All
Target Milestone: --- → Thunderbird1.0
Attached patch the fix — — Splinter Review 
In my fix from yesterday I forgot to handle &quot;
Comment on attachment 166135 [details] [diff] [review]
the fix

jshin, is this conversion to utf-16 before performing the entity replacement
what you had in mind from your e-mail? 

Note: we set the charset on the unicode converter a few lines before this
patch.

This works for me on ascii safe charsets, doing the right thing.
Attachment #166135 - Flags: superreview?(bienvenu)
Attachment #166135 - Flags: review?(jshin)
Attachment #166135 - Flags: superreview?(bienvenu) → superreview+
has patch, nominating.
OS: MacOS X → All
*** Bug 263998 has been marked as a duplicate of this bug. ***
fixed branch and trunk. It was important to get this in for I18N testing. I
didn't want to wait any longer. Will leave the review request open for jshin to
look at when he has time.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Comment on attachment 166135 [details] [diff] [review]
the fix

r=jshin
Attachment #166135 - Flags: review?(jshin) → review+
sorry for the delay. bugzilla again played a trick on me. I've been wondering
why you didn't come up with a patch after my email-reply.
Component: RSS → Feed Reader
Product: Thunderbird → MailNews Core
Target Milestone: Thunderbird1.0 → ---
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: