#2580: RSS feed validation in 0.9.3
----------------------+-----------------------------------------------------
 Reporter:  osimons   |        Owner:  cboos
     Type:  defect    |       Status:  new  
 Priority:  normal    |    Milestone:  0.10 
Component:  timeline  |      Version:  0.9.3
 Severity:  normal    |   Resolution:       
 Keywords:            |  
----------------------+-----------------------------------------------------
Changes (by cboos):

  * status:  reopened => new

Comment:

 After discussing with cmlenz, we concluded that we should produce directly
 a `unicode` object. So I tested the following change:

 {{{
 Index: trac/util/markup.py
 ===================================================================
 --- trac/util/markup.py (revision 3592)
 +++ trac/util/markup.py (working copy)
 @@ -309,8 +309,11 @@
              else:
                  yield escape(child, quotes=False)

 +    def __unicode__(self):
 +        return u''.join(self.serialize())
 +
      def __str__(self):
 -        return Markup(''.join(self.serialize()))
 +        return ''.join(self.serialize())

      def __add__(self, other):
          return Fragment()(self, other)
 }}}

 which fixed the present issue. This change didn't seem to introduce any
 adverse effects, and I tested a big bunch of wiki pages/ macros and all
 the different subsystems, so I think that after some positive user
 feedback, we could safely apply it.

-- 
Ticket URL: <http://trac.edgewall.org/ticket/2580#comment:18>
The Trac Project <http://trac.edgewall.com/>
_______________________________________________
Trac-Tickets mailing list
[email protected]
http://lists.edgewall.com/mailman/listinfo/trac-tickets

Reply via email to