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

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]
** No cross posts or HTML encoding! **
(Related lists - )

Reply via email to