[Zope-CMF] Re: [dev] tseaver-catalog_events branch

2006-05-18 Thread yuppie
Hi! Jens Vagelpohl wrote: I finally found some time to have a look at the tseaver-catalog_events branch. The last failing test is fixed now and I have some questions: Thanks for that, I think I was at a point where I didn't see the forest for the trees. 1.) The ContentishSublocations

[Zope-CMF] Re: [dev] tseaver-catalog_events branch

2006-05-18 Thread Tres Seaver
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 yuppie wrote: Hi! Jens Vagelpohl wrote: I finally found some time to have a look at the tseaver-catalog_events branch. The last failing test is fixed now and I have some questions: Thanks for that, I think I was at a point where I didn't

[Zope-CMF] Re: [dev] tseaver-catalog_events branch

2006-05-18 Thread Jens Vagelpohl
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 I did have an other look at tseaver-catalog_events branch and efge-1.5-five-compatible branch. I propose to split these changes into 2 steps: The first step would be to use the events machinery for dispatching to first class sublocations of

[Zope-CMF] Re: [dev] tseaver-catalog_events branch

2006-05-08 Thread yuppie
Hi! Florent Guillaume wrote: Jens Vagelpohl wrote: 1.) The ContentishSublocations adapter is not registered by default, just for unit tests. Why? 2.) Instead of using a customized ISublocations adapter and _recurseOpaques in handleObjectEvent: Can't we just add a new subscriber that

[Zope-CMF] Re: [dev] tseaver-catalog_events branch

2006-05-07 Thread Florent Guillaume
Jens Vagelpohl wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 I finally found some time to have a look at the tseaver-catalog_events branch. The last failing test is fixed now and I have some questions: Thanks for that, I think I was at a point where I didn't see the forest for the