Dieter Maurer wrote:
Oliver Bleutgen wrote at 2003-6-10 14:54 +0200:
> ...
> (*) if m.find('/'):
> raise 'Redirect', (
> "%s/%s" % (REQUEST['URL1'], m))
> return getattr(self, m)(self, REQUEST)
> > My question is about the marked block. I'd guess that the intent is to > send a redirect if m (== options[0]['action']) contains a '/'.
> > But m.find('/') evaluates to false only if m[0] == '/', otherwise it > yields either -1 (which is true), if there's no '/' in m, or something > greater 0, if there's a slash after the first char.
> > Is this intended behavior or a bug?


It probably a bug. But one, that usually only adds an unnecessary
"redirect".


Usually, but not when m.find('/') == 0, which OTOH happens if you want to give nice, secure absolute paths to the methods.
The attribute error you get in that case is quite irritating.


I'll stick it in the collectior.


thanks, oliver


_______________________________________________
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