heckyel/mediagoblin
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Removing docstring bit about delete_file possibly deleting directories in the future

Browse Source 
I agree that delete_dir as a separate operation is a better way to do
things, especially since there is a non-recursive deletion option that
will politely fail if the directory is not empty.
This commit is contained in:
Christopher Allan Webber 2013-01-22 14:32:00 -06:00
parent 36ae6bcbbb
commit 0ac8317ddc
1 changed files with 0 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
mediagoblin/storage/__init__.py
View File

@ -102,10 +102,6 @@ class StorageInterface(object):
def delete_file(self, filepath): def delete_file(self, filepath):
""" """
Delete or dereference the file (not directory) at filepath. Delete or dereference the file (not directory) at filepath.
TODO: is the below comment correct? AFAIK, we won't clean up empty directories...
This might need to delete directories, buckets, whatever, for
cleanliness. (Be sure to avoid race conditions on that though)
""" """
# Subclasses should override this method. # Subclasses should override this method.
self.__raise_not_implemented() self.__raise_not_implemented()