On Thu, Sep 22, 2011 at 3:14 PM, yuppie <y.2...@wcm-solutions.de> wrote: > My checkin for Products.ZCatalog has an optional dependency on > five.globalrequest. If five.globalrequest is not installed, > Products.ZCatalog behaves exactly as before. I hope we don't need a > Products.ZCatalog 2.14 release for that. Zope 2.13 can use it without > five.globalrequest. > > See http://svn.zope.org/?rev=122892&view=rev > > Is that ok? Should I add an explicit extra in the setup.py of > Products.ZCatalog?
That's ok. One minor nitpick: Instead of writing: request = getattr(self, 'REQUEST', None) I'd write: request = aq_get(self, 'REQUEST', None) with a module import of: from Acquisition import aq_get That way objects which have only an explicit AQ chain or only __parent__ pointers will also work. Hanno _______________________________________________ Zope-CMF maillist - Zope-CMF@zope.org https://mail.zope.org/mailman/listinfo/zope-cmf See https://bugs.launchpad.net/zope-cmf/ for bug reports and feature requests