fix de eliminar elementos

[z.facultad/75.52/sercom.git] / sercom / model.py

diff --git a/sercom/model.py b/sercom/model.py
index bc7388c0edfbd2a5ff2393e0f38c96ab7b47fb0c..d1baac623f8399c856c4a76eef40dc6ff148ba77 100644 (file)
--- a/sercom/model.py
+++ b/sercom/model.py
@@ -159,9 +159,9 @@ class Curso(SQLObject): #{{{
 
     def remove_docente(self, docente):
         if isinstance(docente, Docente):
-            DocenteInscripto.pk.get(curso=self, docente=docente).destroySelf()
+            DocenteInscripto.selectBy(curso=self, docente=docente).getOne().destroySelf()
         else:
-            DocenteInscripto.pk.get(curso=self, docenteID=docente).destroySelf()
+            DocenteInscripto.selectBy(curso=self, docenteID=docente).getOne().destroySelf()
 
     def add_alumno(self, alumno, **kw):
         if isinstance(alumno, Alumno):
@@ -172,9 +172,9 @@ class Curso(SQLObject): #{{{
 
     def remove_alumno(self, alumno):
         if isinstance(alumno, Alumno):
-            AlumnoInscripto.pk.get(curso=self, alumno=alumno).destroySelf()
+            AlumnoInscripto.selectBy(curso=self, alumno=alumno).getOne().destroySelf()
         else:
-            AlumnoInscripto.pk.get(curso=self, alumnoID=alumno).destroySelf()
+            AlumnoInscripto.selectBy(curso=self, alumnoID=alumno).getOne().destroySelf()
 
     def add_grupo(self, nombre, **kw):
         return Grupo(curso=self, nombre=unicode(nombre), **kw)
@@ -703,27 +703,27 @@ class Grupo(Entregador): #{{{
                 self.add_tutor(t)
 
     def add_miembro(self, alumno, **kw):
-        if isinstance(alumno, Alumno):
+        if isinstance(alumno, AlumnoInscripto):
             kw['alumno'] = alumno
         else:
             kw['alumnoID'] = alumno
         return Miembro(grupo=self, **kw)
 
     def remove_miembro(self, alumno):
-        if isinstance(alumno, Alumno):
+        if isinstance(alumno, AlumnoInscripto):
             Miembro.pk.get(grupo=self, alumno=alumno).destroySelf()
         else:
             Miembro.pk.get(grupo=self, alumnoID=alumno).destroySelf()
 
     def add_tutor(self, docente, **kw):
-        if isinstance(docente, Docente):
+        if isinstance(docente, DocenteInscripto):
             kw['docente'] = docente
         else:
             kw['docenteID'] = docente
         return Tutor(grupo=self, **kw)
 
     def remove_tutor(self, docente):
-        if isinstance(docente, Alumno):
+        if isinstance(docente, DocenteInscripto):
             Tutor.pk.get(grupo=self, docente=docente).destroySelf()
         else:
             Tutor.pk.get(grupo=self, docenteID=docente).destroySelf()
@@ -859,6 +859,9 @@ class Correccion(SQLObject): #{{{
     nota            = DecimalCol(size=3, precision=1, default=None)
     observaciones   = UnicodeCol(default=None)
 
+    def _get_entregas(self):
+        return list(Entrega.selectBy(instancia=self.instancia, entregador=self.entregador))
+
     def __repr__(self):
         return 'Correccion(instancia=%s, entregador=%s, entrega=%s, ' \
             'corrector=%s, asignado=%s, corregido=%s, nota=%s, ' \
@@ -868,6 +871,8 @@ class Correccion(SQLObject): #{{{
                     self.corregido, self.nota, self.observaciones)
 
     def shortrepr(self):
+        if not self.corrector:
+            return '%s' % self.entrega.shortrepr()
         return '%s,%s' % (self.entrega.shortrepr(), self.corrector.shortrepr())
 #}}}