Rocky wrote:
On Feb 23, 10:15 am, yuppie <y.20...-E2EsyBC0hj3+aS/[EMAIL PROTECTED]> wrote:
Rocky wrote:
http://svn.zope.org/?rev=72782&view=rev

I just added notify(BeforeTraverseEvent(self, REQUEST)) to DynamicType's
__before_publishing_traverse__.

Hmm... ok, which sounds like we've done away with the need to call
make_objectmanager_site().  And as long as the cmf site provides
IObjectManagerSite (and we decided that it would always provide this
right?)

Yes. IObjectManagerSite is now implemented by the class.

then make_objectmanager_site being accidentally called on it
would be harmless.  make_objectmanager_site will still of course be
useful for turning regular zope folders into sites.

Maybe we should decide if we leave the notify call in DynamicType (in that case turning CMF folders into sub-sites should not add an extra hook) or if we override __before_publishing_traverse__ just for PortalObjectBase.

Cheers, Yuppie


_______________________________________________
Zope-CMF maillist  -  Zope-CMF@lists.zope.org
http://mail.zope.org/mailman/listinfo/zope-cmf

See http://collector.zope.org/CMF for bug reports and feature requests

Reply via email to