Usar bien mensajes flash().

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

diff --git a/sercom/subcontrollers/enunciado/__init__.py b/sercom/subcontrollers/enunciado/__init__.py
index 5016aadd6e4751e28a4b395cc19c612165f9de1f..d953ba21c914823949b68217f5651b9289030cc4 100644 (file)
--- a/sercom/subcontrollers/enunciado/__init__.py
+++ b/sercom/subcontrollers/enunciado/__init__.py
@@ -13,38 +13,43 @@ cls = Enunciado
 name = 'enunciado'
 namepl = name + 's'
 
 name = 'enunciado'
 namepl = name + 's'
 

-def validate_autor(data):
-    autor = data.get('autorID', None)
-    if autor == 0: autor = None
-    if autor is not None:
+fkcls = Docente
+fkname = 'autor'
+fknamepl = fkname + 'es'
+
+def validate_fk(data):
+    fk = data.get(fkname + 'ID', None)
+    if fk == 0: fk = None
+    if fk is not None:

         try:
         try:

-            autor = Docente.get(autor)
+            fk = fkcls.get(fk)

         except LookupError:
         except LookupError:

-            raise redirect('new', tg_flash=_(u'No se pudo crear el nuevo ' \
-                '%s porque el autor con identificador %d no existe.'
-                    % (name, autor)), **data)
-    data.pop('autorID', None)
-    data['autor'] = autor
+            flash(_(u'No se pudo crear el nuevo %s porque el %s con '
+                'identificador %d no existe.' % (name, fkname, fk)))
+            raise redirect('new', **data)
+    data.pop(fkname + 'ID', None)
+    data[fkname] = fk
+    return fk

 
 def validate_get(id):
     return val.validate_get(cls, name, id)
 
 def validate_set(id, data):
 
 def validate_get(id):
     return val.validate_get(cls, name, id)
 
 def validate_set(id, data):

-    validate_autor(data)
+    validate_fk(data)

     return val.validate_set(cls, name, id, data)
 
 def validate_new(data):
     return val.validate_set(cls, name, id, data)
 
 def validate_new(data):

-    validate_autor(data)
+    validate_fk(data)

     return val.validate_new(cls, name, data)
 
 def get_options():
     return val.validate_new(cls, name, data)
 
 def get_options():

-    return [(0, _(u'--'))] + [(a.id, a.shortrepr()) for a in Docente.select()]
+    return [(0, _(u'--'))] + [(fk.id, fk.shortrepr()) for fk in fkcls.select()]

 
 form = TableForm(fields=[
     TextField(name='nombre', label=_(u'Nombre'),
         help_text=_(u'Requerido y único.'),
         validator=validators.UnicodeString(min=5, max=60, strip=True)),
 
 form = TableForm(fields=[
     TextField(name='nombre', label=_(u'Nombre'),
         help_text=_(u'Requerido y único.'),
         validator=validators.UnicodeString(min=5, max=60, strip=True)),

-    SingleSelectField(name='autorID', label=_(u'Autor'),
+    SingleSelectField(name=fkname+'ID', label=_(fkname.capitalize()),

         options=get_options, validator=validators.Int(not_empty=False)),
     TextField(name='descripcion', label=_(u'Descripción'),
         validator=validators.UnicodeString(not_empty=False, max=255, strip=True)),
         options=get_options, validator=validators.Int(not_empty=False)),
     TextField(name='descripcion', label=_(u'Descripción'),
         validator=validators.UnicodeString(not_empty=False, max=255, strip=True)),


@@ -64,18 +69,20 @@ class EnunciadoController(controllers.Controller, identity.SecureResource):
         raise redirect('list')
 
     @expose(template='kid:%s.templates.list' % __name__)
         raise redirect('list')
 
     @expose(template='kid:%s.templates.list' % __name__)

+    @validate(validators=dict(autor=validators.Int))

     @paginate('records')
     @paginate('records')

-    def list(self, **kw):
+    def list(self, autor=None):

         """List records in model"""
         """List records in model"""

-        f = kw.get('tg_flash', None)
-        r = cls.select()
-        return dict(records=r, name=name, namepl=namepl, tg_flash=f)
+        if autor is None:
+            r = cls.select()
+        else:
+            r = cls.selectBy(autorID=autor)
+        return dict(records=r, name=name, namepl=namepl)

 
     @expose(template='kid:%s.templates.new' % __name__)
     def new(self, **kw):
         """Create new records in model"""
 
     @expose(template='kid:%s.templates.new' % __name__)
     def new(self, **kw):
         """Create new records in model"""

-        f = kw.get('tg_flash', None)
-        return dict(name=name, namepl=namepl, form=form, tg_flash=f, values=kw)
+        return dict(name=name, namepl=namepl, form=form, values=kw)

 
     @validate(form=form)
     @error_handler(new)
 
     @validate(form=form)
     @error_handler(new)


@@ -83,14 +90,14 @@ class EnunciadoController(controllers.Controller, identity.SecureResource):
     def create(self, **kw):
         """Save or create record to model"""
         validate_new(kw)
     def create(self, **kw):
         """Save or create record to model"""
         validate_new(kw)

-        raise redirect('list', tg_flash=_(u'Se creó un nuevo %s.') % name)
+        flash(_(u'Se creó un nuevo %s.') % name)
+        raise redirect('list')

 
     @expose(template='kid:%s.templates.edit' % __name__)
     def edit(self, id, **kw):
         """Edit record in model"""
         r = validate_get(id)
 
     @expose(template='kid:%s.templates.edit' % __name__)
     def edit(self, id, **kw):
         """Edit record in model"""
         r = validate_get(id)

-        return dict(name=name, namepl=namepl, record=r, form=form,
-            tg_flash=kw.get('tg_flash', None))
+        return dict(name=name, namepl=namepl, record=r, form=form)

 
     @validate(form=form)
     @error_handler(edit)
 
     @validate(form=form)
     @error_handler(edit)


@@ -98,8 +105,8 @@ class EnunciadoController(controllers.Controller, identity.SecureResource):
     def update(self, id, **kw):
         """Save or create record to model"""
         r = validate_set(id, kw)
     def update(self, id, **kw):
         """Save or create record to model"""
         r = validate_set(id, kw)

-        raise redirect('../list',
-            tg_flash=_(u'El %s fue actualizado.') % name)
+        flash(_(u'El %s fue actualizado.') % name)
+        raise redirect('../list')

 
     @expose(template='kid:%s.templates.show' % __name__)
     def show(self,id, **kw):
 
     @expose(template='kid:%s.templates.show' % __name__)
     def show(self,id, **kw):


@@ -116,6 +123,6 @@ class EnunciadoController(controllers.Controller, identity.SecureResource):
         """Destroy record in model"""
         r = validate_get(id)
         r.destroySelf()
         """Destroy record in model"""
         r = validate_get(id)
         r.destroySelf()

-        raise redirect('../list',
-            tg_flash=_(u'El %s fue eliminado permanentemente.') % name)
+        flash(_(u'El %s fue eliminado permanentemente.') % name)
+        raise redirect('../list')