api/tests/test_skola_autocomplete.py

@@ -11,20 +11,19 @@ class OrgSkolyAutocompleteTestCase(TestCase):
 		sync_skoly('https://mam.mff.cuni.cz/')
 		# Správné školy podle toho, co orgové poslali: (prefix, ID školy)
 		# NOTE: Pozor, jedná se o databázové indexy. Pokud se to někdy rozbije, bude potřeba je přepsat nebo předělat na IZO
-		# TODO: Opravit zakomentované školy.
 		cls.spravna_data = [
 				('gymnázium kolín', 53),
 				('kolín', 53),
-				#('gasoš', 96),
+				('gasoš', 96),
 				('Rokycany', 96),
-				#('gasoš Rokycany', 96),
+				('gasoš Rokycany', 96),
-				#('SPŠE Pardubice', 815),
+				('SPŠE Pardubice', 815),
 				('Jaroše', 164),
-				#("Gymnázium, Brno, tř. Kpt. Jaroše", 164),
+				("Gymnázium, Brno, tř. Kpt. Jaroše", 164),
 				("Jírovcova", 157),
 				('České Budějovice', 157),
 				("Gymnázium, České Budějovice, Jírovcova 8", 157),
-				#("první soukromé", 2),
+				("první soukromé", 2),
 				("Gymnázium Elgartova", 147),
 				("Jihlava", 45),
 				('Milevsko', 223),

odevzdavatko/templates/odevzdavatko/detail.html

@@ -2,7 +2,6 @@
 {% load static %}
 {% load deadliny %}
 {% load mail %}
-{% load jmena %}
 
 {# Přišlo mi to hezčí, než psát všude if. #}
 {% block custom_css %}
@@ -16,17 +15,6 @@
   {% if edit %}
     <script src="{% static 'odevzdavatko/dynamic_formsets_for_detail.js' %}"></script>
     <script src="{% static 'odevzdavatko/check_for_detail.js' %}"></script>
-    <script type="text/javascript">
-      $(document).ready(function () {
-        const zaskrtavatko = document.getElementById('pridat-jmena-resitelu');
-        zaskrtavatko.addEventListener('change', () => {
-          for (var priloha of document.getElementsByClassName("reseni-ke-stazeni")) {
-            let new_download = zaskrtavatko.checked ? priloha.dataset.altFilename : '';
-            priloha.setAttribute('download', new_download);
-          }
-        });
-      });
-    </script>
   {% endif %}
 
 
@@ -59,20 +47,11 @@
 <tr><th>Soubor</th><th>Řešitelova poznámka</th><th>Datum</th></tr>
 {% for priloha in object.prilohy.all %}
 <tr>
-  <td><a class='reseni-ke-stazeni' 
+	<td><a href="{{ priloha.soubor.url }}" download>{{ priloha.split | last }}</a></td>
-         href="{{ priloha.soubor.url }}"
-         download
-         data-alt-filename="{{object.resitele.first.osoba | jmeno_jako_prefix }}_{{ object.id }}_{{ priloha.split | last}}"
-         >{{ priloha.split | last }}</a></td>
 	<td>{{ priloha.res_poznamka }}</td>
 	<td>{{ priloha.vytvoreno }}</td></tr>
 {% endfor %}
 </table>
-{% if edit %} {# FIXME: tohle nesouvisí s editací, ale s tím, jestli je člověk org… #}
-  <br>
-  <input type=checkbox id="pridat-jmena-resitelu">
-  <label class="field-label" for="pridat-jmena-resitelu">Uvést jméno řešitele v názvu souboru při stažení.</label>
-{% endif %}
 {% else %}
 <p>Žádné přílohy</p>
 {% endif %}

odevzdavatko/templatetags/jmena.py

@@ -1,9 +0,0 @@
-from django import template
-register = template.Library()
-
-from personalni.utils import normalizuj_jmeno
-import seminar.models as m # jen kvůli typové anotaci…
-
-@register.filter
-def jmeno_jako_prefix(o: m.Osoba):
-	return normalizuj_jmeno(o).replace(' ', '_')

odevzdavatko/views.py

@@ -114,7 +114,7 @@ class TabulkaOdevzdanychReseniView(ListView):
 		qs = super().get_queryset()
 		if self.jen_neobodovane:
 			qs = qs.filter(body__isnull=True)
-		qs = qs.filter(problem__in=self.problemy, reseni__in=self.reseni, reseni__resitele__in=self.resitele).select_related('reseni', 'problem').prefetch_related('reseni__resitele__osoba').distinct()
+		qs = qs.filter(problem__in=self.problemy, reseni__in=self.reseni, reseni__resitele__in=self.resitele).select_related('reseni', 'problem').prefetch_related('reseni__resitele__osoba')
 		# FIXME tohle je ošklivé, na špatném místě a pomalé. Ale moc mě štvalo, že musím hledat správná místa v tabulce.
 		self.problemy = self.problemy.filter(id__in=qs.values("problem__id"))
 		return qs
@@ -259,8 +259,8 @@ class DetailReseniView(DetailView):
 		return response
 
 	def check_access(self):
-		""" Řešitel musí být součástí řešení, jinak se na něj nemá co dívat. Případně to může být org."""
+		""" Řešitel musí být součástí řešení, jinak se na něj nemá co dívat. """
-		if not self.object.resitele.filter(osoba__user=self.request.user).exists() and not self.request.user.je_org:
+		if not self.object.resitele.filter(osoba__user=self.request.user).exists():
 			raise PermissionDenied()
 
 
@@ -315,8 +315,8 @@ def hodnoceniReseniView(request, pk, *args, **kwargs):
 			zmeny_bodu = [it for it in form.changed_data if it.startswith("body")]
 			if len(zmeny_bodu) == 1:
 				hodnoceni.__setattr__(zmeny_bodu[0], data_for_body[zmeny_bodu[0]])
-			# > jedna změna je špatně, ale 4 "změny" znamenají že nebylo nic zadáno
+			if len(zmeny_bodu) != 1 and len(zmeny_bodu) != 4:
-			if len(zmeny_bodu) > 1 and len(zmeny_bodu) != 4:
+				print(f"Hodnocení {hodnoceni} mělo mít nastavené víc různých bodů: {zmeny_bodu}. Nastavuji -0.1.")
 				logger.warning(f"Hodnocení {hodnoceni} mělo mít nastavené víc různých bodů: {zmeny_bodu}. Nastavuji -0.1.")
 				hodnoceni.body = -0.1
 			hodnoceni.save()

personalni/utils.py

@@ -1,11 +0,0 @@
-import seminar.models as m
-from various.utils import bez_diakritiky_translate
-import re
-
-def normalizuj_jmeno(o: m.Osoba) -> str:
-	# FIXME: Možná není potřeba vázat na model?
-	cele_jmeno = f'{o.jmeno} {o.prijmeni}'
-	cele_jmeno = cele_jmeno.translate(bez_diakritiky_translate)
-	cele_jmeno = re.sub(r'[^a-zA-Z- ]', '', cele_jmeno)
-	return cele_jmeno
-

seminar/templatetags/deadliny.py

@@ -26,7 +26,7 @@ def deadline_html(deadline: m.Deadline):
 		m.Deadline.TYP_PRVNI_A_SOUS: 'sous_deadline',
 		m.Deadline.TYP_CISLA: 'final_deadline',
 		}
-	return mark_safe(f'<span class="{classes[deadline.typ]}" title="{deadline}">{text}</span>')
+	return mark_safe(f'<span class="{classes[deadline.typ]}">{text}</span>')
 
 @register.filter(name='zkrat_nazev_problemu')
 def zkrat_nazev_problemu(nazev,width):

various/utils.py

@@ -1,33 +0,0 @@
-bez_diakritiky = ({}
-	# FIXME: funguje jen pro český a slovenský text, jinak jsou špatně
-	# transliterace. Potenciální řešení:
-	# https://stackoverflow.com/questions/517923/what-is-the-best-way-to-remove-accents-normalize-in-a-python-unicode-string
-	# (ale přidává to další závislosti…)
-
-	# Tisknutelné ASCII
-	| {chr(a): chr(a) for a in range(32, 126+1)}
-	
-	# České, slovenské a blízké diakritiky a divnoznaky
-	| { x: 'a' for x in 'áÁäÄ'}
-	| { x: 'c' for x in 'čČ'}
-	| { x: 'd' for x in 'ďĎ'}
-	| { x: 'e' for x in 'éÉěĚëË'}
-	| { x: 'i' for x in 'íÍ'}
-	| { x: 'l' for x in 'ľĽĺĹ'}
-	| { x: 'n' for x in 'ňŇ'}
-	| { x: 'o' for x in 'óÓöÖôÔ'}
-	| { x: 'r' for x in 'řŘŕŔ'}
-	| { x: 's' for x in 'šŠßẞ'}
-	| { x: 't' for x in 'ťŤ'}
-	| { x: 'u' for x in 'úÚůŮ'}
-	| { x: 'y' for x in 'ýÝ'}
-	| { x: 'z' for x in 'žŽ'}
-	)
-
-# Tabulka pro str.translate
-class _bez_diakritiky_translate:
-	def __getitem__(self, it):
-		return ord(bez_diakritiky.get(chr(it), None))
-bez_diakritiky_translate = _bez_diakritiky_translate()
-
-# TODO: testy?