Bugfix (se corrige bien el bug del from).

[software/sercom-old.git] / src / sc_suwi

diff --git a/src/sc_suwi b/src/sc_suwi
index caf000ecb34b0a35484f2f3399eab2b6ca30ebf7..fe505a6e337cd6dadbd6b7d69796231a1ce1d944 100755 (executable)
--- a/src/sc_suwi
+++ b/src/sc_suwi
@@ -21,10 +21,6 @@ import cgitb; cgitb.enable()
 #XXX HORRIBLE
 PASSWD = conf.get('general', 'cgipw')
 
-def cmp_correccion_padron(e1, e2):
-    'Compara 2 entregas, según el padrón del alumno.'
-    return cmp(e1.inscripto.padron, e2.inscripto.padron)
-
 def http_header_html(req):
     return 'Content-type: text/html\r\n\r\n'
 
@@ -328,7 +324,7 @@ def correcciones(req, entrega_id):
 ''' % (c.inscripto.padron,
         form(req, pruebas, input_pruebas(req, c.intento.id)),
         form(req, intentos, input_intentos(req, c.entrega.id, c.inscripto.id)))
-    def problematico_header():
+    def rechazado_header():
         return '''<table>
     <caption>Entregas rechazadas</caption>
     <thead>
@@ -340,9 +336,9 @@ def correcciones(req, entrega_id):
     <tbody>
 '''
         pass
-    def problematico_footer():
+    def rechazado_footer():
         return '    </tbody>\n</table>\n'
-    def problematico_row(inscripto, entrega):
+    def rechazado_row(inscripto, entrega):
         intentos = int(Intento.selectBy(inscriptoID=inscripto.id,
             entregaID=entrega.id, connection=conn).count())
         return '''
@@ -367,17 +363,20 @@ def correcciones(req, entrega_id):
     r = '<h1>Entrega %d.%d del curso %d-%d-%d</h1>\n' \
         % (e.nroEjercicio, e.entrega, c.anio, c.cuatrimestre, c.curso)
     correcciones = list(e.correcciones)
-    correcciones.sort(cmp_correccion_padron)
+    # Ordena comparando padron
+    correcciones.sort(lambda x, y: cmp(x.inscripto.padron, y.inscripto.padron))
     r += correccion_header()
     for c in correcciones:
         r += correccion_row(c)
     r += correccion_footer()
     inscriptos_ok = set([c.inscripto for c in correcciones])
     inscriptos = set([i.inscripto for i in e.intentos])
-    r += problematico_header()
-    for i in inscriptos - inscriptos_ok:
-        r += problematico_row(i, e)
-    r += problematico_footer()
+    rechazados = list(inscriptos - inscriptos_ok)
+    rechazados.sort(lambda x, y: cmp(x.padron, y.padron))
+    r += rechazado_header()
+    for i in rechazados:
+        r += rechazado_row(i, e)
+    r += rechazado_footer()
     r += footer()
     return r