From 6d539afda68c5c3b14b744225b6037ce4ea11423 Mon Sep 17 00:00:00 2001 From: Jorge Araya Navarro Date: Thu, 5 Jul 2012 22:07:44 -0600 Subject: [PATCH] changing NoSuchFiles for no_such_files --- mediagoblin/tools/files.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) 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) -- 2.25.1