Bugfix: Curso() ponía mal el número de ejercicio.

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

diff --git a/sercom/subcontrollers/grupo/__init__.py b/sercom/subcontrollers/grupo/__init__.py
index 3f092a2c26974ec6e27746361ee851307e61acc2..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_del(id):
+    return val.validate_del(cls, name, id)
 #}}}
 
 #{{{ Formulario
@@ -120,6 +123,18 @@ ajax = u"""
         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)
 
 """
@@ -136,6 +151,7 @@ class GrupoForm(W.TableForm):
 
     fields = Fields()
     javascript = [W.JSSource("MochiKit.DOM.focusOnLoad('curso');"), W.JSSource(ajax)]
+    form_attrs = dict(onsubmit='return onsubmit()')
 
 form = GrupoForm()
 
@@ -209,7 +225,7 @@ class GrupoController(controllers.Controller, identity.SecureResource):
         # TODO : Ver como llenar la lista primero :S
         if r.responsable:
             values.responsable = r.responsable.alumno.padron
-        values.miembros = [1]
+        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)
 
@@ -220,20 +236,16 @@ class GrupoController(controllers.Controller, identity.SecureResource):
         """Save or create record to model"""
         responsable = kw['responsable']
         curso = kw['cursoID']
-        alumno = None
+        resp = kw['responsable']
         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')
-
+        kw['responsable'] = resp
         r = validate_set(id, kw)
-        r.responsable = alumno
         flash(_(u'El %s fue actualizado.') % name)
         raise redirect('../list')
 
@@ -246,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')