[Zope] Re: ERROR(200) SiteError http://localhost/manage_workspace

[Josef Meile]

Ops, I forgot something in the command line test:

> >>> #The first line was the one I forgot
> >>> m='manage_main'
> >>> if m.find('/'):
> ...   print "True"
> ...
> True



----- Original Message ----- 
From: "Josef Meile" <jmeile at hotmail.com>
To: <zope at zope.org>
Sent: Friday, February 27, 2004 6:54 PM
Subject: ERROR(200) SiteError http://localhost/manage_workspace


> Hi,
>
> every time that I hit something on the ZMI (In Zope 2.7.0), I
> got this error on my event log:
>
> 2004-02-27T18:15:55 ERROR(200) SiteError
> http://129.132.39.139/manage_workspace
> Traceback (most recent call last):
>   File "/usr/local/Zope/2.7.0/lib/python/ZPublisher/Publish.py", line 100,
> in publish
>     request, bind=1)
>   File "/usr/local/Zope/2.7.0/lib/python/ZPublisher/mapply.py", line 88,
in
> mapply
>     if debug is not None: return debug(object,args,context)Op
>   File "/usr/local/Zope/2.7.0/lib/python/ZPublisher/Publish.py", line 40,
in
> call_object
>     result=apply(object,args) # Type s<cr> to step into published object.
>   File "/usr/local/Zope/2.7.0/lib/python/App/Management.py", line 86, in
> manage_workspace
>     raise Redirect, (
>     Redirect: http://129.132.39.139/manage_main
>
> I went to the line in question (Management.py, line 86) and saw:
>
> if m.find('/'):
> ----raise Redirect, (
> --------"%s/%s" % (REQUEST['URL1'], m))
>
> So, I just add the following line before the raise command:
> print repr(m)
>
> For my surprise, m was always 'manage_main', it didn't have any '/', so, I
> tested
> in the python command line:
> >>> if m.find('/'):
> ...   print "True"
> ...
> True
>
> As expected it printed "True", so, I concluded that this error will be
> always
> there because m.find('/') is always true, so I think it would be most
> correct
> to do:
> if m.find('/')==-1:
> ----raise Redirect, (
> --------"%s/%s" % (REQUEST['URL1'], m))
>
> Regards,
> Josef
>