Bugfix. Elimina referencia a la clase en el controlador de Docente para que sea más...

[z.facultad/75.52/sercom.git] / sercom / subcontrollers / docente / __init__.py

diff --git a/sercom/subcontrollers/docente/__init__.py b/sercom/subcontrollers/docente/__init__.py
index e4070810738cdee9ed701c8e21a6285edf40c2ea..721ec34613d8ab90fc2c0886478582d0a6f26d35 100644 (file)
--- a/sercom/subcontrollers/docente/__init__.py
+++ b/sercom/subcontrollers/docente/__init__.py
@@ -63,21 +63,39 @@ class DocenteController(controllers.Controller, identity.SecureResource):
     @expose()
     def activate(self, id, activo):
         """Save or create record to model"""
     @expose()
     def activate(self, id, activo):
         """Save or create record to model"""

-        cls.get(int(id)).activo = int(activo)
+        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))

 
         raise redirect('../list')
 
     @expose(template='kid:sercom.subcontrollers.%s.templates.new' % name)
     def new(self, **kw):
         """Create new records in model"""
 
         raise redirect('../list')
 
     @expose(template='kid:sercom.subcontrollers.%s.templates.new' % name)
     def new(self, **kw):
         """Create new records in model"""

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

 
     @validate(form=form)
     @error_handler(new)
     @expose()
     def create(self, **kw):
         """Save or create record to model"""
 
     @validate(form=form)
     @error_handler(new)
     @expose()
     def create(self, **kw):
         """Save or create record to model"""

-        Docente(**kw)
+        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)

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


@@ -85,19 +103,44 @@ class DocenteController(controllers.Controller, identity.SecureResource):
     def edit(self, id, **kw):
         """Edit record in model"""
         try:
     def edit(self, id, **kw):
         """Edit record in model"""
         try:

-            r = cls.get(int(id))
+            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:
         except LookupError:

-            flash = _('No existe el docente con identificador %d.') % id
+            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)
+        return dict(name=name, namepl=namepl, record=r, form=form, tg_flash=f)

 
     @validate(form=form)
     @error_handler(edit)
     @expose()
     def update(self, id, **kw):
         """Save or create record to model"""
 
     @validate(form=form)
     @error_handler(edit)
     @expose()
     def update(self, id, **kw):
         """Save or create record to model"""

-        record = cls.get(int(id))
-        record.set(**kw)
+        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)

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


@@ -105,7 +148,19 @@ class DocenteController(controllers.Controller, identity.SecureResource):
     @expose(template='kid:sercom.subcontrollers.%s.templates.show' % name)
     def show(self,id, **kw):
         """Show record in model"""
     @expose(template='kid:sercom.subcontrollers.%s.templates.show' % name)
     def show(self,id, **kw):
         """Show record in model"""

-        r = cls.get(int(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:
+            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']
 
         return dict(name=name, namepl=namepl, record=r)
         r.obs = publish_parts(r.observaciones, writer_name='html')['html_body']
 
         return dict(name=name, namepl=namepl, record=r)


@@ -113,7 +168,18 @@ class DocenteController(controllers.Controller, identity.SecureResource):
     @expose()
     def delete(self, id):
         """Destroy record in model"""
     @expose()
     def delete(self, id):
         """Destroy record in model"""

-        cls.delete(int(id))
+        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))

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