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 2ba95e15e93f1c4e173fe0ade1901dee888d0f21..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,20 +123,35 @@ 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)
 
 """
+def get_docentes():
+    return [(fk1.id, fk1.shortrepr()) for fk1 in Docente.select()]
 
 class GrupoForm(W.TableForm):
     class Fields(W.WidgetsList):
-        curso = W.SingleSelectField(name='cursoID', label=_(u'Curso'), options = get_cursos,
-        validator = V.Int(not_empty=True))
+        cursoID = W.SingleSelectField(label=_(u'Curso'), options = get_cursos, validator = V.Int(not_empty=True))
         nombre = W.TextField(label=_(u'Nombre'), validator=V.UnicodeString(not_empty=True,strip=True))
         responsable = CustomTextField(label=_(u'Responsable'), validator=V.UnicodeString(not_empty=True), attrs=dict(size='8'))
-        alumnos = AjaxMultiSelect(name='alumnos', label=_(u'Integrantes'), validator=V.Int(), on_add="alumnos_agregar_a_la_lista")
+        miembros = AjaxMultiSelect(label=_(u'Miembros'), validator=V.Int(), on_add="alumnos_agregar_a_la_lista")
+        tutores = W.MultipleSelectField(label=_(u'Tutores'), validator=V.Int(), options=get_docentes)
 
     fields = Fields()
     javascript = [W.JSSource("MochiKit.DOM.focusOnLoad('curso');"), W.JSSource(ajax)]
+    form_attrs = dict(onsubmit='return onsubmit()')
 
 form = GrupoForm()
 
@@ -176,35 +194,17 @@ class GrupoController(controllers.Controller, identity.SecureResource):
     @expose()
     def create(self, **kw):
         """Save or create record to model"""
-        responsable = kw['responsable']
-        curso = kw['cursoID']
-        resp = None
+        resp = kw['responsable']
         try:
             # Busco el alumno inscripto
-            resp = AlumnoInscripto.select(AND(Curso.q.id==curso, Alumno.q.usuario==responsable))
-            if resp.count() > 0:
-                resp = resp[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
 
-        # Busco los alumnos
-        alumnos = []
-        for alumnoid in kw['alumnos']:
-            alumnos.append(Alumno.get(alumnoid))
-        if alumnos == []:
-            flash(_(u'No se pudo crear el grupo. No se han agregado integrantes.'))
-            raise redirect('list')
-
-        del(kw['alumnos'])
-
         r = validate_new(kw)
-        for a in alumnos:
-            r.add_miembro(a)
         flash(_(u'Se creó un nuevo %s.') % name)
         raise redirect('list')
 
@@ -212,7 +212,22 @@ class GrupoController(controllers.Controller, identity.SecureResource):
     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)
@@ -221,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')
 
@@ -247,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')