Commit 7d206e02 by Chris Dodge

remove comment which is no longer valid as well as debug log messages

parent bbea8eb9
......@@ -615,16 +615,10 @@ def delete_item(request):
delete_children = request.POST.get('delete_children', False)
delete_all_versions = request.POST.get('delete_all_versions', False)
logging.debug('delete_all_versions = {0}'.format(delete_all_versions))
store = modulestore()
item = store.get_item(item_location)
# @TODO: this probably leaves draft items dangling. My preferance would be for the semantic to be
# if item.location.revision=None, then delete both draft and published version
# if caller wants to only delete the draft than the caller should put item.location.revision='draft'
if delete_children:
_xmodule_recurse(item, lambda i: store.delete_item(i.location, delete_all_versions))
else:
......
......@@ -701,7 +701,6 @@ class MongoModuleStore(ModuleStoreBase):
location: Something that can be passed to Location
delete_all_versions: is here because the DraftMongoModuleStore needs it and we need to keep the interface the same. It is unused.
"""
logging.debug('delete_all_versions = {0}'.format(delete_all_versions))
# VS[compat] cdodge: This is a hack because static_tabs also have references from the course module, so
# if we add one then we need to also add it to the policy information (i.e. metadata)
# we should remove this once we can break this reference from the course to static tabs
......
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