A unit test would have been nice. :-)

Florent

In article <[EMAIL PROTECTED]> you write:
> Log message for revision 28471:
>         - Collector #1583/ZReST: Fixed handling of the title attribute
>           for non-ascii characters.
>   
> 
> Changed:
>   U   Zope/trunk/doc/CHANGES.txt
>   U   Zope/trunk/lib/python/Products/ZReST/ZReST.py
> 
> -=-
> Modified: Zope/trunk/doc/CHANGES.txt
> ===================================================================
> --- Zope/trunk/doc/CHANGES.txt        2004-11-17 23:06:56 UTC (rev 28470)
> +++ Zope/trunk/doc/CHANGES.txt        2004-11-18 13:18:15 UTC (rev 28471)
> @@ -41,6 +41,9 @@
>  
>      Bugs fixed
>  
> +      - Collector #1583/ZReST: Fixed handling of the title attribute
> +        for non-ascii characters.
> +
>        - Collector #1577: Fixed cryptic error message in ZPublisher if a
>          non-ASCII string is passed to a date, int, long or float property.
>  
> 
> Modified: Zope/trunk/lib/python/Products/ZReST/ZReST.py
> ===================================================================
> --- Zope/trunk/lib/python/Products/ZReST/ZReST.py     2004-11-17 23:06:56 UTC
> (rev 28470)
> +++ Zope/trunk/lib/python/Products/ZReST/ZReST.py     2004-11-18 13:18:15 UTC
> (rev 28471)
> @@ -215,7 +215,7 @@
>          if document.children:
>              item = document.children[0]
>              if item.tagname == 'title':
> -                self.title = str(item.children[0])
> +                self.title = item.children[0].astext()
>  
>          # do the format
>          self.formatted = pub.writer.write(document, pub.destination)
> 
> _______________________________________________
> Zope-Checkins maillist  -  [EMAIL PROTECTED]
> http://mail.zope.org/mailman/listinfo/zope-checkins
> 


-- 
Florent Guillaume, Nuxeo (Paris, France)   CTO, Director of R&D
+33 1 40 33 71 59   http://nuxeo.com   [EMAIL PROTECTED]
_______________________________________________
Zope-Dev maillist  -  [EMAIL PROTECTED]
http://mail.zope.org/mailman/listinfo/zope-dev
**  No cross posts or HTML encoding!  **
(Related lists - 
 http://mail.zope.org/mailman/listinfo/zope-announce
 http://mail.zope.org/mailman/listinfo/zope )

Reply via email to