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