[Mailman-Developers] [ mailman-Patches-646884 ] HyperArch.py multibyte charset

[noreply at sourceforge.net]

Patches item #646884, was opened at 2002-12-02 11:27
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=300103&aid=646884&group_id=103

Category: Pipermail
Group: Mailman 2.1
Status: Open
Resolution: None
Priority: 5
Submitted By: Tokio Kikuchi (tkikuchi)
Assigned to: Nobody/Anonymous (nobody)
Summary: HyperArch.py multibyte charset

Initial Comment:
HyperArch.py in 2.1b5 use Utils.uquote() for
CGIescape() and html_quote() but it does not take
language argument. This  is no good for multilbyte
charset because characters are escaped in separate 8bit
bytes which results in mojibake.
We should rather use Utils.uncanonstr() to honor the
legal characters within the multibyte charset.
This patch fixes other minor bugs in the current code.


----------------------------------------------------------------------

>Comment By: Tokio Kikuchi (tkikuchi)
Date: 2002-12-08 01:51

Message:
Logged In: YES 
user_id=67709

I think I have to generate new translation regarding the
date format problem. Sorry.


----------------------------------------------------------------------

Comment By: Tokio Kikuchi (tkikuchi)
Date: 2002-12-08 01:19

Message:
Logged In: YES 
user_id=67709

Thanks Barry. The patch was successfully applied and things
are worklng better. (a new i18n problem in date format, though)


----------------------------------------------------------------------

Comment By: Martin v. Löwis (loewis)
Date: 2002-12-07 17:10

Message:
Logged In: YES 
user_id=21627

Tokio, can you please explain the problem in more detail?
Utils.uquote will always return 7bit strings only,
potentially with nested HTML character references. Why could
this cause a problem in a multi-byte encoding? Can you refer
to a page that looks incorrectly because of that?

----------------------------------------------------------------------

Comment By: Barry A. Warsaw (bwarsaw)
Date: 2002-12-07 16:27

Message:
Logged In: YES 
user_id=12800

Here's a new version of the patch that applies cleanly to
current cvs.  I'm not sure I'm testing it well enough, so
could you take a look?


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=300103&aid=646884&group_id=103