mamweb/static/css/mamweb.css

@@ -1233,7 +1233,6 @@ div.gdpr {
 .dosla_reseni tr th, .dosla_reseni tr td {
 	padding: 1px 10px 1px 10px;
 	border-collapse: collapse;
-	min-width: 8em; /*Nastřeleno, aby se řádky s řešeními nezalamovaly. Teoreticky není potřeba pro th, ale whatever.*/
 }
 
 .dosla_reseni tr:nth-child(even) {

odevzdavatko/templates/odevzdavatko/tabulka.html

@@ -1,7 +1,6 @@
 {% extends "base.html" %}
 
 {% load utils %} {# Možná by mohlo být někde výš v hierarchii templatů... #}
-{% load barvy_reseni %}
 
 {% block content %}
 
@@ -37,15 +36,12 @@ Do data (včetně): {{ filtr.reseni_do }}
 			{# TODO: Chceme mít view i na řešení konkrétního řešitele ke všem problémům? #}
 			{{ resitel }}
 		</td>
-		{% for soucet,bunka in hodnoty %}
+		{% for hodn in hodnoty %}
 			<td>
-			{% for reseni,hodnoceni in bunka %}
+			{% if hodn %}
-			<a style="color: {{reseni|barva_reseni}};" href="{% url 'odevzdavatko_detail_reseni' pk=reseni.id %}">
+			<a href="{% url 'odevzdavatko_reseni_resitele_k_problemu' problem=hodn.problem_id resitel=hodn.resitel_id %}">
-					{{reseni.cas_doruceni | date:"j. n."}} ({{ hodnoceni.body|default_if_none:"🔨"}} b)
+				{{ hodn.pocet_reseni }} řeš.<br>{{ hodn.body }} b<br>{{ hodn.posledni_odevzdani|kratke_datum|default_if_none:"Nikdy"|default:"???"}}
-				</a><br>
+			</a>
-			{% endfor %}
-			{% if bunka|length > 1 %}
-				<b>Σ: {{soucet}} b</b>
 			{% endif %}
 			</td>
 		{% endfor %}

odevzdavatko/templatetags/barvy_reseni.py

@@ -1,15 +0,0 @@
-from django import template
-register = template.Library()
-
-from functools import cache
-import seminar.models as m
-
-@register.filter
-@cache
-def barva_reseni(r: m.Reseni):
-	"""Vrátí nějakou barvu pro daný problém, ve tvaru '#RRGGBB'
-
-	Efektivně hešujeme do barev."""
-
-	#TODO: ne všechny barvy jsou dobře rozlišitelné a vidět…
-	return f'#{hash(str(r.id)) & 0xffffff:06x}'

odevzdavatko/views.py

@@ -13,7 +13,6 @@ from django.db.models import Q
 
 from dataclasses import dataclass
 import datetime
-from decimal import Decimal
 from itertools import groupby
 import logging
 
@@ -38,6 +37,14 @@ logger = logging.getLogger(__name__)
 # Taky se může hodit:
 # - Tabulka všech řešitelů x všech problémů?
 
+@dataclass
+class SouhrnReseni:
+	"""Dataclass reprezentující data o odevzdaných řešeních pro zobrazení v tabulce."""
+	pocet_reseni : int
+	posledni_odevzdani : datetime.datetime
+	body : float
+
+
 class TabulkaOdevzdanychReseniView(ListView):
 	template_name = 'odevzdavatko/tabulka.html'
 	model = m.Hodnoceni
@@ -113,45 +120,42 @@ class TabulkaOdevzdanychReseniView(ListView):
 		return qs
 
 	def get_context_data(self, *args, **kwargs):
-		# TODO: refactor asi. Přepisoval jsem to jen syntakticky, nejspíš půlka kódu přestala dávat smysl…
 		# self.resitele, self.reseni a self.problemy jsou již nastavené
 
 		ctx = super().get_context_data(*args, **kwargs)
 		ctx['problemy'] = self.problemy
 		ctx['resitele'] = self.resitele
-		tabulka: dict[m.Problem, dict[m.Resitel, list[tuple[m.Reseni, m.Hodnoceni]]]] = dict()
+		tabulka: dict[m.Problem, dict[m.Resitel, SouhrnReseni]] = dict()
-		soucty: dict[m.Problem, dict[m.Resitel, Decimal]] = dict()
 
-		def pridej_reseni(resitel, hodnoceni):
+		def pridej_reseni(problem, resitel, body, cas):
-			problem = hodnoceni.problem
-			body = hodnoceni.body
-			cas = hodnoceni.reseni.cas_doruceni
-			reseni = hodnoceni.reseni
 			if problem not in tabulka:
 				tabulka[problem] = dict()
-				soucty[problem] = dict()
 			if resitel not in tabulka[problem]:
-				tabulka[problem][resitel] = [(reseni, hodnoceni)]
+				tabulka[problem][resitel] = SouhrnReseni(pocet_reseni=1, posledni_odevzdani=cas, body=body)
-				soucty[problem][resitel] = hodnoceni.body or 0 # Neobodované neřešíme
 			else:
-				tabulka[problem][resitel].append((reseni, hodnoceni))
+				tabulka[problem][resitel].posledni_odevzdani = max(tabulka[problem][resitel].posledni_odevzdani, cas)
-				soucty[problem][resitel] += hodnoceni.body or 0 # Neobodované neřešíme
+				# Zvětšení počtu bodů o aktuální počet, pokud se tam někde nevyskytuje None – pak je součet taky None ("Pozor, nezadané body")
+				tabulka[problem][resitel].body = tabulka[problem][resitel].body + body if body is not None and tabulka[problem][resitel].body is not None else None
+				tabulka[problem][resitel].pocet_reseni += 1
+			# Pro jednoduchost template si ještě poznamenáme ID problému a řešitele
+			tabulka[problem][resitel].problem_id = problem.id
+			tabulka[problem][resitel].resitel_id = resitel.id
 		
 		for hodnoceni in self.get_queryset():
 			for resitel in hodnoceni.reseni.resitele.all():
-				pridej_reseni(resitel, hodnoceni)
+				pridej_reseni(hodnoceni.problem, resitel, hodnoceni.body, hodnoceni.reseni.cas_doruceni)
 
-		hodnoty: list[list[tuple[Decimal,list[tuple[m.Reseni, m.Hodnoceni]]]]] = [] # Seznam řádků výsledné tabulky podle self.resitele, v každém řádku buňky v pořadí podle self.problemy + jejich součty, v každé buňce seznam řešení k danému řešiteli a problému.
+		hodnoty: list[list[SouhrnReseni]] = []
 		resitele_do_tabulky: list[m.Resitel] = []
 		for resitel in self.resitele:
 			dostal_body = False
-			resiteluv_radek: list[tuple[Decimal,list[tuple[m.Resitel, m.Hodnoceni]]]] = [] # podle pořadí v self.problemy
+			resiteluv_radek: list[SourhnReseni] = [] # podle pořadí v self.problemy, pokud daný problém neřešil, pak None
 			for problem in self.problemy:
 				if problem in tabulka and resitel in tabulka[problem]:
-					resiteluv_radek.append((soucty[problem][resitel], tabulka[problem][resitel]))
+					resiteluv_radek.append(tabulka[problem][resitel])
 					dostal_body = True
 				else:
-					resiteluv_radek.append((Decimal(0),[]))
+					resiteluv_radek.append(None)
 			if self.chteni_resitele != FiltrForm.RESITELE_RELEVANTNI or dostal_body:
 				hodnoty.append(resiteluv_radek)
 				resitele_do_tabulky.append(resitel)
@@ -170,11 +174,6 @@ class TabulkaOdevzdanychReseniView(ListView):
 
 # Velmi silně inspirováno zdrojáky, FIXME: Nedá se to udělat smysluplněji?
 class ReseniProblemuView(MultipleObjectTemplateResponseMixin, MultipleObjectMixin, View):
-	"""Rozskok mezi více řešeními téhož problému od téhož řešitele.
-	
-	Asi už bude zastaralý v okamžiku, kdy se tenhle komentář nasadí na produkci :-)
-
-	V případě, že takové řešení existuje jen jedno, tak na něj přesměruje."""
 	model = m.Reseni
 	template_name = 'odevzdavatko/seznam.html'