Usar list comprehension en vez de filter().

[software/sercom.git] / sercom / subcontrollers / enunciado / __init__.py

diff --git a/sercom/subcontrollers/enunciado/__init__.py b/sercom/subcontrollers/enunciado/__init__.py
index 3809ef076660b210586b020fec20b34eb272f059..9ef523f7ca1b71b9b4cf98deaeee7dff9269ca63 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,11 @@ 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
+        kw['archivo'] = el_archivo.file.read()
+        kw['archivo_name'] = el_archivo.filename
+        kw['archivo_type'] = el_archivo.type
         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 +178,20 @@ 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(x, 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['archivo'] = el_archivo.file.read()
+            kw['archivo_name'] = el_archivo.filename
+            kw['archivo_type'] = el_archivo.type
         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'])