Opened 14 years ago

Closed 13 years ago

#1076 closed defect (fixed)

Input charset used for htmldoc should not be trac default

Reported by: Tonin Owned by: Tonin
Priority: high Component: PageToPdfPlugin
Severity: normal Keywords: charset
Cc: Trac Release: 0.10


As we know, htmldoc isn't utf-8 compatible. However, quite a lot of Trac site these days are using utf-8 as default_charset. This parameter is needed so that utf-8 encoded comments in subversion are rendered correctly under the source browser.

So using this same default_charset as encoding for input feeded to htmldoc is problematic. We'd better use the output charset for this purpose.

See attached patch as proposed solution. I also added a html meta tag containing this charset so that eventually newer version of htmldoc will correctly use it.

Attachments (1) (1.7 KB) - added by Tonin 14 years ago.

Download all attachments as: .zip

Change History (9)

Changed 14 years ago by Tonin

Attachment: added

comment:1 Changed 14 years ago by anonymous

Cc: Tonin added; anonymous removed

comment:2 Changed 13 years ago by durchanek@…

Priority: normalhigh

I can confirm that this patch is working correctly and I think it should be added to the main tree.

comment:3 Changed 13 years ago by Tonin

Cc: anonymous added; Tonin removed

Alec, if you gave me SVN write access on this directory, I'd be gald to help and commit my changes.

comment:4 Changed 13 years ago by Alec Thomas

Owner: changed from Alec Thomas to Tonin

Good idea, fire away.

comment:5 Changed 13 years ago by Tonin

Status: newassigned

comment:6 Changed 13 years ago by Tonin

Resolution: fixed
Status: assignedclosed

(In [2050]) Fix #1076 better default_charset.

comment:7 Changed 13 years ago by anonymous

Resolution: fixed
Status: closedreopened

There's still a problem in case the page from which we generate the PDF contains characters that cannot be represented in the desired output charset. For info, the supported HTMLDOC charsets currently are

comment:8 Changed 13 years ago by Tonin

Resolution: fixed
Status: reopenedclosed

(In [2100]) Replacing characters without equivalent in the desired output charset (fix #1076).

Modify Ticket

Change Properties
Set your email in Preferences
as closed The owner will remain Tonin.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment

E-mail address and name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.