/me se recurda probas las cosas antes de recordear+sendear

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

diff --git a/sercom/subcontrollers/grupo/__init__.py b/sercom/subcontrollers/grupo/__init__.py
index 532e8bfba2d1d7256f72d8d0cebf11e1e61b82d9..6be17575a0d7b2e2b7869f8bdb21bc98d0808e8a 100644 (file)
--- a/sercom/subcontrollers/grupo/__init__.py
+++ b/sercom/subcontrollers/grupo/__init__.py
@@ -120,6 +120,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 +148,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 +209,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 +233,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)
-            raise redirect('../list')
-
+            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 = 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')