Am 11.11.2008 um 16:52 schrieb yuppie: > AFAICT PortalFolder inherits from CMFCatalogAware to make sure it has > the same manage_afterAdd, manage_afterClone and manage_beforeDelete > methods as other content classes. But these methods are gone, so I > guess > the dependency is no longer needed.
You didn't hear it from me but this is surely abuse of inheritance? Time to call the cops and stand outside our houses in dressing gowns and slippers tutting and looking superior! > There might be some code that expects that *all* content classes > implement ICatalogAware, IWorkflowAware and IOpaqueItemManager, but if > we add deprecation warnings the other code can be cleaned up. +1 It's not as if Folder's behave as content although it is perfectly reasonable in my view for Folderish objects to do so. > In the long run I'd like to get rid of CMFCatalogAware completely. Its > functionality should be moved to adapters and event subscribers. Indeed. And IWorkflowAware should be implemented separately where required, he added quietly still working his way through Workflow code. I've written an almost generic Transition form for IWorkflowAware objects (the workflow should be pumped through the form). Any interest? Currently dependent upon adding ?form.workflow_action=xyz to the transition action which reminds me of add forms and suggests the whole thing probably needs, ahem, revitalising. Charlie -- Charlie Clark Helmholtzstr. 20 Düsseldorf D- 40215 Tel: +49-211-938-5360 GSM: +49-178-782-6226 _______________________________________________ Zope-CMF maillist - [email protected] http://mail.zope.org/mailman/listinfo/zope-cmf See https://bugs.launchpad.net/zope-cmf/ for bug reports and feature requests
