Minor improvements to the processing panel

- It is now possible to actually see what's processing, due to a bug fix
  where __getitem__ was called on the db model.
- Removed DEPRECATED message from the docstring, it wasn't true.
This commit is contained in:
Joar Wandborg
2012-07-10 17:53:37 +02:00
parent 81d3c4cf1c
commit 51eb0267d9
4 changed files with 33 additions and 8 deletions

View File

@@ -24,6 +24,8 @@ from mediagoblin.media_types import get_media_manager
from mediagoblin.processing import mark_entry_failed, BaseProcessingFail
_log = logging.getLogger(__name__)
logging.basicConfig()
_log.setLevel(logging.DEBUG)
################################
@@ -32,8 +34,6 @@ _log = logging.getLogger(__name__)
class ProcessMedia(Task):
"""
DEPRECATED -- This now resides in the individual media plugins
Pass this entry off for processing.
"""
def run(self, media_id):
@@ -44,16 +44,21 @@ class ProcessMedia(Task):
entry = mgg.database.MediaEntry.one(
{'_id': ObjectId(media_id)})
_log.info('Running task {0} on media {1}: {2}'.format(
self.name,
entry._id,
entry.title))
# Try to process, and handle expected errors.
try:
#__import__(entry.media_type)
manager = get_media_manager(entry.media_type)
_log.debug('Processing {0}'.format(entry))
manager['processor'](entry)
except BaseProcessingFail, exc:
except BaseProcessingFail as exc:
mark_entry_failed(entry._id, exc)
return
except ImportError, exc:
except ImportError as exc:
_log.error(
'Entry {0} failed to process due to an import error: {1}'\
.format(