[Zope] AW: [Zope-dev] Re: [Zope] ZCatalog Find Items broken in ZOPE 2.1? FIXED!

[Markus Strickler]

Did you follow the discussion about / and id in ZCatalog?
I haven't checked whether the apply_path bug has been fixed in the cvs as
well.

Markus

> -----Ursprüngliche Nachricht-----
> Von: smpitts@magellan.midsouth.rr.com
> [mailto:smpitts@magellan.midsouth.rr.com]Im Auftrag von Stephen Pitts
> Gesendet am: Mittwoch, 19. Januar 2000 03:59
> An: Markus Strickler
> Cc: Jason Spisak; Brian Lloyd; zope-dev@zope.org; zope@zope.org
> Betreff: [Zope-dev] Re: [Zope] ZCatalog Find Items broken in ZOPE 2.1?
> FIXED!
>
> On Sat, Jan 15, 2000 at 11:30:51PM +0100, Markus Strickler wrote:
> > Hi!
> > I investigated some more on the find problem and think that the
> Find Items
> > option in ZCatalog is broken.
> > Here is what I did:
> > - Installed ZOPE 2.1.2 Win32 Release.
> > - Created a ZCatalog as /Catalog
> > - Created a DTML Document as /test
> > - Selected DTML Document in the Find Items
> > - ZOPE finds the test Document but displays its path as /test (the link
> > leads to http://test)
> >
> > *Ouch*
> >
> > Markus
> >
>
> I just got hit by this as well. The culprit is in line 536 of
> lib/python/Products/ZCatalog/ZCatalog.py:
>
> apply_func(ob, (apply_path+'/'+p))
>
> If apply_path is null, then an extra '/' is prepended to p (the object's
> path) for no apparent reason. A quick fix is to replace this with:
>
> if apply_path:
> 	apply_func(ob, (apply_path+'/'+p))
> else:
> 	apply_func(ob, p)
>
> A patch to that effect is attached
> --
> Stephen Pitts
> smpitts@midsouth.rr.com
> webmaster - http://www.mschess.org
>