Merge branch 'refs/heads/prednasky'

Reviewed-on: #85
Reviewed-by: Pavel Turinský <ksgitea@pokemon.ledoian.cz>

data/groups.json

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

docs/conf.py

@@ -36,6 +36,7 @@ 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

@@ -1,7 +0,0 @@
-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

@@ -1,9 +0,0 @@
-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

@@ -1,65 +0,0 @@
-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

@@ -0,0 +1,14 @@
+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

@@ -1,45 +0,0 @@
-# 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,13 +52,16 @@ class KorekturovanePDF(models.Model):
 
 	stran = models.IntegerField(u'počet stran', help_text='Počet stran PDF',
 								default=0)
-
+	STATUS_PRIDAVANI = 'pridavani'
-	class STATUS(models.TextChoices):
+	STATUS_ZANASENI = 'zanaseni'
-		PRIDAVANI = 'pridavani', 'Přidávání korektur'
+	STATUS_ZASTARALE = 'zastarale'
-		ZANASENI = 'zanaseni', 'Korektury jsou zanášeny'
+	STATUS_CHOICES = (
-		ZASTARALE = 'zastarale', 'Stará verze, nekorigovat'
+			(STATUS_PRIDAVANI, u'Přidávání korektur'),
-
+			(STATUS_ZANASENI, u'Korektury jsou zanášeny'),
-	status = models.CharField(u'stav PDF',max_length=16, choices=STATUS.choices, blank=False, default = STATUS.PRIDAVANI)
+			(STATUS_ZASTARALE, u'Stará verze, nekorigovat'),
+			)
+	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.',
@@ -149,13 +152,32 @@ class Oprava(models.Model):
 	x = models.IntegerField(u'x-ová souřadnice bugu')
 	y = models.IntegerField(u'y-ová souřadnice bugu')
 
-	class STATUS(models.TextChoices):
+	STATUS_K_OPRAVE = 'k_oprave'
-		K_OPRAVE = 'k_oprave', 'K opravě'
+	STATUS_OPRAVENO = 'opraveno'
-		OPRAVENO = 'opraveno', 'Opraveno'
+	STATUS_NENI_CHYBA = 'neni_chyba'
-		NENI_CHYBA = 'neni_chyba', 'Není chyba'
+	STATUS_K_ZANESENI = 'k_zaneseni'
-		K_ZANESENI = 'k_zaneseni', 'K zanesení do TeXu'
+	STATUS_CHOICES = (
+			(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,159 +1,136 @@
-.textzanaseni { display:none; }
+body,
-.textzastarale { display:none; }
+.adding{
-#prekomentar, #preoprava, #prepointer { display: none; }
+	background: #f3f3f3; 
-
+	color: black;
-body {
+}
-	&[data-status="pridavani"] {
+.comitting
-		background: #f3f3f3;
+{
-	}
+    background: yellow;
-
+}
-	&[data-status="zanaseni"] {
+.deprecated {
-		background: yellow;
+    background: red;
-
-		.textzanaseni { display: unset; }
-	}
-
-	&[data-status="zastarale"] {
-		background: red;
-
-		.textzastarale { display: unset; }
-	}
 }
 
 img{background:white;}
 
 /* Barvy korektur */
-[data-opravastatus="k_oprave"] {
+.k_oprave { 
 	--rgb: 255, 0, 0;
-
-	[value="k_oprave"] { display: none }
-	[value="notcomment"] { display: none }
 }
-[data-opravastatus="opraveno"] {
+.opraveno {
 	--rgb: 0, 0, 255;
-
-	[value="opraveno"] { display: none }
-	[value="comment"] { display: none }
 }
-[data-opravastatus="neni_chyba"] {
+.neni_chyba {
 	--rgb: 128, 128, 128;
-
-	[value="neni_chyba"] { display: none }
-	[value="comment"] { display: none }
 }
-[data-opravastatus="k_zaneseni"] {
+.k_zaneseni {
 	--rgb: 0, 255, 0;
-
-	[value="k_zaneseni"] { display: none }
-	[value="notcomment"] { display: none }
 }
 
-/* Skrývání korektur */
+.pointer-hi, 
-[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: 1px solid rgb(var(--rgb),var(--alpha));
+	border-left: 2px solid yellow;
-	border-bottom: 1px solid rgb(var(--rgb),var(--alpha));
+	border-bottom: 2px solid yellow;
-	pointer-events: none;
+	border-color: rgb(var(--rgb),var(--alpha));
-
-	--alpha: 0.35;
-
-	/* Zvýraznění čáry při najetí na korekturu */
-	&[data-highlight="true"] {
-		border-width: 3px;
-		--alpha: 1;
-	}
 }
 
-/* Korektura samotná */
+.pointer {
-.oprava {
+	border-width: 1px;
+	--alpha: 0.35;	
+}
+
+.pointer-hi {
+	border-width: 3px;
+	--alpha: 1;
+}
+
+
+
+.box:hover{
+	border-width:3px;
+	margin: 0px;
+}
+.box {
 	margin: 1px;
 	background-color: white;
-	width: 300px;
+	width:300px;
+	/*position:absolute;*/
 	padding: 3px;
-	border: 2px solid rgb(var(--rgb));
+	border: 2px solid black;
 	border-radius: 10px;
-	position: absolute;
+	border-color: rgb(var(--rgb));
-
-	&:hover {
-		border-width:3px;
-		margin: 0;
-	}
-
-	button, img {
-		border: 1px solid white;
-		background-color:transparent;
-		margin:0;
-		padding: 1px;
-
-		&:hover {
-			border: 1px solid black;
-		}
-	}
-
-	button img { pointer-events: none; }
-
-	.corr-header {
-		overflow: auto;
-	}
-
-	.author {
-		font-weight: bold;
-		float: left;
-		margin-top: 3px;
-	}
-
-	.float-right{
-		float:right;
-	}
 }
-
 form {
 	display:inline;
 }
 
-/* Zobrazované PDF */
+.float-right{
-.imgdiv {
+	float:right;
-	position:relative;
-	left:0;
-	top:0;
 }
 
-/* Přidávání korektury / úprava komentáře */
+.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;	
-/**** ROZLIŠENÍ MEZI LOKÁLNÍM, TESTOVACÍM A PRODUKČNÍM WEBEM ****/
+}
-body.localweb, body.testweb, body.suprodweb {
+
-	&:before, &:after {
+
-		content: "";
+.box button,
-		position: fixed;
+.box img,
-		width: 20px;
+.box-done button,
-		height: 100%;
+.box-done img,
-		top: 0;
+.box-ready button,
-		z-index: -1000;
+.box-ready img,
-	}
+.box-wontfix button,
-
+.box-wontfix img{
-	&:before { left: 0; }
+	border: 1px solid white;
-	&:after { right: 0; }
+	background-color:transparent;
+	margin:0;
+	padding: 1px;
+}
+.box button: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 {
+	height: 0px;
+}
+
+.corr-header {
+    overflow: auto;
+}
+
+.author {
+    font-weight: bold;
+    float: left;
+    margin-top: 3px;
 }
 
-body.localweb { &:before, &:after { background: greenyellow; } }
-body.testweb { &:before, &:after { background: darkorange; } }
-body.suprodweb { &:before, &:after { background: red; } }
-/****************************************************************/

korektury/static/korektury/opraf.js

@@ -1,46 +1,283 @@
-const W_SKIP = 10;
-const H_SKIP = 5;
-const POINTER_MIN_H = 30;
-
 function place_comments_one_div(img_id, comments)
 {
-	const img = document.getElementById("img-"+img_id);
+	var img = document.getElementById(img_id);
-	if( img == null ) return;
+	if( img == null ) { 
-    const comments_sorted = comments.sort((a, b) => a.y - b.y);
+		return;
-
-	const par = img.parentNode;
-	const w = img.clientWidth;
-
-	let bott_max = 0;
-	for (const oprava of comments_sorted) {
-		const x = oprava.x;
-		const y = oprava.y;
-        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);
 	}
+	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;
 
-	if (par.offsetHeight < bott_max) par.style.height = bott_max;
+	var bott_max = 0;
+	var comments_sorted = comments.sort(function (a,b) {
+		return a[2] - b[2];
+		//pokus o hezci kladeni poiteru, ale nic moc
+		if( a[3] < b[3] ) { 
+			return (a[2] + pointer_min_h)- b[2];
+		} else {
+			return (a[2] - pointer_min_h)- b[2];
+		}
+
+	});
+	//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 (let [img_id, opravy] of Object.entries(comments)) {
+    for (var i=0; i < comments.length-1; i++) {
-	    place_comments_one_div(img_id, opravy)
+	    place_comments_one_div(comments[i][0], comments[i][1])
     }
 }
 
+// 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

@@ -1,64 +0,0 @@
-<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

@@ -1,86 +0,0 @@
-{% 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

@@ -1,145 +0,0 @@
-{% 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

@@ -1,50 +0,0 @@
-{% 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

@@ -1,40 +0,0 @@
-{% 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

@@ -1,51 +0,0 @@
-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

@@ -1,44 +0,0 @@
-<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

@@ -1,44 +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" 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

@@ -0,0 +1,244 @@
+{% 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,6 +1,4 @@
 from django.urls import path
-from django.urls import include
-
 from personalni.utils import org_required
 from . import views
 
@@ -9,6 +7,4 @@ 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

@@ -1,53 +0,0 @@
-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,18 +2,24 @@ 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=Oprava.STATUS.K_OPRAVE)),
+		k_oprave_cnt=Count('oprava',distinct=True,filter=Q(oprava__status='k_oprave')),
-		opraveno_cnt=Count('oprava',distinct=True,filter=Q(oprava__status=Oprava.STATUS.OPRAVENO)),
+		opraveno_cnt=Count('oprava',distinct=True,filter=Q(oprava__status='opraveno')),
-		neni_chyba_cnt=Count('oprava',distinct=True,filter=Q(oprava__status=Oprava.STATUS.NENI_CHYBA)),
+		neni_chyba_cnt=Count('oprava',distinct=True,filter=Q(oprava__status='neni_chyba')),
-		k_zaneseni_cnt=Count('oprava',distinct=True,filter=Q(oprava__status=Oprava.STATUS.K_ZANESENI)),
+		k_zaneseni_cnt=Count('oprava',distinct=True,filter=Q(oprava__status='k_zaneseni')),
 		)
 	template_name = 'korektury/seznam.html'
 
@@ -52,15 +58,13 @@ class KorekturySeskupeneListView(KorekturyAktualniListView):
 ### Korektury
 class KorekturyView(generic.TemplateView):
 	model = Oprava
-	template_name = 'korektury/korekturovatko/htmlstrana.html'
+	template_name = 'korektury/opraf.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
@@ -69,27 +73,50 @@ 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:])
-			op = Oprava(x=x,y=y, strana=strana, pdf=self.pdf)
+			pdf = KorekturovanePDF.objects.get(id=q.get('pdf'))
+
+			op = Oprava(x=x,y=y, autor=autor, text=text, strana=strana,pdf = pdf)
 			op.save()
-			kom = Komentar(oprava=op,autor=autor,text=text)
+			self.send_email_notification_komentar(op,autor)
-			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 in Oprava.STATUS.values:
+		elif (action == 'update'):
 			id = int(q.get('id'))
 			op = Oprava.objects.get(id=id)
-			op.status = action
+			text = q.get('txt')
+			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'))
@@ -97,7 +124,7 @@ class KorekturyView(generic.TemplateView):
 			text = q.get('txt')
 			kom = Komentar(oprava=op,autor=autor,text=text)
 			kom.save()
-			send_email_notification_komentar(op, autor, request)
+			self.send_email_notification_komentar(op,autor)
 		elif (action == 'update-comment'):
 			id = int(q.get('id'))
 			kom = Komentar.objects.get(id=id)
@@ -110,23 +137,85 @@ class KorekturyView(generic.TemplateView):
 			kom = Komentar.objects.get(id=id)
 			kom.delete()
 		elif (action == 'set-state'):
-			status = q.get('state')
+			pdf = KorekturovanePDF.objects.get(id=q.get('pdf'))
-			assert status in KorekturovanePDF.STATUS.values
+			if (q.get('state') == 'adding'):
-			self.pdf.status = status
+				pdf.status = pdf.STATUS_PRIDAVANI
-			self.pdf.save()
+			elif (q.get('state') == 'comitting'):
+				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/korekturovatko/htmlstrana.html', context)
+		return render(request, 'korektury/opraf.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)
-		context['pdf'] = self.pdf
+		pdf = get_object_or_404(KorekturovanePDF, id=self.kwargs['pdf'])
-		context['img_prefix'] = self.pdf.get_prefix()
+		context['pdf'] = pdf
+		context['img_prefix'] = pdf.get_prefix()
 		context['img_path'] = settings.KOREKTURY_IMG_DIR
-		context['img_indexes'] = range(self.pdf.stran)
+		context['img_indexes'] = range(pdf.stran) 
-		opravy = Oprava.objects.filter(pdf=self.pdf_id)
+		context['form_oprava'] = OpravaForm()
+		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:
@@ -152,3 +241,6 @@ 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,6 +57,7 @@ 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',
@@ -134,7 +135,6 @@ INSTALLED_APPS = (
 	'tvorba',
 	'galerie',
 	'korektury',
-	'korektury.api',
 	'prednasky',
 	'header_fotky',
 	'various',

mamweb/static/css/layout.css

@@ -435,6 +435,7 @@ 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,5 +503,10 @@ label[for=id_skola] {
 	font-weight: bold;
 }
 
+/* Přednášky */
+.textznalosti, .textprednasky {
+	font-style: italic;
+}
+
 /*******************/
 

mamweb/static/css/rozliseni.css

@@ -0,0 +1,20 @@
+/**** 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 `seminar.models`. Pevně věřím, že to nebylo nikde v kódu. Díky.")
+print("Naimportoval jsi `mamweb.vsechno`. 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,6 +65,9 @@ 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é. 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>
+  <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>
 </ol>
 
 Další poznámky

odevzdavatko/views.py

@@ -222,6 +222,17 @@ 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`. """
@@ -232,18 +243,7 @@ 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):
-			seznam_atributu = [
+			result.append({attr: getattr(hodn, attr) for attr in HODNOCENI_INITIAL_DATA})
-				"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})
 
-	# FIXME: Použit initial i tady a nebastlit hodnocení tak nízkoúrovňově
+	formset = f.OhodnoceniReseniFormSet(request.POST, initial=[
-	# Also: https://docs.djangoproject.com/en/2.2/topics/forms/modelforms/#django.forms.ModelForm
+		{k: getattr(h, k) for k in HODNOCENI_INITIAL_DATA} for h in Hodnoceni.objects.filter(reseni=reseni)
-	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,7 +309,9 @@ 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"])
@@ -320,16 +322,44 @@ 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) == 1:
+			if len(zmeny_bodu) != 0:
-				hodnoceni.__setattr__(zmeny_bodu[0], data_for_body[zmeny_bodu[0]])
+				body_nastaveny: None | tuple[str, object] = None
-			# > jedna změna je špatně, ale 4 "změny" znamenají že nebylo nic zadáno
+				def nastav_body(jake, na_kolik):
-			if len(zmeny_bodu) > 1 and len(zmeny_bodu) != 4 and len(zmeny_bodu) != 2:
+					nonlocal body_nastaveny
-				# 4 znamená vše už vyplněno a nic nezměněno, 2 znamená předvyplnili se součty a nic se nezměnilo
+					if body_nastaveny is not None:
-				logger.warning(f"Hodnocení {hodnoceni} mělo mít nastavené víc různých bodů: {zmeny_bodu}. Nastavuji -0.1.")
+						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.")
-				hodnoceni.body = -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,6 +71,8 @@ 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

@@ -0,0 +1,22 @@
+# 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

@@ -0,0 +1,18 @@
+# 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,6 +250,8 @@ 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

@@ -0,0 +1,34 @@
+.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

@@ -0,0 +1,28 @@
+{% 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,6 +51,7 @@
 </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,4 +33,11 @@ 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
+from .models import Organizator, Osoba 
 
 
 def aktivniOrganizatori(datum=timezone.now()):
@@ -62,6 +62,11 @@ 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:
@@ -230,6 +235,7 @@ 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

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

prednasky/admin.py

@@ -4,11 +4,15 @@ from reversion.admin import VersionAdmin
 from django.utils.safestring import mark_safe
 from django.utils.html import escape
 
-from .models import Prednaska, Seznam, STAV_NAVRH
+from .models import Prednaska, Seznam, Znalost
 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
 
@@ -54,24 +58,57 @@ 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]
+	inlines = [Seznam_PrednaskaInline, Seznam_ZnalostInline]
 
 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 = []
+	search_fields = ['nazev']
 	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=STAV_NAVRH)
+		seznam = Seznam.objects.filter(soustredeni=sous, stav=Seznam.Stav.NAVRH)
 		if len(seznam) == 0:
 			self.message_user(
 				request,
@@ -97,3 +134,14 @@ 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,7 +1,31 @@
 from django import forms
 
-class NewPrednaskyForm(forms.Form):
+from .models import Hlasovani, HlasovaniOZnalostech
-	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

@@ -0,0 +1,39 @@
+# 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

@@ -0,0 +1,18 @@
+# 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

@@ -0,0 +1,24 @@
+# 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,81 +1,134 @@
 from django.db import models
 
 from soustredeni.models import Soustredeni
-from personalni.models import Organizator
+from personalni.models import Organizator, Osoba
-
-STAV_NAVRH = 1
-STAV_BUDE = 2
-
-
-STAV_CHOICES = (
-(STAV_NAVRH, 'Návrh'),
-(STAV_BUDE, 'Bude')
-)
 
 
 class Seznam(models.Model):
-	class Meta:
+	"""
-		db_table = 'prednasky_seznam'
+		Spojuje :py:class:`Přednášky <prednasky.models.Prednaska>`
-		verbose_name = 'Seznam přednášek'
+		se :py:class:`Soustředěními <soustredeni.models.Soustredeni>`,
-		verbose_name_plural = 'Seznamy přednášek'
+		kde by mohly zaznít, nebo zazní/zazněly.
-		ordering = ['soustredeni', 'stav']
+	"""
 
-	id = models.AutoField(primary_key = True) 
+	class Meta:
-	soustredeni = models.ForeignKey(Soustredeni,null = True, default = None, 
+		db_table = "prednasky_seznam"
-		on_delete=models.PROTECT)
+		verbose_name = "Seznam přednášek"
-	stav = models.IntegerField('Stav',choices=STAV_CHOICES,default = STAV_NAVRH)
+		verbose_name_plural = "Seznamy přednášek"
+		ordering = ["soustredeni", "stav"]
+
+	class Stav(models.IntegerChoices):
+		""" Stav seznamu přednášek (NAVRH se používá k hlasování viz :py:func:`daný view <prednasky.views.newPrednaska>`). """
+		NAVRH = 1, "Návrh"
+		BUDE = 2, "Bude"
+
+	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 "Seznam {}přednášek na {}".format("návrhů " 
+		return f"Seznam {'návrhů ' if self.stav == Seznam.Stav.NAVRH else ''}přednášek na {self.soustredeni}"
-					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"]
 
-	id = models.AutoField(primary_key = True) 
+	class Obtiznost(models.IntegerChoices):
-	nazev = models.CharField('Název', max_length = 300)
+		LEHKA = 1, "Lehká"
+		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=CHOICES_OBTIZNOST)
+	obtiznost = models.IntegerField("Obtížnost", choices=Obtiznost.choices) #: :py:class:`Obtížnost <prednasky.models.Prednaska.Obtiznost>` Přednášky
-	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 "{} ({})".format(self.nazev, self.org)
+		return f"{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)
-	body = models.IntegerField('Body', default = 0, choices = CHOICES_BODY)
+	#: Příslušné hlasování: :py:class:`Body <prednasky.models.Hlasovani.Body>`
-	ucastnik = models.CharField('Účastník', max_length = 100)
+	body = models.IntegerField("Body", default=Body.JEDNO, choices=Body.choices)
-	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 "{} dal {} bodů {} v seznamu {}".format(self.ucastnik, 
+		return f"{self.ucastnik} dal {self.body} bodů {self.prednaska} v seznamu {self.seznam}"
-					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,34 +5,36 @@
 
 
 {% block content %}
-<h1>
+<h1>{% block nadpis1a %}Hlasování o přednáškách{% endblock %}</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>
+
-    {% for p, h in prednasky %}
+<h3>Jak moc by ses chtěl(a) zúčastnit následujících přednášek?</h3>
-    <tr><td><label>{{p.org}}: <span style="font-size: 175%">{{p.nazev}}</span></label></td></tr>
+<p>Obtížnost 1 je nejlehčí, 3 nejtěžší.</p>
-    <tr><td><p><i>{{p.anotace}}</i></p></td></tr>
+  {{ form_set_prednasky.management_form }}
-    <tr><td><label>Obor: </label> {{p.obor}}</td></tr>
+  {% for f, p in formy_a_prednasky %}
-    <tr><td><label>Obtížnost: </label> {{p.obtiznost}}</td>   </tr>
+    <h4>{{p.nazev}} ({{p.org}})</h4>
-    {% if p.klicova %}<tr><td><label>Klíčová slova: </label> {{p.klicova}}</td></tr>{% endif%}
+    <p class="textprednasky">{{p.anotace}}</p>
-    <tr><td>Hodnocení:
+    <label>Obor: </label> {{p.obor}}<br>
-        <INPUT TYPE="radio" NAME="q{{p.pk}}" VALUE="-1" {% if h == -1 %} CHECKED="checked" {% endif %} > rozhodně nechci
+    <label>Obtížnost: </label> {{p.obtiznost}}<br>
-        <INPUT TYPE="radio" NAME="q{{p.pk}}" VALUE="0" {% if h == 0 %} CHECKED="checked" {% endif %}> je mi to jedno
+    {% if p.klicova %}<label>Klíčová slova: </label> {{p.klicova}}<br>{% endif%}
-        <INPUT TYPE="radio" NAME="q{{p.pk}}" VALUE="1" {% if h == 1 %} CHECKED="checked" {% endif %}> rozhodně chci
+    <br>
-    </td></tr>
+    {{ f }}
-    <tr><td>&nbsp;</td></tr>
+    <br>
-    {% endfor %}
+  {% empty %}
-    <tr><td><input name="odeslat" type="submit" value="Odeslat"></td><tr>
+    Nejsou žádné přednášky o kterých by šlo hlasovat.
-  </table>
+  {% endfor %}
+
+  {{ form_set_znalosti.management_form }}
+  {% 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

@@ -1,11 +0,0 @@
-{% 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}}/export">Export</a>
+    <a href="/prednasky/seznam_prednasek/{{seznam.id}}/hlasovani.csv">Export</a>
     </li>
   {% endfor %}
   </ul>

prednasky/urls.py

@@ -12,10 +12,15 @@ 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>/export',
+		'prednasky/seznam_prednasek/<int:seznam>/hlasovani.csv',
-		org_required(views.SeznamExportView),
+		org_required(views.PrednaskyExportView),
-		name='seznam-export'
+		name='seznam-export-csv'
 	),
 	path(
 		'prednasky/seznam_prednasek/<int:seznam>/',

prednasky/views.py

@@ -1,67 +1,142 @@
+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.models import Sum
+from django.db import transaction
-from django.forms import Form
 
-from prednasky.models import Prednaska, Hlasovani, Seznam, STAV_NAVRH
+from various.views.pomocne import formularOKView
+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
 
-def newPrednaska(request):
+PREDNASKY_PREFIX = "prednasky"
+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 = Soustredeni.objects.first()
+	sous = Nastaveni.get_solo().aktualni_sous
-	seznam = Seznam.objects.filter(soustredeni = sous, stav = STAV_NAVRH).first()
+	seznam = Seznam.objects.filter(soustredeni = sous, stav=Seznam.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)
+	ucastnik = osoba.plne_jmeno() + ' ' + str(osoba.id) # id, kvůli kolizi jmen
-	# 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
 
-			# TODO v následujících řádcích je zbytečně mnoho dotazů na QuerySet (pokud účastník hlasoval, hlasoval u všech)
+	if request.method == 'POST': # Když to byl POST, tak ukládáme.
-			for i in request.POST:
+		# Načteme data do formsetů
-				if i[0] == 'q':
+		form_set_prednasky = HlasovaniPrednaskaFormSet(request.POST, prefix=PREDNASKY_PREFIX)
-					prednaska = Prednaska.objects.filter(pk=int(i[1:]))[0]
+		form_set_znalosti = HlasovaniZnalostiFormSet(request.POST, prefix=ZNALOSTI_PREFIX)
-					hlasovani = Hlasovani.objects.filter(ucastnik=ucastnik, prednaska=prednaska).first()
+
-					if not hlasovani:
+		if form_set_prednasky.is_valid() and form_set_znalosti.is_valid():
-						hlasovani = Hlasovani()
+			with transaction.atomic():
-						hlasovani.prednaska = prednaska
+				# Místo updatování data prostě smažeme a vytvoříme nová
-						hlasovani.ucastnik = ucastnik
+				seznam.hlasovani_set.filter(ucastnik=ucastnik).delete()
-						hlasovani.seznam = seznam
+				seznam.hlasovanioznalostech_set.filter(ucastnik=osoba).delete()
-					hlasovani.body = int(request.POST[i])
+
-					hlasovani.save()
+				for form in form_set_prednasky:
+					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')
 
-	def prednaska_hodnoceni(prednaska):
+		else: # Pokud je nějaký formset nevalidní, vracíme je k přepracování
-		h = Hlasovani.objects.filter(ucastnik=ucastnik, prednaska=prednaska).first()
+			prednasky = seznam.prednaska_set.all()
-		if h:
+			znalosti = seznam.znalost_set.all()
-			return prednaska, h.body
+			# FIXME Spadnout, pokud nesedí přednáška/znalost s formulářem. (Nějak se mi to nepovedlo.)
-		else:
+			# Může se totiž stát, že se mezitím změnily přednášky (nějaká byla přidána/odebrána)
-			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):
+def Prednaska_hotovo(request: HttpRequest) -> HttpResponse:
-	return render(request, 'prednasky/hotovo.html')
+	""" View po vyplnění :py:func:`hlasování <prednasky.views.newPrednaska>` """
+	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):
@@ -77,7 +152,7 @@ class SeznamListView(generic.ListView):
 
 		# hlasovani se vztahuje k nejnovejsimu soustredeni
 		sous = Soustredeni.objects.first()
-		seznam = Seznam.objects.filter(soustredeni = sous, stav = STAV_NAVRH).first()
+		seznam = Seznam.objects.filter(soustredeni = sous, stav=Seznam.Stav.NAVRH).first()
 	
 		for obj in self.object_list:
 			hlasovani_set = obj.hlasovani_set.filter(seznam=seznam).only('body')
@@ -86,32 +161,86 @@ 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:
+
-		p.body = []
+def PrednaskyExportView(request: HttpRequest, seznam: int, **kwargs) -> HttpResponse:
-		for u in ucastnici:
+	"""
-			try:
+		Vrátí všechna :py:class:`Hlasování <prednasky.models.Hlasovani>`
-				p.body.append(hlasovani.get(ucastnik=u, prednaska=p).body)
+		i :py:class:`HlasováníOZnalostech <prednasky.models.HlasovaniOZnalostech>`
-			except ObjectDoesNotExist:
+		v daném :py:class:`Seznamu <prednasky.models.Seznam>`
-				# účastník nehlasoval
+		jako csv soubor (řádky = účastníci, sloupce = přednášky&znalosti).
-				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:
-		h.ucastnik = hash(h.ucastnik)
+		if h.ucastnik not in table: # Pokud jsme účastníka ještě neviděli, předgenerujeme si jeho řádek
+			table[h.ucastnik] = [h.ucastnik] + ([""] * width)
 
-	return render(
+		if h.prednaska.id in prednasky_map:
-		request,
+			table[h.ucastnik][prednasky_map[h.prednaska.id]] = h.body
-		'prednasky/seznam_prednasek_export.txt',
+		else:
-		{"hlasovani": hlasovani, "prednasky": prednasky, "orgove": orgove},
+			pass # TODO Padat hlasitě?
-		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 # Automatické doplňování (problémů, účastníků, …) ve formulářích
+django-autocomplete-light>=3.9.0,<3.12.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,4 +49,5 @@ lorem
 
 sphinx
 sphinx_rtd_theme
+sphinxcontrib-django
 myst_parser

tvorba/migrations/0008_alter_problem_opravovatele_and_more.py

@@ -0,0 +1,27 @@
+# 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,20 +393,6 @@ 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):
@@ -462,7 +448,7 @@ class Problem(SeminarModelBase,PolymorphicModel):
 		on_delete=models.SET_NULL)
 
 	opravovatele = models.ManyToManyField(Organizator, verbose_name='opravovatelé',
-		blank=True, related_name='opravovatele_%(class)s', through=Problemy_Opravovatele)
+		blank=True, related_name='opravovatele_%(class)s', db_table='seminar_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

@@ -0,0 +1,20 @@
+# 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,6 +26,11 @@ 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