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 c18199475210cd956616fa8186eb9242b239161b..1633a640acb71868e81f3dcf9024dd2214fc9ba9 100644 (file)
--- a/sercom/subcontrollers/enunciado/__init__.py
+++ b/sercom/subcontrollers/enunciado/__init__.py
@@ -12,6 +12,7 @@ from sercom.subcontrollers import validate as val
 from sercom.model import Enunciado, Docente, Curso, Tarea, TareaFuente, TareaPrueba
 from cherrypy import request, response
 from sercom.widgets import *
 from sercom.model import Enunciado, Docente, Curso, Tarea, TareaFuente, TareaPrueba
 from cherrypy import request, response
 from sercom.widgets import *

+from caso_de_prueba import CasoDePruebaController

 #}}}
 
 #{{{ Configuración
 #}}}
 
 #{{{ Configuración


@@ -94,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))
             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',
         tareas_fuente = AjaxDosListasSelect(label=_(u'Tareas Fuente'),
             title_from=u'Disponibles',
             title_to=u'Asignadas',


@@ -117,6 +118,8 @@ class EnunciadoController(controllers.Controller, identity.SecureResource):
     """Basic model admin interface"""
     require = identity.has_permission('entregar')
 
     """Basic model admin interface"""
     require = identity.has_permission('entregar')
 

+    caso_de_prueba = CasoDePruebaController()
+

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


@@ -147,11 +150,10 @@ class EnunciadoController(controllers.Controller, identity.SecureResource):
     @error_handler(new)
     @expose()
     @identity.require(identity.has_permission('admin'))
     @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"""
         """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'])
         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'])


@@ -175,16 +177,18 @@ 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)
     @error_handler(edit)
     @expose()
     @identity.require(identity.has_permission('admin'))
         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"""
         """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'])
         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'])


@@ -225,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)