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 3809ef076660b210586b020fec20b34eb272f059..1633a640acb71868e81f3dcf9024dd2214fc9ba9 100644 (file)
--- a/sercom/subcontrollers/enunciado/__init__.py
+++ b/sercom/subcontrollers/enunciado/__init__.py
@@ -95,7 +95,7 @@ class EnunciadoForm(W.TableForm):
             options=get_options, validator=V.Int(not_empty=False))
         descripcion = W.TextField(label=_(u'Descripción'),
             validator=V.UnicodeString(not_empty=False, max=255, strip=True))
-        archivo = W.FileField(label=_(u'Archivo'))
+        el_archivo = W.FileField(label=_(u'Archivo'))
         tareas_fuente = AjaxDosListasSelect(label=_(u'Tareas Fuente'),
             title_from=u'Disponibles',
             title_to=u'Asignadas',
@@ -150,11 +150,10 @@ class EnunciadoController(controllers.Controller, identity.SecureResource):
     @error_handler(new)
     @expose()
     @identity.require(identity.has_permission('admin'))
-    def create(self, archivo, **kw):
+    def create(self, el_archivo, **kw):
         """Save or create record to model"""
-        kw['archivo'] = archivo.file.read()
-        kw['archivo_name'] = archivo.filename
-        kw['archivo_type'] = 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'])
@@ -178,16 +177,18 @@ class EnunciadoController(controllers.Controller, identity.SecureResource):
     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)
     @error_handler(edit)
     @expose()
     @identity.require(identity.has_permission('admin'))
-    def update(self, id, archivo, **kw):
+    def update(self, id, el_archivo, **kw):
         """Save or create record to model"""
+        if el_archivo.filename:
+            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'])
@@ -228,12 +229,11 @@ class EnunciadoController(controllers.Controller, identity.SecureResource):
     @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)