Commit 20f0492c authored by Sidnei da Silva's avatar Sidnei da Silva

- Forward-port fix from 2.7:

      - WebDAV COPY and MOVE did not call '_notifyOfCopyTo' and
        '_postCopy' hooks like it was done in
        OFS.CopySupport. Additionally added
        'manage_changeOwnershipType' to make MOVE behave even closer
        to OFS.CopySupport.
parent 6c7d7749
...@@ -38,6 +38,12 @@ Zope Changes ...@@ -38,6 +38,12 @@ Zope Changes
- Fixed CMFBTreeFolder for CMF 1.5+ - Fixed CMFBTreeFolder for CMF 1.5+
- WebDAV COPY and MOVE did not call '_notifyOfCopyTo' and
'_postCopy' hooks like it was done in
OFS.CopySupport. Additionally added
'manage_changeOwnershipType' to make MOVE behave even closer
to OFS.CopySupport.
- Collector #1548: Fix 'httplib' usage in ZPublisher.Client. - Collector #1548: Fix 'httplib' usage in ZPublisher.Client.
- Collector #1808: manage_convertIndexes no longer tries to change the - Collector #1808: manage_convertIndexes no longer tries to change the
......
...@@ -373,6 +373,7 @@ class Resource(ExtensionClass.Base, Lockable.LockableItem): ...@@ -373,6 +373,7 @@ class Resource(ExtensionClass.Base, Lockable.LockableItem):
else: else:
raise Locked, 'Destination is locked.' raise Locked, 'Destination is locked.'
self._notifyOfCopyTo(parent, op=0)
ob = self._getCopy(parent) ob = self._getCopy(parent)
ob._setId(name) ob._setId(name)
...@@ -385,6 +386,7 @@ class Resource(ExtensionClass.Base, Lockable.LockableItem): ...@@ -385,6 +386,7 @@ class Resource(ExtensionClass.Base, Lockable.LockableItem):
parent._delObject(name) parent._delObject(name)
parent._setObject(name, ob) parent._setObject(name, ob)
ob = parent._getOb(name) ob = parent._getOb(name)
ob._postCopy(parent, op=0)
ob.manage_afterClone(ob) ob.manage_afterClone(ob)
# We remove any locks from the copied object because webdav clients # We remove any locks from the copied object because webdav clients
# don't track the lock status and the lock token for copied resources # don't track the lock status and the lock token for copied resources
...@@ -481,7 +483,12 @@ class Resource(ExtensionClass.Base, Lockable.LockableItem): ...@@ -481,7 +483,12 @@ class Resource(ExtensionClass.Base, Lockable.LockableItem):
raise PreconditionFailed, 'Source is locked and no '\ raise PreconditionFailed, 'Source is locked and no '\
'condition was passed in.' 'condition was passed in.'
ob=aq_base(self._getCopy(parent)) # try to make ownership explicit so that it gets carried
# along to the new location if needed.
self.manage_changeOwnershipType(explicit=1)
self._notifyOfCopyTo(parent, op=1)
ob = aq_base(self._getCopy(parent))
self.aq_parent._delObject(absattr(self.id)) self.aq_parent._delObject(absattr(self.id))
ob._setId(name) ob._setId(name)
if existing: if existing:
...@@ -489,6 +496,12 @@ class Resource(ExtensionClass.Base, Lockable.LockableItem): ...@@ -489,6 +496,12 @@ class Resource(ExtensionClass.Base, Lockable.LockableItem):
self.dav__validate(object, 'DELETE', REQUEST) self.dav__validate(object, 'DELETE', REQUEST)
parent._delObject(name) parent._delObject(name)
parent._setObject(name, ob) parent._setObject(name, ob)
ob = parent._getOb(name)
ob._postCopy(parent, op=1)
# try to make ownership implicit if possible
ob.manage_changeOwnershipType(explicit=0)
RESPONSE.setStatus(existing and 204 or 201) RESPONSE.setStatus(existing and 204 or 201)
if not existing: if not existing:
RESPONSE.setHeader('Location', dest) RESPONSE.setHeader('Location', dest)
......
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