Se agrega un chequeo para ver y advertir si un alumno tiene intentos pero ninguno...

diff --git a/src/sc_deliver b/src/sc_deliver
index 67ec78d838e637d68a5cbf8602f2913c53d270c6..3b7abc4b68487aeec0c97cdb659fe2f30ac801ce 100755 (executable)
--- a/src/sc_deliver
+++ b/src/sc_deliver
@@ -46,18 +46,27 @@ for entrega in Entrega.getPendientes(conn):
     for inscripto in Inscripto.selectBy(cursoID=entrega.cursoID, activo=True,
             connection=conn):
         log.debug('Procesando inscripto: %s', inscripto)
-        for intento in Intento.select(sercom.sqlo.AND(
-                Intento.q.inscriptoID == inscripto.id, Intento.q.entregaID == entrega.id),
-                orderBy=-Intento.q.numero, connection=conn):
+        intentos = list(Intento.select(
+            sercom.sqlo.AND(Intento.q.inscriptoID == inscripto.id,
+                Intento.q.entregaID == entrega.id),
+            orderBy=-Intento.q.numero, connection=conn))
+        no_entrega = True
+        for intento in intentos:
             log.debug('Procesando intento: %s', intento)
             # Si aprobó las pruebas públicas, está apto para corregir
             if intento.pruebasPublicasPasadas:
                 correccion = Correccion(entrega=entrega, inscripto=inscripto,
                     intento=intento, docente=1, connection=conn) #XXX Docente dummy
                 notificar(correccion)
+                no_entrega = False
                 log.info('Intento %d del alumno %d aceptado como entrega final.',
                     intento.numero, inscripto.padron)
                 break
+        # Si no tiene ningún intento 'entregable', es algo malo.
+        if intentos and no_entrega:
+            log.warn('El alumno con padrón %d tiene intentos para la ' \
+                'entrega %d.%d pero ninguno es aceptable', inscripto.padron,
+                entrega.nroEjercicio, entrega.entrega)
     log.debug('Entrega finalizada')
     entrega.finalizada = True