From: Jorge Araya Navarro Date: Fri, 6 Jul 2012 04:07:44 +0000 (-0600) Subject: changing NoSuchFiles for no_such_files X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=6d539afda68c5c3b14b744225b6037ce4ea11423;p=mediagoblin.git changing NoSuchFiles for no_such_files --- diff --git a/mediagoblin/tools/files.py b/mediagoblin/tools/files.py index 31a7a972..2b4ad4e4 100644 --- a/mediagoblin/tools/files.py +++ b/mediagoblin/tools/files.py @@ -36,21 +36,21 @@ def delete_media_files(media): Arguments: - media: A MediaEntry document """ - noSuchFiles = [] + no_such_files = [] for listpath in media.media_files.itervalues(): try: mg_globals.public_store.delete_file( listpath) except OSError: - noSuchFiles.append(_jointhat(listpath)) + no_such_files.append(_jointhat(listpath)) for attachment in media.attachment_files: try: mg_globals.public_store.delete_file( attachment['filepath']) except OSError: - noSuchFiles.append(_jointhat(attachment)) + no_such_files.append(_jointhat(attachment)) - if noSuchFiles: + if no_such_files: # This breaks pep8 as far as I know raise OSError, ", ".join(noSuchFiles)