Woohoo my first check in ;)
--
  Andy McKay
  Agmweb Consulting
  http://www.agmweb.ca


----- Original Message -----
From: "Casey Duncan" <[EMAIL PROTECTED]>
To: "Andy McKay" <[EMAIL PROTECTED]>; <[EMAIL PROTECTED]>
Sent: Saturday, August 17, 2002 12:14 PM
Subject: Re: [Zope-dev] PathIndex doesn't index last part of path


> Your code looks fine, I think it meshes better with the underlying catalog
> code too. I don't have a problem with this getting checked in, just make
> sure you update IZCatalog.py, help/Catalog.py and add unit tests (that
pass
> ;^).
>
> -Casey
>
> ----- Original Message -----
> From: "Andy McKay" <[EMAIL PROTECTED]>
> To: "Casey Duncan" <[EMAIL PROTECTED]>; <[EMAIL PROTECTED]>
> Sent: Sunday, August 18, 2002 2:54 AM
> Subject: Re: [Zope-dev] PathIndex doesn't index last part of path
>
>
> > > If you call getObject, that actually does traversal anyway.
> >
> > Right, in that situation it would be pointless...
> >
> > You rock, Casey, thanks. I was thinking more about adding:
> >
> > def getMetadataFromPath(self, path):
> >     """ get metadata for an object using its path """
> >     rid = self._catalog.uids[path]
> >     return self._catalog.getMetadataForRID(rid)
> >
> > def getIndexFromPath(self, path):
> >     """ get index for an object using its path """
> >     rid = self._catalog.uids[path]
> >     return self._catalog.getIndexDataForRID(rid)
> >
> > Since this uses the same terminology and returns the same data as
> > getIndexDataForRID and getMetadataDataForRID. Is there any reason why I
> > couldn't checked these in?
> > --
> >   Andy McKay
> >   Agmweb Consulting
> >   http://www.agmweb.ca
> >
> >
> >
> >
>
>
> _______________________________________________
> Zope-Dev maillist  -  [EMAIL PROTECTED]
> http://lists.zope.org/mailman/listinfo/zope-dev
> **  No cross posts or HTML encoding!  **
> (Related lists -
>  http://lists.zope.org/mailman/listinfo/zope-announce
>  http://lists.zope.org/mailman/listinfo/zope )
>



_______________________________________________
Zope-Dev maillist  -  [EMAIL PROTECTED]
http://lists.zope.org/mailman/listinfo/zope-dev
**  No cross posts or HTML encoding!  **
(Related lists - 
 http://lists.zope.org/mailman/listinfo/zope-announce
 http://lists.zope.org/mailman/listinfo/zope )

Reply via email to