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

Just raise standard exception. Pass print statement to gettext

Browse Source
This commit is contained in:
Rodney Ewing 2013-08-01 13:51:36 -07:00
parent 7c1f6a6aee
commit 81d880b16a
1 changed files with 10 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
mediagoblin/gmg_commands/reprocess.py
View File

@ -41,13 +41,6 @@ def reprocess_parser_setup(subparser):
help="The media_entry id(s) you wish to reprocess.") help="The media_entry id(s) you wish to reprocess.")
class MismatchingMediaTypes(Exception):
"""
Error that should be raised if the media_types are not the same
"""
pass
def _set_media_type(args): def _set_media_type(args):
if len(args[0].media_id) == 1: if len(args[0].media_id) == 1:
media_type = MediaEntry.query.filter_by(id=args[0].media_id[0])\ media_type = MediaEntry.query.filter_by(id=args[0].media_id[0])\
@ -56,9 +49,8 @@ def _set_media_type(args):
if not args[0].type: if not args[0].type:
args[0].type = media_type args[0].type = media_type
elif args[0].type != media_type: elif args[0].type != media_type:
raise MismatchingMediaTypes(_('The type that you set does not' raise Exception(_('The type that you set does not match the type'
' match the type of the given' ' of the given media_id.'))
' media_id.'))
elif len(args[0].media_id) > 1: elif len(args[0].media_id) > 1:
media_types = [] media_types = []
@ -67,27 +59,26 @@ def _set_media_type(args):
.media_type.split('.')[-1]) .media_type.split('.')[-1])
for type in media_types: for type in media_types:
if media_types[0] != type: if media_types[0] != type:
raise MismatchingMediaTypes((u'You cannot reprocess different' raise Exception((u'You cannot reprocess different media_types'
' media_types at the same time.')) ' at the same time.'))
if not args[0].type: if not args[0].type:
args[0].type = media_types[0] args[0].type = media_types[0]
elif args[0].type != media_types[0]: elif args[0].type != media_types[0]:
raise MismatchingMediaTypes(_('The type that you set does not' raise Exception(_('The type that you set does not match the type'
' match the type of the given' ' of the given media_ids.'))
' media_ids.'))
elif not args[0].type: elif not args[0].type:
raise MismatchingMediaTypes(_('You must provide either a media_id or' raise Exception(_('You must provide either a media_id or set the'
' set the --type flag')) ' --type flag'))
def _reprocess_all(args): def _reprocess_all(args):
if not args[0].type: if not args[0].type:
if args[0].state == 'failed': if args[0].state == 'failed':
if args[0].available: if args[0].available:
print '\n Available reprocess actions for all failed' \ print _('\n Available reprocess actions for all failed' \
' media_entries: \n \t --initial_processing' ' media_entries: \n \t --initial_processing')
return return
else: else:
#TODO reprocess all failed entries #TODO reprocess all failed entries