diff options
author | Rogelio Mita <rogeliomita@activitycentral.com> | 2013-10-03 03:47:59 (GMT) |
---|---|---|
committer | Rogelio Mita <rogeliomita@activitycentral.com> | 2013-10-03 03:47:59 (GMT) |
commit | 77fca0aa9ab62c7fb21a2f47382a2931530c8b2d (patch) | |
tree | da76295c530e2f06d2d6a2e66e9c1638fa7e031f /webapp/polls/views.py | |
parent | d742d88416b8b6e9bf76cd8df2346acab5ff29c6 (diff) |
Moving file existence logic to a higher up level
Diffstat (limited to 'webapp/polls/views.py')
-rw-r--r-- | webapp/polls/views.py | 28 |
1 files changed, 14 insertions, 14 deletions
diff --git a/webapp/polls/views.py b/webapp/polls/views.py index 578d137..0c8ced0 100644 --- a/webapp/polls/views.py +++ b/webapp/polls/views.py @@ -375,24 +375,24 @@ class UnploadPollResultFormView(TemplateView): msg = u'{0}: No es un .poll_result válido.'.format(fname) messages.add_message(self.request, messages.ERROR, msg) + # comprueba si los archivos ya existían y los excluye + uploaded_files = [] + for index, file in enumerate(valid_files): + prf = PollResultFile(file.temporary_file_path()) + if prf.exists() or not prf.validate(): + uploaded_files.append(file.name) + del valid_files[index] + + if len(uploaded_files): + msg = u'Los siguientes resultados ya se encuentran \ + publicados: %s y no serán procesados.' % ( + ", ".join(uploaded_files)) + messages.add_message(self.request, messages.INFO, msg) + # si se subió algún archivo: if len(valid_files): # Esto hace todo - # comprueba si los archivos ya existían y los excluye - uploaded_files = [] - for index, file in enumerate(valid_files): - prf = PollResultFile(file.temporary_file_path()) - if prf.exists() or not prf.validate(): - uploaded_files.append(file.name) - del valid_files[index] - - if len(uploaded_files): - msg = u'Los siguientes resultados ya se encuentran \ - publicados: %s y no serán procesados.' % ( - ", ".join(uploaded_files)) - messages.add_message(self.request, messages.INFO, msg) - # si no hay más archivos, vuelve al usuario if not len(valid_files): return self.render_to_response(context) |