From 59972c6fd03ea093bd26eca762a7f4fa5a1320dd Mon Sep 17 00:00:00 2001 From: Leandro Lucarella Date: Sat, 10 Feb 2007 00:36:26 +0000 Subject: [PATCH 1/1] =?utf8?q?Reutilizar=20validaci=C3=B3n=20de=20ID=20y?= =?utf8?q?=20existencia=20de=20objetos.=20Se=20mueve=20la=20parte=20de=20v?= =?utf8?q?alidaci=C3=B3n=20de=20id=20y=20de=20existencia=20de=20SQLObjects?= =?utf8?q?=20al=20m=C3=B3dulo=20validate=20de=20subcontrollers.=20De=20est?= =?utf8?q?a=20forma=20se=20puede=20reutilizar=20esta=20validaci=C3=B3n=20e?= =?utf8?q?n=20otros=20controladores.=20En=20el=20controlador=20de=20docent?= =?utf8?q?es=20se=20'bindean'=20las=20funciones=20con=20los=20par=C3=A1met?= =?utf8?q?ros=20de=20clase=20y=20nombre=20para=20que=20sea=20m=C3=A1s=20si?= =?utf8?q?mple=20el=20uso=20(tipo=20aplicaci=C3=B3n=20parcial=20:).?= MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit --- sercom/subcontrollers/docente/__init__.py | 101 ++++------------------ sercom/subcontrollers/validate.py | 32 +++++++ 2 files changed, 50 insertions(+), 83 deletions(-) create mode 100644 sercom/subcontrollers/validate.py diff --git a/sercom/subcontrollers/docente/__init__.py b/sercom/subcontrollers/docente/__init__.py index 721ec34..c93184d 100644 --- a/sercom/subcontrollers/docente/__init__.py +++ b/sercom/subcontrollers/docente/__init__.py @@ -7,14 +7,19 @@ 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 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.'), @@ -57,26 +62,16 @@ class DocenteController(controllers.Controller, identity.SecureResource): """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) @@ -90,97 +85,37 @@ 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) 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) 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 = validate_get(id) 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) diff --git a/sercom/subcontrollers/validate.py b/sercom/subcontrollers/validate.py new file mode 100644 index 0000000..67b749a --- /dev/null +++ b/sercom/subcontrollers/validate.py @@ -0,0 +1,32 @@ +# vim: set et sw=4 sts=4 encoding=utf-8 : + +__all__ = ('validate_get', 'validate_set', 'validate_new') + +from turbogears import redirect + +def validate_get(cls, name, id, url='../list'): + try: + id = int(id) + except ValueError: + raise redirect(url, tg_flash=_(u'Identificador inválido: %s.') % id) + try: + return cls.get(id) + except LookupError: + raise redirect(url, tg_flash=_(u'No existe %s con identificador %d') + % (name, id)) + +def validate_set(cls, name, id, data, url='../edit'): + r = validate_get(cls, name, id) + try: + r.set(**data) + except Exception, e: + raise redirect('%s/%s' % (url, id), tg_flash=_(u'No se pudo ' \ + 'modificar el %s (error: %s).') % (name, e), **data) + +def validate_new(cls, name, data, url='new'): + try: + return cls(**data) + except Exception, e: + raise redirect(url, tg_flash=_(u'No se pudo crear el nuevo %s ' \ + '(error: %s).') % (name, e), **data) + -- 2.43.0