From 3e70bed38777dfe39bffc9fd1a622fc66301ef40 Mon Sep 17 00:00:00 2001 From: Leandro Lucarella Date: Sun, 6 Mar 2005 00:42:38 +0000 Subject: [PATCH] Se agrega un chequeo para ver y advertir si un alumno tiene intentos pero ninguno aceptable para corregir. --- src/sc_deliver | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/src/sc_deliver b/src/sc_deliver index 67ec78d..3b7abc4 100755 --- 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 -- 2.43.0