Collector number 735: http://collector.zope.org/Zope/735

This was prompted by some bugs people found with Plone. For a two character
fix that to me looked like a simple oversight I didn't bother with a unit
test.
--
  Andy McKay

----- Original Message -----
From: "Florent Guillaume" <[EMAIL PROTECTED]>
To: <[EMAIL PROTECTED]>
Cc: <[EMAIL PROTECTED]>
Sent: Sunday, December 22, 2002 11:01 AM
Subject: [Zope-dev] Re: [Zope-Checkins] CVS: Zope/lib/python/TAL -
TALInterpreter.py:1.69.6.10


> What prompted this? What does it fix?
> I don't see that quotes need escaping here.
> Also, there should be a unit test for it.
> (And if it's a fix it should be in HEAD too.)
>
> Florent
>
> In article <[EMAIL PROTECTED]> you write:
> > Modified Files:
> >       Tag: Zope-2_6-branch
> > TALInterpreter.py
> > Log Message:
> > Escape quotes correctly
> >
> >
> > === Zope/lib/python/TAL/TALInterpreter.py 1.69.6.9 => 1.69.6.10 ===
> > --- Zope/lib/python/TAL/TALInterpreter.py:1.69.6.9 Wed Oct  9 10:37:38
2002
> > +++ Zope/lib/python/TAL/TALInterpreter.py Tue Dec 17 19:33:43 2002
> > @@ -486,7 +486,7 @@
> >          if text is self.Default:
> >              self.interpret(stuff[1])
> >              return
> > -        s = escape(text)
> > +        s = escape(text, 1)
> >          self._stream_write(s)
> >          i = s.rfind('\n')
> >          if i < 0:
>
>
> --
> Florent Guillaume, Nuxeo (Paris, France)
> +33 1 40 33 79 87  http://nuxeo.com  mailto:[EMAIL PROTECTED]
>
> _______________________________________________
> Zope-Dev maillist  -  [EMAIL PROTECTED]
> http://lists.zope.org/mailman/listinfo/zope-dev
> **  No cross posts or HTML encoding!  **
> (Related lists -
>  http://lists.zope.org/mailman/listinfo/zope-announce
>  http://lists.zope.org/mailman/listinfo/zope )


_______________________________________________
Zope-Dev maillist  -  [EMAIL PROTECTED]
http://lists.zope.org/mailman/listinfo/zope-dev
**  No cross posts or HTML encoding!  **
(Related lists - 
 http://lists.zope.org/mailman/listinfo/zope-announce
 http://lists.zope.org/mailman/listinfo/zope )

Reply via email to