From 7b1a546cf211f419053f417a509de6add0cd5875 Mon Sep 17 00:00:00 2001
From: Ricardo Markiewicz <rmarkie@fi.uba.ar>
Date: Sat, 17 Mar 2007 22:04:20 +0000
Subject: [PATCH] Pongo en color rojo, amarillo o verde a cada fila de las
 entregas dependiendo como salio. No pude probar el amarillo porque las que
 estan mal de entrada (Entrega.exito == False) no tienen pruebas ya que lo que
 fallo fue la compilacion.

En el caso de Entrega.exito==True no miro que onda las pruebas, porque asumo que si exito==True paso todas
las pruebas bien.
---
 sercom/static/css/style.css                   | 24 ++++++++++-
 .../instancia/templates/entregas.kid          | 42 ++++++++++++++++---
 2 files changed, 59 insertions(+), 7 deletions(-)

diff --git a/sercom/static/css/style.css b/sercom/static/css/style.css
index ad33b17..4fbb2d0 100644
--- a/sercom/static/css/style.css
+++ b/sercom/static/css/style.css
@@ -237,4 +237,26 @@ form.filter table td,form.filter th,form.filter tr {
 form.filter th {
     background: none;
     font-weight:bold;
-}
+}
+
+.list .exitosa {
+	background: #b9ffb9;
+}
+
+.list .noexitosa {
+	background: #ffdddd;
+}
+
+.list .entregaok {
+	background:green;
+}
+
+.list .entregamaso {
+	background:yellow;
+}
+
+.list .entregamal {
+	background:red;
+}
+
+
diff --git a/sercom/subcontrollers/curso/ejercicio/instancia/templates/entregas.kid b/sercom/subcontrollers/curso/ejercicio/instancia/templates/entregas.kid
index cd6316a..b2e5bdb 100644
--- a/sercom/subcontrollers/curso/ejercicio/instancia/templates/entregas.kid
+++ b/sercom/subcontrollers/curso/ejercicio/instancia/templates/entregas.kid
@@ -19,12 +19,42 @@
         <th>Observaciones</th>
 				<th>Operaciones</th>
     </tr>
-    <tr py:for="record in instancia.entregas">
-        <td><span py:if="record.entregador" py:replace="record.entregador.shortrepr()">usuario</span></td>
-        <td><span py:replace="record.exito">fecha asignado</span></td>
-        <td><span py:replace="record.inicio">fecha corregido</span></td>
-        <td><span py:replace="record.fin">fecha corregido</span></td>
-        <td><span py:replace="record.observaciones">nota</span></td>
+		<tr py:for="record in instancia.entregas">
+				<?python
+					def contar_comandos_mal(prueba, publico):
+						total = 0
+						for c in prueba.comandos_ejecutados:
+							if not c.exito and c.publico == publico:
+								total += 1
+						return total
+
+					if not record.exito:
+						# Reviso que tan mal esta
+						# si no hay pruebas, esta mal porque no anduvieron los comandos
+						if len(record.pruebas) == 0:
+							color = "entregamal"
+						else:
+							# Veo que onda con las pruebas
+							pruebas_pub_mal = 0
+							pruebas_priv_mal = 0
+							for prueba in record.pruebas:
+								if contar_comandos_mal(prueba, True) > 0:
+									pruebas_pub_mal += 1
+								if contar_comandos_mal(prueba, False) > 0:
+									pruebas_priv_mal += 1
+							if pruebas_pub_mal > 0:
+								color = "entregamal"
+							if pruebas_priv_mal > 0:
+								color = "entregamal"
+					else:
+						# Todo Ok! 
+						color = "entregaok"
+				?>
+				<td class="${color}"><span py:if="record.entregador" py:replace="record.entregador.shortrepr()">usuario</span></td>
+        <td class="${color}"><span py:replace="record.exito">fecha asignado</span></td>
+        <td class="${color}"><span py:replace="record.inicio">fecha corregido</span></td>
+        <td class="${color}"><span py:replace="record.fin">fecha corregido</span></td>
+        <td class="${color}"><span py:replace="record.observaciones">nota</span></td>
 				<td>
 					<a href="${tg.url('/mis_entregas/corrida/%d' % record.id)}">Corrida</a>
 					<a href="${tg.url('/mis_entregas/get_archivo/%d' % record.id)}">Bajar Archivo</a>
-- 
2.43.0