Commit 21f7f035 authored by Jérome Perrin's avatar Jérome Perrin

websection: Do not accumulate websection traveral hook when copy pasting

Also cleanup existing documents
parent 85ca0d87
...@@ -151,6 +151,21 @@ class WebSection(Domain, DocumentExtensibleTraversableMixin): ...@@ -151,6 +151,21 @@ class WebSection(Domain, DocumentExtensibleTraversableMixin):
BeforeTraverse.registerBeforeTraverse(item, WebSectionTraversalHook(), handle) BeforeTraverse.registerBeforeTraverse(item, WebSectionTraversalHook(), handle)
super(WebSection, self).manage_afterAdd(item, container) super(WebSection, self).manage_afterAdd(item, container)
security.declarePrivate( 'manage_afterClone' )
def manage_afterClone(self, item):
self._cleanupBeforeTraverseHooks()
super(WebSection, self).manage_afterClone(item)
def _cleanupBeforeTraverseHooks(self):
# unregister all before traversal hooks that do not belong to us.
my_handle = self.meta_type + '/' + self.getId()
handle_to_unregister_list = []
for (priority, handle), hook in self.__before_traverse__.items():
if isinstance(hook, WebSectionTraversalHook) and handle != my_handle:
handle_to_unregister_list.append(handle)
for handle in handle_to_unregister_list:
BeforeTraverse.unregisterBeforeTraverse(self, handle)
security.declareProtected(Permissions.AccessContentsInformation, 'getLayoutProperty') security.declareProtected(Permissions.AccessContentsInformation, 'getLayoutProperty')
def getLayoutProperty(self, key, default=None): def getLayoutProperty(self, key, default=None):
""" """
...@@ -427,8 +442,12 @@ class WebSection(Domain, DocumentExtensibleTraversableMixin): ...@@ -427,8 +442,12 @@ class WebSection(Domain, DocumentExtensibleTraversableMixin):
return result return result
def _edit(self, **kw): def _edit(self, **kw):
# migrate beforeTraverse hook if missing if self.getPortalType() == 'Web Section':
if getattr(self, '__before_traverse__', None) is None and self.getPortalType() == 'Web Section': if getattr(self, '__before_traverse__', None) is None:
handle = self.meta_type + '/' + self.getId() # migrate beforeTraverse hook if missing
BeforeTraverse.registerBeforeTraverse(self, WebSectionTraversalHook(), handle) handle = self.meta_type + '/' + self.getId()
BeforeTraverse.registerBeforeTraverse(self, WebSectionTraversalHook(), handle)
else:
# cleanup beforeTraverse hooks that may exist after this document was cloned.
self._cleanupBeforeTraverseHooks()
super(WebSection, self)._edit(**kw) super(WebSection, self)._edit(**kw)
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment