307380 - Multipart Article: body (summary) charset is invalid ("NaN")

Status: RESOLVED FIXED

(MailNews Core :: Feed Reader, defect)

Description

[Mike Cowperthwaite]

Encountered while testing for bug 300145.   TB 1.5a1-0904 and 1.6a1-0904, Win2K

Steps to reproduce:
1) Set up a feed for   <http://www.pro-php.com/podcast.php>   with "show article 
summary" selected.
2) Select an article that shows an attachment, and View | Source
3) Observe the Content-Type header of the body (text/html) portion of the 
article

Actual results:
3)   Content-Type: text/html; charset=NaN

Expected results:
3)   Content-Type: text/html; charset=UTF-8

When viewing a message with this bogus charset, the symptom from bug 253849 is 
seen (and so would, presumably, bug 251634).

Comment 1

[Robert Sayre]

Attachment: fix a typo during the writing of the MIME message

Comment 2

[Scott MacGregor]

fixed branch and trunk.

Thanks Robert.

Comment 3

[Mike Cowperthwaite]

I can't test this fix because the feed is now being reported as invalid
(TB 1.6a1-0928).

Comment 4

[Robert Sayre]

(In reply to comment #3)
> I can't test this fix because the feed is now being reported as invalid
> (TB 1.6a1-0928).

As it should be:

XML Parsing Error: syntax error
Location: http://www.pro-php.com/podcast.php
Line Number 2, Column 1:

Unfortunately I couldn't connect to the database. <br />Access denied for user:
'prophp@localhost' (Using password: YES)
^