Agregar algunos detalles a la TODO.

[software/sercom.git] / sercom / model.py

diff --git a/sercom/model.py b/sercom/model.py
index b2ec70600179d7f36fc6e2eae6d52f33f34e99da..b2bc3a24870e750145edcc9e2cbc9bc569b70a8c 100644 (file)
--- a/sercom/model.py
+++ b/sercom/model.py
@@ -143,7 +143,7 @@ class Curso(SQLObject): #{{{
             for e in Ejercicio.selectBy(curso=self):
                 e.destroySelf()
             for (n, e) in enumerate(ejercicios):
-                self.add_ejercicio(n, e)
+                self.add_ejercicio(n+1, e)
         if alumnos is not None:
             for a in AlumnoInscripto.selectBy(curso=self):
                 a.destroySelf()
@@ -159,9 +159,10 @@ class Curso(SQLObject): #{{{
 
     def remove_docente(self, docente):
         if isinstance(docente, Docente):
-            DocenteInscripto.pk.get(curso=self, docente=docente).destroySelf()
-        else:
-            DocenteInscripto.pk.get(curso=self, docenteID=docente).destroySelf()
+            docente = docente.id
+        # FIXME esto deberian arreglarlo en SQLObject y debería ser
+        # DocenteInscripto.pk.get(self, docente).destroySelf()
+        DocenteInscripto.pk.get(self.id, docente).destroySelf()
 
     def add_alumno(self, alumno, **kw):
         if isinstance(alumno, Alumno):
@@ -172,15 +173,16 @@ class Curso(SQLObject): #{{{
 
     def remove_alumno(self, alumno):
         if isinstance(alumno, Alumno):
-            AlumnoInscripto.pk.get(curso=self, alumno=alumno).destroySelf()
-        else:
-            AlumnoInscripto.pk.get(curso=self, alumnoID=alumno).destroySelf()
+            alumno = alumno.id
+        # FIXME esto deberian arreglarlo en SQLObject
+        AlumnoInscripto.pk.get(self.id, alumno).destroySelf()
 
     def add_grupo(self, nombre, **kw):
         return Grupo(curso=self, nombre=unicode(nombre), **kw)
 
     def remove_grupo(self, nombre):
-        Grupo.pk.get(curso=self, nombre=nombre).destroySelf()
+        # FIXME esto deberian arreglarlo en SQLObject
+        Grupo.pk.get(self.id, nombre).destroySelf()
 
     def add_ejercicio(self, numero, enunciado, **kw):
         if isinstance(enunciado, Enunciado):
@@ -190,7 +192,8 @@ class Curso(SQLObject): #{{{
         return Ejercicio(curso=self, numero=numero, **kw)
 
     def remove_ejercicio(self, numero):
-        Ejercicio.pk.get(curso=self, numero=numero).destroySelf()
+        # FIXME esto deberian arreglarlo en SQLObject
+        Ejercicio.pk.get(self.id, numero).destroySelf()
 
     def __repr__(self):
         return 'Curso(id=%s, anio=%s, cuatrimestre=%s, numero=%s, ' \
@@ -565,8 +568,6 @@ CREATE TABLE instancia_tarea (
     # Joins
     entregas        = MultipleJoin('Entrega', joinColumn='instancia_id')
     correcciones    = MultipleJoin('Correccion', joinColumn='instancia_id')
-    casos_de_prueba = RelatedJoin('CasoDePrueba', # TODO CasoInstancia -> private
-                        addRemoveName='_caso_de_prueba')
 
     def __init__(self, tareas=(), **kw):
         super(InstanciaDeEntrega, self).__init__(**kw)