Hacer que formularios sean subclase de TableForm.

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

diff --git a/sercom/subcontrollers/enunciado/__init__.py b/sercom/subcontrollers/enunciado/__init__.py
index 55df337ee1c9f1aa925829262eefe870a4269a8f..31a4c3021ba48c2975d9f9bc4346e435421c05fb 100644 (file)
--- a/sercom/subcontrollers/enunciado/__init__.py
+++ b/sercom/subcontrollers/enunciado/__init__.py
@@ -1,14 +1,18 @@
-# vim: set et sw=4 sts=4 encoding=utf-8 :
+# vim: set et sw=4 sts=4 encoding=utf-8 foldmethod=marker :
 
+#{{{ Imports
 from turbogears import controllers, expose, redirect
-from turbogears import validate, validators, flash, error_handler
-from turbogears.widgets import *
+from turbogears import validate, flash, error_handler
+from turbogears import validators as V
+from turbogears import widgets as W
 from turbogears import identity
 from turbogears import paginate
 from docutils.core import publish_parts
 from sercom.subcontrollers import validate as val
 from sercom.model import Enunciado, Docente
+#}}}
 
+#{{{ Configuración
 cls = Enunciado
 name = 'enunciado'
 namepl = name + 's'
@@ -16,7 +20,9 @@ namepl = name + 's'
 fkcls = Docente
 fkname = 'autor'
 fknamepl = fkname + 'es'
+#}}}
 
+#{{{ Validación
 def validate_fk(data):
     fk = data.get(fkname + 'ID', None)
     if fk == 0: fk = None
@@ -24,9 +30,9 @@ def validate_fk(data):
         try:
             fk = fkcls.get(fk)
         except LookupError:
-            raise redirect('new', tg_flash=_(u'No se pudo crear el nuevo ' \
-                '%s porque el %s con identificador %d no existe.'
-                    % (name, fkname, fk)), **data)
+            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
@@ -41,20 +47,28 @@ def validate_set(id, data):
 def validate_new(data):
     validate_fk(data)
     return val.validate_new(cls, name, data)
+#}}}
 
+#{{{ Formulario
 def get_options():
     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)),
-    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)),
-])
-
+class EnunciadoForm(W.TableForm):
+    fields = [
+        W.TextField(name='nombre', label=_(u'Nombre'),
+            help_text=_(u'Requerido y único.'),
+            validator=V.UnicodeString(min=5, max=60, strip=True)),
+        W.SingleSelectField(name=fkname+'ID', label=_(fkname.capitalize()),
+            options=get_options, validator=V.Int(not_empty=False)),
+        W.TextField(name='descripcion', label=_(u'Descripción'),
+            validator=V.UnicodeString(not_empty=False, max=255, strip=True)),
+    ]
+    javascript = [W.JSSource("MochiKit.DOM.focusOnLoad('form_nombre');")]
+
+form = EnunciadoForm()
+#}}}
+
+#{{{ Controlador
 class EnunciadoController(controllers.Controller, identity.SecureResource):
     """Basic model admin interface"""
     require = identity.has_permission('admin')
@@ -69,21 +83,20 @@ class EnunciadoController(controllers.Controller, identity.SecureResource):
         raise redirect('list')
 
     @expose(template='kid:%s.templates.list' % __name__)
-    @validate(validators=dict(autorID=validators.Int))
+    @validate(validators=dict(autor=V.Int))
     @paginate('records')
-    def list(self, autorID=None, tg_flash=None):
+    def list(self, autor=None):
         """List records in model"""
-        if autorID is None:
+        if autor is None:
             r = cls.select()
         else:
-            r = cls.selectBy(autorID=autorID)
-        return dict(records=r, name=name, namepl=namepl, tg_flash=tg_flash)
+            r = cls.selectBy(autorID=autor)
+        return dict(records=r, name=name, namepl=namepl, parcial=autor)
 
     @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)
@@ -91,14 +104,14 @@ class EnunciadoController(controllers.Controller, identity.SecureResource):
     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)
-        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)
@@ -106,8 +119,8 @@ class EnunciadoController(controllers.Controller, identity.SecureResource):
     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):
@@ -124,6 +137,7 @@ class EnunciadoController(controllers.Controller, identity.SecureResource):
         """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')
+#}}}