Muevo docente_inscripto dentro de Curso.

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

diff --git a/sercom/finalizer.py b/sercom/finalizer.py
index 0f681204701d4e385c8d0acf3963b97d985d3bc6..1d79a5a8542ea9ed5aa53c7a6dd23a38674a6c31 100644 (file)
--- a/sercom/finalizer.py
+++ b/sercom/finalizer.py
@@ -1,6 +1,6 @@
 # vim: set et sw=4 sts=4 encoding=utf-8 foldmethod=marker :
 
-from sercom.model import InstanciaDeEntrega
+from sercom.model import InstanciaDeEntrega, Entrega, SQLObjectNotFound
 from datetime import datetime
 import logging
 
@@ -53,14 +53,14 @@ def instancia_finalizar(self): #{{{
     docentes = [di.docente for di in curso.docentes if di.corrige]
     curr_docente = 0
     for ai in curso.alumnos:
-        mejor_entrega = None
-        for entrega in Entrega.selectBy(instancia=self, entregador=ai).orderBy(-Entrega.q.fecha):
-            if not mejor_entrega or not mejor_entrega.exito and entrega.exito:
-                mejor_entrega = entrega
-        if mejor_entrega:
-            mejor_entrega.make_correccion(docentes[curr_docente])
+        try:
+            e = Entrega.selectBy(instancia=self, entregador=ai) \
+                .orderBy(-Entrega.q.fecha) \
+                .limit(1) \
+                .getOne()
+            e.make_correccion(docentes[curr_docente])
             curr_docente = (curr_docente + 1) % len(docentes)
-        else:
+        except SQLObjectNotFound:
             log.info(_(u'El alumno inscripto %s no entregó', ai))
 
 InstanciaDeEntrega.finalizar = instancia_finalizar