Commit d336b029 authored by Maurits van Rees's avatar Maurits van Rees

Include the target folder when warning about partial copy.

parent 8c85cda9
...@@ -596,9 +596,9 @@ class CopySource(Base): ...@@ -596,9 +596,9 @@ class CopySource(Base):
# being copied, so it makes no sense to check any of its sub # being copied, so it makes no sense to check any of its sub
# objects. It probably means we are in a test. # objects. It probably means we are in a test.
return ob return ob
return self._cleanupCopy(ob) return self._cleanupCopy(ob, container)
def _cleanupCopy(self, cp): def _cleanupCopy(self, cp, container):
sm = getSecurityManager() sm = getSecurityManager()
ob = aq_base(self) ob = aq_base(self)
if hasattr(ob, 'objectIds'): if hasattr(ob, 'objectIds'):
...@@ -609,9 +609,10 @@ class CopySource(Base): ...@@ -609,9 +609,10 @@ class CopySource(Base):
# events that are needless for objects that are not even in # events that are needless for objects that are not even in
# an Acquisition chain yet. # an Acquisition chain yet.
logger.warn( logger.warn(
'While copying %s, removed %s from copy ' 'While copying %s to %s, removed %s from copy '
'because user is not allowed to view the original.', 'because user is not allowed to view the original.',
'/'.join(self.getPhysicalPath()), '/'.join(self.getPhysicalPath()),
'/'.join(container.getPhysicalPath()),
'/'.join(v.getPhysicalPath()) '/'.join(v.getPhysicalPath())
) )
cp._delOb(k) cp._delOb(k)
...@@ -621,7 +622,7 @@ class CopySource(Base): ...@@ -621,7 +622,7 @@ class CopySource(Base):
i for i in cp._objects if i['id'] != k]) i for i in cp._objects if i['id'] != k])
else: else:
# recursively check # recursively check
v._cleanupCopy(cp._getOb(k)) v._cleanupCopy(cp._getOb(k), container)
return cp return cp
def _postCopy(self, container, op=0): def _postCopy(self, container, op=0):
......
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