Merge branch 'korekturovatko-full' into korekturovatko

data/groups.json

@@ -14,12 +14,12 @@
 					"flatpage"
 				],
 				[
-					"change_flatpage",
+					"delete_flatpage",
 					"flatpages",
 					"flatpage"
 				],
 				[
-					"delete_flatpage",
+					"change_flatpage",
 					"flatpages",
 					"flatpage"
 				],
@@ -34,12 +34,12 @@
 					"galerie"
 				],
 				[
-					"change_galerie",
+					"delete_galerie",
 					"galerie",
 					"galerie"
 				],
 				[
-					"delete_galerie",
+					"change_galerie",
 					"galerie",
 					"galerie"
 				],
@@ -54,12 +54,12 @@
 					"obrazek"
 				],
 				[
-					"change_obrazek",
+					"delete_obrazek",
 					"galerie",
 					"obrazek"
 				],
 				[
-					"delete_obrazek",
+					"change_obrazek",
 					"galerie",
 					"obrazek"
 				],
@@ -104,12 +104,12 @@
 					"komentar"
 				],
 				[
-					"change_komentar",
+					"delete_komentar",
 					"korektury",
 					"komentar"
 				],
 				[
-					"delete_komentar",
+					"change_komentar",
 					"korektury",
 					"komentar"
 				],
@@ -124,12 +124,12 @@
 					"korekturovanepdf"
 				],
 				[
-					"change_korekturovanepdf",
+					"delete_korekturovanepdf",
 					"korektury",
 					"korekturovanepdf"
 				],
 				[
-					"delete_korekturovanepdf",
+					"change_korekturovanepdf",
 					"korektury",
 					"korekturovanepdf"
 				],
@@ -144,12 +144,12 @@
 					"oprava"
 				],
 				[
-					"change_oprava",
+					"delete_oprava",
 					"korektury",
 					"oprava"
 				],
 				[
-					"delete_oprava",
+					"change_oprava",
 					"korektury",
 					"oprava"
 				],
@@ -164,12 +164,12 @@
 					"novinky"
 				],
 				[
-					"change_novinky",
+					"delete_novinky",
 					"novinky",
 					"novinky"
 				],
 				[
-					"delete_novinky",
+					"change_novinky",
 					"novinky",
 					"novinky"
 				],
@@ -204,12 +204,12 @@
 					"prijemce"
 				],
 				[
-					"change_prijemce",
+					"delete_prijemce",
 					"personalni",
 					"prijemce"
 				],
 				[
-					"delete_prijemce",
+					"change_prijemce",
 					"personalni",
 					"prijemce"
 				],
@@ -234,12 +234,12 @@
 					"skola"
 				],
 				[
-					"change_skola",
+					"delete_skola",
 					"personalni",
 					"skola"
 				],
 				[
-					"delete_skola",
+					"change_skola",
 					"personalni",
 					"skola"
 				],
@@ -249,14 +249,24 @@
 					"skola"
 				],
 				[
-					"view_hlasovani",
+					"add_hlasovani",
 					"prednasky",
 					"hlasovani"
 				],
 				[
-					"view_hlasovanioznalostech",
+					"delete_hlasovani",
 					"prednasky",
-					"hlasovanioznalostech"
+					"hlasovani"
+				],
+				[
+					"change_hlasovani",
+					"prednasky",
+					"hlasovani"
+				],
+				[
+					"view_hlasovani",
+					"prednasky",
+					"hlasovani"
 				],
 				[
 					"add_prednaska",
@@ -264,12 +274,12 @@
 					"prednaska"
 				],
 				[
-					"change_prednaska",
+					"delete_prednaska",
 					"prednasky",
 					"prednaska"
 				],
 				[
-					"delete_prednaska",
+					"change_prednaska",
 					"prednasky",
 					"prednaska"
 				],
@@ -284,12 +294,12 @@
 					"seznam"
 				],
 				[
-					"change_seznam",
+					"delete_seznam",
 					"prednasky",
 					"seznam"
 				],
 				[
-					"delete_seznam",
+					"change_seznam",
 					"prednasky",
 					"seznam"
 				],
@@ -298,38 +308,18 @@
 					"prednasky",
 					"seznam"
 				],
-				[
-					"add_znalost",
-					"prednasky",
-					"znalost"
-				],
-				[
-					"change_znalost",
-					"prednasky",
-					"znalost"
-				],
-				[
-					"delete_znalost",
-					"prednasky",
-					"znalost"
-				],
-				[
-					"view_znalost",
-					"prednasky",
-					"znalost"
-				],
 				[
 					"add_konfera",
 					"soustredeni",
 					"konfera"
 				],
 				[
-					"change_konfera",
+					"delete_konfera",
 					"soustredeni",
 					"konfera"
 				],
 				[
-					"delete_konfera",
+					"change_konfera",
 					"soustredeni",
 					"konfera"
 				],
@@ -344,12 +334,12 @@
 					"konfery_ucastnici"
 				],
 				[
-					"change_konfery_ucastnici",
+					"delete_konfery_ucastnici",
 					"soustredeni",
 					"konfery_ucastnici"
 				],
 				[
-					"delete_konfery_ucastnici",
+					"change_konfery_ucastnici",
 					"soustredeni",
 					"konfery_ucastnici"
 				],
@@ -364,12 +354,12 @@
 					"soustredeni"
 				],
 				[
-					"change_soustredeni",
+					"delete_soustredeni",
 					"soustredeni",
 					"soustredeni"
 				],
 				[
-					"delete_soustredeni",
+					"change_soustredeni",
 					"soustredeni",
 					"soustredeni"
 				],
@@ -384,12 +374,12 @@
 					"soustredeni_organizatori"
 				],
 				[
-					"change_soustredeni_organizatori",
+					"delete_soustredeni_organizatori",
 					"soustredeni",
 					"soustredeni_organizatori"
 				],
 				[
-					"delete_soustredeni_organizatori",
+					"change_soustredeni_organizatori",
 					"soustredeni",
 					"soustredeni_organizatori"
 				],
@@ -404,12 +394,12 @@
 					"soustredeni_ucastnici"
 				],
 				[
-					"change_soustredeni_ucastnici",
+					"delete_soustredeni_ucastnici",
 					"soustredeni",
 					"soustredeni_ucastnici"
 				],
 				[
-					"delete_soustredeni_ucastnici",
+					"change_soustredeni_ucastnici",
 					"soustredeni",
 					"soustredeni_ucastnici"
 				],
@@ -424,12 +414,12 @@
 					"tag"
 				],
 				[
-					"change_tag",
+					"delete_tag",
 					"taggit",
 					"tag"
 				],
 				[
-					"delete_tag",
+					"change_tag",
 					"taggit",
 					"tag"
 				],
@@ -444,12 +434,12 @@
 					"taggeditem"
 				],
 				[
-					"change_taggeditem",
+					"delete_taggeditem",
 					"taggit",
 					"taggeditem"
 				],
 				[
-					"delete_taggeditem",
+					"change_taggeditem",
 					"taggit",
 					"taggeditem"
 				],
@@ -464,12 +454,12 @@
 					"cislo"
 				],
 				[
-					"change_cislo",
+					"delete_cislo",
 					"tvorba",
 					"cislo"
 				],
 				[
-					"delete_cislo",
+					"change_cislo",
 					"tvorba",
 					"cislo"
 				],
@@ -484,12 +474,12 @@
 					"clanek"
 				],
 				[
-					"change_clanek",
+					"delete_clanek",
 					"tvorba",
 					"clanek"
 				],
 				[
-					"delete_clanek",
+					"change_clanek",
 					"tvorba",
 					"clanek"
 				],
@@ -519,12 +509,12 @@
 					"pohadka"
 				],
 				[
-					"change_pohadka",
+					"delete_pohadka",
 					"tvorba",
 					"pohadka"
 				],
 				[
-					"delete_pohadka",
+					"change_pohadka",
 					"tvorba",
 					"pohadka"
 				],
@@ -539,12 +529,12 @@
 					"problem"
 				],
 				[
-					"change_problem",
+					"delete_problem",
 					"tvorba",
 					"problem"
 				],
 				[
-					"delete_problem",
+					"change_problem",
 					"tvorba",
 					"problem"
 				],
@@ -559,12 +549,12 @@
 					"rocnik"
 				],
 				[
-					"change_rocnik",
+					"delete_rocnik",
 					"tvorba",
 					"rocnik"
 				],
 				[
-					"delete_rocnik",
+					"change_rocnik",
 					"tvorba",
 					"rocnik"
 				],
@@ -579,12 +569,12 @@
 					"tema"
 				],
 				[
-					"change_tema",
+					"delete_tema",
 					"tvorba",
 					"tema"
 				],
 				[
-					"delete_tema",
+					"change_tema",
 					"tvorba",
 					"tema"
 				],
@@ -599,12 +589,12 @@
 					"uloha"
 				],
 				[
-					"change_uloha",
+					"delete_uloha",
 					"tvorba",
 					"uloha"
 				],
 				[
-					"delete_uloha",
+					"change_uloha",
 					"tvorba",
 					"uloha"
 				],
@@ -619,12 +609,12 @@
 					"nastaveni"
 				],
 				[
-					"change_nastaveni",
+					"delete_nastaveni",
 					"various",
 					"nastaveni"
 				],
 				[
-					"delete_nastaveni",
+					"change_nastaveni",
 					"various",
 					"nastaveni"
 				],

docs/conf.py

@@ -36,7 +36,6 @@ extensions = [
     'sphinx.ext.intersphinx',
     'sphinx.ext.autosectionlabel',
     'myst_parser',
-	'sphinxcontrib_django',
 ]
 
 # Add any paths that contain templates here, relative to this directory.

korektury/api/apps.py

@@ -0,0 +1,7 @@
+from django.apps import AppConfig
+
+
+class ApiConfig(AppConfig):
+	default_auto_field = 'django.db.models.BigAutoField'
+	name = 'korektury.api'
+	label = 'korektury_api' # Protože jedno api už máme.

korektury/api/urls.py

@@ -0,0 +1,9 @@
+from django.urls import path
+from personalni.utils import org_required
+from . import views
+
+urlpatterns = [
+	path('<int:pdf_id>/stav', org_required(views.korektury_stav_view), name='korektury_api_pdf_stav'),
+	path('oprava/stav', org_required(views.oprava_stav_view), name='korektury_api_oprava_stav'),
+	path('<int:pdf_id>/opravy_a_komentare', org_required(views.opravy_a_komentare_view), name='korektury_api_opravy_a_komentare'),
+]

korektury/api/views.py

@@ -0,0 +1,65 @@
+from django.http import HttpResponseForbidden, JsonResponse
+from django.shortcuts import get_object_or_404
+from django.utils.html import linebreaks
+from django.views.decorators.csrf import csrf_exempt
+
+from rest_framework import serializers
+
+from korektury.utils import send_email_notification_komentar
+from korektury.models import Oprava, KorekturovanePDF, Komentar
+from personalni.models import Organizator, Osoba
+
+
+def korektury_stav_view(request, pdf_id: int, **kwargs):
+	q = request.POST
+	pdf = get_object_or_404(KorekturovanePDF, id=pdf_id)
+	status = q.get('state')
+	if status is not None:
+		assert status in KorekturovanePDF.STATUS.values
+		pdf.status = status
+		pdf.save()
+	return JsonResponse({'status': pdf.status})
+
+
+def oprava_stav_view(request, **kwargs):
+	q = request.POST
+	op_id_str = q.get('id')
+	assert op_id_str is not None
+	op_id = int(op_id_str)
+	op = get_object_or_404(Oprava, id=op_id)
+	status = q.get('action')
+	if status is not None:
+		assert status in Oprava.STATUS.values
+		op.status = status
+		op.save()
+	return JsonResponse({'status': op.status})
+
+class KomentarSerializer(serializers.ModelSerializer):
+	class Meta:
+		model = Komentar
+		fields = '__all__'
+
+	def to_representation(self, instance):
+		ret = super().to_representation(instance)
+		ret["autor"] = str(instance.autor)
+		ret["text"] = linebreaks(ret["text"], autoescape=True) # Autora není třeba escapovat, ten se vkládá jako text.
+		return ret
+
+class OpravaSerializer(serializers.ModelSerializer):
+	class Meta:
+		model = Oprava
+		fields = '__all__'
+
+	def to_representation(self, instance):
+		ret = super().to_representation(instance)
+		ret["komentare"] = [KomentarSerializer(komentar).data for komentar in instance.komentar_set.all()]
+		return ret
+
+	# komentar_set = serializers.ListField(child=KomentarSerializer())
+
+def opravy_a_komentare_view(request, pdf_id: int, **kwargs):
+	q = request.POST
+
+	opravy = Oprava.objects.filter(pdf=pdf_id).all()
+	# Serializovat list je prý security vulnerability, tedy je přidán slovník pro bezpečnost
+	return JsonResponse({"context": [OpravaSerializer(oprava).data for oprava in opravy]})

korektury/forms.py

@@ -1,14 +0,0 @@
-from django import forms
-
-class OpravaForm(forms.Form):
-	""" formulář k přidání opravy (:class:`korektury.models.Oprava`) """
-	text = forms.CharField(max_length=256)
-	autor = forms.CharField(max_length=20)
-	x = forms.IntegerField()
-	y = forms.IntegerField()
-	scroll = forms.CharField(max_length=256)
-	pdf = forms.CharField(max_length=256)
-	img_id = forms.CharField(max_length=256)
-	id = forms.CharField(max_length=256)
-	action = forms.CharField(max_length=256)
-

korektury/migrations/0025_remove_oprava_autor_remove_oprava_text.py

@@ -0,0 +1,45 @@
+# Generated by Django 4.2.16 on 2024-12-12 10:25
+
+from django.db import migrations
+
+import datetime
+from django.utils import timezone
+
+
+def oprava2komentar(apps, schema_editor):
+    Oprava = apps.get_model('korektury', 'Oprava')
+    Komentar = apps.get_model('korektury', 'Komentar')
+
+    for o in Oprava.objects.all():
+        Komentar.objects.create(oprava=o, text=o.text, autor=o.autor, cas=timezone.make_aware(datetime.datetime.fromtimestamp(0)))
+
+def komentar2oprava(apps, schema_editor):
+    Oprava = apps.get_model('korektury', 'Oprava')
+    Komentar = apps.get_model('korektury', 'Komentar')
+
+    for o in Oprava.objects.all():
+        k = Komentar.objects.filter(oprava=o).first()
+        o.text = k.text
+        o.autor = k.autor
+        o.save()
+        k.delete()
+
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('korektury', '0024_vic_orgu_k_pdf'),
+    ]
+
+    operations = [
+        migrations.RunPython(oprava2komentar, komentar2oprava),
+        migrations.RemoveField(
+            model_name='oprava',
+            name='autor',
+        ),
+        migrations.RemoveField(
+            model_name='oprava',
+            name='text',
+        ),
+    ]

korektury/models.py

@@ -52,16 +52,13 @@ class KorekturovanePDF(models.Model):
 
 	stran = models.IntegerField(u'počet stran', help_text='Počet stran PDF',
 								default=0)
-	STATUS_PRIDAVANI = 'pridavani'
+
-	STATUS_ZANASENI = 'zanaseni'
+	class STATUS(models.TextChoices):
-	STATUS_ZASTARALE = 'zastarale'
+		PRIDAVANI = 'pridavani', 'Přidávání korektur'
-	STATUS_CHOICES = (
+		ZANASENI = 'zanaseni', 'Korektury jsou zanášeny'
-			(STATUS_PRIDAVANI, u'Přidávání korektur'),
+		ZASTARALE = 'zastarale', 'Stará verze, nekorigovat'
-			(STATUS_ZANASENI, u'Korektury jsou zanášeny'),
+
-			(STATUS_ZASTARALE, u'Stará verze, nekorigovat'),
+	status = models.CharField(u'stav PDF',max_length=16, choices=STATUS.choices, blank=False, default = STATUS.PRIDAVANI)
-			)
-	status = models.CharField(u'stav PDF',max_length=16, choices=STATUS_CHOICES, blank=False,
-			default = STATUS_PRIDAVANI)
 
 	poslat_mail = models.BooleanField('Poslat mail o novém PDF', default=True,
 			help_text='Určuje, zda se má o nově nahraném PDF poslat e-mail do mam-org. Při upravování existujícího souboru už nemá žádný vliv.',
@@ -152,32 +149,13 @@ class Oprava(models.Model):
 	x = models.IntegerField(u'x-ová souřadnice bugu')
 	y = models.IntegerField(u'y-ová souřadnice bugu')
 
-	STATUS_K_OPRAVE = 'k_oprave'
+	class STATUS(models.TextChoices):
-	STATUS_OPRAVENO = 'opraveno'
+		K_OPRAVE = 'k_oprave', 'K opravě'
-	STATUS_NENI_CHYBA = 'neni_chyba'
+		OPRAVENO = 'opraveno', 'Opraveno'
-	STATUS_K_ZANESENI = 'k_zaneseni'
+		NENI_CHYBA = 'neni_chyba', 'Není chyba'
-	STATUS_CHOICES = (
+		K_ZANESENI = 'k_zaneseni', 'K zanesení do TeXu'
-			(STATUS_K_OPRAVE, u'K opravě'),
-			(STATUS_OPRAVENO, u'Opraveno'),
-			(STATUS_NENI_CHYBA, u'Není chyba'),
-			(STATUS_K_ZANESENI, u'K zanesení do TeXu'),
-			)
-	status = models.CharField(u'stav opravy',max_length=16, choices=STATUS_CHOICES, blank=False,
-			default = STATUS_K_OPRAVE)
-
-	autor = models.ForeignKey(Organizator, blank = True,
-			help_text='Autor opravy',
-			null = True, on_delete=models.SET_NULL)
-	
-	text = models.TextField(u'text opravy',blank = True, help_text='Text opravy')
-
-#	def __init__(self,dictionary):
-#		for k,v in dictionary.items():
-#			setattr(self,k,v)
-
-	def __str__(self):
-		return '{} od {}: {}'.format(self.status,self.autor,self.text)
 
+	status = models.CharField(u'stav opravy',max_length=16, choices=STATUS.choices, blank=False, default = STATUS.K_OPRAVE)
 
 
 @reversion.register(ignore_duplicates=True)

korektury/static/korektury/opraf.css

@@ -1,136 +1,159 @@
-body,
+.textzanaseni { display:none; }
-.adding{
+.textzastarale { display:none; }
+#prekomentar, #preoprava, #prepointer { display: none; }
+
+body {
+	&[data-status="pridavani"] {
 		background: #f3f3f3;
-	color: black;
+	}
-}
+
-.comitting
+	&[data-status="zanaseni"] {
-{
 		background: yellow;
-}
+
-.deprecated {
+		.textzanaseni { display: unset; }
+	}
+
+	&[data-status="zastarale"] {
 		background: red;
+
+		.textzastarale { display: unset; }
+	}
 }
 
 img{background:white;}
 
 /* Barvy korektur */
-.k_oprave { 
+[data-opravastatus="k_oprave"] {
 	--rgb: 255, 0, 0;
+
+	[value="k_oprave"] { display: none }
+	[value="notcomment"] { display: none }
 }
-.opraveno {
+[data-opravastatus="opraveno"] {
 	--rgb: 0, 0, 255;
+
+	[value="opraveno"] { display: none }
+	[value="comment"] { display: none }
 }
-.neni_chyba {
+[data-opravastatus="neni_chyba"] {
 	--rgb: 128, 128, 128;
+
+	[value="neni_chyba"] { display: none }
+	[value="comment"] { display: none }
 }
-.k_zaneseni {
+[data-opravastatus="k_zaneseni"] {
 	--rgb: 0, 255, 0;
+
+	[value="k_zaneseni"] { display: none }
+	[value="notcomment"] { display: none }
 }
 
-.pointer-hi, 
+/* Skrývání korektur */
+[data-opravazobrazit="false"] {
+	.corr-body { display: none; }
+	.corr-buttons { display: none; }
+	.toggle-button { transform: rotate(180deg); }
+}
+
+/* Čára od textu k místu korektury */
 .pointer{ 
 	position:absolute;
-	/*border-bottom-left-radius: 10px; */
+	border-bottom-left-radius: 10px;
-	border-left: 2px solid yellow;
+	border-left: 1px solid rgb(var(--rgb),var(--alpha));
-	border-bottom: 2px solid yellow;
+	border-bottom: 1px solid rgb(var(--rgb),var(--alpha));
-	border-color: rgb(var(--rgb),var(--alpha));
+	pointer-events: none;
-}
 
-.pointer {
-	border-width: 1px;
 	--alpha: 0.35;
-}
 
-.pointer-hi {
+	/* Zvýraznění čáry při najetí na korekturu */
+	&[data-highlight="true"] {
 		border-width: 3px;
 		--alpha: 1;
+	}
 }
 
-
+/* Korektura samotná */
-
+.oprava {
-.box:hover{
-	border-width:3px;
-	margin: 0px;
-}
-.box {
 	margin: 1px;
 	background-color: white;
-	width:300px;
+	width: 300px;
-	/*position:absolute;*/
 	padding: 3px;
-	border: 2px solid black;
+	border: 2px solid rgb(var(--rgb));
 	border-radius: 10px;
-	border-color: rgb(var(--rgb));
-}
-form {
-	display:inline;
-}
-
-.float-right{
-	float:right;
-}
-
-.imgdiv {
-	position:relative;
-	left:0px;
-	top:0px;
-}
-#commform-div {
-	display: none;
 	position: absolute;
-	background-color: white;
-	border: 1px solid;
-	padding: 3px;
-	/*
-	width: 310;
-	height: 220;
-	*/
-	z-index: 10;
-	border: 4px solid red;
-	border-radius: 10px;
-	background-color: white;
-	opacity: 80%;
-}
-.close-button{
-	background-color: yellow;	
-}
 
+	&:hover {
+		border-width:3px;
+		margin: 0;
+	}
 
-.box button,
+	button, img {
-.box img,
-.box-done button,
-.box-done img,
-.box-ready button,
-.box-ready img,
-.box-wontfix button,
-.box-wontfix img{
 		border: 1px solid white;
 		background-color:transparent;
 		margin:0;
 		padding: 1px;
-}
+
-.box button:hover,
+		&:hover {
-.box img:hover,
-.box-done img:hover,
-.box-done button:hover,
-.box-ready img:hover,
-.box-ready button:hover,
-.box-wontfix img:hover,
-.box-wontfix button:hover{
 			border: 1px solid black;
-}
+		}
+	}
 
-.comment hr {
+	button img { pointer-events: none; }
-	height: 0px;
-}
 
-.corr-header {
+	.corr-header {
 		overflow: auto;
-}
+	}
 
-.author {
+	.author {
 		font-weight: bold;
 		float: left;
 		margin-top: 3px;
+	}
+
+	.float-right{
+		float:right;
+	}
 }
 
+form {
+	display:inline;
+}
+
+/* Zobrazované PDF */
+.imgdiv {
+	position:relative;
+	left:0;
+	top:0;
+}
+
+/* Přidávání korektury / úprava komentáře */
+#commform-div {
+	position: absolute;
+	background-color: white;
+	padding: 3px;
+	z-index: 10;
+	border: 4px solid red;
+	border-radius: 10px;
+	opacity: 80%;
+}
+
+
+/**** ROZLIŠENÍ MEZI LOKÁLNÍM, TESTOVACÍM A PRODUKČNÍM WEBEM ****/
+body.localweb, body.testweb, body.suprodweb {
+	&:before, &:after {
+		content: "";
+		position: fixed;
+		width: 20px;
+		height: 100%;
+		top: 0;
+		z-index: -1000;
+	}
+
+	&:before { left: 0; }
+	&:after { right: 0; }
+}
+
+body.localweb { &:before, &:after { background: greenyellow; } }
+body.testweb { &:before, &:after { background: darkorange; } }
+body.suprodweb { &:before, &:after { background: red; } }
+/****************************************************************/

korektury/static/korektury/opraf.js

@@ -1,283 +1,46 @@
+const W_SKIP = 10;
+const H_SKIP = 5;
+const POINTER_MIN_H = 30;
+
 function place_comments_one_div(img_id, comments)
 {
-	var img = document.getElementById(img_id);
+	const img = document.getElementById("img-"+img_id);
-	if( img == null ) { 
+	if( img == null ) return;
-		return;
+    const comments_sorted = comments.sort((a, b) => a.y - b.y);
-	}
-	var par = img.parentNode;
-	var w = img.clientWidth;
-	var h = img.clientHeight;
-	var w_skip = 10;
-	var h_skip = 5;
-	var pointer_min_h = 30;
 
-	var bott_max = 0;
+	const par = img.parentNode;
-	var comments_sorted = comments.sort(function (a,b) {
+	const w = img.clientWidth;
-		return a[2] - b[2];
+
-		//pokus o hezci kladeni poiteru, ale nic moc
+	let bott_max = 0;
-		if( a[3] < b[3] ) { 
+	for (const oprava of comments_sorted) {
-			return (a[2] + pointer_min_h)- b[2];
+		const x = oprava.x;
-		} else {
+		const y = oprava.y;
-			return (a[2] - pointer_min_h)- b[2];
+        const htmlElement = oprava.htmlElement;
+		const pointer = oprava.pointer;
+
+		par.appendChild(pointer);
+		par.appendChild(htmlElement);
+
+		const delta_y = (y > bott_max) ?  0: bott_max - y + H_SKIP;
+
+		pointer.style.left = x;
+		pointer.style.top = y;
+		pointer.style.width = w - x + W_SKIP;
+		pointer.style.height = POINTER_MIN_H + delta_y;
+
+		htmlElement.style.left = w + W_SKIP;
+		htmlElement.style.top = y + delta_y;
+
+		bott_max = Math.max(bott_max, htmlElement.offsetTop + htmlElement.offsetHeight);
 	}
 
-	});
+	if (par.offsetHeight < bott_max) par.style.height = bott_max;
-	//console.log("w:" + w);
-	for (c in comments_sorted) {
-		var id = comments_sorted[c][0];
-		var x  = comments_sorted[c][1];
-		var y  = comments_sorted[c][2];
-
-		var el = document.getElementById(id);
-		var elp = document.getElementById(id + "-pointer");
-
-		if( el == null  || elp == null ) { 
-			continue;
-		}
-
-		par.appendChild(elp);
-		par.appendChild(el);
-
-		var delta_y = (y > bott_max) ?  0: bott_max - y + h_skip;
-
-		elp.style.left = x;
-		elp.style.top = y ;
-		elp.style.width = w - x  + w_skip;
-		elp.style.height = pointer_min_h + delta_y;
-		elp.img_id = img_id;
-		el.img_id = img_id;
-
-		el.style.position = 'absolute';
-		el.style.left = w + w_skip;
-		el.style.top = y + delta_y;
-
-		var bott =  el.offsetTop + el.offsetHeight;
-		bott_max = ( bott_max > bott ) ?  bott_max : bott;
-
-		//console.log( "par.w:"  + par.style.width);
-
-	}
-	if( par.offsetHeight < bott_max ) {
-		//par.style.height = bott_max;
-		//alert("preteklo to:"+ par.offsetHeight +",mx:" + bott_max );
-		par.style.height = bott_max;
-
-	}
 }
 
 function place_comments() {
-    for (var i=0; i < comments.length-1; i++) {
+    for (let [img_id, opravy] of Object.entries(comments)) {
-	    place_comments_one_div(comments[i][0], comments[i][1])
+	    place_comments_one_div(img_id, opravy)
     }
 }
 
-// ctrl-enter submits form
-function textarea_onkey(ev)
-{
-	//console.log("ev:" + ev.keyCode + "," + ev.ctrlKey);
-	if( (ev.keyCode  == 13 || ev.keyCode == 10 )  && ev.ctrlKey ) {
-		var form = document.getElementById('commform');
-		if( form ) { 
-			save_scroll(form);
-			//form.action ='';
-			form.submit();
-		}
-		return true;
-	}
-	return false;
-}
-
-//hide comment  form
-function close_commform() {
-	var formdiv = document.getElementById('commform-div');
-	if( formdiv == null ) {
-		alert("form null");
-		return true;
-	}
-	formdiv.style.display = 'none';
-	return false;
-}
-
-// show comment form, when clicked to image
-function img_click(element, ev) {
-	var body_class = document.body.className;
-	switch(body_class){
-		case "comitting":
-			if (!confirm("Právě jsou zanášeny korektury, opravdu chcete přidat novou?"))
-				return;
-			break;
-		case "deprecated":
-			if (!confirm("Toto PDF je již zastaralé, opravdu chcete vytvořit korekturu?"))
-				return;
-			break;
-	}
-
-	var dx, dy;
-	var par = element.parentNode;
-	if( ev.pageX != null ) { 
-		dx = ev.pageX - par.offsetLeft;
-		dy = ev.pageY - par.offsetTop;
-	} else { //IE
-		dx = ev.offsetX;
-		dy = ev.offsetY;
-	}
-	var img_id = element.id;
-	if( element.img_id != null ) {
-		// click was to '-pointer'
-		img_id = element.img_id;
-	}
-	return show_form(img_id, dx, dy, '', '', '', '');
-}
-// hide or show text of correction
-function toggle_visibility(oid){
-	var buttondiv = document.getElementById(oid+'-buttons')
-	var text = document.getElementById(oid+'-body');
-	if (text.style.display == 'none'){
-		text.style.display = 'block';
-		buttondiv.style.display = 'inline-block';
-	}else {
-		text.style.display = 'none';
-		buttondiv.style.display = 'none';
-	}
-	for (var i=0;i<comments.length-1;i++){
-		 place_comments_one_div(comments[i][0], comments[i][1])
-	}
-} 
-
-// show comment form, when 'edit' or 'comment' button pressed
-function box_edit(oid, action)
-{
-	var divpointer = document.getElementById(oid + '-pointer');
-
-	var text;
-	if (action == 'update') {
-		var text_el = document.getElementById(oid + '-text');
-		text = text_el.textContent; // FIXME původně tu bylo innerHTML.unescapeHTML()
-
-	} else {
-		text = '';
-	}
-
-	var dx = parseInt(divpointer.style.left);
-	var dy = parseInt(divpointer.style.top);
-	var divbox = document.getElementById(oid);
-	//alert('not yet 2:' + text + text_el); // + divpointer.style.top "x" + divpo );
-	id = oid.substring(2);
-	return show_form(divbox.img_id, dx, dy, id, text, action);
-}
-
-// show comment form when 'update-comment' button pressed
-function update_comment(oid,ktid)
-{
-	var divpointer = document.getElementById(oid + '-pointer');
-	var dx = parseInt(divpointer.style.left);
-	var dy = parseInt(divpointer.style.top);
-	
-	var divbox = document.getElementById(oid);
-	var text = document.getElementById(ktid).textContent; // FIXME původně tu bylo innerHTML.unescapeHTML()
-
-	return show_form(divbox.img_id, dx, dy, ktid.substring(2), text, 'update-comment');
-}
-
-//fill up comment form and show him
-function show_form(img_id, dx, dy, id, text, action) {
-	var form = document.getElementById('commform');
-	var formdiv = document.getElementById('commform-div');
-	var textarea = document.getElementById('commform-text');
-	var inputX  = document.getElementById('commform-x');
-	var inputY  = document.getElementById('commform-y');
-	var inputImgId  = document.getElementById('commform-img-id');
-	var inputId  = document.getElementById('commform-id');
-	var inputAction  = document.getElementById('commform-action');
-	var img = document.getElementById(img_id);
-
-	if( formdiv == null || textarea == null ) {
-		alert("form null");
-		return 1;
-	}
-
-	//form.action = "#" +  img_id;
-
-	// set hidden values
-	inputX.value = dx;
-	inputY.value = dy;
-	inputImgId.value = img_id;
-	inputId.value = id;
-	inputAction.value = action;
-	textarea.value = text;
-
-	//textarea.value = "dxy:"+ dx + "x" + dy + "\n" + 'id:' + img_id;
-
-	// show form
-	formdiv.style.display = 'block';
-	formdiv.style.left = dx;
-	formdiv.style.top = dy;
-	
-	img.parentNode.appendChild(formdiv);
-
-	textarea.focus();
-
-	return true;
-}
-
-function box_onmouseover(box)
-{
-	var id = box.id;
-	var pointer = document.getElementById(box.id + '-pointer');
-	pointer.classList.remove('pointer');
-	pointer.classList.add('pointer-hi');
-}
-
-function box_onmouseout(box)
-{
-	var id = box.id;
-	var pointer = document.getElementById(box.id + '-pointer');
-	pointer.classList.remove('pointer-hi');
-	pointer.classList.add('pointer');
-}
-
-function save_scroll(form)
-{
-	//alert('save_scroll:' + document.body.scrollTop);
-	form.scroll.value =  document.body.scrollTop;
-	//alert('save_scroll:' + form.scroll.value);
-	return true;
-}
-
-function toggle_corrections(aclass)
-{
-	var stylesheets = document.styleSheets;
-	var ssheet = null;
-	for (var i=0;i<stylesheets.length; i++){
-		if (stylesheets[i].title === "opraf-css"){
-			ssheet = stylesheets[i];
-			break;
-		}
-	}
-	if (! ssheet){
-		return;	
-	}
-	for (var i=0;i<ssheet.cssRules.length;i++){
-		var rule = ssheet.cssRules[i];	
-		if (rule.selectorText === '.'+aclass){
-			if (rule.style.display === ""){
-				rule.style.display = "none";
-			} else {
-				rule.style.display = "";
-			}
-		}
-	}
-	place_comments();
-		
-}
-
-
-String.prototype.unescapeHTML = function () {                                       
-        return(                                                                 
-            this.replace(/&amp;/g,'&').                                         
-                replace(/&gt;/g,'>').                                           
-                replace(/&lt;/g,'<').                                           
-                replace(/&quot;/g,'"')                                         
-        );                                                                     
-};
 

korektury/templates/korektury/korekturovatko/__edit_komentar.html

@@ -0,0 +1,64 @@
+<div id="commform-div" style="display: none">
+    <form action='' id="commform" method="POST">
+        {% csrf_token %}
+        <input size="24" name="au" value="{{user.first_name}} {{user.last_name}}" readonly/>
+        <input type=submit value="Oprav!"/>
+        <button type="button" onclick="close_commform()">Zavřít</button>
+        <br/>
+        <textarea id="commform-text" cols=40 rows=10 name="txt"></textarea>
+        <br/>
+        <input type="hidden" size="3" id="commform-x" name="x"/>
+        <input type="hidden" size="3" id="commform-y" name="y"/>
+        <input type="hidden" size="3" id="commform-img-id" name="img-id"/>
+        <input type="hidden" size="3" id="commform-id" name="id"/>
+        <input type="hidden" size="3" id="commform-action" name="action"/>
+    </form>
+</div>
+
+<script>
+    const commform = document.getElementById('commform');
+    commform._div = document.getElementById('commform-div');
+    commform._text = document.getElementById('commform-text');
+    commform._x  = document.getElementById('commform-x');
+    commform._y  = document.getElementById('commform-y');
+    commform._imgID  = document.getElementById('commform-img-id');
+    commform._id  = document.getElementById('commform-id');
+    commform._action  = document.getElementById('commform-action');
+
+  // ctrl-enter submits form
+  commform._text.addEventListener("keydown", ev => {
+      if (ev.code === "Enter" && ev.ctrlKey) commform.submit();
+  });
+
+  //hide comment  form
+  function close_commform() {
+    commform._div.style.display = 'none';
+    return false;
+  }
+
+  //fill up comment form and show him
+  function show_form(img_id, dx, dy, id, text, action) {
+    const img = document.getElementById("img-" + img_id);
+
+    if (commform._div.style.display !== 'none' && commform._text.value !== "" && !confirm("Zavřít předchozí okénko přidávání korektury / editace komentáře?")) return 1;
+
+    // set hidden values
+    commform._x.value = dx;
+    commform._y.value = dy;
+    commform._imgID.value = img_id;
+    commform._id.value = id;
+    commform._action.value = action;
+    commform._text.value = text;
+
+    // show form
+    commform._div.style.display = 'block';
+    commform._div.style.left = dx;
+    commform._div.style.top = dy;
+
+    img.parentNode.appendChild(commform._div);
+
+    commform._text.focus();
+
+    return true;
+  }
+</script>

korektury/templates/korektury/korekturovatko/__komentar.html

@@ -0,0 +1,86 @@
+{% load static %}
+
+<div class='comment' id='prekomentar' {# id='k{{k.id}}' #}>
+  <div class='corr-header'>
+    <div class='author'>{# {{k.autor}} #}</div>
+
+    <div class='float-right'>
+      <button type='button' style='display: none' class='del-comment' title='Smaž komentář'>
+        <img src='{% static "korektury/imgs/delete.png" %}' alt='del'/>
+      </button>
+
+      <button type='button' class='update-comment' title='Uprav komentář'>
+        <img src='{% static "korektury/imgs/edit.png"%}' alt='edit'/>
+      </button>
+    </div>
+
+  </div>
+
+  <div class='komtext'>{# {{k.text|linebreaks}} #}</div>
+</div>
+
+
+
+<script>
+  const prekomentar = document.getElementById('prekomentar');
+  const komentare = {};
+
+  class Komentar {
+    static update_or_create(komentar_data, oprava) {
+      const id = komentar_data['id'];
+      if (id in komentare) komentare[id].update(komentar_data);
+      else new Komentar(komentar_data, oprava);
+    }
+
+    #autor; #text;
+    htmlElement;
+    id; oprava; {# komentar_data; #}
+
+    /**
+     *
+     * @param komentar_data
+     * @param {Oprava} oprava
+     */
+    constructor(komentar_data, oprava) {
+      this.htmlElement = prekomentar.cloneNode(true);
+      this.#autor = this.htmlElement.getElementsByClassName('author')[0];
+      this.#text = this.htmlElement.getElementsByClassName('komtext')[0];
+
+      this.id = komentar_data['id'];
+      this.htmlElement.id = 'k' + this.id;
+
+      this.oprava = oprava;
+      this.oprava.add_komentar_htmlElement(this.htmlElement);
+
+      this.update(komentar_data);
+
+      this.htmlElement.getElementsByClassName('update-comment')[0].addEventListener('click', _ => this.#update_comment());
+      this.htmlElement.getElementsByClassName('del-comment')[0].addEventListener('click', _ => this.#delete_comment());
+
+      komentare[this.id] = this;
+    }
+
+    update(komentar_data) {
+      {# this.komentar_data = komentar_data; #}
+      this.set_autor(komentar_data['autor']);
+      this.set_text(komentar_data['text']);
+    };
+
+    set_autor(autor) {this.#autor.textContent=autor;};
+    set_text(text) {
+      this.#text.innerHTML=text;
+    };
+
+
+    // show comment form when 'update-comment' button pressed
+    #update_comment() {
+      return show_form(this.oprava.img_id, this.oprava.x, this.oprava.y, this.id, this.#text.textContent, 'update-comment');
+    }
+
+    #delete_comment() {
+      if (confirm('Opravdu smazat komentář?')) {
+        throw {name : 'NotImplementedError', message: '(Webaři jsou) too lazy to implement'};
+      }
+    }
+  }
+</script>

korektury/templates/korektury/korekturovatko/__oprava.html

@@ -0,0 +1,145 @@
+{% load static %}
+
+<div id='prepointer' {# id='op{{o.id}}-pointer' #}
+     class='pointer'
+     data-highlight='false'
+     {# data-opravastatus='{{o.status}}' #}
+></div>
+
+<div id='preoprava' {# name='op{{o.id}}' id='op{{o.id}}' #}
+     class='oprava'
+     {# data-opravastatus='{{o.status}}' #}
+     data-opravazobrazit='true'
+>
+  <div class='corr-body'>
+    {# {% for k in o.komentare %} {% include "korektury/korekturovatko/__komentar.html" %} <hr> {% endfor %} #}
+  </div>
+
+  <div class='corr-header'>
+    <span class='float-right'>
+      <span class='corr-buttons'>
+        <button type='button' style='display: none' class='del' title='Smaž opravu'>
+          <img src='{% static "korektury/imgs/delete.png"%}' alt='🗑️'/>
+        </button>
+        <button type='button' class='action' value='k_oprave' title='Označ jako neopravené'>
+          <img src='{% static "korektury/imgs/undo.png"%}' alt='↪'/>
+        </button>
+        <button type='button' class='action' value='opraveno' title='Označ jako opravené'>
+          <img src='{% static "korektury/imgs/check.png"%}' alt='✔️'/>
+        </button>
+        <button type='button' class='action' value='neni_chyba' title='Označ, že se nebude měnit'>
+          <img src='{% static "korektury/imgs/cross.png" %}' alt='❌'/>
+        </button>
+        <button type='button' class='action' value='k_zaneseni' title='Označ jako připraveno k zanesení'>
+          <img src='{% static "korektury/imgs/tex.png" %}' alt='TeX'/>
+        </button>
+
+        <button type='button' class='notcomment' title='Korekturu nelze komentovat, protože už je uzavřená' disabled=''>
+          <img src='{% static "korektury/imgs/comment-gr.png" %}' alt='💭'/>
+				</button>
+        <button type='button' class='comment' title='Komentovat'>
+          <img src='{% static "korektury/imgs/comment.png" %}' alt='💭'/>
+				</button>
+      </span>
+
+      <button type='button' class='toggle-vis' title='Skrýt/Zobrazit'>
+        <img class='toggle-button' src='{% static "korektury/imgs/hide.png" %}' alt='⬆'/>
+      </button>
+
+			</span>
+  </div>
+</div>
+
+<script>
+  const preoprava = document.getElementById('preoprava');
+  const prepointer = document.getElementById('prepointer');
+  const opravy = {};
+
+  class Oprava {
+    static update_or_create(oprava_data) {
+      const id = oprava_data['id'];
+      if (id in opravy) return opravy[id].update(oprava_data);
+      else return new Oprava(oprava_data);
+    }
+
+    #komentare;
+    htmlElement; pointer;
+    id; x; y; img_id; zobrazit = true; {# oprava_data; #}
+
+    constructor(oprava_data) {
+      this.htmlElement = preoprava.cloneNode(true);
+      this.pointer = prepointer.cloneNode(true);
+      this.#komentare = this.htmlElement.getElementsByClassName('corr-body')[0];
+
+      this.id = oprava_data['id'];
+      this.htmlElement.id = 'op' + this.id;
+      this.pointer.id = 'op' + this.id + '-pointer';
+
+      this.x = oprava_data['x'];
+      this.y = oprava_data['y'];
+      this.img_id = oprava_data['strana'];
+
+      this.update(oprava_data);
+
+      this.htmlElement.getElementsByClassName('toggle-vis')[0].addEventListener('click', _ => this.#toggle_visibility());
+      for (const button of this.htmlElement.getElementsByClassName('action'))
+        button.addEventListener('click', async event => this.#zmenStavKorektury(event));
+      this.htmlElement.getElementsByClassName('comment')[0].addEventListener('click', _ => this.#comment())
+      this.htmlElement.getElementsByClassName('del')[0].addEventListener('click', _ => this.#delete());
+
+      this.htmlElement.addEventListener('mouseover', _ => this.pointer.dataset.highlight = 'true');
+      this.htmlElement.addEventListener('mouseout', _ => this.pointer.dataset.highlight = 'false');
+
+      opravy[this.id] = this;
+      if (this.img_id in comments) comments[this.img_id].push(this); else alert("Někdo korekturoval stranu, která neexistuje. Dejte vědět webařům :)");
+    }
+
+    update(oprava_data) {
+      {# this.oprava_data = oprava_data; #}
+      this.set_status(oprava_data['status']);
+      return this;
+    };
+
+    set_status(status) {
+      this.htmlElement.dataset.opravastatus=status;
+      this.pointer.dataset.opravastatus=status;
+    };
+
+    add_komentar_htmlElement(htmlElement) {
+      this.#komentare.appendChild(htmlElement);
+      this.#komentare.appendChild(document.createElement('hr'));
+    }
+
+
+
+
+    // hide or show text of correction
+    #toggle_visibility(){
+      this.zobrazit = !this.zobrazit;
+      this.htmlElement.dataset.opravazobrazit = String(this.zobrazit);
+      place_comments()
+    }
+
+    // show comment form, when 'comment' button pressed
+    #comment() { return show_form(this.img_id, this.x, this.y, this.id, "", "comment"); }
+
+    #zmenStavKorektury(event) {
+      const data = new FormData(CSRF_FORM);
+      data.append('id', this.id);
+      data.append('action', event.target.value);
+
+      fetch('{% url "korektury_api_oprava_stav" %}', {method: 'POST', body: data})
+        .then(response => {
+          if (!response.ok) {alert('Něco se nepovedlo:' + response.statusText);}
+          else response.json().then(data => this.set_status(data['status']));
+        })
+        .catch(error => {alert('Něco se nepovedlo:' + error);});
+    }
+
+    #delete() {
+      if (confirm('Opravdu smazat korekturu?')) {
+        throw {name : 'NotImplementedError', message: '(Webaři jsou) too lazy to implement'};
+      }
+    }
+  }
+</script>

korektury/templates/korektury/korekturovatko/__stranky.html

@@ -0,0 +1,50 @@
+{% for i in img_indexes %}
+  <div class='imgdiv'>
+    <img
+      id='img-{{i}}'
+      width='1021' height='1448'
+      src='/media/korektury/img/{{img_prefix}}-{{i}}.png'
+      alt='Strana {{ i|add:1 }}'
+      class="strana"
+    />
+  </div>
+
+  <hr/>
+{% endfor %}
+
+<script>
+  // Mapování stránka -> korektury
+  const comments = {
+    {% for s in opravy_strany %}
+      {{s.strana}}: []{% if not forloop.last %},{% endif %}
+    {% endfor %}
+  };
+
+  // show comment form, when clicked to image
+  for (const image of document.getElementsByClassName('strana')) {
+    image.addEventListener('click', ev => {
+      switch (document.body.dataset.status) {
+        case 'zanaseni':
+          if (!confirm('Právě jsou zanášeny korektury, opravdu chcete přidat novou?'))
+            return;
+          break;
+        case 'zastarale':
+          if (!confirm('Toto PDF je již zastaralé, opravdu chcete vytvořit korekturu?'))
+            return;
+          break;
+      }
+
+      let dx, dy;
+      const par = image.parentNode;
+      if (ev.pageX != null) {
+        dx = ev.pageX - par.offsetLeft;
+        dy = ev.pageY - par.offsetTop;
+      } else { //IE a další
+        dx = ev.offsetX;
+        dy = ev.offsetY;
+      }
+      const img_id = image.id;
+      return show_form(img_id, dx, dy, '', '', '');
+    });
+  }
+</script>

korektury/templates/korektury/korekturovatko/_main.html

@@ -0,0 +1,40 @@
+{% include "korektury/korekturovatko/__edit_komentar.html" %}
+
+{% include "korektury/korekturovatko/__stranky.html" %}
+
+{# {% for o in opravy %} {% include "korektury/korekturovatko/__oprava.html" %} {% endfor %} #}
+{% include "korektury/korekturovatko/__oprava.html" %}
+{% include "korektury/korekturovatko/__komentar.html" %}
+
+<script>
+  /**
+   *
+   * @param {RequestInit} data
+   * @param {Boolean} catchError
+   */
+  function update_all(data={}, catchError=true) { // FIXME není mi jasné, zda v {} nemá být `cache: "no-store"`, aby prohlížeč necachoval GET.
+    fetch('{% url "korektury_api_opravy_a_komentare" pdf.id %}', data)
+      .then(response => {
+        if (!response.ok && catchError) {alert('Něco se nepovedlo:' + response.statusText);}
+        else response.json().then(data => {
+          for (const oprava_data of data["context"]) {
+            const oprava = Oprava.update_or_create(oprava_data);
+            for (const komentar_data of oprava_data["komentare"]) {
+              Komentar.update_or_create(komentar_data, oprava);
+            }
+          }
+
+          place_comments();
+        });
+      })
+      .catch(error => {if (catchError) alert('Něco se nepovedlo:' + error);});
+  }
+
+  update_all();
+</script>
+
+<form id='CSRF_form' style='display: none'>{% csrf_token %}</form>
+
+<script>
+  const CSRF_FORM = document.getElementById('CSRF_form');
+</script>

korektury/templates/korektury/korekturovatko/_schovani_korektur.html

@@ -0,0 +1,51 @@
+Zobrazit:
+<input type="checkbox"
+       id="k_oprave_checkbox"
+       name="k_oprave_checkbox"
+       onchange="toggle_corrections('k_oprave')" checked>
+<label for="k_oprave_checkbox">K opravě ({{k_oprave_cnt}})</label>
+<input type="checkbox"
+       id="opraveno_checkbox"
+       name="opraveno_checkbox"
+       onchange="toggle_corrections('opraveno')" checked>
+<label for="opraveno_checkbox">Opraveno ({{opraveno_cnt}})</label>
+<input type="checkbox"
+       id="neni_chyba_checkbox"
+       name="neni_chyba_checkbox"
+       onchange="toggle_corrections('neni_chyba')" checked>
+<label for="neni_chyba_checkbox">Není chyba ({{neni_chyba_cnt}})</label>
+<input type="checkbox"
+       id="k_zaneseni_checkbox"
+       name="k_zaneseni_checkbox"
+       onchange="toggle_corrections('k_zaneseni')" checked>
+<label for="k_zaneseni_checkbox">K zanesení ({{k_zaneseni_cnt}})</label>
+
+<hr/>
+
+<script>
+  function toggle_corrections(aclass)
+  {
+    const stylesheets = document.styleSheets;
+    let ssheet = null;
+    for (let i=0; i<stylesheets.length; i++){
+      if (stylesheets[i].title === "opraf-css"){
+        ssheet = stylesheets[i];
+        break;
+      }
+    }
+    if (! ssheet){
+      return;
+    }
+    for (let i=0; i<ssheet.cssRules.length; i++){
+      const rule = ssheet.cssRules[i];
+      if (rule.selectorText === '[data-opravastatus="'+aclass+'"]'){
+        if (rule.style.display === ""){
+          rule.style.display = "none";
+        } else {
+          rule.style.display = "";
+        }
+      }
+    }
+    place_comments();
+  }
+</script>

korektury/templates/korektury/korekturovatko/_zmena_stavu.html

@@ -0,0 +1,44 @@
+<h4>Změnit stav PDF:</h4>
+<i>Aktuální: {{pdf.status}}</i>
+<br>
+<form method="post" id="PDFSTAV_FORM">
+  {% csrf_token %}
+  <input type="radio" name="state" value="{{ pdf.STATUS.PRIDAVANI }}" {% if pdf.status == pdf.STATUS.PRIDAVANI %} checked {% endif %}>Přidávání korektur
+  <br>
+  <input type="radio" name="state" value="{{ pdf.STATUS.ZANASENI }}" {% if pdf.status == pdf.STATUS.ZANASENI %} checked {% endif %}>Zanášení korektur
+  <br>
+  <input type="radio" name="state" value="{{ pdf.STATUS.ZASTARALE }}" {%	if pdf.status == pdf.STATUS.ZASTARALE %} checked {% endif %}>Zastaralé, nekorigovat
+  <br>
+  <input type='submit' value='Změnit stav PDF'/>
+</form>
+
+<script>
+  const pdfstav_form = document.getElementById('PDFSTAV_FORM');
+
+  /**
+   *
+   * @param {RequestInit} data
+   * @param {Boolean} catchError
+   */
+  function fetchStav(data, catchError=true) {
+    fetch("{% url 'korektury_api_pdf_stav' pdf.id %}", data
+    )
+      .then(response => {
+        if (!response.ok) { if (catchError) alert("Něco se nepovedlo:" + response.statusText);}
+        else response.json().then(data => document.body.dataset.status = data["status"]);
+      })
+      .catch(error => {if (catchError) alert("Něco se nepovedlo:" + error);});
+  }
+
+  pdfstav_form.addEventListener('submit', async event => {
+    event.preventDefault();
+
+    const data = new FormData(pdfstav_form);
+    fetchStav({method: "POST", body: data});
+  });
+
+
+  // FIXME není mi jasné, zda v {} nemá být `cache: "no-store"`, aby prohlížeč necachoval get.
+  setInterval(() => fetchStav({}, false), 30000); // Každou půl minutu fetchni stav
+</script>
+

korektury/templates/korektury/korekturovatko/htmlstrana.html

@@ -0,0 +1,44 @@
+{% load static %}
+
+<html>
+<head>
+  <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
+  <link rel="stylesheet" title="opraf-css" type="text/css" media="screen, projection" href="{% static "korektury/opraf.css"%}" />
+  <link href="{% static 'css/rozliseni.css' %}?version=1" rel="stylesheet">
+  <script src="{% static "korektury/opraf.js"%}"></script>
+  <title>Korektury {{pdf.nazev}}</title>
+</head>
+<body class="{{ LOCAL_TEST_PROD }}web" data-status="{{ pdf.status }}" onload='place_comments()'>
+
+  <h1>Korektury {{pdf.nazev}}</h1>
+
+  <h2 class="textzanaseni"> Probíhá zanášení korektur, zvažte, zda chcete přidávat nové </h2>
+  <h2 class="textzastarale"> Toto PDF je již zastaralé, nepřidávejte nové korektury </h2>
+
+  <i>{{pdf.komentar}}</i>
+  <br>
+  <i>Klikni na chybu, napiš komentář</i>  |
+  <a href="{{pdf.pdf.url}}">stáhnout PDF (bez korektur)</a> |
+  <a href="../">seznam souborů</a> |
+  <a href="/admin/korektury/korekturovanepdf/">Spravovat PDF</a> |
+  <a href="../help">nápověda</a> |
+  &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;|
+  <a href="/">hlavní stránka</a> |
+  <a href="https://mam.mff.cuni.cz/wiki">wiki</a> |
+  <hr />
+  
+  {% include "korektury/korekturovatko/_schovani_korektur.html" %}
+  
+  {% include "korektury/korekturovatko/_main.html" %}
+
+  {% include "korektury/korekturovatko/_zmena_stavu.html" %}
+
+  <hr/>
+  <p>
+    Děkujeme opravovatelům:
+    {% for z in zasluhy %}
+      {{z.autor}} ({{z.pocet}}){% if not forloop.last %},{% endif %}
+    {% endfor %}</p>
+  <hr>
+</body>
+</html>

korektury/templates/korektury/opraf.html

@@ -1,244 +0,0 @@
-{% load static %}
-
-<html>
-<head>
-	<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
-  <link rel="stylesheet" title="opraf-css" type="text/css" media="screen, projection" href="{% static "korektury/opraf.css"%}" />
-    <link href="{% static 'css/rozliseni.css' %}?version=1" rel="stylesheet">
-  <script src="{% static "korektury/opraf.js"%}"></script>
-	<title>Korektury {{pdf.nazev}}</title>
-</head>
-<body class="{{ LOCAL_TEST_PROD }}web{% if pdf.status == 'zanaseni'%} comitting{% elif pdf.status == 'zastarale' %} deprecated{% endif %}" onload='place_comments()'>
-	<h1>Korektury {{pdf.nazev}}</h1>
-	{% if pdf.status == 'zanaseni' %} <h2> Probíhá zanášení korektur, zvažte, zda chcete přidávat nové </h2> {% endif %}
-	{% if pdf.status == 'zastarale' %} <h2> Toto PDF je již zastaralé, nepřidávejte nové korektury </h2> {% endif %}
-	<i>{{pdf.komentar}}</i>
-	<br>
-	<i>Klikni na chybu, napiš komentář</i>  |
-	<a href="{{pdf.pdf.url}}">stáhnout PDF (bez korektur)</a> |
-	<a href="../">seznam souborů</a> |
-	<a href="/admin/korektury/korekturovanepdf/">Spravovat PDF</a> |
-	<a href="../help">nápověda</a> |
-	&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;|
-	<a href="/">hlavní stránka</a> |
-	<a href="https://mam.mff.cuni.cz/wiki">wiki</a> |
-	<hr />
-	Zobrazit: 
-	<input type="checkbox" 
-		id="k_oprave_checkbox" 
-		name="k_oprave_checkbox" 
-		onchange="toggle_corrections('k_oprave')" checked>
-	<label for="k_oprave_checkbox">K opravě ({{k_oprave_cnt}})</label>
-	<input type="checkbox"
-		id="opraveno_checkbox"
-		name="opraveno_checkbox" 
-		onchange="toggle_corrections('opraveno')" checked>
-	<label for="opraveno_checkbox">Opraveno ({{opraveno_cnt}})</label>
-	<input type="checkbox"
-		id="neni_chyba_checkbox"
-		name="neni_chyba_checkbox"
-		onchange="toggle_corrections('neni_chyba')" checked>
-	<label for="neni_chyba_checkbox">Není chyba ({{neni_chyba_cnt}})</label>
-	<input type="checkbox"
-		id="k_zaneseni_checkbox"
-		name="k_zaneseni_checkbox"
-		onchange="toggle_corrections('k_zaneseni')" checked>
-	<label for="k_zaneseni_checkbox">K zanesení ({{k_zaneseni_cnt}})</label>
-
-	<hr/>
-
-	<div id="commform-div">
-		<!-- Pridat korekturu / komentar !-->
-		<form action='' onsubmit='save_scroll(this)' id="commform" method="POST">
-			{% csrf_token %}
-			<input size="24" name="au" value="{{user.first_name}} {{user.last_name}}" readonly/>
-			<input type=submit value="Oprav!"/>
-			<button type="button" onclick="close_commform()">Zavřít</button>
-	  		<br/>
-	  		<textarea onkeypress="textarea_onkey(event);" id="commform-text" cols=40 rows=10 name="txt"></textarea>
-	  		<br/>
-	  		<input type="hidden" size="3" name="pdf" value='{{pdf.id}}'/>
-	  		<input type="hidden" size="3" id="commform-x" name="x"/>
-	  		<input type="hidden" size="3" id="commform-y" name="y"/>
-	  		<input type="hidden" size="3" id="commform-img-id" name="img-id"/>
-	  		<input type="hidden" size="3" id="commform-id" name="id"/>
-	  		<input type="hidden" size="3" id="commform-action" name="action"/>
-	  		<input type="hidden" size="3" id="commform-action" name="scroll"/>
-		</form>
-		<!-- /Pridat korekturu / komentar !-->
-	</div>
-
-	{% for i in img_indexes %}
-	<div class='imgdiv'>
-		<img width='1021' height='1448' 
-			onclick='img_click(this,event)' id='img-{{i}}' 
-			src='/media/korektury/img/{{img_prefix}}-{{i}}.png'/>
-	</div>
-	<hr/>
-	{% endfor %}
-
-	<h4>Změnit stav PDF:</h4>
-	<i>Aktuální: {{pdf.status}}</i>
-	<br>
-	<!-- Zmenit stav PDF !-->
-	<form method="post">
-		{% csrf_token %}
-		<input type='hidden' name='action' value='set-state'/>
-		<input type='hidden' name='pdf' value='{{pdf.id}}'/>
-		<input type="radio" name="state" value="adding" {% if pdf.status == 'pridavani' %} checked {% endif %}>Přidávání korektur
-		<br>
-		<input type="radio" name="state" value="comitting" {% if pdf.status == 'zanaseni' %} checked {% endif %}>Zanášení korektur
-		<br>
-		<input type="radio" name="state" value="deprecated" {%	if pdf.status == 'zastarale' %} checked {% endif %}>Zastaralé, nekorigovat
-		<br>
-		<input type='submit' value='Změnit stav PDF'/>
-	</form>
-	
-	<!-- /Zmenit stav PDF !-->
-	<hr/>
-	<p>
-  Děkujeme opravovatelům:
-    {% for z in zasluhy %}
-      {{z.autor}} ({{z.pocet}}){% if not forloop.last %},{% endif %}
-    {% endfor %}</p>
-	<hr>
-
-	{% for o in opravy %}
-	<div onclick='img_click(this,event)'
-		id='op{{o.id}}-pointer'
-		class='pointer {{o.status}}'>
-	</div>
-	<div name='op{{o.id}}' id='op{{o.id}}' 
-	    class='box {{o.status}}' 
-	    onmouseover='box_onmouseover(this)' 
-	    onmouseout='box_onmouseout(this)'> 
-
-    	<div class='corr-header'> 
-			<span class='author' id='op{{o.id}}-autor'>{{o.autor}}</span>
-			<span class='float-right'>
-				<span id='op{{o.id}}-buttons'>
-				<!-- Existujici korektura !-->
-	  			<form  action='' onsubmit='save_scroll(this)' method='POST'>
-	   				{% csrf_token %}
-	   				<input type='hidden' name="au" value="{{o.autor}}"/>
-	   				<input type='hidden' name='pdf' value='{{pdf.id}}'>
-	   				<input type='hidden' name='id' value='{{o.id}}'>
-	   				<input type='hidden' name='scroll'>
-	   				
-					{% if o.komentare %}
-	   				<button name='action' value='del' type='button'
-            			title="Opravu nelze smazat &ndash; už ji někdo okomentoval">
-                  <img src="{% static "korektury/imgs/delete-gr.png"%}"/>
-	   				</button>
-					{% else %}
-					<button type='submit' name='action' value='del' title='Smaž opravu'>
-            <img src="{% static "korektury/imgs/delete.png"%}"/>
-					</button>
-					{% endif %} 
-					{% if o.status != 'k_oprave' %}
-					<button type='submit' name='action' value='undone' title='Označ jako neopravené'>
-            <img src="{% static "korektury/imgs/undo.png"%}"/>
-					</button>
-					{% endif %}
-					{% if o.status != 'opraveno' %}
-					<button type='submit' name='action' value='done' title='Označ jako opravené'>
-            <img src="{% static "korektury/imgs/check.png"%}"/>
-					</button>
-					{% endif %}
-					{% if o.status != 'neni_chyba' %} 
-					<button type='submit' name='action' value='wontfix' title='Označ, že se nebude měnit'>
-            <img src="{% static "korektury/imgs/cross.png" %}"/>
-					</button>
-					{% endif %}
-					{% if o.status != 'k_zaneseni' %} 
-					<button type='submit' name='action' value='ready' title='Označ jako připraveno k zanesení'>
-            <img src="{% static "korektury/imgs/tex.png" %}"/>
-					</button>
-					{% endif %}
-				</form>
-				<!-- /Existujici korektura !-->
-						
-				{% if o.komentare %}
-				<button type='button' title="Korekturu nelze upravit &ndash; už ji někdo okomentoval">
-          <img src="{% static "korektury/imgs/edit-gr.png" %}"/>
-				</button>
-				{% else %}
-				<button type='button' onclick='box_edit("op{{o.id}}","update");' title='Oprav opravu'>
-          <img src="{% static "korektury/imgs/edit.png" %}"/>
-				</button>
-				{% endif %}
-				{% if o.status == 'opraveno' or o.status == 'neni_chyba'  %}
-				<button type='button' title='Korekturu nelze komentovat, protože už je uzavřená'>
-          <img src="{% static "korektury/imgs/comment-gr.png" %}"/>
-				</button>
-				{% else %}
-				<button type='button' onclick='box_edit("op{{o.id}}", "comment");' title='Komentovat'>
-          <img src="{% static "korektury/imgs/comment.png" %}"/>
-				</button>
-				{% endif %}
-
-				</span>
-				<button type='button' onclick='toggle_visibility("op{{o.id}}");' title='Skrýt/Zobrazit'>
-          <img src="{% static "korektury/imgs/hide.png" %}"/>
-				</button>
-
-			</span>
-		</div>
-		<div class='corr-body' id='op{{o.id}}-body'>
-		<div id='op{{o.id}}-text'>{{o.text|linebreaks}}</div>
-
-		{% for k in o.komentare %}
-    	<hr>
-		<div class='comment' id='k{{k.id}}'>
-    		<div class='corr-header'>
-        		<div class='author'>{{k.autor}}</div>
-            	<div class="float-right">
-		<!-- Komentar !-->
-                    <form  action='' onsubmit='save_scroll(this)' method='POST'>
-						{% csrf_token %}
-                    	<input type='hidden' name='pdf' value='{{pdf.id}}'>
-						<input type='hidden' name='id' value='{{k.id}}'>
-                    	<input type='hidden' name='scroll'>
-                        {% if forloop.last %}
-                    	  <button type='submit' name='action' value='del-comment' title='Smaž komentář'
-                      	  onclick='return confirm("Opravdu smazat komentář?")'>
-                            <img src="{% static "korektury/imgs/delete.png" %}"/>
-					 	  </button>
-                        {% else %}
-                          <button name='action' value='del-comment' type='button'
-            			  title="Komentář nelze smazat &ndash; existuje novější">
-                            <img src="{% static "korektury/imgs/delete-gr.png"%}"/>
-                          </button>
-                        {% endif %}
-                    </form>
-					<!-- /Komentar !-->
-                    {% if forloop.last %}
-					  <button type='button' onclick="update_comment('op{{o.id}}','kt{{k.id}}');" title='Uprav komentář'>
-                        <img src="{% static "korektury/imgs/edit.png"%}"/>
-					  </button>
-                    {% else %}
-                      <button type='button' title="Komentář nelze upravit &ndash; existuje novější">
-                        <img src="{% static "korektury/imgs/edit-gr.png" %}"/>
-                      </button>
-                    {% endif %}
-				</div>
-			</div>
-			<div id='kt{{k.id}}'>{{k.text|linebreaks}}</div>
-		</div>
-		{% endfor %} 
-		</div>
-	</div>	
-	{% endfor %}
-
-	<script>
-	var comments = [
-	{% for s in opravy_strany %}
-		["img-{{s.strana}}", [{% for o in s.op_id %}["op{{o.id}}",{{o.x}},{{o.y}}],{% endfor %}[]]],
-	{% endfor %}
-	[]]
-	{% if scroll %}
-		window.scrollTo(0,{{scroll}});
-	{% endif %}
-	</script>
-</body>
-</html>

korektury/urls.py

@@ -1,4 +1,6 @@
 from django.urls import path
+from django.urls import include
+
 from personalni.utils import org_required
 from . import views
 
@@ -7,4 +9,6 @@ urlpatterns = [
 	path('korektury/neseskupene/', org_required(views.KorekturyAktualniListView.as_view()), name='korektury_neseskupene_list'),
 	path('korektury/zastarale/', org_required(views.KorekturyZastaraleListView.as_view()), name='korektury_stare_list'),
 	path('korektury/<int:pdf>/', org_required(views.KorekturyView.as_view()), name='korektury'),
+
+	path('korektury/api/', include('korektury.api.urls')),
 ]

korektury/utils.py

@@ -0,0 +1,53 @@
+from django.core.mail import EmailMessage
+from django.http import HttpRequest
+from django.urls import reverse
+
+from korektury.models import Komentar, Oprava
+from personalni.models import Organizator
+
+
+def send_email_notification_komentar(oprava: Oprava, autor: Organizator, request: HttpRequest):
+	''' Rozesle e-mail pri pridani komentare / opravy,
+		ktery obsahuje text vlakna opravy.
+	'''
+
+	# parametry e-mailu
+	#odkaz = "https://mam.mff.cuni.cz/korektury/{}/".format(oprava.pdf.pk)
+	odkaz = request.build_absolute_uri(reverse('korektury', kwargs={'pdf': oprava.pdf.pk}))
+	odkaz = f"{odkaz}#op{oprava.id}-pointer"
+	from_email = 'korekturovatko@mam.mff.cuni.cz'
+	subject = 'Nová korektura od {} v {}'.format(autor, oprava.pdf.nazev)
+	texty = []
+	for kom in Komentar.objects.filter(oprava=oprava):
+		texty.append((kom.autor.osoba.plne_jmeno(),kom.text))
+	optext = "\n\n\n".join([": ".join(t) for t in texty])
+	text = u"Text komentáře:\n\n{}\n\n=== Konec textu komentáře ===\n\
+				\nodkaz do korekturovátka: {}\n\
+				\nVaše korekturovátko\n".format(optext, odkaz)
+
+	# Prijemci e-mailu
+	emails = set()
+
+	# nalezeni e-mailu na autory komentaru
+	for komentar in oprava.komentar_set.all():
+		email_komentujiciho = komentar.autor.osoba.email
+		if email_komentujiciho:
+			emails.add(email_komentujiciho)
+
+	# zodpovedni orgove
+	for org in oprava.pdf.orgove.all():
+		email_zobpovedny = org.osoba.email
+		if email_zobpovedny:
+			emails.add(email_zobpovedny)
+
+	# odstran e-mail autora opravy
+	email = autor.osoba.email
+	if email:
+		emails.discard(email)
+
+	EmailMessage(
+		subject=subject,
+		body=text,
+		from_email=from_email,
+		to=list(emails),
+	).send()

korektury/views.py

@@ -2,24 +2,18 @@ from django.shortcuts import get_object_or_404, render
 from django.views import generic
 from django.conf import settings
 from django.http import HttpResponseForbidden
-from django.core.mail import EmailMessage
 from django.db.models import Count,Q
 
+from .utils import send_email_notification_komentar
 from .models import Oprava,Komentar,KorekturovanePDF, Organizator
-from .forms import OpravaForm
-
-import subprocess
-import shutil
-import os
 
 class KorekturyListView(generic.ListView):
 	model = KorekturovanePDF
-	# Nefunguje, filtry se vubec nepouziji
 	queryset = KorekturovanePDF.objects.annotate(
-		k_oprave_cnt=Count('oprava',distinct=True,filter=Q(oprava__status='k_oprave')),
+		k_oprave_cnt=Count('oprava',distinct=True,filter=Q(oprava__status=Oprava.STATUS.K_OPRAVE)),
-		opraveno_cnt=Count('oprava',distinct=True,filter=Q(oprava__status='opraveno')),
+		opraveno_cnt=Count('oprava',distinct=True,filter=Q(oprava__status=Oprava.STATUS.OPRAVENO)),
-		neni_chyba_cnt=Count('oprava',distinct=True,filter=Q(oprava__status='neni_chyba')),
+		neni_chyba_cnt=Count('oprava',distinct=True,filter=Q(oprava__status=Oprava.STATUS.NENI_CHYBA)),
-		k_zaneseni_cnt=Count('oprava',distinct=True,filter=Q(oprava__status='k_zaneseni')),
+		k_zaneseni_cnt=Count('oprava',distinct=True,filter=Q(oprava__status=Oprava.STATUS.K_ZANESENI)),
 		)
 	template_name = 'korektury/seznam.html'
 
@@ -58,13 +52,15 @@ class KorekturySeskupeneListView(KorekturyAktualniListView):
 ### Korektury
 class KorekturyView(generic.TemplateView):
 	model = Oprava
-	template_name = 'korektury/opraf.html'
+	template_name = 'korektury/korekturovatko/htmlstrana.html'
-	form_class = OpravaForm
+
+	def setup(self, request, *args, **kwargs):
+		super().setup(request, *args, **kwargs)
+		self.pdf_id = self.kwargs["pdf"]
+		self.pdf = get_object_or_404(KorekturovanePDF, id=self.pdf_id)
 
 	def post(self, request, *args, **kwargs):
-		form = self.form_class(request.POST)
 		q = request.POST
-		scroll = q.get('scroll')
 
 		# prirazeni autora podle prihlaseni
 		autor_user = request.user
@@ -73,50 +69,27 @@ class KorekturyView(generic.TemplateView):
 		if not autor:
 			return HttpResponseForbidden()
 
-		if not scroll:
-			scroll = 0
-
 		action = q.get('action')
 		if (action == ''):  # Přidej
 			x = int(q.get('x'))
 			y = int(q.get('y'))
 			text = q.get('txt')
 			strana = int(q.get('img-id')[4:])
-			pdf = KorekturovanePDF.objects.get(id=q.get('pdf'))
+			op = Oprava(x=x,y=y, strana=strana, pdf=self.pdf)
-
-			op = Oprava(x=x,y=y, autor=autor, text=text, strana=strana,pdf = pdf)
 			op.save()
-			self.send_email_notification_komentar(op,autor)
+			kom = Komentar(oprava=op,autor=autor,text=text)
+			kom.save()
+			send_email_notification_komentar(op, autor, request)
 		elif (action == 'del'):
 			id = int(q.get('id'))
 			op = Oprava.objects.get(id=id)
+			for k in Komentar.objects.filter(oprava=op):
+				k.delete()
 			op.delete()
-		elif (action == 'update'):
+		elif action in Oprava.STATUS.values:
 			id = int(q.get('id'))
 			op = Oprava.objects.get(id=id)
-			text = q.get('txt')
+			op.status = action
-			op.autor = autor
-			op.text = text
-			op.save()
-		elif (action == 'undone'):
-			id = int(q.get('id'))
-			op = Oprava.objects.get(id=id)
-			op.status = op.STATUS_K_OPRAVE
-			op.save()
-		elif (action == 'done'):
-			id = int(q.get('id'))
-			op = Oprava.objects.get(id=id)
-			op.status = op.STATUS_OPRAVENO
-			op.save()
-		elif (action == 'ready'):
-			id = int(q.get('id'))
-			op = Oprava.objects.get(id=id)
-			op.status = op.STATUS_K_ZANESENI
-			op.save()
-		elif (action == 'wontfix'):
-			id = int(q.get('id'))
-			op = Oprava.objects.get(id=id)
-			op.status = op.STATUS_NENI_CHYBA
 			op.save()
 		elif (action == 'comment'):
 			id = int(q.get('id'))
@@ -124,7 +97,7 @@ class KorekturyView(generic.TemplateView):
 			text = q.get('txt')
 			kom = Komentar(oprava=op,autor=autor,text=text)
 			kom.save()
-			self.send_email_notification_komentar(op,autor)
+			send_email_notification_komentar(op, autor, request)
 		elif (action == 'update-comment'):
 			id = int(q.get('id'))
 			kom = Komentar.objects.get(id=id)
@@ -137,85 +110,23 @@ class KorekturyView(generic.TemplateView):
 			kom = Komentar.objects.get(id=id)
 			kom.delete()
 		elif (action == 'set-state'):
-			pdf = KorekturovanePDF.objects.get(id=q.get('pdf'))
+			status = q.get('state')
-			if (q.get('state') == 'adding'):
+			assert status in KorekturovanePDF.STATUS.values
-				pdf.status = pdf.STATUS_PRIDAVANI
+			self.pdf.status = status
-			elif (q.get('state') == 'comitting'):
+			self.pdf.save()
-				pdf.status = pdf.STATUS_ZANASENI
-			elif (q.get('state') == 'deprecated'):
-				pdf.status = pdf.STATUS_ZASTARALE
-			pdf.save()
 		context = self.get_context_data()
-		context['scroll'] = scroll
 		context['autor'] = autor
-		return render(request, 'korektury/opraf.html',context)
+		return render(request, 'korektury/korekturovatko/htmlstrana.html', context)
-
-	def send_email_notification_komentar(self, oprava, autor):
-		''' Rozesle e-mail pri pridani komentare / opravy,
-			ktery obsahuje text vlakna opravy.
-		'''
-
-		# parametry e-mailu
-		#odkaz = "https://mam.mff.cuni.cz/korektury/{}/".format(oprava.pdf.pk)
-		from django.urls import reverse
-		odkaz = self.request.build_absolute_uri(reverse('korektury', kwargs={'pdf': oprava.pdf.pk}))
-		odkaz = f"{odkaz}#op{oprava.id}-pointer"
-		from_email = 'korekturovatko@mam.mff.cuni.cz'
-		subject = 'Nová korektura od {} v {}'.format(autor, oprava.pdf.nazev)
-		texty = [(oprava.autor.osoba.plne_jmeno(),oprava.text)]
-		for kom in Komentar.objects.filter(oprava=oprava):
-			texty.append((kom.autor.osoba.plne_jmeno(),kom.text))
-		optext = "\n\n\n".join([": ".join(t) for t in texty])
-		text = u"Text komentáře:\n\n{}\n\n=== Konec textu komentáře ===\n\
-				\nodkaz do korekturovátka: {}\n\
-				\nVaše korekturovátko\n".format(optext, odkaz)
-
-		# Prijemci e-mailu
-		emails = set()
-		# e-mail autora korektury
-		email = oprava.autor.osoba.email
-		if email:
-			emails.add(email)
-
-		# nalezeni e-mailu na autory komentaru
-		for komentar in oprava.komentar_set.all():
-			email_komentujiciho = komentar.autor.osoba.email
-			if email_komentujiciho:
-				emails.add(email_komentujiciho)
-
-		# zodpovedni orgove
-		for org in oprava.pdf.orgove.all():
-			email_zobpovedny = org.osoba.email
-			if email_zobpovedny:
-				emails.add(email_zobpovedny)
-
-		# odstran e-mail autora opravy
-		email = autor.osoba.email
-		if email:
-			emails.discard(email)
-
-		EmailMessage(
-			subject=subject,
-			body=text,
-			from_email=from_email,
-			to=list(emails),
-		).send()
 
 	def get_context_data(self, **kwargs):
 		context = super().get_context_data(**kwargs)
-		pdf = get_object_or_404(KorekturovanePDF, id=self.kwargs['pdf'])
+		context['pdf'] = self.pdf
-		context['pdf'] = pdf
+		context['img_prefix'] = self.pdf.get_prefix()
-		context['img_prefix'] = pdf.get_prefix()
 		context['img_path'] = settings.KOREKTURY_IMG_DIR
-		context['img_indexes'] = range(pdf.stran) 
+		context['img_indexes'] = range(self.pdf.stran)
-		context['form_oprava'] = OpravaForm()
+		opravy = Oprava.objects.filter(pdf=self.pdf_id)
-		opravy = Oprava.objects.filter(pdf=self.kwargs['pdf'])
 		zasluhy = {}
 		for o in opravy:
-			if o.autor in zasluhy:
-				zasluhy[o.autor]+=1
-			else:
-				zasluhy[o.autor]=1
 			o.komentare = o.komentar_set.all()
 			for k in o.komentare:
 				if k.autor in zasluhy:
@@ -241,6 +152,3 @@ class KorekturyView(generic.TemplateView):
 		context['zasluhy'] = zasluhy
 		return context
 
-	def form_valid(self,form):
-		return super().form_valid(form)
-

make/lib.sh

@@ -95,7 +95,7 @@ function safe_checkout_branch {
 		echo >&2 "Změna v $SCRIPT, prosím pullni manuálně"
 		exit 1
 	fi
-	git checkout "$BRANCH" --
+	git checkout "$BRANCH"
 	git pull
 	git clean -f
 }

mamweb/settings_common.py

@@ -57,7 +57,6 @@ DOBA_ODHLASENI_PRI_ZASKRTNUTI_NEODHLASOVAT = 365 * 24 * 3600  # rok
 CSRF_FAILURE_VIEW = 'various.views.csrf.csrf_error'
 
 # Modules configuration
-FORM_RENDERER = "django.forms.renderers.DjangoDivFormRenderer"
 
 AUTHENTICATION_BACKENDS = (
 	'django.contrib.auth.backends.ModelBackend',
@@ -135,6 +134,7 @@ INSTALLED_APPS = (
 	'tvorba',
 	'galerie',
 	'korektury',
+	'korektury.api',
 	'prednasky',
 	'header_fotky',
 	'various',

mamweb/static/css/layout.css

@@ -435,7 +435,6 @@ body.localweb, body.testweb, body.suprodweb {
 		height: 100%;
 		top: 0;
 		z-index: -1000;
-		opacity: 0.7;
 	}
 
 	&:before { left: 0; }

mamweb/static/css/modules.css

@@ -503,10 +503,5 @@ label[for=id_skola] {
 	font-weight: bold;
 }
 
-/* Přednášky */
-.textznalosti, .textprednasky {
-	font-style: italic;
-}
-
 /*******************/
 

mamweb/static/css/rozliseni.css

@@ -1,20 +0,0 @@
-/**** ROZLIŠENÍ MEZI LOKÁLNÍM, TESTOVACÍM A PRODUKČNÍM WEBEM ****/
-body.localweb, body.testweb, body.suprodweb {
-	&:before, &:after {
-		content: "";
-		position: fixed;
-		width: 20px;
-		height: 100%;
-		top: 0;
-		z-index: -1000;
-		opacity: 0.7;
-	}
-
-	&:before { left: 0; }
-	&:after { right: 0; }
-}
-
-body.localweb { &:before, &:after { background: greenyellow; } }
-body.testweb { &:before, &:after { background: darkorange; } }
-body.suprodweb { &:before, &:after { background: red; } }
-/****************************************************************/

mamweb/vsechno.py

@@ -1,6 +1,6 @@
 # Tento soubor slouží pouze pro shell a podobné. Nikde neimportovat v kódu!
 
-print("Naimportoval jsi `mamweb.vsechno`. Pevně věřím, že to nebylo nikde v kódu. Díky.")
+print("Naimportoval jsi `seminar.models`. Pevně věřím, že to nebylo nikde v kódu. Díky.")
 
 from galerie.models import *
 from header_fotky.models import *

odevzdavatko/models.py

@@ -65,9 +65,6 @@ class Reseni(SeminarModelBase):
 	def absolute_url(self):
 		return "https://" + str(get_current_site(None)) + self.verejne_url()
 
-	def resitel_url(self):
-		return f'https://{get_current_site(None)}{reverse_lazy("odevzdavatko_resitel_reseni", args=[self.id])}'
-
 	# má OneToOneField s:
 	# Konfera
 

odevzdavatko/templates/odevzdavatko/detail.html

@@ -191,7 +191,7 @@ Sloupce:
     </ul>
   </li>
   <li>Pokud nemáš důvod, deadline neměň. Sloupeček s deadlinem znamená, do kterého deadlinu se započítají body (nemusí se shodovat s deadlinem řešení).</li>
-  <li>Poslední sloupec je na zpětnou vazbu řešiteli, tedy (na rozdíl od Neveřejné poznámky, která je určena pro synchronizaci orgů) ji uvidí řešitelé. Změníte-li u nějakého hodnocení toto políčko, řešitel bude upozorněn emailem, pokud si tuto možnost nevypl ve svém profilu. Pohled řešitele si můžete prohlédnout <a href="{% url 'odevzdavatko_resitel_reseni' reseni.id %}">zde</a>. Pokud chcete z nějakého důvodu napsat řešitelům e-mail, klikněte na „Poslat mail všem řešitelům“.</li>
+  <li>Poslední sloupec je na zpětnou vazbu řešiteli, tedy (na rozdíl od Neveřejné poznámky, která je určena pro synchronizaci orgů) ji uvidí řešitelé. Zatím jen pasivně (nechodí e-mail). Pohled řešitele si můžete prohlédnout <a href="{% url 'odevzdavatko_resitel_reseni' reseni.id %}">zde</a>. Pokud chcete z nějakého důvodu napsat řešitelům e-mail, klikněte na „Poslat mail všem řešitelům“.</li>
 </ol>
 
 Další poznámky

odevzdavatko/views.py

@@ -222,17 +222,6 @@ class ReseniProblemuView(MultipleObjectTemplateResponseMixin, MultipleObjectMixi
 		ctx["problem_id"] = self.kwargs['problem']
 		return ctx
 
-HODNOCENI_INITIAL_DATA = [
-	"problem",
-	"body",
-	"body_celkem",
-	"body_neprepocitane",
-	"body_neprepocitane_celkem",
-	"body_max",
-	"body_neprepocitane_max",
-	"deadline_body",
-	"feedback",
-]
 ## XXX: https://docs.djangoproject.com/en/3.1/topics/class-based-views/mixins/#avoid-anything-more-complex
 class DetailReseniView(DetailView):
 	""" Náhled na řešení. Editace je v :py:class:`EditReseniView`. """
@@ -243,7 +232,18 @@ class DetailReseniView(DetailView):
 		self.reseni = get_object_or_404(Reseni, id=self.kwargs['pk'])
 		result = [] # Slovníky s klíči problem, body, deadline_body -- initial data pro f.OhodnoceniReseniFormSet
 		for hodn in Hodnoceni.objects.filter(reseni=self.reseni):
-			result.append({attr: getattr(hodn, attr) for attr in HODNOCENI_INITIAL_DATA})
+			seznam_atributu = [
+				"problem",
+				"body",
+				"body_celkem",
+				"body_neprepocitane",
+				"body_neprepocitane_celkem",
+				"body_max",
+				"body_neprepocitane_max",
+				"deadline_body",
+				"feedback",
+			]
+			result.append({attr: getattr(hodn, attr) for attr in seznam_atributu})
 		return result
 
 	def get_context_data(self, **kw):
@@ -291,9 +291,9 @@ def hodnoceniReseniView(request, pk, *args, **kwargs):
 	reseni = get_object_or_404(Reseni, pk=pk)
 	success_url = reverse('odevzdavatko_detail_reseni', kwargs={'pk': pk})
 
-	formset = f.OhodnoceniReseniFormSet(request.POST, initial=[
+	# FIXME: Použit initial i tady a nebastlit hodnocení tak nízkoúrovňově
-		{k: getattr(h, k) for k in HODNOCENI_INITIAL_DATA} for h in Hodnoceni.objects.filter(reseni=reseni)
+	# Also: https://docs.djangoproject.com/en/2.2/topics/forms/modelforms/#django.forms.ModelForm
-	])
+	formset = f.OhodnoceniReseniFormSet(request.POST)
 	poznamka_form = f.PoznamkaReseniForm(request.POST, instance=reseni)
 	# TODO: Napsat validaci formuláře a formsetu
 	if not (formset.is_valid() and poznamka_form.is_valid()):
@@ -309,9 +309,7 @@ def hodnoceniReseniView(request, pk, *args, **kwargs):
 		qs.delete()
 
 		# Vyrobíme nová podle formsetu
-		notifikace = False
 		for form in formset:
-			notifikace |= 'feedback' in form.changed_data
 			data_for_hodnoceni = form.cleaned_data
 			data_for_body = data_for_hodnoceni.copy()
 			del(data_for_hodnoceni["body_celkem"])
@@ -322,44 +320,16 @@ def hodnoceniReseniView(request, pk, *args, **kwargs):
 					**form.cleaned_data,
 					)
 			logger.info(f"Creating Hodnoceni: {hodnoceni}")
-			# FIXME následující kód má velmi vysokou šanci se rozbít, vymyslet, jak to udělat jinak
 			zmeny_bodu = [it for it in form.changed_data if it.startswith("body")]
-			if len(zmeny_bodu) != 0:
+			if len(zmeny_bodu) == 1:
-				body_nastaveny: None | tuple[str, object] = None
+				hodnoceni.__setattr__(zmeny_bodu[0], data_for_body[zmeny_bodu[0]])
-				def nastav_body(jake, na_kolik):
+			# > jedna změna je špatně, ale 4 "změny" znamenají že nebylo nic zadáno
-					nonlocal body_nastaveny
+			if len(zmeny_bodu) > 1 and len(zmeny_bodu) != 4 and len(zmeny_bodu) != 2:
-					if body_nastaveny is not None:
+				# 4 znamená vše už vyplněno a nic nezměněno, 2 znamená předvyplnili se součty a nic se nezměnilo
-						logger.warning(f"Hodnocení {hodnoceni} s id {hodnoceni.id} k řešení {reseni.id} mělo mít nastavené kromě {body_nastaveny[0]} na {body_nastaveny[1]} ještě další body: {jake} na {na_kolik}. 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
-					else:
-						body_nastaveny = (jake, na_kolik)
-						hodnoceni.__setattr__(jake, na_kolik)
-
-				for key, value in data_for_body.items():
-					if key.startswith("body") and value is not None:
-						nastav_body(key, value)
-
-				# Něco se změnilo, ale nic není nastavené = něco bylo smazáno
-				if body_nastaveny is None:
-					hodnoceni.body = None
 			hodnoceni.save()
 
-		adresati = reseni.resitele.filter(upozornovat_na_opravy_reseni=True).values_list('osoba__email', flat=True)
-		if notifikace and adresati:
-			email = EmailMessage(
-					subject='Změna hodnocení odevzdaného řešení',
-					body=f"""Milá řešitelko, milý řešiteli,
-
-došlo ke změně zpětné vazby k Tebou odevzdanému řešení. Zobrazit si ji můžeš na {reseni.resitel_url()}.
-
-Tvoji organizátoři M&M
----
-Nechceš-li tato upozornění dostávat, můžeš si to nastavit ve svém profilu.""",
-					from_email='odevzdavatko@mam.mff.cuni.cz',
-					bcc=adresati,
-					)
-			email.send()
-
 	return redirect(success_url)
 
 

personalni/forms.py

@@ -71,8 +71,6 @@ class UdajeForm(forms.Form):
 	zasilat_cislo_emailem = forms.BooleanField(label='Chci dostávat e-mailem upozornění na vydání nového čísla', required=False, initial=True)
 	spam = forms.BooleanField(label='Souhlasím se zasíláním propagačních materiálů od MFF UK', required=False)
 
-	upozornovat_na_opravy_reseni = forms.BooleanField(label='Chci dostávat emailová upozornění na změnu zpětné vazby k mým řešením', required=False, initial=True)
-
 	def clean_prezdivka_resitele(self):
 		prezdivka_resitele = self.cleaned_data.get('prezdivka_resitele')
 		if prezdivka_resitele == '':

personalni/migrations/0018_resitel_upozorneni.py

@@ -1,22 +0,0 @@
-# Generated by Django 4.2.16 on 2024-12-03 19:08
-
-from django.db import migrations, models
-
-def vypnuti_upozorneni_na_opravy_reseni(apps, schema_editor):
-    Resitel = apps.get_model('personalni', 'Resitel')
-    Resitel.objects.update(upozorneni=False)
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('personalni', '0017_odstrel_treenode_post'),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name='resitel',
-            name='upozorneni',
-            field=models.BooleanField(default=True, verbose_name='zasílat upozornění na změnu zpětné vazby k řešení emailem'),
-        ),
-        migrations.RunPython(vypnuti_upozorneni_na_opravy_reseni),
-    ]

personalni/migrations/0019_rename_upozorneni_resitel_upozornovat_na_opravy_reseni.py

@@ -1,18 +0,0 @@
-# Generated by Django 4.2.18 on 2025-01-14 19:48
-
-from django.db import migrations
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('personalni', '0018_resitel_upozorneni'),
-    ]
-
-    operations = [
-        migrations.RenameField(
-            model_name='resitel',
-            old_name='upozorneni',
-            new_name='upozornovat_na_opravy_reseni',
-        ),
-    ]

personalni/models.py

@@ -250,8 +250,6 @@ class Resitel(SeminarModelBase):
 	poznamka = models.TextField('neveřejná poznámka', blank=True,
 		help_text='Neveřejná poznámka k řešiteli (plain text)')
 
-	upozornovat_na_opravy_reseni = models.BooleanField('zasílat upozornění na změnu zpětné vazby k řešení emailem', default=True)
-
 
 	def export_row(self):
 		"Slovnik pro pouziti v AESOP exportu"

personalni/static/personalni/jak_se_dozvedeli.css

@@ -1,34 +0,0 @@
-.seznam {
-	display: flex;
-	flex-direction: column;
-	gap: 0.3em;
-}
-
-.hint {
-	border: 1px solid #ccc;
-	padding: 0.3em 1em;
-	border-radius: 5px;
-	margin-bottom: 1em;
-}
-
-.osoba {
-	display: flex;
-	justify-content: space-between;
-	gap: 0.5em;
-
-	.uno {
-		flex: 2;
-	}
-
-	.dos {
-		flex: 2;
-	}
-
-	.tres {
-		flex: 1;
-	}
-
-	.grey {
-		opacity: 0.5;
-	}
-}

personalni/templates/personalni/jak_se_dozvedeli.html

@@ -1,28 +0,0 @@
-{% extends "base.html" %}
-
-{% block custom_css %}
-{% load static %}
-<link href="{% static 'personalni/jak_se_dozvedeli.css' %}" rel="stylesheet">
-{% endblock %}
-
-
-
-{% block content %}
-<div class="seznam">
-  <div class="osoba hint">
-    <div class="uno">Jméno</div>
-    <div class="dos">Jak se dozvěděli</div>
-    <div class="tres">Datum registrace</div>
-  </div>
-
-  {% for osoba in object_list %}
-  <div class="osoba">
-    <div class="uno">{{ osoba.jmeno }} {{ osoba.prijmeni }}</div>
-    <div class="dos {% if not osoba.jak_se_dozvedeli %}grey{% endif %}">{% if osoba.jak_se_dozvedeli %} {{osoba.jak_se_dozvedeli}} {% else %} NEZADÁNO {% endif %}</div>
-    <div class="tres">{{ osoba.datum_registrace }}</div>
-  </div>
-  {% endfor %}
-</div>
-{% endblock%}
-
-

personalni/templates/personalni/udaje/udaje.html

@@ -51,7 +51,6 @@
 </h4>
 <table class="form">
   {% include "personalni/udaje/prihlaska_field.html" with field=form.zasilat_cislo_emailem %}
-  {% include "personalni/udaje/prihlaska_field.html" with field=form.upozornovat_na_opravy_reseni %}
   {% include "personalni/udaje/prihlaska_field.html" with field=form.zasilat_cislo_papirove %}
   {% include "personalni/udaje/prihlaska_field.html" with field=form.spam %}
   {% include "personalni/udaje/prihlaska_field.html" with field=form.zasilat %}

personalni/urls.py

@@ -33,11 +33,4 @@ urlpatterns = [
 		name='stari_organizatori'
 	),
 
-	# Zpřístupnění dat z "jak jste se o nás dozvěděli" pro orgy propagace
-    path(
-        'org/propagace/jak-se-dozvedeli/',
-        org_required(views.JakSeDozvedeliView.as_view()),
-        name='jak_se_dozvedeli'
-	)
-
 ]

personalni/views.py

@@ -34,7 +34,7 @@ from various.autentizace.utils import posli_reset_hesla
 
 from django.forms.models import model_to_dict
 
-from .models import Organizator, Osoba 
+from .models import Organizator
 
 
 def aktivniOrganizatori(datum=timezone.now()):
@@ -62,11 +62,6 @@ class CojemamOrganizatoriStariView(generic.ListView):
 		id__in=aktivniOrganizatori()
 	).order_by('-organizuje_do')
 
-class JakSeDozvedeliView(generic.ListView):
-	model = Osoba
-	template_name = 'personalni/jak_se_dozvedeli.html'
-	queryset = Osoba.objects.order_by('-datum_registrace')
-
 
 def obalkyView(request, resitele):
 	if len(resitele) == 0:
@@ -235,7 +230,6 @@ def resitelEditView(request):
 				resitel_edit.zasilat = fcd['zasilat']
 				resitel_edit.zasilat_cislo_emailem = fcd['zasilat_cislo_emailem']
 				resitel_edit.zasilat_cislo_papirove = fcd['zasilat_cislo_papirove']
-				resitel_edit.upozornovat_na_opravy_reseni = fcd['upozornovat_na_opravy_reseni']
 				if fcd.get('skola'):
 					resitel_edit.skola = fcd['skola']
 				else:

prednasky/__init__.py

@@ -1,3 +0,0 @@
-"""
-Aplikace umožňující orgům vypisovat si přednášky a účastníkům o nich hlasovat.
-"""

prednasky/admin.py

@@ -4,15 +4,11 @@ from reversion.admin import VersionAdmin
 from django.utils.safestring import mark_safe
 from django.utils.html import escape
 
-from .models import Prednaska, Seznam, Znalost
+from .models import Prednaska, Seznam, STAV_NAVRH
 from soustredeni.models import Soustredeni
 
 
 class Seznam_PrednaskaInline(admin.TabularInline):
-	"""
-		Pomůcka pro :py:class:`prednasky.admin.SeznamAdmin` zobrazující hezky :py:class:`Přednášky <prednasky.models.Prednaska>`
-		v adminu :py:class:`Seznamu <prednasky.models.Seznam>`.
-	"""
 	model = Prednaska.seznamy.through
 	extra = 0
 
@@ -58,57 +54,24 @@ class Seznam_PrednaskaInline(admin.TabularInline):
 	def has_add_permission(self, req, obj): return False
 
 
-class Seznam_ZnalostInline(admin.TabularInline):
-	"""
-		Pomůcka pro :py:class:`prednasky.admin.SeznamAdmin` zobrazující hezky :py:class:`Znalosti <prednasky.models.Znalost>`
-		v adminu :py:class:`Seznamu <prednasky.models.Seznam>`.
-	"""
-	model = Znalost.seznamy.through
-	extra = 0
-
-	def znalost__nazev(self, obj):
-		return mark_safe(
-			f"<a href='/admin/prednasky/znalost/{obj.znalost.id}'>{obj.znalost.nazev}</a>"
-		)
-
-	def znalost__text(self, obj):
-		return mark_safe(
-			f"<div style='width: 200px'>{escape(obj.znalost.text)}</div>"
-		)
-
-	znalost__nazev.short_description = u'Přednáška'
-	znalost__text.short_description = u'Popis pro orgy'
-
-	readonly_fields = [
-		'znalost__nazev',
-		'znalost__text',
-	]
-	exclude = ['znalost']
-
-	def has_add_permission(self, req, obj): return False
-
-
 class SeznamAdmin(VersionAdmin):
-	""" Admin pro :py:class:`Seznam <prednasky.models.Seznam>` """
 	list_display = ['soustredeni', 'stav']
-	inlines = [Seznam_PrednaskaInline, Seznam_ZnalostInline]
+	inlines = [Seznam_PrednaskaInline]
 
 admin.site.register(Seznam, SeznamAdmin)
 
 
 class PrednaskaAdmin(VersionAdmin):
-	""" Admin pro :py:class:`Přednášku <prednasky.models.Prednaska> """
 	list_display = ['nazev', 'org', 'obor']
 	list_filter = ['org', 'obor']
-	search_fields = ['nazev']
+	search_fields = []
 	filter_horizontal = ('seznamy', )
 
 	actions = ['move_to_soustredeni']
 
 	def move_to_soustredeni(self, request, queryset):
-		""" Přidá dané přednášky do seznamu, o kterém se právě hlasuje """
 		sous = Soustredeni.objects.first()
-		seznam = Seznam.objects.filter(soustredeni=sous, stav=Seznam.Stav.NAVRH)
+		seznam = Seznam.objects.filter(soustredeni=sous, stav=STAV_NAVRH)
 		if len(seznam) == 0:
 			self.message_user(
 				request,
@@ -134,14 +97,3 @@ class PrednaskaAdmin(VersionAdmin):
 
 
 admin.site.register(Prednaska, PrednaskaAdmin)
-
-
-class ZnalostAdmin(PrednaskaAdmin): # Trochu hack, ať nemusím vypisovat všechno znovu
-	"""
-		Admin pro :py:class:`Znalost <prednasky.models.Znalost>
-		TODO předělat, aby nedědila z :py:class:`prednasky.admin.PrednaskaAdmin`, ale společné věci byly zvlášť
-	"""
-	list_display = ("__str__",)
-	list_filter = ()
-
-admin.site.register(Znalost, ZnalostAdmin)

prednasky/forms.py

@@ -1,31 +1,7 @@
 from django import forms
 
-from .models import Hlasovani, HlasovaniOZnalostech
+class NewPrednaskyForm(forms.Form):
+	ucastnik = forms.CharField(label = 'Tvoje jméno', max_length = 100)
 
-class HlasovaniPrednaskaForm(forms.Form):
-	""" :py:class:`Formulář <django.forms.Form>` pro pro :py:class:`Hlasování <prednasky.models.Hlasovani>` o jedné :py:class:`Přednášce <prednasky.models.Prednaska>`
-	(neobsahuje téměř nic, většina se musí doplnit jiným způsobem)
-	"""
 
-	#: ID :py:class:`Přednášky <prednasky.models.Prednaska>`, o které se hlasuje
-	prednaska_id = forms.IntegerField(widget=forms.HiddenInput)
-	#: :py:class:`Hodnocení (Body) <prednasky.models.Hlasovani.Body>` této přednášky
-	body = forms.ChoiceField(label=False, widget=forms.RadioSelect, choices=Hlasovani.Body.choices, initial=Hlasovani.Body.JEDNO)
 
-#: Množina formulářů (:py:class:`formset <django.forms.formsets.BaseFormSet>` :py:class:`HlasovaniPrednaskaFormů <prednasky.forms.HlasovaniPrednaskaForm>`)
-#: pro :py:class:`Hlasování <prednasky.models.Hlasovani>` o množině :py:class:`Přednášek <prednasky.models.Prednaska>`
-HlasovaniPrednaskaFormSet = forms.formset_factory(HlasovaniPrednaskaForm, extra=0)
-
-class HlasovaniZnalostiForm(forms.Form):
-	""" :py:class:`Formulář <django.forms.Form>` pro pro :py:class:`HlasováníOZnalostech <prednasky.models.HlasovaniOZnalostech>` o jedné :py:class:`Znalosti <prednasky.models.Znalost>`
-	(neobsahuje téměř nic, většina se musí doplnit jiným způsobem)
-	"""
-
-	#: ID :py:class:`Znalosti <prednasky.models.Znalost>`, o které hlasujeme
-	znalost_id = forms.IntegerField(widget=forms.HiddenInput)
-	#: :py:class:`Odpověď <prednasky.models.HlasovaniOZnalostech.Odpoved>` na tuto znalost
-	odpoved = forms.ChoiceField(label=False, widget=forms.RadioSelect, choices=HlasovaniOZnalostech.Odpoved.choices)
-
-#: Množina formulářů (:py:class:`formset <django.forms.formsets.BaseFormSet>` :py:class:`HlasovaniZnalostiFormů <prednasky.forms.HlasovaniZnalostiForm>`)
-#: pro :py:class:`HlasováníOZnalostech <prednasky.models.HlasovaniOZnalostech>` o množině :py:class:`Znalostí <prednasky.models.Znalost>`
-HlasovaniZnalostiFormSet = forms.formset_factory(HlasovaniZnalostiForm, extra=0)

prednasky/migrations/0019_znalost_hlasovanioznalostech.py

@@ -1,39 +0,0 @@
-# Generated by Django 4.2.16 on 2025-01-24 13:41
-
-from django.db import migrations, models
-import django.db.models.deletion
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('personalni', '0019_rename_upozorneni_resitel_upozornovat_na_opravy_reseni'),
-        ('prednasky', '0018_post_split_soustredeni'),
-    ]
-
-    operations = [
-        migrations.CreateModel(
-            name='Znalost',
-            fields=[
-                ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
-                ('nazev', models.CharField(help_text='Např. Neuronové sítě', max_length=200, verbose_name='Nadpis')),
-                ('text', models.TextField(blank=True, help_text='Např. Perceptron, vrstevnatá síť, forward a backward propagation', null=True, verbose_name='Detailní popis')),
-                ('seznamy', models.ManyToManyField(to='prednasky.seznam')),
-            ],
-            options={
-                'verbose_name': 'Znalost k přednáškám',
-                'verbose_name_plural': 'Znalosti k přednáškám',
-                'db_table': 'prednasky_znalost',
-            },
-        ),
-        migrations.CreateModel(
-            name='HlasovaniOZnalostech',
-            fields=[
-                ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
-                ('odpoved', models.CharField(choices=[(-1, 'Tohle celkem umím'), (0, 'Už jsem o tom slyšel, ale neřekl bychm, že to úplně umím'), (1, 'Tohle vůbec neznám')], max_length=16, verbose_name='odpověď')),
-                ('seznam', models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.SET_NULL, to='prednasky.seznam')),
-                ('ucastnik', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to='personalni.osoba')),
-                ('znalost', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to='prednasky.znalost')),
-            ],
-        ),
-    ]

prednasky/migrations/0020_alter_hlasovani_body.py

@@ -1,18 +0,0 @@
-# Generated by Django 4.2.16 on 2025-01-24 20:04
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('prednasky', '0019_znalost_hlasovanioznalostech'),
-    ]
-
-    operations = [
-        migrations.AlterField(
-            model_name='hlasovani',
-            name='body',
-            field=models.IntegerField(choices=[(-1, 'rozhodně nechci'), (0, 'je mi to jedno'), (1, 'rozhodně chci')], default=0, verbose_name='Body'),
-        ),
-    ]

prednasky/migrations/0021_alter_hlasovanioznalostech_odpoved.py

@@ -1,24 +0,0 @@
-# Generated by Django 4.2.16 on 2025-02-04 20:09
-
-from django.db import migrations, models
-
-def zmena_bodu(apps, _schema_editor):
-    HlasovaniOZnalostech = apps.get_model('prednasky','HlasovaniOZnalostech')
-    for h in HlasovaniOZnalostech.objects.all():
-        h.odpoved = -int(h.odpoved)
-        h.save()
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('prednasky', '0020_alter_hlasovani_body'),
-    ]
-
-    operations = [
-        migrations.AlterField(
-            model_name='hlasovanioznalostech',
-            name='odpoved',
-            field=models.IntegerField(choices=[(1, 'Tohle celkem umím'), (0, 'Už jsem o tom slyšel, ale neřekl bychm, že to úplně umím'), (-1, 'Tohle vůbec neznám')], verbose_name='odpověď'),
-        ),
-        migrations.RunPython(zmena_bodu, reverse_code=zmena_bodu),
-    ]

prednasky/models.py

@@ -1,134 +1,81 @@
 from django.db import models
 
 from soustredeni.models import Soustredeni
-from personalni.models import Organizator, Osoba
+from personalni.models import Organizator
+
+STAV_NAVRH = 1
+STAV_BUDE = 2
+
+
+STAV_CHOICES = (
+(STAV_NAVRH, 'Návrh'),
+(STAV_BUDE, 'Bude')
+)
 
 
 class Seznam(models.Model):
-	"""
-		Spojuje :py:class:`Přednášky <prednasky.models.Prednaska>`
-		se :py:class:`Soustředěními <soustredeni.models.Soustredeni>`,
-		kde by mohly zaznít, nebo zazní/zazněly.
-	"""
-
 	class Meta:
-		db_table = "prednasky_seznam"
+		db_table = 'prednasky_seznam'
-		verbose_name = "Seznam přednášek"
+		verbose_name = 'Seznam přednášek'
-		verbose_name_plural = "Seznamy přednášek"
+		verbose_name_plural = 'Seznamy přednášek'
-		ordering = ["soustredeni", "stav"]
+		ordering = ['soustredeni', 'stav']
 
-	class Stav(models.IntegerChoices):
+	id = models.AutoField(primary_key = True) 
-		""" Stav seznamu přednášek (NAVRH se používá k hlasování viz :py:func:`daný view <prednasky.views.newPrednaska>`). """
+	soustredeni = models.ForeignKey(Soustredeni,null = True, default = None, 
-		NAVRH = 1, "Návrh"
+		on_delete=models.PROTECT)
-		BUDE = 2, "Bude"
+	stav = models.IntegerField('Stav',choices=STAV_CHOICES,default = STAV_NAVRH)
-
-	id = models.AutoField(primary_key=True)
-	soustredeni = models.ForeignKey(Soustredeni, null=True, default=None, on_delete=models.PROTECT)
-	stav = models.IntegerField("Stav", choices=Stav.choices, default=Stav.NAVRH) #: :py:class:`Stav <prednasky.models.Seznam.Stav>` Seznamu
 
 	def __str__(self):
-		return f"Seznam {'návrhů ' if self.stav == Seznam.Stav.NAVRH else ''}přednášek na {self.soustredeni}"
+			return "Seznam {}přednášek na {}".format("návrhů " 
+					if self.stav == STAV_NAVRH else "", self.soustredeni)
 
 
+CHOICES_OBTIZNOST = (
+				(1, 'Lehká'),
+				(2, 'Střední'),
+				(3, 'Těžká'),
+				)
+
+CHOICES_BODY = (
+		(-1, '-1'),
+		(0, '0'),
+		(1, '1'),
+		)
+
 class Prednaska(models.Model):
-	"""
-		Reprezentuje přednášku, kterou si org může vypsat a účastník o ní hlasovat.
-		(Viz :py:class:`Hlasování <prednasky.models.Hlasovani>`.)
-	"""
 	class Meta:
-		db_table = "prednasky_prednaska"
+		db_table = 'prednasky_prednaska'
-		verbose_name = "Přednáška"
+		verbose_name = 'Přednáška'
-		verbose_name_plural = "Přednášky"
+		verbose_name_plural = 'Přednášky'
-		ordering = ["org", "nazev"]
+		ordering = ['org', 'nazev']
 
-	class Obtiznost(models.IntegerChoices):
+	id = models.AutoField(primary_key = True) 
-		LEHKA = 1, "Lehká"
+	nazev = models.CharField('Název', max_length = 300)
-		STREDNI = 2, "Střední"
-		TEZKA = 3, "Těžká"
-
-	id = models.AutoField(primary_key=True)
-	nazev = models.CharField("Název", max_length=300)
 	org = models.ForeignKey(Organizator, on_delete=models.PROTECT) 
-	popis = models.TextField("Popis pro orgy", null=True, blank=True, help_text="Neveřejný popis pro ostatní orgy")
+	popis = models.TextField('Popis pro orgy',null = True, blank = True,help_text = 'Neveřejný popis pro ostatní orgy')
-	anotace = models.TextField("Anotace", null=True, blank=True, help_text="Veřejná anotace v hlasování")
+	anotace = models.TextField('Anotace',null = True, blank = True, help_text = 'Veřejná anotace v hlasování')
-	obtiznost = models.IntegerField("Obtížnost", choices=Obtiznost.choices) #: :py:class:`Obtížnost <prednasky.models.Prednaska.Obtiznost>` Přednášky
+	obtiznost = models.IntegerField('Obtížnost', choices=CHOICES_OBTIZNOST)
-	obor = models.CharField("Obor", max_length=5, help_text="Podmnožina MFIOB")
+	obor = models.CharField('Obor', max_length = 5, help_text = 'Podmnožina MFIOB')
-	klicova = models.CharField("Klíčová slova", max_length=200, null=True, blank=True)
+	klicova = models.CharField('Klíčová slova', max_length = 200, null = True, blank = True)
 	seznamy = models.ManyToManyField(Seznam)
 
 	def __str__(self):
-		return f"{self.nazev} ({self.org})"
+		return "{} ({})".format(self.nazev, self.org)
 
 
 class Hlasovani(models.Model):
-	"""
-		Reprezentuje hlasování jednoho účastníka
-		o jedné :py:class:`Přednášce <prednasky.models.Prednaska>`
-		v jednom :py:class:`Seznamu <prednasky.models.Seznam>` (účastníkův pohled se totiž mezi sousy změnit)
-	"""
 	class Meta:
-		db_table = "prednasky_hlasovani"
+		db_table = 'prednasky_hlasovani'
-		verbose_name = "Hlasování"
+		verbose_name = 'Hlasování'
-		verbose_name_plural = "Hlasování"
+		verbose_name_plural = 'Hlasování'
-		ordering = ["ucastnik", "prednaska"]
+		ordering = ['ucastnik', 'prednaska']
-
+	id = models.AutoField(primary_key = True) 
-	class Body(models.IntegerChoices):
-		""" Ohodnocení přednášky v daném Hlasování (větší číslo = víc chci) """
-		NECHCI = -1, "rozhodně nechci"
-		JEDNO = 0, "je mi to jedno"
-		CHCI = 1, "rozhodně chci"
-
-	id = models.AutoField(primary_key=True)
 	prednaska = models.ForeignKey(Prednaska, on_delete=models.CASCADE)
-	#: Příslušné hlasování: :py:class:`Body <prednasky.models.Hlasovani.Body>`
+	body = models.IntegerField('Body', default = 0, choices = CHOICES_BODY)
-	body = models.IntegerField("Body", default=Body.JEDNO, choices=Body.choices)
+	ucastnik = models.CharField('Účastník', max_length = 100)
-
+	seznam = models.ForeignKey(Seznam,null=True,on_delete=models.SET_NULL)
-	#: Účastník, který hlasoval. Pouze string:
-	#: *(přechod z jména na objekt Osoby nějak kape na tom,
-	#: že všechna předchozí hlasování zde mají náhodný string…)
-	#: TODO Změnit to na Osobu*
-	ucastnik = models.CharField("Účastník", max_length=100)
-	seznam = models.ForeignKey(Seznam, null=True, on_delete=models.SET_NULL)
 
 	def __str__(self):
-		return f"{self.ucastnik} dal {self.body} bodů {self.prednaska} v seznamu {self.seznam}"
+		return "{} dal {} bodů {} v seznamu {}".format(self.ucastnik, 
-
+					self.body, self.prednaska, self.seznam)
-
-class Znalost(models.Model):
-	"""
-		Reprezentuje znalost, na kterou se můžeme účastníka ptát (nechat je hlasovat).
-		(Viz :py:class:`HlasováníOZnalostech <prednasky.models.HlasovaniOZnalostech>`.)
-	"""
-	class Meta:
-		db_table = "prednasky_znalost"
-		verbose_name = "Znalost k přednáškám"
-		verbose_name_plural = "Znalosti k přednáškám"
-
-	nazev = models.CharField("Nadpis", max_length=200, blank=False, null=False, help_text="Např. Neuronové sítě")
-	text = models.TextField("Detailní popis", blank=True, null=True, help_text="Např. Perceptron, vrstevnatá síť, forward a backward propagation")
-	seznamy = models.ManyToManyField(Seznam)
-
-	def __str__(self):
-		return self.nazev
-
-
-class HlasovaniOZnalostech(models.Model):
-	"""
-		Reprezentuje hlasování jednoho účastníka
-		o jedné :py:class:`Znalosti <prednasky.models.Znalost>`
-		v jednom :py:class:`Seznamu <prednasky.models.Seznam>` (účastníkův pohled se totiž mezi sousy změnit)
-	"""
-	class Odpoved(models.IntegerChoices):
-		""" Na kolik danou znalost účastník ovládá v daném Hlasování (větší číslo = víc zná) """
-		UMIM = 1, "Tohle celkem umím"
-		CIRCA = 0, "Už jsem o tom slyšel, ale neřekl bychm, že to úplně umím"
-		NEUMIM = -1, "Tohle vůbec neznám"
-
-	odpoved = models.IntegerField(u"odpověď", choices=Odpoved.choices, blank=False, null=False) #: :py:class:`Odpověď <prednasky.models.Prednaska.Odpoved>` na HlasováníOZnalostech
-	znalost = models.ForeignKey(Znalost, on_delete=models.CASCADE, blank=False, null=False)
-	ucastnik = models.ForeignKey(Osoba, on_delete=models.CASCADE, blank=False, null=False)
-	seznam = models.ForeignKey(Seznam, on_delete=models.SET_NULL, blank=True, null=True)
-
-	def __str__(self):
-		return f"{self.ucastnik} dal {self.znalost} bodů {self.znalost} v seznamu {self.seznam}"
 

prednasky/templates/prednasky/base.html

@@ -5,36 +5,34 @@
 
 
 {% block content %}
-<h1>{% block nadpis1a %}Hlasování o přednáškách{% endblock %}</h1>
+<h1>
+{% block nadpis1a %}Hlasování o přednáškách{% endblock %}
+</h1>
+
+<p>
+Jak moc by ses chtěl(a) zúčastnit následujících přednášek?
+<br>
+<span style="font-size: 75%">Obtížnost 1 je nejlehčí, 3 nejtěžší.</span>
+</p>
 
 <form enctype="multipart/form-data" action="." method="post">
   {% csrf_token %}
-
+  <table>
-<h3>Jak moc by ses chtěl(a) zúčastnit následujících přednášek?</h3>
+    {% for p, h in prednasky %}
-<p>Obtížnost 1 je nejlehčí, 3 nejtěžší.</p>
+    <tr><td><label>{{p.org}}: <span style="font-size: 175%">{{p.nazev}}</span></label></td></tr>
-  {{ form_set_prednasky.management_form }}
+    <tr><td><p><i>{{p.anotace}}</i></p></td></tr>
-  {% for f, p in formy_a_prednasky %}
+    <tr><td><label>Obor: </label> {{p.obor}}</td></tr>
-    <h4>{{p.nazev}} ({{p.org}})</h4>
+    <tr><td><label>Obtížnost: </label> {{p.obtiznost}}</td>   </tr>
-    <p class="textprednasky">{{p.anotace}}</p>
+    {% if p.klicova %}<tr><td><label>Klíčová slova: </label> {{p.klicova}}</td></tr>{% endif%}
-    <label>Obor: </label> {{p.obor}}<br>
+    <tr><td>Hodnocení:
-    <label>Obtížnost: </label> {{p.obtiznost}}<br>
+        <INPUT TYPE="radio" NAME="q{{p.pk}}" VALUE="-1" {% if h == -1 %} CHECKED="checked" {% endif %} > rozhodně nechci
-    {% if p.klicova %}<label>Klíčová slova: </label> {{p.klicova}}<br>{% endif%}
+        <INPUT TYPE="radio" NAME="q{{p.pk}}" VALUE="0" {% if h == 0 %} CHECKED="checked" {% endif %}> je mi to jedno
-    <br>
+        <INPUT TYPE="radio" NAME="q{{p.pk}}" VALUE="1" {% if h == 1 %} CHECKED="checked" {% endif %}> rozhodně chci
-    {{ f }}
+    </td></tr>
-    <br>
+    <tr><td>&nbsp;</td></tr>
-  {% empty %}
-    Nejsou žádné přednášky o kterých by šlo hlasovat.
     {% endfor %}
-
+    <tr><td><input name="odeslat" type="submit" value="Odeslat"></td><tr>
-  {{ form_set_znalosti.management_form }}
+  </table>
-  {% for f, z in formy_a_znalosti %}
-    {% if forloop.first %}<hr/><h3>Jak moc znáš následující?</h3>{% endif %}
-    <h4>{{z.nazev}}</h4>
-    <p class="textznalosti">{{z.text}}</p>
-    {{ f }}
-    <br>
-  {% endfor %}
-  <input type="submit" value="Odeslat"/>
 </form>
    
 {% endblock %}

prednasky/templates/prednasky/hotovo.html

@@ -0,0 +1,11 @@
+{% extends 'base.html' %}
+
+{% load humanize %}
+{% load static %}
+
+
+{% block content %}
+
+<h1> Děkujeme. </h1>
+   
+{% endblock %}

prednasky/templates/prednasky/metaseznam_prednasek.html

@@ -14,7 +14,7 @@
     {% else %}
         <a href="/prednasky/seznam_prednasek/{{seznam.id}}">Seznam přednášek na soustředění {{seznam.soustredeni.misto}} </a>
     {% endif %}
-    <a href="/prednasky/seznam_prednasek/{{seznam.id}}/hlasovani.csv">Export</a>
+    <a href="/prednasky/seznam_prednasek/{{seznam.id}}/export">Export</a>
     </li>
   {% endfor %}
   </ul>

prednasky/urls.py

@@ -12,15 +12,10 @@ urlpatterns = [
 		'prednasky/metaseznam_prednasek',
 		org_required(views.MetaSeznamListView.as_view()),
 		name='metaseznam-list'),
-	# path(
-	# 	'prednasky/seznam_prednasek/<int:seznam>/export',
-	# 	org_required(views.SeznamExportView),
-	# 	name='seznam-export'
-	# ),
 	path(
-		'prednasky/seznam_prednasek/<int:seznam>/hlasovani.csv',
+		'prednasky/seznam_prednasek/<int:seznam>/export',
-		org_required(views.PrednaskyExportView),
+		org_required(views.SeznamExportView),
-		name='seznam-export-csv'
+		name='seznam-export'
 	),
 	path(
 		'prednasky/seznam_prednasek/<int:seznam>/',

prednasky/views.py

@@ -1,142 +1,67 @@
-import csv
-import http
-import logging
-
-from django.http import HttpResponse, HttpRequest
 from django.shortcuts import render, get_object_or_404
 from django.views import generic
 from django.shortcuts import HttpResponseRedirect
 from django.core.exceptions import ObjectDoesNotExist
-from django.db import transaction
+from django.db.models import Sum
+from django.forms import Form
 
-from various.views.pomocne import formularOKView
+from prednasky.models import Prednaska, Hlasovani, Seznam, STAV_NAVRH
-from .forms import HlasovaniPrednaskaFormSet, HlasovaniZnalostiFormSet
-
-from various.models import Nastaveni
-from prednasky.models import Prednaska, Hlasovani, Znalost, HlasovaniOZnalostech, Seznam
 from soustredeni.models import Soustredeni
 from personalni.models import Osoba
 
-PREDNASKY_PREFIX = "prednasky"
+def newPrednaska(request):
-ZNALOSTI_PREFIX = "znalosti"
-
-logger = logging.getLogger(__name__)
-
-def newPrednaska(request: HttpRequest) -> HttpResponse:
-	"""
-		View zobrazující a ukládající účastnické hlasování
-		(:py:class:`Hlasování <prednasky.models.Hlasovani>`
-		a :py:class:`HlasováníOZnalostech <prednasky.models.HlasovaniOZnalostech>`)
-		o :py:class:`Přednáškách <prednasky.models.Prednaska>`
-		a :py:class:`Znalostech <prednasky.models.Znalost>`
-	"""
 	# hlasovani se vztahuje k nejnovejsimu soustredeni
-	sous = Nastaveni.get_solo().aktualni_sous
+	sous = Soustredeni.objects.first()
-	seznam = Seznam.objects.filter(soustredeni = sous, stav=Seznam.Stav.NAVRH).first()
+	seznam = Seznam.objects.filter(soustredeni = sous, stav = STAV_NAVRH).first()
-	if sous is None or seznam is None:
-		return render(request, 'universal.html', {
-			'title': "Nelze hlasovat",
-			'text': "Není žádný seznam přednášek, o kterém by se dalo hlasovat.",
-		}, status=http.HTTPStatus.NOT_FOUND)
-
 	osoba = Osoba.objects.filter(user=request.user).first()
-	ucastnik = osoba.plne_jmeno() + ' ' + str(osoba.id) # id, kvůli kolizi jmen
+	ucastnik = osoba.plne_jmeno() + ' ' + str(osoba.id)
+	# obsluha formulare
+	if request.method == 'POST':
+		form = Form(request.POST, request.FILES)
+		if form.is_valid():
+			# id z důvodu duplicitních jmen (přechod z jména na objekt Osoby nějak kape na tom,
+			# že všechna předchozí hlasování zde mají náhodný string…)
+			# TODO Změnit to na Osobu
 
-	if request.method == 'POST': # Když to byl POST, tak ukládáme.
+			# TODO v následujících řádcích je zbytečně mnoho dotazů na QuerySet (pokud účastník hlasoval, hlasoval u všech)
-		# Načteme data do formsetů
+			for i in request.POST:
-		form_set_prednasky = HlasovaniPrednaskaFormSet(request.POST, prefix=PREDNASKY_PREFIX)
+				if i[0] == 'q':
-		form_set_znalosti = HlasovaniZnalostiFormSet(request.POST, prefix=ZNALOSTI_PREFIX)
+					prednaska = Prednaska.objects.filter(pk=int(i[1:]))[0]
-
+					hlasovani = Hlasovani.objects.filter(ucastnik=ucastnik, prednaska=prednaska).first()
-		if form_set_prednasky.is_valid() and form_set_znalosti.is_valid():
+					if not hlasovani:
-			with transaction.atomic():
+						hlasovani = Hlasovani()
-				# Místo updatování data prostě smažeme a vytvoříme nová
+						hlasovani.prednaska = prednaska
-				seznam.hlasovani_set.filter(ucastnik=ucastnik).delete()
+						hlasovani.ucastnik = ucastnik
-				seznam.hlasovanioznalostech_set.filter(ucastnik=osoba).delete()
+						hlasovani.seznam = seznam
-
+					hlasovani.body = int(request.POST[i])
-				for form in form_set_prednasky:
+					hlasovani.save()
-					prednaska_id = form.cleaned_data['prednaska_id']
-					prednaska = Prednaska.objects.filter(id=prednaska_id).first()
-					if prednaska is None:
-						logger.error(f"Účastník {ucastnik} hodnotil neexistující přednášku {prednaska_id} číslem {form.cleaned_data['body']}")
-						continue
-
-					Hlasovani.objects.create(
-						prednaska=prednaska,
-						body=form.cleaned_data['body'],
-						ucastnik=ucastnik,
-						seznam=seznam,
-					)
-
-				for form in form_set_znalosti:
-					znalost_id = form.cleaned_data['znalost_id']
-					znalost = Znalost.objects.filter(id=znalost_id).first()
-					if znalost is None:
-						logger.error(f"Účastník {ucastnik} hodnotil neexistující znalost {znalost_id} číslem {form.cleaned_data['odpoved']}")
-						continue
-
-					HlasovaniOZnalostech.objects.create(
-						odpoved=form.cleaned_data['odpoved'],
-						znalost=znalost,
-						ucastnik=osoba,
-						seznam=seznam,
-					)
 
+			# presmerovani na prave vzniklou galerii
 			return HttpResponseRedirect('./hotovo')
 
-		else: # Pokud je nějaký formset nevalidní, vracíme je k přepracování
+	def prednaska_hodnoceni(prednaska):
-			prednasky = seznam.prednaska_set.all()
+		h = Hlasovani.objects.filter(ucastnik=ucastnik, prednaska=prednaska).first()
-			znalosti = seznam.znalost_set.all()
+		if h:
-			# FIXME Spadnout, pokud nesedí přednáška/znalost s formulářem. (Nějak se mi to nepovedlo.)
+			return prednaska, h.body
-			# Může se totiž stát, že se mezitím změnily přednášky (nějaká byla přidána/odebrána)
+		else:
+			return prednaska, 0
 
-	else: # Když to nebyl POST, tak inicializujeme (pokud už o přednášce/znalosti účastník hlasoval, předvyplníme mu to).
-		def odpoved_prednasky(p: Prednaska) -> Hlasovani.Body:
-			hlasovani = p.hlasovani_set.filter(ucastnik=ucastnik).first()
-			return hlasovani.body if hlasovani else Hlasovani.Body.JEDNO
-
-		def odpoved_znalosti(z: Znalost) -> HlasovaniOZnalostech.Odpoved:
-			hlasovani = z.hlasovanioznalostech_set.filter(ucastnik=osoba).first()
-			return hlasovani.odpoved if hlasovani else HlasovaniOZnalostech.Odpoved.CIRCA
-
-		prednasky = seznam.prednaska_set.all()
-		znalosti = seznam.znalost_set.all()
-
-		form_set_prednasky = HlasovaniPrednaskaFormSet(initial=[
-			{"prednaska_id": p.id, "body": odpoved_prednasky(p)} for p in prednasky
-		], prefix=PREDNASKY_PREFIX)
-
-		form_set_znalosti = HlasovaniZnalostiFormSet(initial=[
-			{"znalost_id": z.id, "odpoved": odpoved_znalosti(z)} for z in znalosti
-		], prefix=ZNALOSTI_PREFIX)
-
-
-	# V případě nePOSTu nebo chyby při ukládání vracíme hlasování
 	return render(
 		request,
 		'prednasky/base.html',
-		{
+		{'prednasky': map(prednaska_hodnoceni, seznam.prednaska_set.all())}
-			'form_set_prednasky': form_set_prednasky, 'form_set_znalosti': form_set_znalosti,
-			'formy_a_prednasky': zip(form_set_prednasky, prednasky),
-			'formy_a_znalosti': zip(form_set_znalosti, znalosti),
-		}
 	)
 
 
-def Prednaska_hotovo(request: HttpRequest) -> HttpResponse:
+def Prednaska_hotovo(request):
-	""" View po vyplnění :py:func:`hlasování <prednasky.views.newPrednaska>` """
+	return render(request, 'prednasky/hotovo.html')
-	return formularOKView(request, "Děkujeme za vyplnění hlasování o přednáškách a těšíme se na soustředění.")
 
 class MetaSeznamListView(generic.ListView):
-	""" Seznam všech :py:class:`Seznamů <prednasky.models.Seznam>` s odkazy na exporty """
 	model = Seznam
 	template_name = 'prednasky/metaseznam_prednasek.html'
 
 
 class SeznamListView(generic.ListView):
-	"""
-		Náhled na to, kolik má která přednáška v :py:class:`Seznamu <prednasky.models.Seznam>` :py:class:`hlasů <prednasky.models.Hlasovani.Body>`.
-		(Je otázka, zda tento View vůbec chceme. Pokud ano, hodilo by se do něj přidat i znalosti.)
-	"""
 	template_name = 'prednasky/seznam_prednasek.html'
 
 	def get_queryset(self):
@@ -152,7 +77,7 @@ class SeznamListView(generic.ListView):
 
 		# hlasovani se vztahuje k nejnovejsimu soustredeni
 		sous = Soustredeni.objects.first()
-		seznam = Seznam.objects.filter(soustredeni = sous, stav=Seznam.Stav.NAVRH).first()
+		seznam = Seznam.objects.filter(soustredeni = sous, stav = STAV_NAVRH).first()
 	
 		for obj in self.object_list:
 			hlasovani_set = obj.hlasovani_set.filter(seznam=seznam).only('body')
@@ -161,86 +86,32 @@ class SeznamListView(generic.ListView):
 		return context
 
 
-# def SeznamExportView(request, seznam):
+def SeznamExportView(request, seznam):
-# 	"""Vypíše výsledky hlasování ve formátu pro prologovský optimalizátor"""
+	"""Vypíše výsledky hlasování ve formátu pro prologovský optimalizátor"""
-# 	# TODO zřejmě se nepoužívá, časem vyřadit? nahradit tabulkou vhodnější pro
+	# TODO zřejmě se nepoužívá, časem vyřadit? nahradit tabulkou vhodnější pro
-# 	# lidi?
+	# lidi?
-# 	hlasovani = Hlasovani.objects.filter(seznam=seznam)
+	hlasovani = Hlasovani.objects.filter(seznam=seznam)
-# 	prednasky = Prednaska.objects.filter(seznamy=seznam)
+	prednasky = Prednaska.objects.filter(seznamy=seznam)
-# 	orgove = set(p.org for p in prednasky)
+	orgove = set(p.org for p in prednasky)
-# 	ucastnici = set(h.ucastnik for h in hlasovani)
+	ucastnici = set(h.ucastnik for h in hlasovani)
-#
-# 	for p in prednasky:
-# 		p.body = []
-# 		for u in ucastnici:
-# 			try:
-# 				p.body.append(hlasovani.get(ucastnik=u, prednaska=p).body)
-# 			except ObjectDoesNotExist:
-# 				# účastník nehlasoval
-# 				p.body.append("?")
-#
-# 	for h in hlasovani:
-# 		h.ucastnik = hash(h.ucastnik)
-#
-# 	return render(
-# 		request,
-# 		'prednasky/seznam_prednasek_export.txt',
-# 		{"hlasovani": hlasovani, "prednasky": prednasky, "orgove": orgove},
-# 		content_type="text/plain"
-# 	)
 
-
+	for p in prednasky:
-def PrednaskyExportView(request: HttpRequest, seznam: int, **kwargs) -> HttpResponse:
+		p.body = []
-	"""
+		for u in ucastnici:
-		Vrátí všechna :py:class:`Hlasování <prednasky.models.Hlasovani>`
+			try:
-		i :py:class:`HlasováníOZnalostech <prednasky.models.HlasovaniOZnalostech>`
+				p.body.append(hlasovani.get(ucastnik=u, prednaska=p).body)
-		v daném :py:class:`Seznamu <prednasky.models.Seznam>`
+			except ObjectDoesNotExist:
-		jako csv soubor (řádky = účastníci, sloupce = přednášky&znalosti).
+				# účastník nehlasoval
-
+				p.body.append("?")
-		:param seznam: ID daného :py:class:`Seznamu <prednasky.models.Seznam>`
-	"""
-	hlasovani = Hlasovani.objects.filter(seznam=seznam).select_related("prednaska")
-	hlasovani_o_znalostech = HlasovaniOZnalostech.objects.filter(seznam=seznam).select_related('ucastnik', 'znalost')
-
-	# Inicializujeme sloupce
-	prednasky = list(Prednaska.objects.filter(seznamy=seznam))
-	znalosti = list(Znalost.objects.filter(seznamy=seznam))
-
-	prednasky_map: dict[int, int] = {p.id: i for i, p in enumerate(prednasky, 1)}
-	offset = len(prednasky_map)
-	znalosti_map: dict[int, int] = {z.id: i for i, z in enumerate(znalosti, offset + 1)}
-	width = offset + len(znalosti_map)
-
-	# A po inicializaci sloupců vyplníme tabulku
-	table: [str, list[str|Prednaska|Znalost,]] = {}
 
 	for h in hlasovani:
-		if h.ucastnik not in table: # Pokud jsme účastníka ještě neviděli, předgenerujeme si jeho řádek
+		h.ucastnik = hash(h.ucastnik)
-			table[h.ucastnik] = [h.ucastnik] + ([""] * width)
 
-		if h.prednaska.id in prednasky_map:
+	return render(
-			table[h.ucastnik][prednasky_map[h.prednaska.id]] = h.body
+		request,
-		else:
+		'prednasky/seznam_prednasek_export.txt',
-			pass # TODO Padat hlasitě?
+		{"hlasovani": hlasovani, "prednasky": prednasky, "orgove": orgove},
-
+		content_type="text/plain"
-	for h in hlasovani_o_znalostech:
+	)
-		ucastnik = str(h.ucastnik) + ' ' + str(h.ucastnik.id) # id, kvůli kolizi jmen
-		if ucastnik not in table: # Pokud jsme účastníka ještě neviděli, předgenerujeme si jeho řádek
-			table[ucastnik] = [ucastnik] + ([""] * width)
-
-		if h.znalost.id in znalosti_map:
-			table[ucastnik][znalosti_map[h.znalost.id]] = h.odpoved
-		else:
-			pass # TODO Padat hlasitě?
-
-
-	response = HttpResponse(content_type="text/csv", charset="utf-8")
-	response["Content-Disposition"] = 'attachment; filename="hlasovani.csv"'
-
-	writer = csv.writer(response)
-	writer.writerow(["jména \\ přednáška|znalost"] + list(map(str, prednasky + znalosti)))
-	for row in table.values():
-		writer.writerow(list(map(str, row)))
-	return response
 
 

requirements.txt

@@ -17,7 +17,7 @@ django-solo # Singleton model (speciálně Nastavení)
 django-ckeditor-5 # Editor htmlka (hlavně v adminu u flatpages)
 django-cleanup  # Uklízí media/ od smazaných „databázových“ souborů
 django-taggit # Taggy v djangu (speciálně zaměření problémů)
-django-autocomplete-light>=3.9.0,<3.12.0 # Automatické doplňování (problémů, účastníků, …) ve formulářích
+django-autocomplete-light>=3.9.0 # Automatické doplňování (problémů, účastníků, …) ve formulářích
 django-imagekit # Všechny možné obrázky v Djangu
 django-polymorphic # Polymorfismus na django modelech (hlavně Problém nebo treenode)
 django-sitetree # Struktura stránek, hlavně pro meníčko
@@ -49,5 +49,4 @@ lorem
 
 sphinx
 sphinx_rtd_theme
-sphinxcontrib-django
 myst_parser

tvorba/migrations/0008_alter_problem_opravovatele_and_more.py

@@ -1,27 +0,0 @@
-# Generated by Django 4.2.16 on 2025-01-21 20:54
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('personalni', '0019_rename_upozorneni_resitel_upozornovat_na_opravy_reseni'),
-        ('tvorba', '0007_alter_deadline_typ'),
-    ]
-
-    operations = [
-        migrations.SeparateDatabaseAndState(
-            database_operations=[],
-            state_operations=[
-                migrations.AlterField(
-                    model_name='problem',
-                    name='opravovatele',
-                    field=models.ManyToManyField(blank=True, db_table='seminar_problemy_opravovatele', related_name='opravovatele_%(class)s', to='personalni.organizator', verbose_name='opravovatelé'),
-                ),
-                migrations.DeleteModel(
-                    name='Problemy_Opravovatele',
-                ),
-            ]
-        ),
-    ]

tvorba/models.py

@@ -393,6 +393,20 @@ class ZmrazenaVysledkovka(SeminarModelBase):
 
 	html = models.TextField(null=False, blank=False)
 
+class Problemy_Opravovatele(SeminarModelBase):
+	"""Jen vazebná tabulka pro opravovatele.
+
+	Ona stejně existovala, při přesunu mezi aplikacemi jen potřebujeme zajistit nepřejmenování DB tabulky.
+	Proto taky nepotřebuje žádná specifika, ze :py:class:SeminarModelBase: dědí ze zvyku než že by to k něčemu kdy měo být.
+	"""
+	class Meta:
+		db_table = 'seminar_problemy_opravovatele'
+	
+	id = models.AutoField(primary_key = True)
+
+	problem = models.ForeignKey('Problem', on_delete=models.CASCADE)
+	organizator = models.ForeignKey(Organizator, on_delete=models.CASCADE)
+
 @reversion.register(ignore_duplicates=True)
 # Pozor na následující řádek. *Nekrmit, asi kouše!*
 class Problem(SeminarModelBase,PolymorphicModel):
@@ -448,7 +462,7 @@ class Problem(SeminarModelBase,PolymorphicModel):
 		on_delete=models.SET_NULL)
 
 	opravovatele = models.ManyToManyField(Organizator, verbose_name='opravovatelé',
-		blank=True, related_name='opravovatele_%(class)s', db_table='seminar_problemy_opravovatele')
+		blank=True, related_name='opravovatele_%(class)s', through=Problemy_Opravovatele)
 
 	kod = models.CharField('lokální kód', max_length=32, blank=True, default='',
 		help_text='Číslo/kód úlohy v čísle nebo kód tématu/článku/seriálu v ročníku')

various/migrations/0007_nastaveni_aktualni_sous.py

@@ -1,20 +0,0 @@
-# Generated by Django 4.2.16 on 2025-01-21 20:34
-
-from django.db import migrations, models
-import django.db.models.deletion
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('soustredeni', '0013_alter_soustredeni_kontaktnicek_pdf_and_more'),
-        ('various', '0006_tvorba_post'),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name='nastaveni',
-            name='aktualni_sous',
-            field=models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.PROTECT, to='soustredeni.soustredeni', verbose_name='Aktuálně připravovaný sous'),
-        ),
-    ]

various/models.py

@@ -26,11 +26,6 @@ class Nastaveni(SingletonModel):
 									verbose_name="Účastnický poplatek za soustředění",
 									default=1000)
 
-	aktualni_sous = models.ForeignKey(
-		"soustredeni.Soustredeni", verbose_name='Aktuálně připravovaný sous',
-		null=True, blank=True, on_delete=models.PROTECT,
-	)
-
 	@property
 	def aktualni_rocnik(self):
 		return self.aktualni_cislo.rocnik