X-Git-Url: https://git.llucax.com/software/sercom.git/blobdiff_plain/c319838b39466b11a0cf2e7f2e31d8fe23bb6352..5b8a71a4d21a5ee35694718804dda7820f13f768:/sercom/subcontrollers/docente/__init__.py diff --git a/sercom/subcontrollers/docente/__init__.py b/sercom/subcontrollers/docente/__init__.py index 721ec34..d7d58bc 100644 --- a/sercom/subcontrollers/docente/__init__.py +++ b/sercom/subcontrollers/docente/__init__.py @@ -2,19 +2,24 @@ from turbogears import controllers, expose, redirect from turbogears import validate, validators, flash, error_handler -from sercom.model import Docente from turbogears.widgets import * 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 Docente cls = Docente name = 'docente' namepl = name + 's' +def validate_get(id): return val.validate_get(cls, name, id) +def validate_set(id, data): return val.validate_set(cls, name, id, data) +def validate_new(data): return val.validate_new(cls, name, data) + form = TableForm(fields=[ TextField(name='usuario', label=_(u'Usuario'), - help_text=_(u'Requerido.'), + help_text=_(u'Requerido y único.'), validator=validators.UnicodeString(min=3, max=10, strip=True)), TextField(name='nombre', label=_(u'Nombre'), help_text=_(u'Requerido.'), @@ -51,35 +56,25 @@ class DocenteController(controllers.Controller, identity.SecureResource): def index(self): raise redirect('list') - @expose(template='kid:sercom.subcontrollers.%s.templates.list' % name) + @expose(template='kid:%s.templates.list' % __name__) @paginate('records') def list(self, **kw): """List records in model""" f = kw.get('tg_flash', None) r = cls.select() - return dict(records=r, name=name, namepl=namepl, tg_flash=f) @expose() def activate(self, id, activo): """Save or create record to model""" + r = validate_get(id) try: - id = int(id) - except ValueError: - raise redirect('../list', - tg_flash=_(u'Identificador inválido: %s.') % id) - - try: - r = cls.get(id) - except LookupError: - f = _(u'No existe el %s con identificador %d.') % (name, id) - raise redirect('../list', tg_flash=f) - - r.activo = bool(int(activo)) - + r.activo = bool(int(activo)) + except ValueError, e: + raise redirect('../list', tg_flash=_(u'Acción inválida.')) raise redirect('../list') - @expose(template='kid:sercom.subcontrollers.%s.templates.new' % name) + @expose(template='kid:%s.templates.new' % __name__) def new(self, **kw): """Create new records in model""" f = kw.get('tg_flash', None) @@ -90,97 +85,40 @@ class DocenteController(controllers.Controller, identity.SecureResource): @expose() def create(self, **kw): """Save or create record to model""" - try: - cls(**kw) - except Exception, e: - raise redirect('new', tg_flash=_(u'No se pudo crear el nuevo %s, ' \ - 'probablemente ya existe uno con el mismo usuario (error: %s).' - % (name, e)), **kw) - + validate_new(kw) raise redirect('list', tg_flash=_(u'Se creó un nuevo %s.') % name) - @expose(template='kid:sercom.subcontrollers.%s.templates.edit' % name) + @expose(template='kid:%s.templates.edit' % __name__) def edit(self, id, **kw): """Edit record in model""" - try: - id = int(id) - except ValueError: - raise redirect('../list', - tg_flash=_(u'Identificador inválido: %s.') % id) - - f = kw.get('tg_flash', None) - try: - r = cls.get(id) - except LookupError: - f = _(u'No existe el %s con identificador %d.') % (name, id) - raise redirect('../list', tg_flash=f) - - return dict(name=name, namepl=namepl, record=r, form=form, tg_flash=f) + r = validate_get(id) + return dict(name=name, namepl=namepl, record=r, form=form, + tg_flash=kw.get('tg_flash', None)) @validate(form=form) @error_handler(edit) @expose() def update(self, id, **kw): """Save or create record to model""" - try: - id = int(id) - except ValueError: - raise redirect('../list', - tg_flash=_(u'Identificador inválido: %s.') % id) - - try: - record = cls.get(id) - except LookupError: - raise redirect('../list', - tg_flash=_(u'No existe el %s con identificador %d.') - % (name, id)) - - try: - record.set(**kw) - except Exception, e: - raise redirect('../edit/%d' % id, tg_flash=_(u'No se pudo ' \ - 'modificar el %s, probablemente ya existe uno con el mismo ' \ - 'usuario (error: %s).' % (name, e)), **kw) - + r = validate_set(id, kw) raise redirect('../list', tg_flash=_(u'El %s fue actualizado.') % name) - @expose(template='kid:sercom.subcontrollers.%s.templates.show' % name) + @expose(template='kid:%s.templates.show' % __name__) def show(self,id, **kw): """Show record in model""" - try: - id = int(id) - except ValueError: - raise redirect('../list', - tg_flash=_(u'Identificador inválido: %s.') % id) - - try: - r = cls.get(id) - except LookupError: - raise redirect('../list', - tg_flash=_(u'No existe el %s con identificador %d.') - % (name, id)) - - r.obs = publish_parts(r.observaciones, writer_name='html')['html_body'] - + 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() def delete(self, id): """Destroy record in model""" - try: - id = int(id) - except ValueError: - raise redirect('../list', - tg_flash=_(u'Identificador inválido: %s.') % id) - - try: - cls.delete(id) - except LookupError: - raise redirect('../list', - tg_flash=_(u'No existe el %s con identificador %d.') - % (name, id)) - + r = validate_get(id) + r.destroySelf() raise redirect('../list', tg_flash=_(u'El %s fue eliminado permanentemente.') % name)