Bugfix. Corregir un par de MultipleJoin y herencia incorrecta.

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

diff --git a/sercom/subcontrollers/grupo/__init__.py b/sercom/subcontrollers/grupo/__init__.py
index 532e8bfba2d1d7256f72d8d0cebf11e1e61b82d9..77e21c537575e4c274f7b54f84023cee17d5a594 100644 (file)
--- 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_new(data):
     validate_fk(data)
     return val.validate_new(cls, name, data)

+
+def validate_del(id):
+    return val.validate_del(cls, name, id)

 #}}}
 
 #{{{ Formulario
 #}}}
 
 #{{{ Formulario


@@ -120,6 +123,18 @@ ajax = u"""
         connect('form_responsable', 'onblur', buscar_alumno);
     }
 
         connect('form_responsable', 'onblur', buscar_alumno);
     }
 

+    function onsubmit()
+    {
+        /* TODO : Validar datos y evitar el submit si no esta completo */
+
+        /* 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;
+        }
+        return true; // Dejo hacer el submit
+    }
+

     MochiKit.DOM.addLoadEvent(prepare)
 
 """
     MochiKit.DOM.addLoadEvent(prepare)
 
 """


@@ -136,6 +151,7 @@ class GrupoForm(W.TableForm):
 
     fields = Fields()
     javascript = [W.JSSource("MochiKit.DOM.focusOnLoad('curso');"), W.JSSource(ajax)]
 
     fields = Fields()
     javascript = [W.JSSource("MochiKit.DOM.focusOnLoad('curso');"), W.JSSource(ajax)]

+    form_attrs = dict(onsubmit='return onsubmit()')

 
 form = GrupoForm()
 
 
 form = GrupoForm()
 


@@ -196,7 +212,22 @@ class GrupoController(controllers.Controller, identity.SecureResource):
     def edit(self, id, **kw):
         """Edit record in model"""
         r = validate_get(id)
     def edit(self, id, **kw):
         """Edit record in model"""
         r = validate_get(id)

-        return dict(name=name, namepl=namepl, record=r, form=form)
+        # TODO : No encontre mejor forma de pasar cosas al form
+        # de manera comoda y facil de formatear segun lo que espera la UI (que
+        # en este caso es super particular). Ese EmptyClass no se si hay algo estandar
+        # en python que usar, puse {} y [] pero cuando quiero hacer values.id = XX explota.
+        class EmptyClass:
+            pass
+        values = EmptyClass()
+        values.id = r.id
+        values.cursoID = r.cursoID
+        values.nombre = r.nombre
+        # TODO : Ver como llenar la lista primero :S
+        if r.responsable:
+            values.responsable = r.responsable.alumno.padron
+        values.miembros = [{"id":i.alumno.id, "label":i.alumno.alumno.nombre} for i in r.miembros]
+        values.tutores = [a.docenteID for a in r.tutores]
+        return dict(name=name, namepl=namepl, record=values, form=form)

 
     @validate(form=form)
     @error_handler(edit)
 
     @validate(form=form)
     @error_handler(edit)


@@ -205,20 +236,16 @@ class GrupoController(controllers.Controller, identity.SecureResource):
         """Save or create record to model"""
         responsable = kw['responsable']
         curso = kw['cursoID']
         """Save or create record to model"""
         responsable = kw['responsable']
         curso = kw['cursoID']

-        alumno = None
+        resp = kw['responsable']

         try:
             # Busco el alumno inscripto
         try:
             # Busco el alumno inscripto

-            alumno = AlumnoInscripto.select(AND(Curso.q.id==curso, Alumno.q.usuario==responsable))
-            if alumno.count() > 0:
-                alumno = alumno[0]
-            else:
-                raise Exception
-        except Exception, (inst):
-            flash(_(u'El responsable %s no existe') % responsable)
+            resp = AlumnoInscripto.selectBy(cursoID=kw['cursoID'],
+                alumno=Alumno.byPadron(kw['responsable'])).getOne()
+        except SQLObjectNotFound:
+            flash(_(u'El responsable %s no existe') % resp)

             raise redirect('../list')
             raise redirect('../list')

-
+        kw['responsable'] = resp

         r = validate_set(id, kw)
         r = validate_set(id, kw)

-        r.responsable = alumno

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


@@ -231,8 +258,7 @@ class GrupoController(controllers.Controller, identity.SecureResource):
     @expose()
     def delete(self, id):
         """Destroy record in model"""
     @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')
 
         flash(_(u'El %s fue eliminado permanentemente.') % name)
         raise redirect('../list')