Bugfix: faltó el validator :S

[software/sercom.git] / sercom / subcontrollers / tarea_prueba / comandos / __init__.py

diff --git a/sercom/subcontrollers/tarea_prueba/comandos/__init__.py b/sercom/subcontrollers/tarea_prueba/comandos/__init__.py
index 52f14d8395d5ccfa7d5cae34ba5f0066c10e5c3c..f8d6584f33ca930ab4a87acea7ff9ff460d2bc5d 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))
-        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()
@@ -83,22 +83,20 @@ class ComandoPruebaController(controllers.Controller, identity.SecureResource):
         """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
-        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)
@@ -115,6 +113,21 @@ class ComandoPruebaController(controllers.Controller, identity.SecureResource):
     @expose()
     def update(self, id, **kw):
         """Save or create record to model"""
+        orden = kw['orden']
+        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
+        del kw['archivos_guardar']
         r = validate_set(id, kw)
         flash(_(u'El %s fue actualizado.') % name)
         raise redirect('../list/%d' % r.tarea.id)
@@ -123,10 +136,6 @@ class ComandoPruebaController(controllers.Controller, identity.SecureResource):
     def show(self,id, **kw):
         """Show record in model"""
         r = validate_get(id)
-        if r.observaciones is None:
-            r.obs = ''
-        else:
-            r.obs = publish_parts(r.observaciones, writer_name='html')['html_body']
         return dict(name=name, namepl=namepl, record=r)
 
     @expose()
@@ -139,19 +148,17 @@ class ComandoPruebaController(controllers.Controller, identity.SecureResource):
         raise redirect('../list/%d' % tareaID)
 
     @expose()
-    def get_archivos_entrada(self, id):
+    def file(self, name, id):
         from cherrypy import request, response
         r = validate_get(id)
         response.headers["Content-Type"] = "application/zip"
-        response.headers["Content-disposition"] = "attachment;filename=archivos_entrada.zip"
-        return r.archivos_entrada
-
-    @expose()
-    def get_archivos_a_comparar(self, id):
-        from cherrypy import request, response
-        r = validate_get(id)
-        response.headers["Content-Type"] = "application/zip"
-        response.headers["Content-disposition"] = "attachment;filename=archivos_a_comparar.zip"
-        return r.archivos_a_comparar
+        response.headers["Content-disposition"] = "attachment;filename=%s_%d.zip" % (name, r.id)
+        if name == "archivos_entrada":
+            ret = r.archivos_entrada
+        elif name == "archivos_a_comparar":
+            ret = r.archivos_a_comparar
+        else:
+            raise NotFound
+        return ret
 #}}}