seminar/utils.py

aesop/views.py

@@ -8,7 +8,7 @@ from django.utils.encoding import force_str
 from .utils import default_ovvpfile
 from seminar.models import Rocnik, Soustredeni
 from vysledkovky import utils
-from seminar.utils import aktivniResitele
+from tvorba.utils import aktivniResitele
 
 class ExportIndexView(generic.View):
 	def get(self, request):

api/tests/test_skola_autocomplete.py

@@ -1,7 +1,7 @@
 from django.test import TestCase, tag
 from django.urls import reverse
 import seminar.models as m
-from seminar.utils import sync_skoly
+from personalni.utils import sync_skoly
 
 @tag('stejny-model-na-produkci')
 class OrgSkolyAutocompleteTestCase(TestCase):

api/urls.py

@@ -1,6 +1,6 @@
 from django.urls import path
 from . import views
-from seminar.utils import org_required
+from personalni.utils import org_required
 
 urlpatterns = [
 	# Export škol

docs/zapisy/2021-12-06-testovani_dokumentace_codereview.md

@@ -116,7 +116,7 @@ Aktuálně: Jakýsi coding style zhruba existuje, není popsaný, šíří se li
 - Nesmí být striktně vynucovaný
 - Musel by být hodně nastavitelný
     - Nechceme mít kód plný `#NOQA: WTF42`
-- Nejspíš vždycky bude mít false positives (`seminar.utils.roman_numerals`) i false negatives (`seminar.models.tvorba.Cislo.posli_cislo_mailem`)
+- Nejspíš vždycky bude mít false positives (`tvorba.utils.roman_numerals`) i false negatives (`seminar.models.tvorba.Cislo.posli_cislo_mailem`)
     - Možná dobrý sluha, ale určitě špatný pán (also: špatná zkušenost ☺)
 - __Důsledek:__ Hrozí, že těch falešných varování bude moc, čímž to ztratí smysl úplně
     - Potenciálně by šlo aplikovat jen lokálně na změny?

galerie/urls.py

@@ -1,5 +1,5 @@
 from django.urls import path
-from seminar.utils import org_required
+from personalni.utils import org_required
 from . import views
 
 urlpatterns = [

korektury/urls.py

@@ -1,5 +1,5 @@
 from django.urls import path
-from seminar.utils import org_required
+from personalni.utils import org_required
 from . import views
 
 urlpatterns = [

odevzdavatko/urls.py

@@ -1,7 +1,7 @@
 from django.urls import path
 
-from seminar.utils import org_required, resitel_required, viewMethodSwitch, \
+from personalni.utils import org_required, resitel_required, resitel_or_org_required
-	resitel_or_org_required
+from various.views.generic import viewMethodSwitch
 from . import views
 
 urlpatterns = [

odevzdavatko/utils.py

@@ -0,0 +1,11 @@
+import decimal
+
+
+def vzorecek_na_prepocet(body, resitelu):
+	""" Vzoreček na přepočet plných bodů na parciálni, když má řešení více řešitelů. """
+	return body * 3 / (resitelu + 2)
+
+
+def inverze_vzorecku_na_prepocet(body: decimal.Decimal, resitelu) -> decimal.Decimal:
+	""" Vzoreček na přepočet parciálních bodů na plné, když má řešení více řešitelů. """
+	return round(body * (resitelu + 2) / 3, 1)

odevzdavatko/views.py

@@ -20,7 +20,7 @@ import logging
 import seminar.models as m
 from . import forms as f
 from .forms import OdevzdavatkoTabulkaFiltrForm as FiltrForm
-from seminar.utils import resi_v_rocniku
+from tvorba.utils import resi_v_rocniku
 from various.views.pomocne import formularOKView
 
 logger = logging.getLogger(__name__)

personalni/urls.py

@@ -1,7 +1,7 @@
 from django.urls import path
 from django.contrib.auth.decorators import login_required
 from . import views
-from seminar.utils import org_required
+from personalni.utils import org_required
 
 urlpatterns = [
 	path(

personalni/utils.py

@@ -2,10 +2,183 @@ import seminar.models as m
 from various.utils import bez_diakritiky_translate
 import re
 
-def normalizuj_jmeno(o: m.Osoba) -> str:
+from django.contrib.auth import get_user_model
+from django.contrib.auth.decorators import permission_required, user_passes_test
+from django.contrib.auth.models import AnonymousUser
+from django.db import transaction
+
+import seminar.models as m
+import soustredeni.models
+
+from .models import Osoba, Organizator, Skola, Resitel, Prijemce
+
+
+org_required = permission_required('auth.org')
+resitel_required = permission_required('auth.resitel')
+
+
+# inspirováno django.contrib.auth.decorators permission_required
+def check_perms(user):
+	if user.has_perms(('auth.resitel',)):
+		return True
+	if user.has_perms(('auth.org',)):
+		return True
+	return False
+
+
+resitel_or_org_required = user_passes_test(check_perms)
+
+User = get_user_model()
+# Není to úplně hezké, ale budeme doufat, že to je funkční...
+User.je_org = property(lambda self: self.has_perm('auth.org'))
+User.je_resitel = property(lambda self: self.has_perm('auth.resitel'))
+AnonymousUser.je_org = False
+AnonymousUser.je_resitel = False
+
+def normalizuj_jmeno(o: Osoba) -> str:
 	# FIXME: Možná není potřeba vázat na model?
 	cele_jmeno = f'{o.jmeno} {o.prijmeni}'
 	cele_jmeno = cele_jmeno.translate(bez_diakritiky_translate)
 	cele_jmeno = re.sub(r'[^a-zA-Z- ]', '', cele_jmeno)
 	return cele_jmeno
 
+
+def sync_skoly(base_url):
+	"""Stáhne všechny školy z mamwebu na adrese <base_url> a uloží je do databáze"""
+	from django.urls import reverse
+	full_url = base_url.rstrip('/') + reverse('export_skoly')
+	import requests
+	from django.core import serializers
+	json =  requests.get(full_url, stream=True).content
+	for skola in serializers.deserialize('json', json):
+		skola.save()
+
+@transaction.atomic
+def merge_resitele(cilovy, zdrojovy):
+	"""Spojí dva řešitelské objekty do cílového.
+
+	Pojmenování "zdrojový" je silně nepřiléhající, ale co už…"""
+
+	# Postup:
+	# Sjednotit / upravit informace cílového řešitele
+	print('Upravuji data modelu')
+	fieldy_shoda = ['skola', 'rok_maturity', 'zasilat', 'zasilat_cislo_emailem', 'zasilat_cislo_papirove']
+
+	for f in fieldy_shoda:
+		zf = getattr(zdrojovy, f)
+		cf = getattr(cilovy, f)
+		if cf == zf:
+			print(f' Údaj {f} je shodný ({zf})')
+		else:
+			if zf is None:
+				print(f' Údaj {f} je pouze v cílovém, používám')
+				continue
+			if cf is None:
+				setattr(cilovy, f, zf)
+				cilovy.poznamka += f'\nDEBUG: Merge: doplnéný údaj {f} ze zdrojového: {zf}'
+				print(f" Přiřazuji {f} ze zdrojového: {zf}")
+				continue
+			# Jsou fakt různé…
+			# FIXME: chybí možnost na vlastní úpravu…
+			verdikt = input(f"\n\n Údaj {f} se u řešitele {cilovy} ({cilovy.id}) liší:\n  Zdrojový: {zf}\n  Cílový: {cf}\n Který použít, [z]drojový, [c]ílový? ")
+			verdikt = verdikt[0].casefold()
+			if verdikt == 'z':
+				setattr(cilovy, f, zf)
+				cilovy.poznamka += f'\nDEBUG: Merge: pro {f} použit údaj {zf} (zdrojový), nepoužit {cf} (cílový)'
+			elif verdikt == 'c':
+				cilovy.poznamka += f'\nDEBUG: Merge: pro {f} použit údaj {cf} (cílový), nepoužit {zf} (zdrojový)'
+			else: raise ValueError('Špatná odpověď, řešitel pravděpodobně neuložen')
+	# poznámku chceme nezahodit…
+	cilovy.poznamka += f'\nDEBUG: Merge: Původní poznámka: {zdrojovy.poznamka}'
+	print(f' Výsledný řešitel: {cilovy.__dict__}, ukládám')
+	cilovy.save()
+
+
+	# Přepojit všechny vazby ze zdrojového na cílového
+	print('Přepojuji vazby')
+	# Vazby: Škola (hotovo), Řešení_Řešitelé, Konfery_Účastníci, Soustředění_Účastníci, Osoba (vyřeší se později, nejde přepojit)
+	ct = m.Reseni_Resitele.objects.filter(resitele=zdrojovy).update(resitele=cilovy)
+	print(f' Přepojeno {ct} řešení')
+	ct = soustredeni.models.Konfery_Ucastnici.objects.filter(resitel=zdrojovy).update(resitel=cilovy)
+	print(f' Přepojeno {ct} konfer')
+	ct = soustredeni.models.Soustredeni_Ucastnici.objects.filter(resitel=zdrojovy).update(resitel=cilovy)
+	print(f' Přepojeno {ct} sousů')
+
+	# Teď by na zdrojovém řešiteli nemělo nic viset, smazat ho, pamatujíce si jeho Osobu
+	zdrosoba = zdrojovy.osoba
+	print(f'Mažu zdrojového řešitele {zdrojovy.__dict__}')
+	zdrojovy.delete()
+	# Spojit osoby (separátní funkce).
+	merge_osoby(cilovy.osoba, zdrosoba)
+
+	input("Potvrdit transakci řešitelů (^C pro zrušení) ")
+
+@transaction.atomic
+def merge_osoby(cilova, zdrojova):
+	""" Spojí dvě osoby do cílové
+
+	Nehlídá omezení typu "max 1 řešitel na osobu", to by měla hlídat databáze (OneToOneField)."""
+	# Sjednocení dat
+	print('Sjednocuji data osob')
+	# ID, User neřešíme, poznámku vyřešíme separátně.
+	fieldy = ['datum_narozeni', 'datum_registrace', 'datum_souhlasu_udaje',
+			  'datum_souhlasu_zasilani', 'email', 'foto', 'jmeno', 'mesto',
+			  'osloveni', 'prezdivka', 'prijmeni', 'psc', 'stat', 'telefon', 'ulice']
+	for f in fieldy:
+		zf = getattr(zdrojova, f)
+		cf = getattr(cilova, f)
+		if cf == zf:
+			print(f' Údaj {f} je shodný ({zf})')
+		else:
+			if zf is None:
+				print(f' Údaj {f} je pouze v cílové, používám')
+				continue
+			if cf is None:
+				setattr(cilova, f, zf)
+				cilova.poznamka += f'\nDEBUG: Merge: doplnéný údaj {f} ze zdrojové: {zf}'
+				print(f" Přiřazuji {f} ze zdrojové: {zf}")
+				continue
+			# Jsou fakt různé…
+			# FIXME: chybí možnost na vlastní úpravu…
+			verdikt = input(f"\n\n Údaj {f} se u osoby {cilova} ({cilova.id}) liší:\n  Zdrojový: {zf}\n  Cílový: {cf}\n Který použít, [z]drojový, [c]ílový? ")
+			verdikt = verdikt[0].casefold()
+			if verdikt == 'z':
+				setattr(cilova, f, zf)
+				cilova.poznamka += f'\nDEBUG: Merge: pro {f} použit údaj {zf} (zdrojová), nepoužit {cf} (cílová)'
+			elif verdikt == 'c':
+				cilova.poznamka += f'\nDEBUG: Merge: pro {f} použit údaj {cf} (cílová), nepoužit {zf} (zdrojová)'
+			else: raise ValueError('Špatná odpověď, řešitel pravděpodobně neuložen')
+	# poznámku chceme nezahodit…
+	cilova.poznamka += f'\nDEBUG: Merge: Původní poznámka: {zdrojova.poznamka}'
+	print(f' Výsledná osoba: {cilova.__dict__}, ukládám')
+	cilova.save()
+
+	# Vazby: Řešitel, User, Příjemce, Organizátor, Škola.kontaktní_osoba
+	print('Přepojuji vazby')
+	ct = Skola.objects.filter(kontaktni_osoba=zdrojova).update(kontaktni_osoba=cilova)
+	print(f' Přepojeno {ct} kontaktních osob')
+	# Ostatní vazby vyřeší OneToOneFieldy, ale někdy nemusí existovat…
+	ct = Resitel.objects.filter(osoba=zdrojova).update(osoba=cilova)
+	print(f' Přepojeno {ct} řešitelů')
+	ct = Prijemce.objects.filter(osoba=zdrojova).update(osoba=cilova)
+	print(f' Přepojeno {ct} příjemců')
+	ct = Organizator.objects.filter(osoba=zdrojova).update(osoba=cilova)
+	print(f' Přepojeno {ct} organizátorů')
+	# Uživatelé vedou opačným směrem, radši chceme zkontrolovat, že jsou různí ručně:
+	if zdrojova.user != cilova.user:
+		# Jeden z nich může být nenastavený…
+		if zdrojova.user is None:
+			print('Uživatel je již v cílové osobě')
+		elif cilova.user is None:
+			print('Používám uživatele zdrojové osoby')
+			cilova.user = zdrojova.user
+		# Teď nemůžeme uložit, protože kolize uživatelů. Ukládat cílovou budeme až po smazání zdrojové.
+		else: raise ValueError('Osoby mají obě uživatele, radši padám')
+
+	# Uložení a mazání
+	print(f'Mažu zdrojovou osobu {zdrojova.__dict__}')
+	zdrojova.delete()
+	print(f'Ukládám cílovou osobu {cilova.__dict__}')
+	cilova.save()
+
+	input("Potvrdit transakci osob (^C pro zrušení) ")

prednasky/urls.py

@@ -1,5 +1,5 @@
 from django.urls import path
-from seminar.utils import org_required, resitel_or_org_required
+from personalni.utils import org_required, resitel_or_org_required
 from . import views
 
 urlpatterns = [

seminar/models/odevzdavatko.py

@@ -13,7 +13,7 @@ from seminar.models import tvorba as am
 from seminar.models import treenode as tm
 from seminar.models import base as bm
 
-from seminar.utils import vzorecek_na_prepocet, inverze_vzorecku_na_prepocet
+from odevzdavatko.utils import vzorecek_na_prepocet, inverze_vzorecku_na_prepocet
 from personalni.models import Resitel
 
 

seminar/models/tvorba.py

@@ -23,7 +23,7 @@ from taggit.managers import TaggableManager
 
 from reversion import revisions as reversion
 
-from seminar.utils import roman
+from tvorba.utils import roman, aktivniResitele
 from treenode import treelib
 
 from unidecode import unidecode # Používám pro získání ID odkazu (ještě je to někde po někom zakomentované)
@@ -31,7 +31,6 @@ from unidecode import unidecode # Používám pro získání ID odkazu (ještě
 from polymorphic.models import PolymorphicModel
 
 from django.core.mail import EmailMessage
-from seminar.utils import aktivniResitele
 
 from personalni.models import Prijemce, Organizator
 

seminar/utils.py

@@ -1,387 +0,0 @@
-import datetime
-import decimal
-
-from django.contrib.auth import get_user_model
-from django.contrib.auth.decorators import permission_required, \
-	user_passes_test
-from django import views as DjangoViews
-
-from django.db import transaction
-
-from django.contrib.auth.models import AnonymousUser
-from django.contrib.contenttypes.models import ContentType
-from django.core.exceptions import ObjectDoesNotExist
-
-import logging
-
-import seminar.models as m
-import treenode.treelib as t
-
-logger = logging.getLogger(__name__)
-
-org_required = permission_required('auth.org')
-resitel_required = permission_required('auth.resitel')
-
-
-# inspirováno django.contrib.auth.decorators permission_required
-def check_perms(user):
-	if user.has_perms(('auth.resitel',)):
-		return True
-	if user.has_perms(('auth.org',)):
-		return True
-	return False
-
-
-resitel_or_org_required = user_passes_test(check_perms)
-
-User = get_user_model()
-# Není to úplně hezké, ale budeme doufat, že to je funkční...
-User.je_org = property(lambda self: self.has_perm('auth.org'))
-User.je_resitel = property(lambda self: self.has_perm('auth.resitel'))
-AnonymousUser.je_org = False
-AnonymousUser.je_resitel = False
-
-
-def vzorecek_na_prepocet(body, resitelu):
-	""" Vzoreček na přepočet plných bodů na parciálni, když má řešení více řešitelů. """
-	return body * 3 / (resitelu + 2)
-
-
-def inverze_vzorecku_na_prepocet(body: decimal.Decimal, resitelu) -> decimal.Decimal:
-	""" Vzoreček na přepočet parciálních bodů na plné, když má řešení více řešitelů. """
-	return round(body * (resitelu + 2) / 3, 1)
-
-
-def histogram(seznam):
-	d = {}
-	for i in seznam:
-		if i not in d:
-			d[i] = 0
-		d[i] += 1
-	return d
-
-# Pozor: zarovnáno velmi netradičně pro přehlednost
-roman_numerals = zip((1000, 900, 500, 400,  100, 90,   50,  40,   10,  9,    5,   4,    1),
-                     ('M', 'CM', 'D', 'CD', 'C', 'XC', 'L', 'XL', 'X', 'IX', 'V', 'IV', 'I'))
-
-
-def roman(num):
-	res = ""
-	for i, n in roman_numerals:
-		res += n * (num // i)
-		num %= i
-	return res
-
-
-def from_roman(rom):
-	if not rom:
-		return 0
-	for i, n in roman_numerals:
-		if rom.upper().startswith(n):
-			return i + from_roman(rom[len(n):])
-	raise Exception('Invalid roman numeral: "%s"', rom)
-
-
-def seznam_problemu():
-	"""Funkce pro hledání nekonzistencí v databázi a dalších nežádoucích stavů webu/databáze.
-
-	Nijak nesouvisí s Problémy zadanými řešitelům."""
-	# FIXME: přejmenovat funkci?
-	# FIXME: Tak, jak je napsaná, asi spíš patří někam k views a ne do utils (?)
-	problemy = []
-
-	# Pomocna fce k formatovani problemovych hlasek
-	def prb(cls, msg, objs=None):
-		s = '<b>%s:</b> %s' % (cls.__name__, msg)
-		if objs:
-			s += ' ['
-			for o in objs:
-				try:
-					url = o.admin_url()
-				except:
-					url = None
-				if url:
-					s += '<a href="%s">%s</a>, ' % (url, o.pk,)
-				else:
-					s += '%s, ' % (o.pk,)
-			s = s[:-2] + ']'
-		problemy.append(s)
-
-	# Duplicita jmen
-	jmena = {}
-	for r in m.Resitel.objects.all():
-		j = r.osoba.plne_jmeno()
-		if j not in jmena:
-			jmena[j] = []
-		jmena[j].append(r)
-	for j in jmena:
-		if len(jmena[j]) > 1:
-			prb(m.Resitel, 'Duplicitní jméno "%s"' % (j,), jmena[j])
-
-	# Data maturity a narození
-	for r in m.Resitel.objects.all():
-		if not r.rok_maturity:
-			prb(m.Resitel, 'Neznámý rok maturity', [r])
-		if r.rok_maturity and (r.rok_maturity < 1990 or r.rok_maturity > datetime.date.today().year + 10):
-			prb(m.Resitel, 'Podezřelé datum maturity', [r])
-		if r.osoba.datum_narozeni and (
-				r.osoba.datum_narozeni.year < 1970 or r.osoba.datum_narozeni.year > datetime.date.today().year - 12):
-			prb(m.Resitel, 'Podezřelé datum narození', [r])
-#		if not r.email:
-#			prb(Resitel, u'Neznámý email', [r])
-
-	## Kontroly konzistence databáze a TreeNodů
-
-	# Články
-	for clanek in m.Clanek.objects.all():
-		# získáme řešení svázané se článkem a z něj node ve stromě
-		reseni = clanek.reseni_set
-		if (reseni.count() != 1):
-			raise ValueError("Článek k sobě má nejedno řešení!")
-		r = reseni.first()
-		clanek_node = r.text_cely	# vazba na ReseniNode z Reseni
-		# content type je věc pomáhající rozeznávat různé typy objektů v django-polymorphic
-		# protože isinstance vrátí vždy jen TreeNode
-		# https://django-polymorphic.readthedocs.io/en/stable/migrating.html
-		cislonode_ct = ContentType.objects.get_for_model(m.CisloNode)
-		node = clanek_node
-		while node is not None:
-			node_ct = node.polymorphic_ctype
-			if node_ct == cislonode_ct:	# dostali jsme se k CisloNode
-				# zkontrolujeme, že stromové číslo odpovídá atributu
-				# .cislonode je opačná vazba k treenode_ptr, abychom z TreeNode dostali
-				# CisloNode
-				if clanek.cislo != node.cislonode.cislo:
-					prb(m.Clanek, "Číslo otištění uložené u článku nesedí s "
-						"číslem otištění podle struktury treenodů.", [clanek])
-				break
-			node = t.get_parent(node)
-
-	return problemy
-
-
-### Generovani obalek
-def resi_v_rocniku(rocnik, cislo=None):
-	""" Vrátí seznam řešitelů, co vyřešili nějaký problém v daném ročníku, do daného čísla.
-	Parametry:
-		rocnik (typu Rocnik)	ročník, ze kterého chci řešitele, co něco odevzdali
-		cislo (typu Cislo)	číslo, do kterého včetně se počítá, že v daném
-					ročníku řešitel něco poslal.
-					Pokud není zadané, počítají se všechna řešení z daného ročníku.
-	Výstup:
-		QuerySet objektů typu Resitel """
-
-	if cislo is None:
-		# filtrujeme pouze podle ročníku
-		return m.Resitel.objects.filter(rok_maturity__gte=rocnik.druhy_rok(),
-										reseni__hodnoceni__deadline_body__cislo__rocnik=rocnik).distinct()
-	else:  # filtrujeme podle ročníku i čísla
-		return m.Resitel.objects.filter(rok_maturity__gte=rocnik.druhy_rok(),
-										reseni__hodnoceni__deadline_body__cislo__rocnik=rocnik,
-										reseni__hodnoceni__deadline_body__cislo__poradi__lte=cislo.poradi).distinct()
-
-
-def aktivniResitele(cislo, pouze_letosni=False):
-	""" Vrací QuerySet aktivních řešitelů, což jsou ti, co ještě neodmaturovali
-	a letos něco poslali (anebo loni něco poslali, pokud jde o první tři čísla).
-	Parametry:
-		cislo (typu Cislo)	číslo, o které se jedná
-		pouze_letosni		jen řešitelé, kteří tento rok něco poslali
-
-	"""
-	letos = cislo.rocnik
-
-	# detekujeme, zda jde o první tři čísla či nikoli (tj. zda spamovat řešitele z minulého roku)
-	zacatek_rocniku = True
-	try:
-		if int(cislo.poradi) > 3:
-			zacatek_rocniku = False
-	except ValueError:
-		# if cislo.poradi != '7-8':
-		# 	raise ValueError(f'{cislo} je neplatné číslo čísla (není int a není 7-8)')
-		zacatek_rocniku = False
-
-	# nehledě na číslo chceme jen řešitele, kteří letos něco odevzdali
-	if pouze_letosni:
-		zacatek_rocniku = False
-
-	try:
-		loni = m.Rocnik.objects.get(rocnik=letos.rocnik - 1)
-	except ObjectDoesNotExist:
-		# Pro první ročník neexistuje ročník předchozí
-		zacatek_rocniku = False
-
-	if not zacatek_rocniku:
-		return resi_v_rocniku(letos, cislo).filter(rok_maturity__gte=letos.druhy_rok())
-	else:
-		# spojíme querysety s řešiteli loni a letos do daného čísla
-		return (resi_v_rocniku(loni) | resi_v_rocniku(letos, cislo)).distinct().filter(rok_maturity__gte=letos.druhy_rok())
-
-def viewMethodSwitch(get, post):
-	"""
-	Vrátí view, který zavolá různé jiné views podle toho, kterou metodou je zavolán.
-
-	Inspirováno https://docs.djangoproject.com/en/3.1/topics/class-based-views/mixins/#an-alternative-better-solution, jen jsem to udělal genericky.
-
-	Parametry:
-		post	view pro metodu POST
-		get	view pro metodu GET
-	
-	V obou případech se míní už view jakožto funkce, takže u class-based views se už má použít .as_view()
-
-	TODO: Podpora i pro metodu HEAD? A možná i pro FILES?
-	"""
-
-	theGetView = get
-	thePostView = post
-
-	class NewView(DjangoViews.View):
-		def get(self, request, *args, **kwargs):
-			return theGetView(request, *args, **kwargs)
-		def post(self, request, *args, **kwargs):
-			return thePostView(request, *args, **kwargs)
-	
-	return NewView.as_view()
-
-
-def sync_skoly(base_url):
-	"""Stáhne všechny školy z mamwebu na adrese <base_url> a uloží je do databáze"""
-	from django.urls import reverse
-	full_url = base_url.rstrip('/') + reverse('export_skoly')
-	import requests
-	from django.core import serializers
-	json =  requests.get(full_url, stream=True).content
-	for skola in serializers.deserialize('json', json):
-		skola.save()
-
-@transaction.atomic
-def merge_resitele(cilovy, zdrojovy):
-	"""Spojí dva řešitelské objekty do cílového.
-
-	Pojmenování "zdrojový" je silně nepřiléhající, ale co už…"""
-
-	# Postup:
-	# Sjednotit / upravit informace cílového řešitele
-	print('Upravuji data modelu')
-	fieldy_shoda = ['skola', 'rok_maturity', 'zasilat', 'zasilat_cislo_emailem', 'zasilat_cislo_papirove']
-	
-	for f in fieldy_shoda:
-		zf = getattr(zdrojovy, f)
-		cf = getattr(cilovy, f)
-		if cf == zf:
-			print(f' Údaj {f} je shodný ({zf})')
-		else:
-			if zf is None:
-				print(f' Údaj {f} je pouze v cílovém, používám')
-				continue
-			if cf is None:
-				setattr(cilovy, f, zf)
-				cilovy.poznamka += f'\nDEBUG: Merge: doplnéný údaj {f} ze zdrojového: {zf}'
-				print(f" Přiřazuji {f} ze zdrojového: {zf}")
-				continue
-			# Jsou fakt různé…
-			# FIXME: chybí možnost na vlastní úpravu…
-			verdikt = input(f"\n\n Údaj {f} se u řešitele {cilovy} ({cilovy.id}) liší:\n  Zdrojový: {zf}\n  Cílový: {cf}\n Který použít, [z]drojový, [c]ílový? ")
-			verdikt = verdikt[0].casefold()
-			if verdikt == 'z':
-				setattr(cilovy, f, zf)
-				cilovy.poznamka += f'\nDEBUG: Merge: pro {f} použit údaj {zf} (zdrojový), nepoužit {cf} (cílový)'
-			elif verdikt == 'c':
-				cilovy.poznamka += f'\nDEBUG: Merge: pro {f} použit údaj {cf} (cílový), nepoužit {zf} (zdrojový)'
-			else: raise ValueError('Špatná odpověď, řešitel pravděpodobně neuložen')
-	# poznámku chceme nezahodit…
-	cilovy.poznamka += f'\nDEBUG: Merge: Původní poznámka: {zdrojovy.poznamka}'
-	print(f' Výsledný řešitel: {cilovy.__dict__}, ukládám')
-	cilovy.save()
-
-
-	# Přepojit všechny vazby ze zdrojového na cílového
-	print('Přepojuji vazby')
-	# Vazby: Škola (hotovo), Řešení_Řešitelé, Konfery_Účastníci, Soustředění_Účastníci, Osoba (vyřeší se později, nejde přepojit)
-	ct = m.Reseni_Resitele.objects.filter(resitele=zdrojovy).update(resitele=cilovy)
-	print(f' Přepojeno {ct} řešení')
-	ct = m.Konfery_Ucastnici.objects.filter(resitel=zdrojovy).update(resitel=cilovy)
-	print(f' Přepojeno {ct} konfer')
-	ct = m.Soustredeni_Ucastnici.objects.filter(resitel=zdrojovy).update(resitel=cilovy)
-	print(f' Přepojeno {ct} sousů')
-
-	# Teď by na zdrojovém řešiteli nemělo nic viset, smazat ho, pamatujíce si jeho Osobu
-	zdrosoba = zdrojovy.osoba
-	print(f'Mažu zdrojového řešitele {zdrojovy.__dict__}')
-	zdrojovy.delete()
-	# Spojit osoby (separátní funkce).
-	merge_osoby(cilovy.osoba, zdrosoba)
-
-	input("Potvrdit transakci řešitelů (^C pro zrušení) ")
-
-@transaction.atomic
-def merge_osoby(cilova, zdrojova):
-	""" Spojí dvě osoby do cílové
-
-	Nehlídá omezení typu "max 1 řešitel na osobu", to by měla hlídat databáze (OneToOneField)."""
-	# Sjednocení dat
-	print('Sjednocuji data osob')
-	# ID, User neřešíme, poznámku vyřešíme separátně.
-	fieldy = ['datum_narozeni', 'datum_registrace', 'datum_souhlasu_udaje',
-			'datum_souhlasu_zasilani', 'email', 'foto', 'jmeno', 'mesto',
-			'osloveni', 'prezdivka', 'prijmeni', 'psc', 'stat', 'telefon', 'ulice']
-	for f in fieldy:
-		zf = getattr(zdrojova, f)
-		cf = getattr(cilova, f)
-		if cf == zf:
-			print(f' Údaj {f} je shodný ({zf})')
-		else:
-			if zf is None:
-				print(f' Údaj {f} je pouze v cílové, používám')
-				continue
-			if cf is None:
-				setattr(cilova, f, zf)
-				cilova.poznamka += f'\nDEBUG: Merge: doplnéný údaj {f} ze zdrojové: {zf}'
-				print(f" Přiřazuji {f} ze zdrojové: {zf}")
-				continue
-			# Jsou fakt různé…
-			# FIXME: chybí možnost na vlastní úpravu…
-			verdikt = input(f"\n\n Údaj {f} se u osoby {cilova} ({cilova.id}) liší:\n  Zdrojový: {zf}\n  Cílový: {cf}\n Který použít, [z]drojový, [c]ílový? ")
-			verdikt = verdikt[0].casefold()
-			if verdikt == 'z':
-				setattr(cilova, f, zf)
-				cilova.poznamka += f'\nDEBUG: Merge: pro {f} použit údaj {zf} (zdrojová), nepoužit {cf} (cílová)'
-			elif verdikt == 'c':
-				cilova.poznamka += f'\nDEBUG: Merge: pro {f} použit údaj {cf} (cílová), nepoužit {zf} (zdrojová)'
-			else: raise ValueError('Špatná odpověď, řešitel pravděpodobně neuložen')
-	# poznámku chceme nezahodit…
-	cilova.poznamka += f'\nDEBUG: Merge: Původní poznámka: {zdrojova.poznamka}'
-	print(f' Výsledná osoba: {cilova.__dict__}, ukládám')
-	cilova.save()
-
-	# Vazby: Řešitel, User, Příjemce, Organizátor, Škola.kontaktní_osoba
-	print('Přepojuji vazby')
-	ct = m.Skola.objects.filter(kontaktni_osoba=zdrojova).update(kontaktni_osoba=cilova)
-	print(f' Přepojeno {ct} kontaktních osob')
-	# Ostatní vazby vyřeší OneToOneFieldy, ale někdy nemusí existovat…
-	ct = m.Resitel.objects.filter(osoba=zdrojova).update(osoba=cilova)
-	print(f' Přepojeno {ct} řešitelů')
-	ct = m.Prijemce.objects.filter(osoba=zdrojova).update(osoba=cilova)
-	print(f' Přepojeno {ct} příjemců')
-	ct = m.Organizator.objects.filter(osoba=zdrojova).update(osoba=cilova)
-	print(f' Přepojeno {ct} organizátorů')
-	# Uživatelé vedou opačným směrem, radši chceme zkontrolovat, že jsou různí ručně:
-	if zdrojova.user != cilova.user:
-		# Jeden z nich může být nenastavený…
-		if zdrojova.user is None:
-			print('Uživatel je již v cílové osobě')
-		elif cilova.user is None:
-			print('Používám uživatele zdrojové osoby')
-			cilova.user = zdrojova.user
-			# Teď nemůžeme uložit, protože kolize uživatelů. Ukládat cílovou budeme až po smazání zdrojové.
-		else: raise ValueError('Osoby mají obě uživatele, radši padám')
-	
-	# Uložení a mazání
-	print(f'Mažu zdrojovou osobu {zdrojova.__dict__}')
-	zdrojova.delete()
-	print(f'Ukládám cílovou osobu {cilova.__dict__}')
-	cilova.save()
-
-	input("Potvrdit transakci osob (^C pro zrušení) ")
-
-

sifrovacka/urls.py

@@ -1,6 +1,6 @@
 from django.urls import path
 
-from seminar.utils import org_required, resitel_or_org_required
+from personalni.utils import org_required, resitel_or_org_required
 from .views import SifrovackaView, SifrovackaListView, NapovedaView, NapovedaListView, PreskoceniView
 
 urlpatterns = [

soustredeni/urls.py

@@ -1,6 +1,6 @@
 from django.urls import path, include
 from . import views
-from seminar.utils import org_required
+from personalni.utils import org_required
 
 urlpatterns = [
 	path(

tvorba/urls.py

@@ -1,6 +1,6 @@
 from django.urls import path, include, re_path
 from . import views
-from seminar.utils import org_required
+from personalni.utils import org_required
 
 urlpatterns = [
 #	path('aktualni/temata/', views.TemataRozcestnikView),

tvorba/utils.py

@@ -0,0 +1,89 @@
+from django.core.exceptions import ObjectDoesNotExist
+
+import personalni.models
+
+import seminar.models as m
+
+
+def resi_v_rocniku(rocnik, cislo=None):
+	""" Vrátí seznam řešitelů, co vyřešili nějaký problém v daném ročníku, do daného čísla.
+	Parametry:
+		rocnik (typu Rocnik)	ročník, ze kterého chci řešitele, co něco odevzdali
+		cislo (typu Cislo)	číslo, do kterého včetně se počítá, že v daném
+					ročníku řešitel něco poslal.
+					Pokud není zadané, počítají se všechna řešení z daného ročníku.
+	Výstup:
+		QuerySet objektů typu Resitel """
+
+	if cislo is None:
+		# filtrujeme pouze podle ročníku
+		return personalni.models.Resitel.objects.filter(
+			rok_maturity__gte=rocnik.druhy_rok(),
+			reseni__hodnoceni__deadline_body__cislo__rocnik=rocnik
+		).distinct()
+	else:  # filtrujeme podle ročníku i čísla
+		return personalni.models.Resitel.objects.filter(
+			rok_maturity__gte=rocnik.druhy_rok(),
+			reseni__hodnoceni__deadline_body__cislo__rocnik=rocnik,
+			reseni__hodnoceni__deadline_body__cislo__poradi__lte=cislo.poradi
+		).distinct()
+
+
+def aktivniResitele(cislo, pouze_letosni=False):
+	""" Vrací QuerySet aktivních řešitelů, což jsou ti, co ještě neodmaturovali
+	a letos něco poslali (anebo loni něco poslali, pokud jde o první tři čísla).
+	Parametry:
+		cislo (typu Cislo)	číslo, o které se jedná
+		pouze_letosni		jen řešitelé, kteří tento rok něco poslali
+
+	"""
+	letos = cislo.rocnik
+
+	# detekujeme, zda jde o první tři čísla či nikoli (tj. zda spamovat řešitele z minulého roku)
+	zacatek_rocniku = True
+	try:
+		if int(cislo.poradi) > 3:
+			zacatek_rocniku = False
+	except ValueError:
+		# if cislo.poradi != '7-8':
+		# 	raise ValueError(f'{cislo} je neplatné číslo čísla (není int a není 7-8)')
+		zacatek_rocniku = False
+
+	# nehledě na číslo chceme jen řešitele, kteří letos něco odevzdali
+	if pouze_letosni:
+		zacatek_rocniku = False
+
+	try:
+		loni = m.Rocnik.objects.get(rocnik=letos.rocnik - 1)
+	except ObjectDoesNotExist:
+		# Pro první ročník neexistuje ročník předchozí
+		zacatek_rocniku = False
+
+	if not zacatek_rocniku:
+		return resi_v_rocniku(letos, cislo).filter(rok_maturity__gte=letos.druhy_rok())
+	else:
+		# spojíme querysety s řešiteli loni a letos do daného čísla
+		return (resi_v_rocniku(loni) | resi_v_rocniku(letos, cislo))\
+			.distinct().filter(rok_maturity__gte=letos.druhy_rok())
+
+
+# Pozor: zarovnáno velmi netradičně pro přehlednost
+roman_numerals = zip((1000, 900, 500, 400,  100, 90,   50,  40,   10,  9,    5,   4,    1),  # noqa
+					 ('M', 'CM', 'D', 'CD', 'C', 'XC', 'L', 'XL', 'X', 'IX', 'V', 'IV', 'I'))  # noqa
+
+
+def roman(num):
+	res = ""
+	for i, n in roman_numerals:
+		res += n * (num // i)
+		num %= i
+	return res
+
+
+def from_roman(rom):
+	if not rom:
+		return 0
+	for i, n in roman_numerals:
+		if rom.upper().startswith(n):
+			return i + from_roman(rom[len(n):])
+	raise Exception('Invalid roman numeral: "%s"', rom)

tvorba/views/views_all.py

@@ -15,7 +15,6 @@ from seminar.models import Problem, Cislo, Reseni, Nastaveni, Rocnik, \
 	Resitel, Novinky, Tema, Clanek, \
 	Deadline  # Tohle je stare a chceme se toho zbavit. Pouzivejte s.ToCoChci
 #from .models import VysledkyZaCislo, VysledkyKCisluZaRocnik, VysledkyKCisluOdjakziva
-from seminar import utils
 from treenode import treelib
 import treenode.templatetags as tnltt
 import treenode.serializers as vr
@@ -32,9 +31,10 @@ import unicodedata
 import logging
 import time
 
-from seminar.utils import aktivniResitele
 import personalni.views
 
+from .. import utils
+
 # ze starého modelu
 #def verejna_temata(rocnik):
 #	"""
@@ -368,7 +368,7 @@ class OdmenyView(generic.TemplateView):
 		context = super().get_context_data(**kwargs)
 		fromcislo = get_object_or_404(Cislo, rocnik=self.kwargs.get('frocnik'), poradi=self.kwargs.get('fcislo'))
 		tocislo = get_object_or_404(Cislo, rocnik=self.kwargs.get('trocnik'), poradi=self.kwargs.get('tcislo'))
-		resitele = aktivniResitele(tocislo)
+		resitele = utils.aktivniResitele(tocislo)
 
 		def get_diff(from_deadline: Deadline, to_deadline: Deadline):
 			frombody = body_resitelu(resitele=resitele, jen_verejne=False, do=from_deadline)
@@ -481,7 +481,7 @@ class RocnikVysledkovkaView(RocnikView):
 
 def cisloObalkyView(request, rocnik, cislo):
 	realne_cislo = get_object_or_404(Cislo, poradi=cislo, rocnik__rocnik=rocnik)
-	return personalni.views.obalkyView(request, aktivniResitele(realne_cislo))
+	return personalni.views.obalkyView(request, utils.aktivniResitele(realne_cislo))
 
 
 

various/urls.py

@@ -1,6 +1,6 @@
 from django.urls import path
 from .views.final import TitulniStranaView, JakResitView, StavDatabazeView
-from seminar.utils import org_required
+from personalni.utils import org_required
 
 urlpatterns = [
 	path('', TitulniStranaView.as_view(), name='titulni_strana'),

various/views/final.py

@@ -4,13 +4,15 @@ Stránky, které se mi nepovedlo lépe zařadit.
 Oproti `./pomocne.py` se tyto views používají přímo ve various
 a naopak importují spoustu věcí odjinud
 """
+import datetime
 
+from django.contrib.contenttypes.models import ContentType
 from django.shortcuts import get_object_or_404, render
 from django.utils import timezone
 from django.views import generic
 
 import novinky.views
-import seminar.utils
+import treenode.treelib as t
 import tvorba.views
 from personalni.models import Resitel
 from seminar import models as m
@@ -56,9 +58,94 @@ class JakResitView(generic.ListView):
 
 
 ### Status
+def histogram(seznam):
+	d = {}
+	for i in seznam:
+		if i not in d:
+			d[i] = 0
+		d[i] += 1
+	return d
+
+
+def seznam_problemu():
+	"""Funkce pro hledání nekonzistencí v databázi a dalších nežádoucích stavů webu/databáze.
+
+	Nijak nesouvisí s Problémy zadanými řešitelům."""
+	# FIXME: přejmenovat funkci?
+	problemy = []
+
+	# Pomocna fce k formatovani problemovych hlasek
+	def prb(cls, msg, objs=None):
+		s = '<b>%s:</b> %s' % (cls.__name__, msg)
+		if objs:
+			s += ' ['
+			for o in objs:
+				try:
+					url = o.admin_url()
+				except:
+					url = None
+				if url:
+					s += '<a href="%s">%s</a>, ' % (url, o.pk,)
+				else:
+					s += '%s, ' % (o.pk,)
+			s = s[:-2] + ']'
+		problemy.append(s)
+
+	# Duplicita jmen
+	jmena = {}
+	for r in m.Resitel.objects.all():
+		j = r.osoba.plne_jmeno()
+		if j not in jmena:
+			jmena[j] = []
+		jmena[j].append(r)
+	for j in jmena:
+		if len(jmena[j]) > 1:
+			prb(m.Resitel, 'Duplicitní jméno "%s"' % (j,), jmena[j])
+
+	# Data maturity a narození
+	for r in m.Resitel.objects.all():
+		if not r.rok_maturity:
+			prb(m.Resitel, 'Neznámý rok maturity', [r])
+		if r.rok_maturity and (r.rok_maturity < 1990 or r.rok_maturity > datetime.date.today().year + 10):
+			prb(m.Resitel, 'Podezřelé datum maturity', [r])
+		if r.osoba.datum_narozeni and (
+				r.osoba.datum_narozeni.year < 1970 or r.osoba.datum_narozeni.year > datetime.date.today().year - 12):
+			prb(m.Resitel, 'Podezřelé datum narození', [r])
+	#		if not r.email:
+	#			prb(Resitel, u'Neznámý email', [r])
+
+	## Kontroly konzistence databáze a TreeNodů
+
+	# Články
+	for clanek in m.Clanek.objects.all():
+		# získáme řešení svázané se článkem a z něj node ve stromě
+		reseni = clanek.reseni_set
+		if (reseni.count() != 1):
+			raise ValueError("Článek k sobě má nejedno řešení!")
+		r = reseni.first()
+		clanek_node = r.text_cely	# vazba na ReseniNode z Reseni
+		# content type je věc pomáhající rozeznávat různé typy objektů v django-polymorphic
+		# protože isinstance vrátí vždy jen TreeNode
+		# https://django-polymorphic.readthedocs.io/en/stable/migrating.html
+		cislonode_ct = ContentType.objects.get_for_model(m.CisloNode)
+		node = clanek_node
+		while node is not None:
+			node_ct = node.polymorphic_ctype
+			if node_ct == cislonode_ct:	# dostali jsme se k CisloNode
+				# zkontrolujeme, že stromové číslo odpovídá atributu
+				# .cislonode je opačná vazba k treenode_ptr, abychom z TreeNode dostali
+				# CisloNode
+				if clanek.cislo != node.cislonode.cislo:
+					prb(m.Clanek, "Číslo otištění uložené u článku nesedí s "
+								  "číslem otištění podle struktury treenodů.", [clanek])
+				break
+			node = t.get_parent(node)
+
+	return problemy
+
 def StavDatabazeView(request):
 	# nastaveni = Nastaveni.objects.get()
-	problemy = seminar.utils.seznam_problemu()
+	problemy = seznam_problemu()
 	muzi = Resitel.objects.filter(osoba__osloveni=m.Osoba.OSLOVENI_MUZSKE)
 	zeny = Resitel.objects.filter(osoba__osloveni=m.Osoba.OSLOVENI_ZENSKE)
 	return render(request, 'various/stav_databaze.html', {
@@ -68,6 +155,6 @@ def StavDatabazeView(request):
 		'resitele': Resitel.objects.all(),
 		'muzi': muzi,
 		'zeny': zeny,
-		'jmena_muzu': seminar.utils.histogram([r.osoba.jmeno for r in muzi]),
+		'jmena_muzu': histogram([r.osoba.jmeno for r in muzi]),
-		'jmena_zen': seminar.utils.histogram([r.osoba.jmeno for r in zeny]),
+		'jmena_zen': histogram([r.osoba.jmeno for r in zeny]),
 	})

various/views/generic.py

@@ -0,0 +1,29 @@
+import django.views
+
+
+def viewMethodSwitch(get, post):
+	"""
+	Vrátí view, který zavolá různé jiné views podle toho, kterou metodou je zavolán.
+
+	Inspirováno https://docs.djangoproject.com/en/3.1/topics/class-based-views/mixins/#an-alternative-better-solution, jen jsem to udělal genericky.
+
+	Parametry:
+		post	view pro metodu POST
+		get	view pro metodu GET
+
+	V obou případech se míní už view jakožto funkce, takže u class-based views se už má použít .as_view()
+
+	TODO: Podpora i pro metodu HEAD? A možná i pro FILES?
+	"""
+
+	theGetView = get
+	thePostView = post
+
+	class NewView(django.views.View):
+		def get(self, request, *args, **kwargs):
+			return theGetView(request, *args, **kwargs)
+
+		def post(self, request, *args, **kwargs):
+			return thePostView(request, *args, **kwargs)
+
+	return NewView.as_view()

vyroci/urls.py

@@ -1,6 +1,6 @@
 from django.urls import path
 
-from seminar.utils import org_required
+from personalni.utils import org_required
 from .views import VyrociView, VyrociListView
 
 urlpatterns = [

vysledkovky/utils.py

@@ -4,7 +4,7 @@ from typing import Union, Iterable  # TODO: s pythonem 3.10 přepsat na '|'
 
 import seminar.models as m
 from django.db.models import Q, Sum
-from seminar.utils import resi_v_rocniku
+from tvorba.utils import resi_v_rocniku
 
 ROCNIK_ZRUSENI_TEMAT = 25