[Zope] a simple design problem, a Zope bug and a fix

Ivan Kurmanov iku@tut.by
Sat, 6 Oct 2001 00:46:39 +0300


Hi,

For everybody who is interested, the patch that I've posted here few
days ago is not a fix, because it creates another absolute_url()
problem under certain conditions.  Sorry.  I'll keep trying to fix it.

Ivan

On Fri, Oct 05, 2001 at 05:23:18AM +0300, Ivan Kurmanov wrote:

[skipped a lot]
> Secondly, the Bug. 
> 
> And now we came to a bug description: I claim that absolute_url()
> method of the Traversable class has a bug, at least in Zope 2.4.1
> 
[skipped a bit more] 
> 
> Finally, a fix: I'm far from well-understanding the change I made to
> the Zope code, but it works better for me now.  I attach to this
> message a patch for Traversable.py file.
> 
> 
> Any comments welcome.
> 
> Cheers!
> 
> -- 
> Ivan Kurmanov
> 

> --- /home/ivan/master-copy/Zope-2.4.1-src/lib/python/OFS/Traversable.py	Thu Aug 30 16:07:49 2001
> +++ /home/ivan/zope/lib/python/OFS/Traversable.py	Fri Oct  5 04:59:50 2001
> @@ -130,7 +130,7 @@
>          '''
>          path = (self.getId(),)
>          
> -        p = aq_parent(aq_inner(self))
> +        p = aq_parent( aq_base(aq_inner(self)) ) 
>          if p is not None: 
>              path = p.getPhysicalPath() + path
>