Bugfix: El zip se subía roto en las entregas.

[z.facultad/75.52/sercom.git] / sercom / subcontrollers / misentregas / __init__.py

diff --git a/sercom/subcontrollers/misentregas/__init__.py b/sercom/subcontrollers/misentregas/__init__.py
index 8ea647f016137b4a85121412da59ae23380af16b..31c76783293a28a790a810ad1a978e2550fb3d22 100644 (file)
--- a/sercom/subcontrollers/misentregas/__init__.py
+++ b/sercom/subcontrollers/misentregas/__init__.py
@@ -105,6 +105,8 @@ class MisEntregasController(controllers.Controller, identity.SecureResource):
     """Basic model admin interface"""
     require = identity.has_permission('entregar')
 
     """Basic model admin interface"""
     require = identity.has_permission('entregar')
 

+    hide_to_admin = 1
+

     @expose()
     def default(self, tg_errors=None):
         """handle non exist urls"""
     @expose()
     def default(self, tg_errors=None):
         """handle non exist urls"""


@@ -138,10 +140,14 @@ class MisEntregasController(controllers.Controller, identity.SecureResource):
     @expose()
     def create(self, archivo, ejercicio, **kw):
         """Save or create record to model"""
     @expose()
     def create(self, archivo, ejercicio, **kw):
         """Save or create record to model"""

+        archivo = archivo.file.read()

         try:
         try:

-            zfile = ZipFile(archivo.file)
+            zfile = ZipFile(StringIO(archivo), 'r')

         except BadZipfile:
         except BadZipfile:

-            flash(_(u'El archivo ZIP no es valido'))
+            flash(_(u'El archivo ZIP no es válido'))
+            raise redirect('list')
+        if zfile.testzip() is not None:
+            flash(_(u'El archivo ZIP tiene errores de CRC'))

             raise redirect('list')
 
         # por defecto el entregador es el user loggeado
             raise redirect('list')
 
         # por defecto el entregador es el user loggeado


@@ -169,7 +175,7 @@ class MisEntregasController(controllers.Controller, identity.SecureResource):
                 raise redirect('list')
 
             entregador = m.grupo
                 raise redirect('list')
 
             entregador = m.grupo

-        kw['archivos'] = archivo.file.read()
+        kw['archivos'] = archivo

         kw['entregador'] = entregador
         validate_new(kw)
         flash(_(u'Se creó una nueva %s.') % name)
         kw['entregador'] = entregador
         validate_new(kw)
         flash(_(u'Se creó una nueva %s.') % name)