On 1/13/11 14:01 , Godefroid Chapelle wrote:
>
>
> Le 13/01/11 12:07, Wichert Akkerman a écrit :
>> On 1/13/11 12:04 , Godefroid Chapelle wrote:
>>> Hi,
>>>
>>> I have fixed an exportimport bug on branch 2.2 : see tests in revision
>>> 119560.
>>
>> Aren't you risking double encoding now? That patch looks like it makes
>> things worse, not better.
>
> I am not sure I understand what you mean.
>
> I did try the patch on one of my sites where export was raising an
> exception. Export did work and the files produced make sense to me.
>
> In the branch, prior to apply the patch, I added tests that mimic my
> site setup and do not pass.

But do you know why? The original code was more robust than your 
version, so I am extremely curious why it was failing for you and is not 
now.

Wichert.
_______________________________________________
Zope-CMF maillist  -  Zope-CMF@zope.org
https://mail.zope.org/mailman/listinfo/zope-cmf

See https://bugs.launchpad.net/zope-cmf/ for bug reports and feature requests

Reply via email to