InstanciaDeEntrega.observaciones no puede ser None para simplificar concatenación.

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

diff --git a/sercom/subcontrollers/enunciado/__init__.py b/sercom/subcontrollers/enunciado/__init__.py
index a81c9e316b83b4397532b1e78727940bbbcf5e53..1633a640acb71868e81f3dcf9024dd2214fc9ba9 100644 (file)
--- a/sercom/subcontrollers/enunciado/__init__.py
+++ b/sercom/subcontrollers/enunciado/__init__.py
@@ -152,9 +152,8 @@ class EnunciadoController(controllers.Controller, identity.SecureResource):
     @identity.require(identity.has_permission('admin'))
     def create(self, el_archivo, **kw):
         """Save or create record to model"""
     @identity.require(identity.has_permission('admin'))
     def create(self, el_archivo, **kw):
         """Save or create record to model"""

-        kw['archivo'] = el_archivo.file.read()
-        kw['archivo_name'] = el_archivo.filename
-        kw['archivo_type'] = el_archivo.type
+        if el_archivo.filename:
+            kw['archivos'] = el_archivo.file.read() # TODO verificar que es ZIP

         if 'tareas_fuente_to' in kw.keys() and 'tareas_prueba_to' in kw.keys():
             kw['tareas'] = list(kw['tareas_fuente_to']) + list(kw['tareas_prueba_to'])
             del(kw['tareas_fuente_to'])
         if 'tareas_fuente_to' in kw.keys() and 'tareas_prueba_to' in kw.keys():
             kw['tareas'] = list(kw['tareas_fuente_to']) + list(kw['tareas_prueba_to'])
             del(kw['tareas_fuente_to'])


@@ -178,8 +177,8 @@ class EnunciadoController(controllers.Controller, identity.SecureResource):
     def edit(self, id, **kw):
         """Edit record in model"""
         r = validate_get(id)
     def edit(self, id, **kw):
         """Edit record in model"""
         r = validate_get(id)

-        r.tareas_fuente = [{"id":t.id, "label":t.shortrepr()} for t in filter(lambda x: isinstance(x, TareaFuente), r.tareas)]
-        r.tareas_prueba = [{"id":t.id, "label":t.shortrepr()} for t in filter(lambda x: isinstance(x, TareaPrueba), r.tareas)]
+        r.tareas_fuente = [{"id":t.id, "label":t.shortrepr()} for t in r.tareas if isinstance(t, TareaFuente)]
+        r.tareas_prueba = [{"id":t.id, "label":t.shortrepr()} for t in r.tareas if isinstance(t, TareaPrueba)]

         return dict(name=name, namepl=namepl, record=r, form=form)
 
     @validate(form=form)
         return dict(name=name, namepl=namepl, record=r, form=form)
 
     @validate(form=form)


@@ -189,9 +188,7 @@ class EnunciadoController(controllers.Controller, identity.SecureResource):
     def update(self, id, el_archivo, **kw):
         """Save or create record to model"""
         if el_archivo.filename:
     def update(self, id, el_archivo, **kw):
         """Save or create record to model"""
         if el_archivo.filename:

-            kw['archivo'] = el_archivo.file.read()
-            kw['archivo_name'] = el_archivo.filename
-            kw['archivo_type'] = el_archivo.type
+            kw['archivos'] = el_archivo.file.read()

         if 'tareas_fuente_to' in kw.keys() and 'tareas_prueba_to' in kw.keys():
             kw['tareas'] = list(kw['tareas_fuente_to']) + list(kw['tareas_prueba_to'])
             del(kw['tareas_fuente_to'])
         if 'tareas_fuente_to' in kw.keys() and 'tareas_prueba_to' in kw.keys():
             kw['tareas'] = list(kw['tareas_fuente_to']) + list(kw['tareas_prueba_to'])
             del(kw['tareas_fuente_to'])


@@ -232,12 +229,11 @@ class EnunciadoController(controllers.Controller, identity.SecureResource):
     @expose()
     def files(self, id):
         r = validate_get(id)
     @expose()
     def files(self, id):
         r = validate_get(id)

-        response.headers["Content-Type"] = r.archivo_type
-        response.headers["Content-disposition"] = "attachment;filename=%s" % (r.archivo_name)
-        flash(_(u'El %s fue eliminado permanentemente.') % name)
-        return r.archivo
+        response.headers["Content-Type"] = 'application/zip'
+        response.headers["Content-disposition"] = 'attachment;filename=enunciado.zip'
+        return r.archivos

 
 

-    @expose("json")
+    @expose('json')

     @identity.require(identity.has_permission('admin'))
     def de_curso(self, curso_id):
         c = Curso.get(curso_id)
     @identity.require(identity.has_permission('admin'))
     def de_curso(self, curso_id):
         c = Curso.get(curso_id)