diff options
author | Elrond <elrond+mediagoblin.org@samba-tng.org> | 2013-03-22 15:45:21 +0100 |
---|---|---|
committer | Elrond <elrond+mediagoblin.org@samba-tng.org> | 2013-03-29 12:01:23 +0100 |
commit | 2ef2f46e73845dcd55666cad49c5a17908bf5b46 (patch) | |
tree | 7ac26a4013d48dfe7e77dfa754a2a2036ed03c86 /mediagoblin/submit/views.py | |
parent | 9924cd0fb6f2a9d53a083ce68d1bbf872ee57d9b (diff) | |
download | mediagoblin-2ef2f46e73845dcd55666cad49c5a17908bf5b46.tar.lz mediagoblin-2ef2f46e73845dcd55666cad49c5a17908bf5b46.tar.xz mediagoblin-2ef2f46e73845dcd55666cad49c5a17908bf5b46.zip |
Refactor file field checking.
When uploading, the file field needs some checks, it seems.
So refactor them into check_file_field and use around.
Diffstat (limited to 'mediagoblin/submit/views.py')
-rw-r--r-- | mediagoblin/submit/views.py | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/mediagoblin/submit/views.py b/mediagoblin/submit/views.py index def7e839..1e47d259 100644 --- a/mediagoblin/submit/views.py +++ b/mediagoblin/submit/views.py @@ -22,7 +22,6 @@ import logging _log = logging.getLogger(__name__) -from werkzeug.datastructures import FileStorage from mediagoblin.tools.text import convert_to_tag_list_of_dicts from mediagoblin.tools.translate import pass_to_ugettext as _ @@ -32,7 +31,8 @@ from mediagoblin.submit import forms as submit_forms from mediagoblin.messages import add_message, SUCCESS from mediagoblin.media_types import sniff_media, \ InvalidFileType, FileTypeNotSupported -from mediagoblin.submit.lib import run_process_media, prepare_queue_task +from mediagoblin.submit.lib import check_file_field, prepare_queue_task, \ + run_process_media @require_active_login @@ -44,9 +44,7 @@ def submit_start(request): license=request.user.license_preference) if request.method == 'POST' and submit_form.validate(): - if not ('file' in request.files - and isinstance(request.files['file'], FileStorage) - and request.files['file'].stream): + if not check_file_field(request, 'file'): submit_form.file.errors.append( _(u'You must provide a file.')) else: |