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

We don't need to save entries during processing... also adding comments explaining such

Browse Source
This commit is contained in:
Christopher Allan Webber 2012-12-12 10:13:56 -06:00
parent af6a43d140
commit 6af6bc05bb
3 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
mediagoblin/media_types/audio/processing.py
View File

@ -144,7 +144,5 @@ def process_audio(entry):
mgg.queue_store.delete_file(queued_filepath) mgg.queue_store.delete_file(queued_filepath)
entry.save()
# clean up workbench # clean up workbench
workbench.destroy_self() workbench.destroy_self()

3
mediagoblin/media_types/video/processing.py
View File

@ -135,6 +135,3 @@ def process_video(entry):
entry.media_files['original'] = original_filepath entry.media_files['original'] = original_filepath
mgg.queue_store.delete_file(queued_filepath) mgg.queue_store.delete_file(queued_filepath)
# Save the MediaEntry
entry.save()

3
mediagoblin/processing/task.py
View File

@ -51,8 +51,11 @@ class ProcessMedia(Task):
_log.debug('Processing {0}'.format(entry)) _log.debug('Processing {0}'.format(entry))
# run the processing code
entry.media_manager['processor'](entry) entry.media_manager['processor'](entry)
# We set the state to processed and save the entry here so there's
# no need to save at the end of the processing stage, probably ;)
entry.state = u'processed' entry.state = u'processed'
entry.save() entry.save()