aboutsummaryrefslogtreecommitdiffstats
path: root/mediagoblin/process_media
diff options
context:
space:
mode:
authorNathan Yergler <nathan@yergler.net>2011-10-01 15:10:02 -0700
committerNathan Yergler <nathan@yergler.net>2011-10-01 15:10:02 -0700
commit243c3843bd574129caa7663e25d1a843b2d2dd30 (patch)
tree0171de5e911010b76e86c689bb5f5f8ce86845db /mediagoblin/process_media
parent573aba86b58c2ab064d0d57ed0bbae6bdf9a2819 (diff)
downloadmediagoblin-243c3843bd574129caa7663e25d1a843b2d2dd30.tar.lz
mediagoblin-243c3843bd574129caa7663e25d1a843b2d2dd30.tar.xz
mediagoblin-243c3843bd574129caa7663e25d1a843b2d2dd30.zip
Whitespace and formatting cleanup.
* Removed trailing whitespace * Line length < 80 where possible * Honor conventions on number of blank lines * Honor conventions about spaces around :, =
Diffstat (limited to 'mediagoblin/process_media')
-rw-r--r--mediagoblin/process_media/__init__.py18
-rw-r--r--mediagoblin/process_media/errors.py9
2 files changed, 15 insertions, 12 deletions
diff --git a/mediagoblin/process_media/__init__.py b/mediagoblin/process_media/__init__.py
index 2b9eed6e..9a7d5c39 100644
--- a/mediagoblin/process_media/__init__.py
+++ b/mediagoblin/process_media/__init__.py
@@ -65,9 +65,10 @@ class ProcessMedia(Task):
"""
If the processing failed we should mark that in the database.
- Assuming that the exception raised is a subclass of BaseProcessingFail,
- we can use that to get more information about the failure and store that
- for conveying information to users about the failure, etc.
+ Assuming that the exception raised is a subclass of
+ BaseProcessingFail, we can use that to get more information
+ about the failure and store that for conveying information to
+ users about the failure, etc.
"""
entry_id = args[0]
mark_entry_failed(entry_id, exc)
@@ -80,10 +81,10 @@ def mark_entry_failed(entry_id, exc):
"""
Mark a media entry as having failed in its conversion.
- Uses the exception that was raised to mark more information. If the
- exception is a derivative of BaseProcessingFail then we can store extra
- information that can be useful for users telling them why their media failed
- to process.
+ Uses the exception that was raised to mark more information. If
+ the exception is a derivative of BaseProcessingFail then we can
+ store extra information that can be useful for users telling them
+ why their media failed to process.
Args:
- entry_id: The id of the media entry
@@ -164,7 +165,8 @@ def process_image(entry):
with queued_file:
original_filepath = create_pub_filepath(entry, queued_filepath[-1])
- with mgg.public_store.get_file(original_filepath, 'wb') as original_file:
+ with mgg.public_store.get_file(original_filepath, 'wb') \
+ as original_file:
original_file.write(queued_file.read())
mgg.queue_store.delete_file(queued_filepath)
diff --git a/mediagoblin/process_media/errors.py b/mediagoblin/process_media/errors.py
index 156f0a01..cb236154 100644
--- a/mediagoblin/process_media/errors.py
+++ b/mediagoblin/process_media/errors.py
@@ -16,17 +16,18 @@
from mediagoblin.util import lazy_pass_to_ugettext as _
+
class BaseProcessingFail(Exception):
"""
Base exception that all other processing failure messages should
subclass from.
-
+
You shouldn't call this itself; instead you should subclass it
and provid the exception_path and general_message applicable to
this error.
"""
general_message = u''
-
+
@property
def exception_path(self):
return u"%s:%s" % (
@@ -34,8 +35,8 @@ class BaseProcessingFail(Exception):
def __init__(self, **metadata):
self.metadata = metadata or {}
-
-
+
+
class BadMediaFail(BaseProcessingFail):
"""
Error that should be raised when an inappropriate file was given