X-Git-Url: https://git.llucax.com/software/sercom.git/blobdiff_plain/26995a1dc8037021ee106a884e4311cdf78cabfd..6ce4d4e8ad92583bb1187385f30f52ade8beee9e:/sercom/subcontrollers/grupo/__init__.py

diff --git a/sercom/subcontrollers/grupo/__init__.py b/sercom/subcontrollers/grupo/__init__.py
index 532e8bf..28abe65 100644
--- a/sercom/subcontrollers/grupo/__init__.py
+++ b/sercom/subcontrollers/grupo/__init__.py
@@ -196,7 +196,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 = [1]
+        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)
@@ -205,20 +220,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)
-            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.responsable = alumno
         flash(_(u'El %s fue actualizado.') % name)
         raise redirect('../list')