Commit 8c85cda9 authored by Maurits van Rees's avatar Maurits van Rees

Log a warning when making a partial copy.

parent f58a2968
...@@ -16,6 +16,7 @@ ...@@ -16,6 +16,7 @@
from cgi import escape from cgi import escape
from marshal import dumps from marshal import dumps
from marshal import loads from marshal import loads
import logging
import re import re
import sys import sys
import tempfile import tempfile
...@@ -61,8 +62,8 @@ class CopyError(Exception): ...@@ -61,8 +62,8 @@ class CopyError(Exception):
pass pass
copy_re = re.compile('^copy([0-9]*)_of_(.*)') copy_re = re.compile('^copy([0-9]*)_of_(.*)')
logger = logging.getLogger('OFS')
_marker=[] _marker = []
class CopyContainer(Base): class CopyContainer(Base):
...@@ -607,6 +608,12 @@ class CopySource(Base): ...@@ -607,6 +608,12 @@ class CopySource(Base):
# We do not use cp._delObject, because this would fire # We do not use cp._delObject, because this would fire
# 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(
'While copying %s, removed %s from copy '
'because user is not allowed to view the original.',
'/'.join(self.getPhysicalPath()),
'/'.join(v.getPhysicalPath())
)
cp._delOb(k) cp._delOb(k)
# We need to cleanup the internal objects list, even when # We need to cleanup the internal objects list, even when
# in some implementations this is always an empty tuple. # in some implementations this is always an empty tuple.
......
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