From: tailor <ndimov@gmail.com>
Date: Tue, 6 Mar 2007 00:02:54 +0000 (+0000)
Subject: excepciones en la eliminacion
X-Git-Tag: 0_9~122
X-Git-Url: https://git.llucax.com/software/sercom.git/commitdiff_plain/7905ce201dac1dcb582645501ec44b08f9f25a1b?ds=sidebyside

excepciones en la eliminacion

** atajo las excepciones de las eliminaciones y mando mensaje de error (poco
feliz)
---

diff --git a/sercom/subcontrollers/caso_de_prueba/__init__.py b/sercom/subcontrollers/caso_de_prueba/__init__.py
index 6fca789..b3dc041 100644
--- a/sercom/subcontrollers/caso_de_prueba/__init__.py
+++ b/sercom/subcontrollers/caso_de_prueba/__init__.py
@@ -48,6 +48,9 @@ def validate_set(id, data):
 def validate_new(data):
     validate_fk(data)
     return val.validate_new(cls, name, data)
+
+def validate_del(id):
+    return val.validate_del(cls, name, id)
 #}}}
 
 #{{{ Formulario
@@ -150,8 +153,7 @@ class CasoDePruebaController(controllers.Controller, identity.SecureResource):
     @expose()
     def delete(self, id):
         """Destroy record in model"""
-        r = validate_get(id)
-        r.destroySelf()
+        validate_del(id)
         flash(_(u'El %s fue eliminado permanentemente.') % name)
         raise redirect('../list')
 #}}}
diff --git a/sercom/subcontrollers/curso/__init__.py b/sercom/subcontrollers/curso/__init__.py
index c17b3cf..9d1c1f3 100644
--- a/sercom/subcontrollers/curso/__init__.py
+++ b/sercom/subcontrollers/curso/__init__.py
@@ -97,6 +97,9 @@ def validate_set(id, data):
 
 def validate_new(data):
     return val.validate_new(cls, name, data)
+
+def validate_del(id):
+    return val.validate_del(cls, name, id)
 #}}}
 
 def get_ejercicios():
@@ -111,6 +114,7 @@ def get_alumnos():
 def get_grupos():
     return [(fk1.id, fk1.shortrepr()) for fk1 in Grupo.select()]
 
+
 #{{{ Formulario
 class CursoForm(W.TableForm):
     class Fields(W.WidgetsList):
@@ -284,12 +288,7 @@ class CursoController(controllers.Controller, identity.SecureResource):
     @expose()
     def delete(self, id):
         """Destroy record in model"""
-        try:
-            r = validate_get(id)
-            r.destroySelf()
-        except Exception, e:
-            flash(_(u'No se pudo eliminar el curso: %s' % e))
-            raise redirect('../list')
+        validate_del(id)
         flash(_(u'El %s fue eliminado permanentemente.') % name)
         raise redirect('../list')
 
diff --git a/sercom/subcontrollers/ejercicio/__init__.py b/sercom/subcontrollers/ejercicio/__init__.py
index d04dde1..cacbad5 100644
--- a/sercom/subcontrollers/ejercicio/__init__.py
+++ b/sercom/subcontrollers/ejercicio/__init__.py
@@ -71,6 +71,9 @@ def validate_new(data):
     validate_fk(data)
     validate_fk1(data)
     return val.validate_new(cls, name, data)
+
+def validate_del(id):
+    return val.validate_del(cls, name, id)
 #}}}
 
 #{{{ Formulario
@@ -221,8 +224,7 @@ class EjercicioController(controllers.Controller, identity.SecureResource):
     @expose()
     def delete(self, id):
         """Destroy record in model"""
-        r = validate_get(id)
-        r.destroySelf()
+        validate_del(id)
         flash(_(u'El %s fue eliminado permanentemente.') % name)
         raise redirect('../list')
 
diff --git a/sercom/subcontrollers/grupo/__init__.py b/sercom/subcontrollers/grupo/__init__.py
index e6e3e49..77e21c5 100644
--- a/sercom/subcontrollers/grupo/__init__.py
+++ b/sercom/subcontrollers/grupo/__init__.py
@@ -52,6 +52,9 @@ def validate_set(id, data):
 def validate_new(data):
     validate_fk(data)
     return val.validate_new(cls, name, data)
+
+def validate_del(id):
+    return val.validate_del(cls, name, id)
 #}}}
 
 #{{{ Formulario
@@ -126,8 +129,8 @@ ajax = u"""
 
         /* Selecciono todos los miembros si no, no llegan al controllere*/
         l = MochiKit.DOM.getElement('form_miembros');
-        for (i=0; i<l.options.length; i++) { 
-            l.options[i].selected = true; 
+        for (i=0; i<l.options.length; i++) {
+            l.options[i].selected = true;
         }
         return true; // Dejo hacer el submit
     }
@@ -255,8 +258,7 @@ class GrupoController(controllers.Controller, identity.SecureResource):
     @expose()
     def delete(self, id):
         """Destroy record in model"""
-        r = validate_get(id)
-        r.destroySelf()
+        validate_del(id)
         flash(_(u'El %s fue eliminado permanentemente.') % name)
         raise redirect('../list')
 
diff --git a/sercom/subcontrollers/validate.py b/sercom/subcontrollers/validate.py
index 524f311..9b4a96a 100644
--- a/sercom/subcontrollers/validate.py
+++ b/sercom/subcontrollers/validate.py
@@ -43,3 +43,11 @@ def validate_new(cls, name, data, url='new'):
             u'inválido (error: %s).') % (name, e))
         raise redirect(url, **data)
 
+def validate_del(cls, name, id):
+    try:
+        id = int(id)
+        r = validate_get(cls, name, id)
+        r.destroySelf()
+    except Exception, e:
+        flash(_(u'No se pudo eliminar el %s: %s' % (name, e)))
+        raise redirect('../list')