Hi,
I have fixed an exportimport bug on branch 2.2 : see tests in revision
119560.
I'd like some review before merging the fix into trunk.
I do not know for sure that I can remove the type check.
Thanks
Gotcha
Le 13/01/11 12:00, Godefroid Chapelle a écrit :
> Log message for revision 119561:
> remove type check that seem useless
>
> Changed:
> U Products.CMFCore/branches/2.2/Products/CMFCore/exportimport/content.py
>
> -=-
> Modified:
> Products.CMFCore/branches/2.2/Products/CMFCore/exportimport/content.py
> ===================================================================
> --- Products.CMFCore/branches/2.2/Products/CMFCore/exportimport/content.py
> 2011-01-13 10:44:41 UTC (rev 119560)
> +++ Products.CMFCore/branches/2.2/Products/CMFCore/exportimport/content.py
> 2011-01-13 11:00:16 UTC (rev 119561)
> @@ -101,15 +101,9 @@
> parser = ConfigParser()
>
> title = self.context.Title()
> - if isinstance(title, unicode):
> - title_str = title.encode(self._encoding)
> - else:
> - title_str = title
> + title_str = title.encode(self._encoding)
> description = self.context.Description()
> - if isinstance(description, unicode):
> - description_str = description.encode(self._encoding)
> - else:
> - description_str = description
> + description_str = description.encode(self._encoding)
> parser.set('DEFAULT', 'Title', title_str)
> parser.set('DEFAULT', 'Description', description_str)
--
Godefroid Chapelle (aka __gotcha) http://bubblenet.be
_______________________________________________
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
