Bugfix: edición de attachments explotaba porque qlee el .zip como unicode string.

diff --git a/sercom/subcontrollers/enunciado/caso_de_prueba/__init__.py b/sercom/subcontrollers/enunciado/caso_de_prueba/__init__.py
index ef814ed206d320c7246bc4aab16084762c75d45f..d65078e4db2d90d033429bbae46ce9900c22f012 100644 (file)
--- a/sercom/subcontrollers/enunciado/caso_de_prueba/__init__.py
+++ b/sercom/subcontrollers/enunciado/caso_de_prueba/__init__.py
@@ -63,18 +63,18 @@ class CasoDePruebaForm(W.TableForm):
         nombre = W.TextField(label=_(u'Nombre'), validator=V.UnicodeString(min=3, max=255, strip=True))
         comando = W.TextField(label=_(u'Comando'), validator=V.UnicodeString(min=3, max=255, strip=True))
         descripcion = W.TextField(label=_(u'Descripcion'), validator=V.UnicodeString(min=3, max=255, strip=True))
         nombre = W.TextField(label=_(u'Nombre'), validator=V.UnicodeString(min=3, max=255, strip=True))
         comando = W.TextField(label=_(u'Comando'), validator=V.UnicodeString(min=3, max=255, strip=True))
         descripcion = W.TextField(label=_(u'Descripcion'), validator=V.UnicodeString(min=3, max=255, strip=True))

-        retorno = W.TextField(label=_(u'Retorno'), help_text=u"Codigo de retorno esperado",validator=V.Int(if_empty=0))
-        max_tiempo_cpu = W.TextField(label=_(u'CPU'), help_text=u"Maximo tiempo de CPU que puede utilizar [seg]",validator=V.Int())
-        max_memoria = W.TextField(label=_(u'Memoria'), help_text=u"Maximo cantidad de memoria que puede utilizar [MB]",validator=V.Int())
-        max_tam_archivo = W.TextField(label=_(u'Tam. Archivo'), help_text=u"Maximo tamanio de archivo a crear [MB]",validator=V.Int())
-        max_cant_archivos = W.TextField(label=_(u'Archivos'),validator=V.Int())
-        max_cant_procesos = W.TextField(label=_(u'Procesos'),validator=V.Int())
-        max_locks_memoria = W.TextField(label=_(u'Mem. Locks'),validator=V.Int())
+        retorno = W.TextField(label=_(u'Retorno'), help_text=u"Codigo de retorno esperado",validator=V.Int)
+        max_tiempo_cpu = W.TextField(label=_(u'CPU'), help_text=u"Maximo tiempo de CPU que puede utilizar [seg]",validator=V.Int)
+        max_memoria = W.TextField(label=_(u'Memoria'), help_text=u"Maximo cantidad de memoria que puede utilizar [MB]",validator=V.Int)
+        max_tam_archivo = W.TextField(label=_(u'Tam. Archivo'), help_text=u"Maximo tamanio de archivo a crear [MB]",validator=V.Int)
+        max_cant_archivos = W.TextField(label=_(u'Archivos'),validator=V.Int)
+        max_cant_procesos = W.TextField(label=_(u'Procesos'),validator=V.Int)
+        max_locks_memoria = W.TextField(label=_(u'Mem. Locks'),validator=V.Int)

         terminar_si_falla = W.CheckBox(label=_(u'Terminar si falla'), default=1, validator=V.Bool(if_empty=1))
         rechazar_si_falla = W.CheckBox(label=_(u'Rechazar si falla'), default=1, validator=V.Bool(if_empty=1))
         publico = W.CheckBox(label=_(u'Es público?'), default=1, validator=V.Bool(if_empty=1))
         terminar_si_falla = W.CheckBox(label=_(u'Terminar si falla'), default=1, validator=V.Bool(if_empty=1))
         rechazar_si_falla = W.CheckBox(label=_(u'Rechazar si falla'), default=1, validator=V.Bool(if_empty=1))
         publico = W.CheckBox(label=_(u'Es público?'), default=1, validator=V.Bool(if_empty=1))

-        archivos_entrada = W.FileField(label=_(u'Archivos Entrada'))
-        archivos_a_comparar = W.FileField(label=_(u'Archivos a Comparar'))
+        los_archivos_entrada = W.FileField(label=_(u'Archivos Entrada'))
+        los_archivos_a_comparar = W.FileField(label=_(u'Archivos a Comparar'))

         archivos_guardar = W.TextField(label=_(u'Archivos a Guardar'))
         activo = W.CheckBox(label=_(u'Activo'), default=1, validator=V.Bool(if_empty=1))
     fields = Fields()
         archivos_guardar = W.TextField(label=_(u'Archivos a Guardar'))
         activo = W.CheckBox(label=_(u'Activo'), default=1, validator=V.Bool(if_empty=1))
     fields = Fields()


@@ -110,22 +110,20 @@ class CasoDePruebaController(controllers.Controller, identity.SecureResource):
         """Save or create record to model"""
         t = Enunciado.get(kw['enunciadoID'])
         del(kw['enunciadoID'])
         """Save or create record to model"""
         t = Enunciado.get(kw['enunciadoID'])
         del(kw['enunciadoID'])

-        if kw['archivos_entrada'].filename:
-            kw['archivos_entrada'] = kw['archivos_entrada'].file.read()
-        else:
-            kw['archivos_entrada'] =  None
-        if kw['archivos_a_comparar'].filename:
-            kw['archivos_a_comparar'] = kw['archivos_a_comparar'].file.read()
-        else:
-            kw['archivos_a_comparar'] =  None
+        if kw['los_archivos_entrada'].filename:
+            kw['archivos_entrada'] = kw['los_archivos_entrada'].file.read()
+        del kw['los_archivos_entrada']
+        if kw['los_archivos_a_comparar'].filename:
+            kw['archivos_a_comparar'] = kw['los_archivos_a_comparar'].file.read()
+        del kw['los_archivos_a_comparar']

         # TODO : Hacer ventanita mas amigable para cargar esto.
         try:
             kw['archivos_a_guardar'] = tuple(kw['archivos_guardar'].split(','))
         except AttributeError:
             pass
         # TODO : Hacer ventanita mas amigable para cargar esto.
         try:
             kw['archivos_a_guardar'] = tuple(kw['archivos_guardar'].split(','))
         except AttributeError:
             pass

-        del(kw['archivos_guardar'])
+        del kw['archivos_guardar']

         nombre = kw['nombre'];
         nombre = kw['nombre'];

-        del(kw['nombre'])
+        del kw['nombre']

         t.add_caso_de_prueba(nombre, **kw)
         flash(_(u'Se creó un nuevo %s.') % name)
         raise redirect('list/%d' % t.id)
         t.add_caso_de_prueba(nombre, **kw)
         flash(_(u'Se creó un nuevo %s.') % name)
         raise redirect('list/%d' % t.id)


@@ -143,20 +141,18 @@ class CasoDePruebaController(controllers.Controller, identity.SecureResource):
     @expose()
     def update(self, id, **kw):
         """Save or create record to model"""
     @expose()
     def update(self, id, **kw):
         """Save or create record to model"""

-        if kw['archivos_entrada'].filename:
-            kw['archivos_entrada'] = kw['archivos_entrada'].file.read()
-        else:
-            kw['archivos_entrada'] =  None
-        if kw['archivos_a_comparar'].filename:
-            kw['archivos_a_comparar'] = kw['archivos_a_comparar'].file.read()
-        else:
-            kw['archivos_a_comparar'] =  None
+        if kw['los_archivos_entrada'].filename:
+            kw['archivos_entrada'] = kw['los_archivos_entrada'].file.read()
+        del kw['los_archivos_entrada']
+        if kw['los_archivos_a_comparar'].filename:
+            kw['archivos_a_comparar'] = kw['los_archivos_a_comparar'].file.read()
+        del kw['los_archivos_a_comparar']

         # TODO : Hacer ventanita mas amigable para cargar esto.
         try:
             kw['archivos_a_guardar'] = tuple(kw['archivos_guardar'].split(','))
         except AttributeError:
             pass
         # TODO : Hacer ventanita mas amigable para cargar esto.
         try:
             kw['archivos_a_guardar'] = tuple(kw['archivos_guardar'].split(','))
         except AttributeError:
             pass

-        del(kw['archivos_guardar'])
+        del kw['archivos_guardar']

         r = validate_set(id, kw)
         flash(_(u'El %s fue actualizado.') % name)
         raise redirect('../list/%d' % r.enunciado.id)
         r = validate_set(id, kw)
         flash(_(u'El %s fue actualizado.') % name)
         raise redirect('../list/%d' % r.enunciado.id)

diff --git a/sercom/subcontrollers/tarea_fuente/comandos/__init__.py b/sercom/subcontrollers/tarea_fuente/comandos/__init__.py
index 12bdee2f6677f397535fcdea041e246f19eae9c9..00bae186e5222f334074695506d7ffe2cc209973 100644 (file)
--- a/sercom/subcontrollers/tarea_fuente/comandos/__init__.py
+++ b/sercom/subcontrollers/tarea_fuente/comandos/__init__.py
@@ -48,8 +48,8 @@ class ComandoFuenteForm(W.TableForm):
         terminar_si_falla = W.CheckBox(label=_(u'Terminar si falla'), default=1, validator=V.Bool(if_empty=1))
         rechazar_si_falla = W.CheckBox(label=_(u'Rechazar si falla'), default=1, validator=V.Bool(if_empty=1))
         publico = W.CheckBox(label=_(u'Es público?'), default=1, validator=V.Bool(if_empty=1))
         terminar_si_falla = W.CheckBox(label=_(u'Terminar si falla'), default=1, validator=V.Bool(if_empty=1))
         rechazar_si_falla = W.CheckBox(label=_(u'Rechazar si falla'), default=1, validator=V.Bool(if_empty=1))
         publico = W.CheckBox(label=_(u'Es público?'), default=1, validator=V.Bool(if_empty=1))

-        archivos_entrada = W.FileField(label=_(u'Archivos Entrada'))
-        archivos_a_comparar = W.FileField(label=_(u'Archivos a Comparar'))
+        los_archivos_entrada = W.FileField(label=_(u'Archivos Entrada'))
+        los_archivos_a_comparar = W.FileField(label=_(u'Archivos a Comparar'))

         archivos_guardar = W.TextField(label=_(u'Archivos a Guardar'))
         activo = W.CheckBox(label=_(u'Activo'), default=1, validator=V.Bool(if_empty=1))
     fields = Fields()
         archivos_guardar = W.TextField(label=_(u'Archivos a Guardar'))
         activo = W.CheckBox(label=_(u'Activo'), default=1, validator=V.Bool(if_empty=1))
     fields = Fields()


@@ -83,22 +83,20 @@ class ComandoFuenteController(controllers.Controller, identity.SecureResource):
         """Save or create record to model"""
         t = TareaFuente.get(kw['tareaID'])
         orden = kw['orden']
         """Save or create record to model"""
         t = TareaFuente.get(kw['tareaID'])
         orden = kw['orden']

-        del(kw['orden'])
-        del(kw['tareaID'])
-        if kw['archivos_entrada'].filename:
-            kw['archivos_entrada'] = kw['archivos_entrada'].file.read()
-        else:
-            kw['archivos_entrada'] =  None
-        if kw['archivos_a_comparar'].filename:
-            kw['archivos_a_comparar'] = kw['archivos_a_comparar'].file.read()
-        else:
-            kw['archivos_a_comparar'] =  None
+        del kw['orden']
+        del kw['tareaID']
+        if kw['los_archivos_entrada'].filename:
+            kw['archivos_entrada'] = kw['los_archivos_entrada'].file.read()
+        del kw['los_archivos_entrada']
+        if kw['los_archivos_a_comparar'].filename:
+            kw['archivos_a_comparar'] = kw['los_archivos_a_comparar'].file.read()
+        del kw['los_archivos_a_comparar']

         # TODO : Hacer ventanita mas amigable para cargar esto.
         try:
             kw['archivos_a_guardar'] = tuple(kw['archivos_guardar'].split(','))
         except AttributeError:
             pass
         # TODO : Hacer ventanita mas amigable para cargar esto.
         try:
             kw['archivos_a_guardar'] = tuple(kw['archivos_guardar'].split(','))
         except AttributeError:
             pass

-        del(kw['archivos_guardar'])
+        del kw['archivos_guardar']

         t.add_comando(orden, **kw)
         flash(_(u'Se creó un nuevo %s.') % name)
         raise redirect('list/%d' % t.id)
         t.add_comando(orden, **kw)
         flash(_(u'Se creó un nuevo %s.') % name)
         raise redirect('list/%d' % t.id)

diff --git a/sercom/subcontrollers/tarea_prueba/comandos/__init__.py b/sercom/subcontrollers/tarea_prueba/comandos/__init__.py
index 52f14d8395d5ccfa7d5cae34ba5f0066c10e5c3c..f7fbe670a359730d217150b5cbcaa22fcc006619 100644 (file)
--- a/sercom/subcontrollers/tarea_prueba/comandos/__init__.py
+++ b/sercom/subcontrollers/tarea_prueba/comandos/__init__.py
@@ -48,8 +48,8 @@ class ComandoPruebaForm(W.TableForm):
         terminar_si_falla = W.CheckBox(label=_(u'Terminar si falla?'), default=1, validator=V.Bool(if_empty=1))
         rechazar_si_falla = W.CheckBox(label=_(u'Rechazar si falla?'), default=1, validator=V.Bool(if_empty=1))
         publico = W.CheckBox(label=_(u'Es público?'), default=1, validator=V.Bool(if_empty=1))
         terminar_si_falla = W.CheckBox(label=_(u'Terminar si falla?'), default=1, validator=V.Bool(if_empty=1))
         rechazar_si_falla = W.CheckBox(label=_(u'Rechazar si falla?'), default=1, validator=V.Bool(if_empty=1))
         publico = W.CheckBox(label=_(u'Es público?'), default=1, validator=V.Bool(if_empty=1))

-        archivos_entrada = W.FileField(label=_(u'Archivos Entrada'))
-        archivos_a_comparar = W.FileField(label=_(u'Archivos a Comparar'))
+        los_archivos_entrada = W.FileField(label=_(u'Archivos Entrada'))
+        los_archivos_a_comparar = W.FileField(label=_(u'Archivos a Comparar'))

         archivos_guardar = W.TextField(label=_(u'Archivos a Guardar'))
         activo = W.CheckBox(label=_(u'Activo'), default=1, validator=V.Bool(if_empty=1))
     fields = Fields()
         archivos_guardar = W.TextField(label=_(u'Archivos a Guardar'))
         activo = W.CheckBox(label=_(u'Activo'), default=1, validator=V.Bool(if_empty=1))
     fields = Fields()


@@ -83,22 +83,20 @@ class ComandoPruebaController(controllers.Controller, identity.SecureResource):
         """Save or create record to model"""
         t = TareaPrueba.get(kw['tareaID'])
         orden = kw['orden']
         """Save or create record to model"""
         t = TareaPrueba.get(kw['tareaID'])
         orden = kw['orden']

-        del(kw['orden'])
-        del(kw['tareaID'])
-        if kw['archivos_entrada'].filename:
-            kw['archivos_entrada'] = kw['archivos_entrada'].file.read()
-        else:
-            kw['archivos_entrada'] =  None
-        if kw['archivos_a_comparar'].filename:
-            kw['archivos_a_comparar'] = kw['archivos_a_comparar'].file.read()
-        else:
-            kw['archivos_a_comparar'] =  None
+        del kw['orden']
+        del kw['tareaID']
+        if kw['los_archivos_entrada'].filename:
+            kw['archivos_entrada'] = kw['los_archivos_entrada'].file.read()
+        del kw['los_archivos_entrada']
+        if kw['los_archivos_a_comparar'].filename:
+            kw['archivos_a_comparar'] = kw['los_archivos_a_comparar'].file.read()
+        del kw['los_archivos_a_comparar']

         # TODO : Hacer ventanita mas amigable para cargar esto.
         try:
             kw['archivos_a_guardar'] = tuple(kw['archivos_guardar'].split(','))
         except AttributeError:
             pass
         # TODO : Hacer ventanita mas amigable para cargar esto.
         try:
             kw['archivos_a_guardar'] = tuple(kw['archivos_guardar'].split(','))
         except AttributeError:
             pass

-        del(kw['archivos_guardar'])
+        del kw['archivos_guardar']

         t.add_comando(orden, **kw)
         flash(_(u'Se creó un nuevo %s.') % name)
         raise redirect('list/%d' % t.id)
         t.add_comando(orden, **kw)
         flash(_(u'Se creó un nuevo %s.') % name)
         raise redirect('list/%d' % t.id)