[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Nuxeo-localizer] Re: [Lleu-commits] CVS: Localizer __init__.py,1.46,1.4
From: |
Florent Guillaume |
Subject: |
[Nuxeo-localizer] Re: [Lleu-commits] CVS: Localizer __init__.py,1.46,1.46.2.1 |
Date: |
16 Oct 2002 17:23:03 +0200 |
On Mon, 2002-10-14 at 18:12, Juan David Ibáñez Palomar wrote:
> Florent Guillaume wrote:
>
> >Update of /cvsroot/lleu/Localizer
> >In directory usw-pr-cvs1:/tmp/cvs-serv11851
> >
> >Modified Files:
> > Tag: branch-1-0
> > __init__.py
> >Log Message:
> >Updated to take into account the fact that Zope 2.6 now is completely
> >Unicode-aware, no need to patch it.
> >
> >We shouldn't even need to patch the StringIO, but I left what we had for
> >now. If the envvar LOCALIZER_USE_ZOPE_UNICODE is defined, then no
> >patching at all is done. I think that this should be the default, but
> >I'm not the one to decide.
>
> And I agree, if it works.
>
> My only concerns is, will it work with Zope 2.6b1 or is it
> required a CVS version? I don't want to wait Zope 2.6b2 to
> release Localizer 0.9.2
It works with b1 and b2.
I'd like Zope Unicode handling can be the default, but there is a case
where it fails which I want to fix before 2.6 final. If a ZPT page
contains non-ASCII characters in the local charset (for instance 'é')
and it uses Unicode strings with tal:content="python:u'foo'" or using
i18n:translate (which calls Localizer through TranslationService) then
Zope attempts to join 'é' and u'foo' and fails. Localizer does it
correctly because its StringIO converts everything to the final output
charset *before* doing the join, however it's slow.
It could be argued that a page using Localizer should *never* have any
plain 8bit string, because it would mean it isn't translated, but I'm
not sure it's ok for everyone. For instance I don't know how
LocalContent works, maybe it outputs 8bit strings.
Florent
--
Florent Guillaume, Nuxeo (Paris, France)
+33 1 40 33 79 87 http://nuxeo.com mailto:address@hidden
- [Nuxeo-localizer] Re: [Lleu-commits] CVS: Localizer __init__.py,1.46,1.46.2.1,
Florent Guillaume <=