Merge branch 'master' into vylepseni_odevzdavatka

# Conflicts:
#	odevzdavatko/templates/odevzdavatko/detail.html

.editorconfig

@@ -1,17 +1,20 @@
+# Univerzální popis našich konvencí pro nastavení editorů.
+# Vizte https://editorconfig.org pro detaily
+
 root = true
 
 [*]
 charset = utf-8
-# Unix-y lines
+# Unixové řádky
 end_of_line = lf
 insert_final_newline = true
 
 [*.{py,css}]
 indent_style = tab
-# I do not think we prescribe, how big a tab is.
+# Nenařizujeme konkrétní šířku tabulátoru
 indent_size = unset
 
-# Migrations are auto-generated and thus follow PEP-8, let's not fight with that
+# Automaticky generované migrace dodržují PEP-8, nemá smysl s tím moc bojovat.
 [*/migrations/*.py]
 indent_style = space
 indent_size = 4

Makefile

@@ -1,154 +1,9 @@
-PYTHON ?= python3
+# Existence tohohle Makefile je tu jen proto, aby fungovala svalová paměť. Pokud můžete, použijte rovnou `make/…`
-VENV ?= ${PYTHON} -m venv
+%:
-# Všechny flagy, které se s venvem/virtualenvem/... mají volat patří sem. Volá se "${VENV} cesta"
+	# Používání make jako příkazu je zastaralé, prosím používej radši skripty ze složky make. Spouštím následující příkaz:
-VENV_PATH ?= env
+	make/$*
-# Musí být definovaná, i kdyby to měla být "."
 
-.PHONY: all venv_check clean install install_web install_venv clean_venv clean_schema run test deploy_test deploy_prod sync_test_media sync_test_db sync_test sync_local_media sync_local_db sync_local
+default:
+	@cat make/README.md
 
-# activate by mělo být předpokladem ke všemu, co volá webový python (i.e. python nasazený do ${VENV}u kvůli webu, např. manage.py)
+.PHONY: default
-all:
-	@# Just echo:
-	# Install je trochu magický:
-	# Spusť následující posloupnost příkazů:
-	#	make install_venv
-	#	. ${VENV_PATH}/bin/activate
-	#	make install_web
-	#
-	# Pokud install_web říká Error: pg_config executable not found. nainstaluj si libpq-dev
-	# Pokud chybová hláška obsahuje #include <Python.h>, nainstaluj si python3-dev
-	#
-	# Až skončíš s vývojem webu, spusť "deactivate". Tím zmizí '(${VENV_PATH})' ze začátku promptu.
-
-venv_check:
-	@# Pokud org nemá zapnutý venv, poradíme mu, aby si ho zapnul a spadneme. Jinak nic.
-	@expr $$PATH : ".*:*$(shell pwd)/${VENV_PATH}/bin" > /dev/null && exit 0 || echo 'Není zapnutý venv, spusť ". ${VENV_PATH}/bin/activate".\nPokud není venv nainstalovaný, spusť "make install_venv"' && false
-
-clean: clean_venv clean_schema
-
-install: install_web
-
-install_web: venv_check
-	@# venv může být příšerně starý, takže nejdříve upgradujeme venvové věci
-	pip install --upgrade pip
-	pip install --upgrade setuptools
-	# Instalace závislostí webu
-	pip install -r requirements.txt --upgrade
-	# Pro vygenerování tesdat spusť ./manage.py testdata
-	# Po vygenerování testdat spusť  ./manage.py loaddata data/*, ať máš menu a další modely
-	# Pro synchronizaci flatpages spusť make sync_prod_flatpages
-
-install_venv:
-	${VENV} ${VENV_PATH}
-
-clean_venv:
-	# Možná není 100% foolproof...
-	@test ! ${VENV_PATH} = . || ! echo "Smaž si prosím venv sám, nebudu mazat celý web"
-	rm -rfv ${VENV_PATH}
-	rm -f pip-selfcheck.json
-clean_schema:
-	rm -f schema_seminar.pdf schema_all.pdf
-
-run: venv_check
-	./manage.py runserver
-
-test: venv_check
-	./manage.py test -v2 seminar mamweb
-
-# DB schemata
-
-schema: schema_seminar.pdf schema_all.pdf
-
-schema_seminar.pdf: venv_check
-	./manage.py graph_models seminar | dot -Tpdf > schema_seminar.pdf
-
-schema_all.pdf: venv_check
-	./manage.py graph_models -a -g | dot -Tpdf > schema_all.pdf
-
-# Deploy to current *mamweb-test* directory
-deploy_test: venv_check
-	@if [ ${USER} != "mam-web" ]; then echo "Only possible by user mam-web"; exit 1; fi
-	@if [ `readlink -f .` != "/aux/akce/mam/www/mamweb-test" ]; then echo "Only possible in directory mamweb-test"; exit 1; fi
-	@echo "Installing version from origin/test ..."
-	git pull origin test
-	git clean -f
-	make install
-	./manage.py migrate 
-	./manage.py collectstatic --noinput
-	(chown -R :mam . || true )
-	(chmod -R g+rX,go-w . || true )
-	@echo Restarting systemd unit
-	systemctl --user restart mamweb-test.service
-	@echo Done.
-
-# Deploy to current *mamweb-prod* directory
-deploy_prod: venv_check
-	@if [ ${USER} != "mam-web" ]; then echo "Only possible by user mam-web"; exit 1; fi
-	@if [ `readlink -f .` != "/aux/akce/mam/www/mamweb-prod" ]; then echo "Only possible in directory mamweb-prod"; exit 1; fi
-	@echo "Backing up production DB ..."
-	( cd -P .. && ./backup_prod_db.sh )
-	@echo "Installing version from origin/master ..."
-	git pull origin master
-	git clean -f
-	make install
-	./manage.py migrate 
-	./manage.py collectstatic --noinput
-	(chown -R :mam . || true )
-	(chmod -R g+rX,go-w . || true )
-	@echo Restarting systemd user unit for MaM web
-	systemctl --user restart mamweb-prod.service
-	@echo Done.
-
-
-sync_prod_flatpages: venv_check 
-	@echo Downloading current version of flatpages from mamweb-prod.
-	ssh mam-web@gimli.ms.mff.cuni.cz \
-	"cd /akce/mam/www/mamweb-prod; . env/bin/activate; ./manage.py dumpdata flatpages --indent=2 > flat.json; ./fix_json.py flat.json flat_fixed.json"
-	rsync -ave ssh mam-web@gimli.ms.mff.cuni.cz:/akce/mam/www/mamweb-prod/flat_fixed.json data/flat.json
-	@echo "Applying downloaded flatpages."
-	./manage.py loaddata data/flat.json
-	@echo "Done."
-
-# Sync test media directory with production
-sync_test_media:
-	@if [ ${USER} != "mam-web" ]; then echo "Only possible by user mam-web"; exit 1; fi
-	@if [ `readlink -f .` != "/aux/akce/mam/www/mamweb-test" ]; then echo "Only possible in /akce/mam/www/mamweb-test"; exit 1; fi
-	rsync -av --delete /akce/mam/www/mamweb-prod/media/ ./media
-
-# Sync (with drop) test database with production database
-sync_test_db_aggressive:
-	@if [ ${USER} != "mam-web" ]; then echo "Only possible by user mam-web"; exit 1; fi
-	pg_dump mam_test > dump-test-`date +"%Y%m%d_%H%M"`.sql
-	pg_dump -Fc mam_prod > dump-prod.sql
-	@# I am not sure which shell is used, so I am calling bash to make sure
-	psql mam_test -c 'DROP OWNED BY "mam-web";'
-	pg_restore -c --if-exists -d mam_test dump-prod.sql
-	rm dump-prod.sql
-	psql mam_test -c "UPDATE django_site SET name='MaMweb (test)', domain='mam-test.ks.matfyz.cz' WHERE id=1"
-	@echo Done.
-
-# Sync test with production
-# HACK ALERT: using aggressive variant, due to the schemas being too different.
-sync_test: sync_test_media sync_test_db_aggressive
-
-
-# Sync media directory with atrey. Useful for local development with production database
-# Does not sync Galerie and CACHE (too huge). 
-sync_local_media:
-	rsync -ave ssh --exclude Galerie --exclude CACHE\
-		mam-web@gimli.ms.mff.cuni.cz:/akce/mam/www/mamweb-prod/media/ ./media/
-# Downloads and restores production database to local database. PostgreSQL only. 
-sync_local_db:
-	scp mam-web@gimli.ms.mff.cuni.cz:`ssh mam-web@gimli.ms.mff.cuni.cz 'ls -v /akce/mam/www/backups/mam_prod-*\.pgdump.xz | tail -n 1'` \
-		./last.pgdump.xz
-	xz -fd last.pgdump.xz
-	pg_restore -c -d mam-prod  last.pgdump
-
-# Sync database and media. See above lines
-sync_local: sync_local_media sync_local_db
-
-# Push local compiled Vue to gimli test site
-push_compiled_vue_to_test:
-	scp vue_frontend/webpack-stats.json mam-web@gimli:/akce/mam/www/mamweb-test/vue_frontend/
-	rsync -ave ssh seminar/static/seminar/vue mam-web@gimli:/akce/mam/www/mamweb-test/seminar/static/seminar/
-	ssh mam-web@gimli.ms.mff.cuni.cz 'cd /akce/mam/www/mamweb-test/ && . env/bin/activate && ./manage.py collectstatic --noinput'

README.md

@@ -1,77 +1,58 @@
-Basic commands for web development
+Web M&M
-==================================
+======
 
-After you clone this repository, run `make`. It will download, locally install
+Tohle je repozitář s kódem M&Mího webu. Pokud zde hledáte web samotný nebo
-and setup virtualenv and pip, and then locally install all required packages
+informace o semináři, najdete je na <https://mam.matfyz.cz> (a upřímně nechápu,
-from `requirements.txt`.
+jak jste se dostali k tomuhle textu :-D)
 
-When working with the code, always use the binaries in `./bin/`, such as
+Pokud jste tu zůstali, tak vás beztak zajímá vývoj webu (a jestli ne, tak
-`bin/pip`, `bin/python`, ... never the global python, pip, ...
+budeme rádi, když začne :-)).
-Use `make` and `./manage.py` for most things.
 
-Use git :-)
+Co je M&Mweb uvnitř
+------
+Celý náš web je napsaný v [Pythonu](https://www.python.org) ve frameworku
+[Django](https://www.djangoproject.com/). Web běží na serveru zvaném Gimli,
+jako databázi používá PostgreSQL (pro lokální vývoj naopak SQLite) a všechen
+náš kód je uložený v [Gitu](https://git-scm.com/) na [téhle
+gitee](https://gitea.ks.matfyz.cz/). Pro dokumentaci používáme
+[Sphinx](https://www.sphinx-doc.org).
 
-Quickstart
+<!--TODO: Z odstavce výše by ideálně měla být zachována jen první věta a zbytek
-----------
+by měl být někde v docs s podrobnějším popisem…-->
-Run the following commands:
-	make install_venv
-	. env/bin/activate
-	make install_web
 
-Pokud install_web říká Error: pg_config executable not found. nainstaluj si libpq-dev
+Jak si web pořídit
-Pokud chybová hláška obsahuje #include <Python.h>, nainstaluj si python3-dev
+------
+Prosím přečti si podrobnější návod v <docs/vyvoj.rst> (tady by bylo zbytečné
+ho duplikovat).
 
-After finishing development, run "deactivate".
+Jak web vyvíjet
+----
+<!--TODO: Napsat obšírněji, asi zase do docs/-->
 
-Make commands
+Na webu je mnoho věcí k dělání, některé ani nevyžadují kódění (třeba uhánění
--------------
+orgů, aby si psali medailonky, aktualizace fotek, …), některé se naopak týkají
+infrastruktury pod kódem (Gitea, Gimli, …). Je proto těžké mít na to úplně
+obecný návod, tak tady je zhruba návod na úpravy kódu a pokud se něco z toho
+nedá aplikovat, tak to prostě zkus nějak udělat jinak, po svém. (Omlouvám se
+neinformatikům, ale líp to teď nesepíšu :-))
 
-* `make install` (or `make`) - locally install and setup virtualpy, install
+1. Nejprve si stáhni repozitář a rozběhni si lokální web u sebe (viz <docs/vyvoj.rst>).
-  required packages. Ran again installs missing packages. Run after changing
+1. Najdi si problém v Kanboardu (klikni na „Issues“ na Gitee) a/nebo se domluv
-  `requirements.txt`.
+   s webaři, na čem bys tak mohl/a pracovat.
-
+1. Najdi místo, kde se to dá opravit a zkus to tam opravit. Uznávám, že tenhle
-* `make clean` - remove local python packages.
+   bod je otravně obecný, pokud tápeš, zkus se zeptat zkušenějších webařů nebo
-
+   podívat do dokumentace.
-* `make veryclean` - remove local packages and virtualpy enviroment and
+1. Vyzkoušej, že ti to lokálně funguje tak, jak má.
-  binaries.
+1. Zvládneš-li a máš-li čas, zkus to i zdokumentovat a/nebo napsat testy (TODO: chybí návod)
-
+1. Po dohodě s webaři to vyzkoušej na testwebu
-* `make run` - runs "./manage.py runserver_plus"
+1. Pošli pull-request a případně zkus reagovat na komentáře
-
+1. Až se změna začlení do hlavní větve (`master`) a nasadí se web na produkci,
-* `make push_test` - pushes the last commited version to test location.
+   můžeš mít radost, že se web bude používat lépe Tobě i ostatním orgům :-)
-  Only git-commited changes are pushed! Rest is re-generated from scratch.
-  At test server, the media data and database are kept the same.
-  Everything else not in .gitignore is deleted/overwritten on the test server.
-
-* `make schema` - generates graph of seminar and all schemas as PDF. Supercool!
-
-* `make sync_prod_flatpages` - downloads and applies static/flat pages from mamweb-prod
-
-./manage.py commands
---------------------
-
-* `./manage.py migrate` - update the database schema, initialise the database.
-  You need to run this in the beginning.
-
-* `./manage.py runserver_plus` - run a debugging server for the web. Slightly
-  enhanced compared to `./manage.py runserver`.
-  Open [127.0.0.1:8000](127.0.0.1:8000).
-
-* `./manage.py testdata` - create pseudo-random seminar data and admin/admin
-  user.
-
-* `./manage.py test` - run the tests.
-
-* `./manage.py shell` - run commands, list elemements of database, check syntax
-  by importing files, etc.
-
-Configurations
---------------
-
-* `mamweb/settings_common.py` contains most configuration options.
-* `mamweb/settings.py` is used only for local development.
-* `mamweb/settings_test.py` is used for testing on atrey.
-* `mamweb/settings_prod.py` is used in production deployment.
-
-These are automatically switched by `make`.
 
+### Proč pull-requesty?
+<!--FIXME: Tohle ale už úplně patří do docs a ne sem, jen je zatím nemám prozkoumané…-->
 
+Účelů pull-requestů je několik. Jednak doufáme, že pomůže webařům se orientovat
+v kódu, jednak tím umožňujeme dělat experimenty a dávat si zpětnou vazbu. V
+neposlední řadě pomáhají držet aspoň trochu konzistentní kód, což má pomoci
+pohodě při programování… (A asi jsem na něco zapomněl :-))

api/urls.py

@@ -24,6 +24,7 @@ urlpatterns = [
 	path('api/autocomplete/resitel/', org_required(views.ResitelAutocomplete.as_view()), name='autocomplete_resitel'),
 	path('api/autocomplete/resitel_public/', views.PublicResitelAutocomplete.as_view(), name='autocomplete_resitel_public'),
 	path('api/autocomplete/problem/odevzdatelny', views.OdevzdatelnyProblemAutocomplete.as_view(), name='autocomplete_problem_odevzdatelny'),
+	path('api/autocomplete/problem/vsechny', views.ProblemAutocomplete.as_view(), name='autocomplete_problem'),
 
 	# Ceka na autocomplete v3
 	# path('autocomplete/organizatori/',

api/views/autocomplete.py

@@ -86,6 +86,21 @@ class OdevzdatelnyProblemAutocomplete(autocomplete.Select2QuerySetView):
 					Q(nazev__icontains=self.q))
 		return qs
 
+class ProblemAutocomplete(autocomplete.Select2QuerySetView):
+	""" View k :mod:`dal.autocomplete` pro vyhledávání problémů především v odevzdávátku. """
+	def get_queryset(self):
+		# FIXME i starší úlohy
+		nastaveni = get_object_or_404(m.Nastaveni)
+		rocnik = nastaveni.aktualni_rocnik
+		temaQ = Q(Tema___rocnik = rocnik)
+		ulohaQ = Q(Uloha___cislo_zadani__rocnik=rocnik)
+		clanekQ = Q(Clanek___cislo__rocnik=rocnik)
+		qs = m.Problem.objects.filter(temaQ | ulohaQ | clanekQ).order_by("-stav", "nazev")
+		if self.q:
+			qs = qs.filter(
+					Q(nazev__icontains=self.q))
+		return qs
+
 # Ceka na autocomplete v3
 # class OrganizatorAutocomplete(autocomplete.Select2QuerySetView):
 # 	def get_queryset(self):

docs/index.rst

@@ -13,6 +13,7 @@ Vítejte v dokumentaci M&Mího webu!
    vyvoj
    sphinx
    dalsi_soubory
+   skripty
    modules/modules
    zapisy/zapisy
 

docs/skripty.rst

@@ -0,0 +1,108 @@
+Skripty pro práci s repozitářem
+===================
+
+Máme dvě hlavní sady skriptů/příkazů na ovládání webu a repozitáře. Skripty pro
+práci s webem psané v Pythonu jsou uložené ve složkách
+``<aplikace>/management/commands/``, případně vestavěné, a volají se pomocí
+``./manage.py <příkaz>``. Oproti tomu skripty pro práci s repozitářem a pro
+úpravy databáze a souborů „zvenčí“ se nejčastěji nacházejí ve složce ``make/``
+a volají se pomocí cesty: ``make/<příkaz>``.
+
+Občas existují i nějaké další skripty na různých jiných místech. Všechny by
+měly být ideálně popsány asi tady.
+
+Make skripty
+----
+
+Skripty v ``make/`` se označují jako „Make skripty“. Slouží často k velkým
+úkonům s repozitářem, jako je nasazení celého webu, zprovoznění lokálního webu
+a podobně.
+
+.. note:: Označení pro tyto skripty je dáno tím, že byly původně volány pomocí
+   make (tj. z Makefile). Ve skutečnosti je lze stále volat i jako ``make
+   <skript>``, ale pak není možné předávat parametry a obecně je tato cesta
+   zastaralá a existuje jen pro zpětnou kompatibilitu se svalovou pamětí.
+
+Tyto skripty jsou samonosné, dají se spustit rovnou a v případě problémů si
+budou hlasitě stěžovat. Pro účely debugování různých věcí jsou ale (bohužel?)
+hlasité i při normálním spuštění, konkrétně vypisují všechny příkazy, které se
+spouštějí (\ ``set -x``). Tyto příkazy jsou vidět za jedním či více plusky (\ ``+``).
+
+.. tip:: Pokud některý make skript selže, tak by na konci měl vypsat, že se něco nepovedlo.
+
+
+Knihovna ``make/lib.sh``
+^^^^^^
+
+Pro pohodlí při psaní velká část z nich využívá knihovnu uloženou
+v ``make/lib.sh``. Jsou zde definované užitečné proměnné, kontroly a společný
+kód. Kromě toho při inicializaci otestuje, že je skript spuštěn z kořene
+repozitáře (takže to pak není potřeba zkoumat v ostatních skriptech).
+
+Proměnné
+"""""
+
+Popsány jsou jen užitečné proměnné, ve skutečnosti jich je definovaných víc,
+ale jsou triviální a samopopisné.
+
+``VENV_PATH``
+  Cesta virtuálního prostředí. Též lze přepsat.
+``REPO``
+  Cesta ke gitovému repozitáři na serveru, rovnou použitelná v ``git clone``
+``GIMLI_LOGIN``
+  Přihlašovací údaje ke Gimlimu
+``PRODWEB`` a ``TESTWEB``
+  Cesty ke složkám s produkčním a testovacím webem
+
+Funkce a další zkratky
+""""""
+
+``ensure_venv``
+  Zajistí, že se zbytek skriptu spustí ve virtuálním prostředí, a pokud neexistuje, tak jej založí.
+``ensure_web_installed``
+  Vyzkouší, že je web (django) aspoň elementárně zprovozněno a pokud ne, tak vyzve uživatele, aby to spravil.
+``gimli_only``
+  Otestuje, že je příkaz spuštěn na Gimlim, pokud tomu tak není, zeptá se, jestli si uživatel skutečně přeje zbytek skriptu vykonat
+``only_in_directory <složka>``
+  Otestuje, že skript běží z konkrétní složky. Zejména použitelné s ``gimli_only`` a ``$TESTWEB``
+``safe_checkout_branch <větev>``
+  Bezpečně přepne repozitář na jinou větev. Pokud by mělo dojít k přepsání
+  knihovny nebo volajícího make skriptu, vyzve uživatele, aby přepnul ručně.
+``install_everything``
+  Společná část kódu pro nasazování produkce a testwebu.
+
+Skripty pro lokální vývoj
+^^^^^^^
+
+``make/install_web`` (nebo ekvivalentně ``make/install``)
+  Vytvoří virtualenv a nainstaluje do něj závislosti webu podle ``requirements.txt``. Následně popíše, jak vyrobit zbytek lokálního webu.
+``make/run``
+  Spustí lokální web (ekvivalentní s ``./manage.py runserver``)
+``make/schema``
+  Vykreslí závislosti a atributy modelů
+``make/sync_prod_flatpages``
+  Stáhne z produkce aktuální statické stránky a uloží je do složky ``data/``
+``make/test``
+  Spustí testy (ekvivalentní s ``./manage.py test -v2``)
+``make/init_local``
+  Zkratka za posloupnost ``make/install_web``, ``./manage.py testdata``, ``./manage.py loaddata data/*``, ``make/sync_prod_flatpages``
+
+Práce s testwebem
+^^^^^^^
+
+``make/deploy``
+  Nasadí testweb. Volitelně bere jako parametr jméno větve, kterou má nasadit.
+  Rovnou nastaví přihlašování a vygeneruje příslušnou verzi dokumentace `sem <https://mam-test.ks.matfyz.cz/docs>`_.
+``make/push_compiled_vue_to_test``
+  **Neotestováno** Nahraje Vue z lokálního počítače na testweb. (Gimli často má moc starou verzi Node.js, takže nejde zkompilovat tam)
+``make/sync_test_db_aggressive``
+  Zkopíruje databázi z produkčního webu.
+``make/sync_test_media``
+  Zkopíruje média (obrázky, nahrané soubory) z produkčního webu.
+``make/sync_test``
+  Zkratka za ``make/sync_test_db_aggressive`` + ``make/sync_test_media``.
+
+Nasazení produkce
+^^^^
+
+``make/deploy_prod``. Před samotným nasazením zálohuje databázi a zkontroluje, že se nasazuje větev ``master``.

docs/tabulka_prerekvizit.rst

@@ -0,0 +1,25 @@
+.. Není odkázaná z menu, je to záměr
+
+Tabulka prerekvizit v různých distribucích
+=========
+
+.. admonition:: Metodika
+
+   Na čistém repozitáři (``git clean -fxd``) a čistém systému spouštíme
+   ``make/init_local``. Když to spadne, tak do tabulky zapíšeme, co jsme
+   přiinstalovali. Protože větev ``makefiles`` aktuálně není mergenutá do
+   masteru, nefunguje synchronizace flatpages (a stejně nemáme SSH klíč), takže
+   tam ``make/init_local`` sestřelíme a vyzkoušíme, že ``make/test`` spustí
+   testy.
+
+.. Grafické tabulky (grid-tables, simple-tables) jsou strašný porod vyrábět, dlabu na to a cpu to do CSV…
+
+.. csv-table:: Prerekvizity v jednotlivých distribucích
+   :header: Distribuce / OS, Repozitář s Py3.9, venv, py knihovny, PostgreSQL knihovna, poznámky
+
+   Ubuntu 22.10, ??, ``python3-venv``, ``python3-dev``, ``libpq-dev``, "Je potřeba zapnout zdroj ``universe`` a nainstalovat kompilátor C (``gcc``)?"
+   Linux Mint 21, ??, ``python3-venv``, ``python3-dev``, ``libpq-dev``, ""
+   Archlinux 2022.11.01, AUR, vestavěný, vestavěné, ``postgresql-libs``, "Je potřeba céčkový kompilátor (``gcc``)"
+   openSUSE Leap 15.4, oficiální (``python39``), předinstalovaný?, ``python39-devel``, ??FIXME!!, "Výchozí verze pythonu je 3.6 a ta je moc stará, potřeba instalovat ``gcc``. Nevím jak sehnat pg_config."
+   Debian 11, "oficiální, výchozí", ??, ??, ??, "Určitě to tam rozběhat jde, protože Gimli. Nejspíš bude relativně podobné Ubuntu."
+

docs/vyvoj.rst

@@ -1,20 +1,184 @@
 Lokální vývoj mamwebu
 =====================
-Stačí spustit::
 
-    ## Nahradte svym gimli username
+Asi hlavní část vývoje většiny webu probíhá lokálně. Každý tak může pracovat na
-    git clone USER@gimli.ms.mff.cuni.cz:/akce/mam/git/mamweb.git mamweb
+vlastních úpravách nezávisle na ostatních.
-    cd mamweb
-    ## Instalace je trochu magická, spusť následující posloupnost příkazů:
-    make install_venv
-    . env/bin/activate
-    make install_web
 
-    ## Vygeneruje nejaka testovaci data (spis chuda)
+Potřebné vybavení
-    ./manage.py testdata
+-------
-    ## Nahraje statické stránky, menu a obrázky v pozadí menu
+
-    ./manage.py loaddata data/*
+Tento soupis cílí na Linuxáky. Jistě je
-    ## Spusti testovaci server na http://127.0.0.1:8000/
+spousta dalších možností, které zde nejsou postihnuty – poraď se s webaři,
-    ./manage.py runserver
+pokud si nejsi jistý. (Speciálně lze nějak vyvíjet na Windows, leč lze často
+narazit na odlišné chování od Linuxu.)
+
+Motivace cílení na Linux je to, že Gimli je Linuxový stroj, takže je vývojové
+prostředí pak podobné produkci a zmenšuje to množství odlišného chování.
+
+.. TODO: Na dokumentaci odlišného chování (Postgres vs. SQLite, Win vs. Linux, …)
+   by to asi chtělo výhledově separátní stránku, ale teď píšu tuhle :-)
+
+Nutné
+^^^^
+
+- `Git <https://git-scm.com>`_
+- `Python <https://python.org>`_
+
+  - Ideálně ve verzi 3.9 (to je to, co je aktuálně (2022) na Gimlim)
+  - Včetně pip-u (na Ubuntu balíček ``python3-pip``), venvu (``python3-venv``) a knihoven pro kompilaci
+    Cčkových rozšíření (``python3-dev``)
+- Knihovna pro práci s PostgreSQL (``libpq-dev``, protože jsou potřeba všechny db backendy)
+- Webový prohlížeč
+- \*NIXový shell (typicky ``bash``)
+
+.. TODO: Pokud tu něco chybí, tak to dopiš :-)
+
+Kromě toho je potřeba mít účet na `Gitee <https://gitea.ks.matfyz.cz>`_, kde
+bydlí gitový repozitář s kódem.
+
+.. tip:: Potřebné balíčky v různých distribucích jsou sepsané v :ref:`tabulce
+   prerekvizit <Tabulka prerekvizit v různých distribucích>`.
+
+Doporučené
+^^^^^^^^^^
+
+- Python wheel (možná řeší problémy s potřebou ``-dev`` balíčků…)
+- Editor / IDE podporující `Editorconfig <https://editorconfig.org/>`_
+- Uživatelská zkušenost s `produkční verzí webu <https://mam.matfyz.cz>`_
+- Účet v `Kanci <https://kanboard.ledoian.cz>`_
+
+.. TODO: A nejspíš další věci, na které jsem si teď nevzpomněl.
+
+
+Zprovoznění
+-------
+
+Nejprve je potřeba stáhnout si repozitář. To se provede příkazem ``git clone
+https://gitea.ks.matfyz.cz/mam/mamweb.git``, případně ``git clone
+git@gitea.ks.matfyz.cz:mam/mamweb.git``, pokud už máš nahraný SSH klíč na
+Giteu. (Obě adresy se dají zkopírovat ze `stránky repozitáře
+<https://gitea.ks.matfyz.cz/mam/mamweb>`_.) To vyrobí složku ``mamweb``, přepni
+se do ní (``cd mamweb``)
+
+O zprovoznění webu se stará skript ``make/install_web``, stačí ho spustit. Ten
+vytvoří virtualenv (neexistuje-li) a nainstaluje do něj závislosti webu (podle
+souboru ``requirements.txt``).
+
+.. FIXME: Novowebaři zmínka o requirements.txt tady moc nepomůže, to má být na
+   stránce o celkové stavbě repozitáře a stacku…
+
+Následně je potřeba nahrát další data do databáze, což uděláš pomocí příkazů
+``./manage.py testdata`` a ``./manage.py loaddata data/*``. Skript
+``make/install_web`` to kdyžtak na konci připomene.
+
+.. caution:: Zatímco skripty v ``make/`` to nepotřebují, pro použití skriptu
+  ``./manage.py`` (a dalších) se potřebuješ přepnout do virtuálního prostředí.
+  To uděláš velmi pravděpodobně spuštěním ``source env/bin/activate``, před
+  začátkem *promptu* by se mělo objevit ``(env)``. Pro opuštění spusť
+  ``deactivate``.
+
+Samotný web se spustí třeba pomocí ``make/run``, nebo ekvivalentně
+``./manage.py runserver`` a pak je vidět na `<http://127.0.0.1:8000>`_.
+
+Časté problémy
+^^^^^^
+
+- ``make/install_web`` vypíše ``Error: pg_config executable not found.``:
+  Chybí ``libpq-dev``
+- Chybová hláška obsahuje ``#include <Python.h>``: chybí ``python3-dev``
+- Na webu není vidět meníčko: spusť ``./manage.py loaddata data/*``
+- ``locale.Error: unsupported locale setting``: Chybí podpora pro příslušný
+  jazyk ve tvém systému. Odkomentuj příslušnou lokalizaci v ``/etc/locale.gen``
+  a spusť ``locale-gen`` jako root, tím se to spraví.
+
+S dalšími problémy se zkus obrátit na další webaře, třeba někdo bude vědět :-)
+
+Příkazy pro ovládání webu
+-------
+
+Příkazy se dělí do několika skupin. Některé souvisí přímo s webem, Djangem,
+databází a podobně, ty typicky používají ``./manage.py``. Skripty pro
+obhospodařování repozitáře a webu „zvenku“ typicky bydlí ve složce ``make/``.
+Ostatní skripty jsou na náhodných místech :-)
+
+Tady jsou rozebrány jen příkazy relevantní pro lokální web a univerzálně
+užitečné, ostatní najdeš v :ref:`Skripty pro práci s repozitářem`.
+
+Make skripty
+^^^^^^^
+
+- ``make/install_web`` nainstaluje závislosti webu
+- ``make/run`` spustí web (ekvivalentní s ``./manage.py runserver``)
+- ``make/schema`` nakreslí schéma vazeb modelů (může se hodit pro referenci a představu)
+- ``make/test`` spustí testy (ale moc jich zatím není; ekvivalentní s ``./manage.py test``)
+- ``make/sync_prod_flatpages`` stáhne statické stránky z produkčního webu a
+  uloží je do souboru v gitu, což umožňuje jejich verzování
+
+Manage.py skripty
+^^^^^
+
+.. note:: Je potřeba je spouštět ve virtuálním prostředí, viz výše.
+
+Všechny skripty kdyžtak mají ``--help``, dá se tak zjistit, co všechno umějí.
+
+- ``./manage.py testdata`` vygeneruje spíše chudá testovací data, aby bylo na
+  čem testovat web.
+- ``./manage.py loaddata <soubor/y>`` nahraje data ze souborů do databáze
+- ``./manage.py dumpdata <model>`` naopak z databáze vyrobí textovou reprezentaci
+- ``./manage.py shell`` spustí interaktivní pythoní shell, ze kterého lze
+  interagovat s webem / Djangem.
+- ``./manage.py dbshell`` spustí databázový shell (typicky používá SQL)
+
+- ``./manage.py makemigrations`` vyrobí popis migrací, ``./manage.py migrate``
+  je spustí, ``showmigrations`` ukáže, které migrace jsou aplikované a které
+  ne.
+- ``./manage.py runserver_plus`` spouští o něco lepší vývojový server (ale
+  nikdy jsem asi ty lepší featury nepoužil)
+
+Může se hodit vědět, že spuštění ``./manage.py`` bez parametrů vypíše seznam
+všech příkazů, které lze spustit.
+
+Dokumentace djangových příkazů je v `dokumentaci Djanga
+<https://docs.djangoproject.com/en/3.2/ref/django-admin/#available-commands>`_
+
+Ostatní užitečné příkazy
+^^^^^
+
+- ``git status`` je univerzální nápověda na aktuální stav repozitáře a co s tím
+  lze dělat.
+- ``git clean -fxd`` smaže všechny soubory, které nejsou uložené v gitu (včetně
+  ignorovaných). **Nebezpečný příkaz**, zamysli se, než ho spustíš
+
+Specifika lokálního webu
+-------
+
+Lokální uživatelé
+^^^^^^^
+
+Přihlašovací údaje jsou psány jako ``login:heslo``
+
+- Superuživatel: ``admin:admin``
+- Orgovské účty: ``o:o``, ``o1:o`` až ``o3:o``
+- Řešitelské účty: ``r:r``, ``r1:r`` až ``r3:r``
+
+Všechny tyto účty jsou (?) svázané s nějakými fiktivními osobami, není ale zřejmé se
+kterými, budeš to muset vyzkoušet a pak tady zdokumentovat :-)
+
+E-maily
+^^^^^
+
+Posílání e-mailů se lokálně dá zkoušet, e-mail se vypíše do terminálu, kde je
+web spuštěn (e.g. pomocí ``make/run``).
+
+Pruhy
+^^^^
+
+To, že má lokální web po stranách zelené pruhy je normální a správně, slouží to
+k vizuálnímu odlišení lokálního webu.
+
+.. TODO: Mít někde popis všech tří instancí a tady na něj pak odkázat.
+.. - Tahák k používání gitových větví: do workflow, ne sem…
+.. - Užitečné odkazy – kam se kouknout
+  (dohledávání views podle URL, settings_*, )
+.. - Zpříjemnění práce (ssh-klíče, tea, --help, …)
 
-Když si lokálně spustíte web, běží na http://127.0.0.1:8000/, admin najdete na http://127.0.0.1:8000/admin/ (admin/admin) Až skončíš s vývojem webu, spusť “deactivate”. Tím zmizí ‘(env)’ ze začátku promptu.

korektury/migrations/0019_auto_20221205_2014.py

@@ -0,0 +1,18 @@
+# Generated by Django 2.2.28 on 2022-12-05 19:14
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('korektury', '0018_korekturovanepdf_poslat_mail'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='korekturovanepdf',
+            name='nazev',
+            field=models.CharField(help_text='Název (např. 22.1 verze 4) korekturovaného PDF', max_length=50, verbose_name='název PDF'),
+        ),
+    ]

korektury/models.py

@@ -55,7 +55,7 @@ class KorekturovanePDF(models.Model):
 
 	cas = models.DateTimeField(u'čas vložení PDF',default=timezone.now,help_text = 'Čas vložení PDF')
 
-	nazev = models.CharField(u'název PDF',blank = True,max_length=50, help_text='Název (např. 22.1 verze 4) korekturovaného PDF')
+	nazev = models.CharField(u'název PDF',blank = False,max_length=50, help_text='Název (např. 22.1 verze 4) korekturovaného PDF')
 
 	komentar = models.TextField(u'komentář k PDF',blank = True, help_text='Komentář ke korekturovanému PDF (např. na co se zaměřit)')
 

make/README.md

@@ -0,0 +1,8 @@
+Milý člověče, M&Mí web tě vítá. Prosím, neděs se, zkusím tě provést lokálním zprovozněním webu.
+Pro detaily a nápovědu si prosím přečti dokumentaci v docs/vyvoj.rst.
+
+TL;DR: Web vyrobíš pomocí následující posloupnosti příkazů:
+	make/init_local
+	make/run
+a web potom najdeš na <http://127.0.0.1:8000>
+

make/deploy

@@ -0,0 +1,31 @@
+#!/bin/bash
+
+set -exuo pipefail
+. make/lib.sh
+
+gimli_only
+only_in_directory "$TESTWEB"
+
+CURRENT_BRANCH="$(git branch --show-current)"
+BRANCH="${1:-$CURRENT_BRANCH}"
+
+safe_checkout_branch "$BRANCH"
+
+# Teď máme správnou větev, jdeme vše nainstalovat
+install_everything
+systemctl --user restart mamweb-test.service
+
+# Přihlášení
+htpasswd -bc .htpasswd test lisakjelisak
+setfacl -m u:www-data:r .htpasswd
+
+# Build dokumentace
+ensure_venv
+(
+	cd docs
+	make html
+)
+# Oprava práv k dokumentaci
+setfacl -m u:www-data:x . docs docs/_build
+setfacl -R -m u:www-data:rX docs/_build/html
+setfacl -R -m default:u:www-data:rX docs/_build/html

make/deploy_prod

@@ -0,0 +1,25 @@
+#!/bin/bash
+
+set -exuo pipefail
+. make/lib.sh
+
+gimli_only
+only_in_directory "$PRODWEB"
+
+CURRENT_BRANCH="$(git branch --show-current)"
+BRANCH="${1:-$CURRENT_BRANCH}"
+
+if test "$BRANCH" != master
+then
+	echo "Pozor, nasazuješ na produkci větev, která není master ($BRANCH), chceš pokračovat? Pokud ne, sestřel tento skript."
+	read
+fi
+
+# Záloha DB
+( cd -P .. && ./backup_prod_db.sh )
+
+safe_checkout_branch "$BRANCH"
+
+# Teď máme správnou větev, jdeme vše nainstalovat
+install_everything
+systemctl --user restart mamweb-prod.service

make/init_local

@@ -0,0 +1,10 @@
+#!/bin/bash
+
+set -exuo pipefail
+. make/lib.sh
+
+make/install_web
+ensure_venv
+./manage.py testdata
+./manage.py loaddata data/*
+make/sync_prod_flatpages

make/install

@@ -0,0 +1 @@
+install_web

make/install_web

@@ -0,0 +1,16 @@
+#!/bin/bash
+
+set -exuo pipefail
+. make/lib.sh
+
+ensure_venv
+
+# Aktualizace toolchainu
+pip install --upgrade pip setuptools
+# Instalace závislostí webu
+pip install -r requirements.txt --upgrade
+
+# XXX: Připomínka z původního Makefile:
+echo 'Pro vygenerování tesdat spusť ./manage.py testdata
+Po vygenerování testdat spusť  ./manage.py loaddata data/*, ať máš menu a další modely
+Pro synchronizaci flatpages spusť make/sync_prod_flatpages'

make/lib.sh

@@ -0,0 +1,104 @@
+#!/bin/false Tohle je knihovna, nemá se spouštět, ale načítat pomocí source(1) nebo '.'
+
+PYTHON="${PYTHON:-python3}"
+VENV="${VENV:-${PYTHON} -m venv}"
+VENV_PATH="${VENV_PATH:-env}"
+BRANCH="${BRANCH:-master}"
+
+REPO="${REPO:-git@gitea.ks.matfyz.cz:mam/mamweb.git}"
+GIMLI='gimli.ms.mff.cuni.cz'
+GIMLI_LOGIN="mam-web@$GIMLI"
+# Skutečné cesty, jak je vrátí `realpath`
+PRODWEB="/aux/akce/mam/www/mamweb-prod"
+TESTWEB="/aux/akce/mam/www/mamweb-test"
+
+function die {
+	echo "$@" >&2
+	exit 1
+}
+
+trap 'echo Něco se nepovedlo :-/' ERR
+
+# Vždycky chceme zajistit, že běžíme z rootu repozitáře
+# TODO: chceme? Nechceme naopak umět to spouštět odkudkoliv, aspoň u většiny targetů?
+test -e '.git' || die "Make skript spuštěn ve špatné složce, spusť ho z kořenového adresáře repozitáře."
+
+function ensure_venv {
+	test -f "$VENV_PATH/bin/activate" || $VENV "$VENV_PATH"
+	. "$VENV_PATH/bin/activate"
+	# To ale není všechno Horste – ten venv nemusí fungovat, chceme to ověřit a případně spadnout.
+	local SPRAVNA_CESTA="$(readlink -f "$VENV_PATH/bin/python")"
+	local SKUTECNA_CESTA="$(readlink -f "$(which python)")"
+	test "$SPRAVNA_CESTA" == "$SKUTECNA_CESTA" || die "Venv asi nefunguje. Prosím smaž si ho a zkus to znovu."
+	python -c 'print()' > /dev/null || die "Python ve venvu je rozbitý. Prosím smaž venv a zkus to znovu."
+}
+
+function ensure_web_installed {
+	ensure_venv
+	python -c 'import django; print(django.__version__)' > /dev/null && return
+	echo >&2 "POZOR: Web nevypadá nainstalovaně, instaluji."
+	make/install_web
+}
+
+function gimli_only {
+	# Rovnou zkontrolujeme i uživatele
+	if test "$HOSTNAME" != gimli -o "$USER" != mam-web
+	then
+		echo "Tento příkaz se má spouštět na gimlim, chceš pokračovat? Pokud ne, sestřel tento skript."
+		read
+	fi
+}
+
+function only_in_directory {
+	local DIR="$1"
+	local CURRENT="$(readlink -f .)"
+	if test "$CURRENT" != "$DIR"
+	then
+		echo "Tento příkaz se má spouštět ve složce $DIR, chceš pokračovat? Pokud ne, sestřel tento skript."
+		read
+	fi
+}
+
+function safe_checkout_branch {
+	if test "$#" -ne 1
+	then
+		echo >&2 "Použití: $0 <branch>"
+		return 1
+	fi
+
+	local BRANCH="$1"
+	local SCRIPT="$0"
+
+	git fetch --all
+	# Od teď si musíme dát pozor, abychom nezměnili kód, který právě běží.
+	# Zkontrolujeme, že se nemění tahle knihovna a skript, který běží.
+	# `git rev-parse` dává SHA-1 hashe objektů, vizte manuálovou stránku pro pochopení.
+	# Pozor: tohle porovnává jen verze commitnuté do gitu. Lokální změny udělají něco náhodného…
+	if test "$(git rev-parse @:make/lib.sh)" != "$(git rev-parse "$BRANCH@{u}":make/lib.sh)"
+	then
+		echo >&2 "Změna v make/lib.sh, prosím pullni manuálně"
+		exit 1
+	fi
+	if test "$(git rev-parse @:"$SCRIPT")" != "$(git rev-parse "$BRANCH@{u}":"$SCRIPT")"
+	then
+		echo >&2 "Změna v $SCRIPT, prosím pullni manuálně"
+		exit 1
+	fi
+	git checkout "$BRANCH"
+	git pull
+	git clean -f
+}
+
+function install_everything {
+	# Skoro celé nasazení webu je stejné pro testweb i pro produkci, tak je to tady dohromady
+	gimli_only
+	ensure_venv
+	make/install
+	./manage.py migrate
+	./manage.py collectstatic --noinput
+	setfacl -R -m default:u:www-data:rX media static
+	setfacl -R -m u:www-data:rX media static
+	chown -R :mam . || true
+	chmod -R g+rX,go-w . || true
+}
+

make/push_compiled_vue_to_test

@@ -0,0 +1,14 @@
+#!/bin/bash
+
+set -exuo pipefail
+. make/lib.sh
+
+scp vue_frontend/webpack-stats.json "$GIMLI_LOGIN:$TESTWEB/vue_frontend/"
+rsync -ave ssh seminar/static/seminar/vue "$GIMLI_LOGIN:$TESTWEB/seminar/static/seminar/"
+ssh "$GIMLI_LOGIN" "
+	set -euxo pipefail
+	cd $TESTWEB
+	. make/lib.sh
+	ensure_venv
+	./manage.py collectstatic --noinput
+	"

make/run

@@ -0,0 +1,8 @@
+#!/bin/bash
+
+set -exuo pipefail
+. make/lib.sh
+
+ensure_web_installed
+
+./manage.py runserver "$@"

make/schema

@@ -0,0 +1,9 @@
+#!/bin/bash
+
+set -exuo pipefail
+. make/lib.sh
+
+ensure_web_installed
+
+./manage.py graph_models seminar | dot -Tpdf > schema_seminar.pdf
+./manage.py graph_models -a -g | dot -Tpdf > schema_all.pdf

make/sync_prod_flatpages

@@ -0,0 +1,20 @@
+#!/bin/bash
+
+set -exuo pipefail
+. make/lib.sh
+
+ensure_web_installed
+
+# TODO: This is very ugly, will fix in a future PR (hopefully)
+ssh "$GIMLI_LOGIN" "
+	set -euxo pipefail
+	cd $PRODWEB
+	. make/lib.sh
+	ensure_venv
+	./manage.py dumpdata flatpages --indent=2 > flat.json
+	./fix_json.py flat.json flat_fixed.json
+	"
+rsync -ave ssh "$GIMLI_LOGIN:$PRODWEB/flat_fixed.json" data/flat.json
+
+./manage.py loaddata data/flat.json
+

make/sync_test

@@ -0,0 +1,11 @@
+#!/bin/bash
+
+set -exuo pipefail
+. make/lib.sh
+
+# Prerekvizity nekontrolujeme, dokud voláme další make skripty, tak by akorát
+# vedly k víc dotazům na stejnou věc a bylo by to otravné. Pokud tu někdy bude
+# něco jiného, tak pak ať tu prerekvizity zmíněné jsou.
+
+make/sync_test_db_aggressive
+make/sync_test_media

make/sync_test_db_aggressive

@@ -0,0 +1,18 @@
+#!/bin/bash
+
+set -exuo pipefail
+. make/lib.sh
+
+gimli_only
+# Teoreticky není potřeba, ale stejně jinde make skripty nejsou a pouštět to z
+# produkce nezní jako běžný stav, kromě toho to aktuálně vyrábí pomocné soubory
+# v aktuální složce (FIXME do budoucna) a to na produkci nechceme
+only_in_directory "$TESTWEB"
+
+pg_dump mam_test > "dump-test-$(date +"%Y%m%d_%H%M").sql"
+pg_dump -Fc mam_prod > dump-prod.sql
+psql mam_test -c 'DROP OWNED BY "mam-web";'
+pg_restore -c --if-exists -d mam_test dump-prod.sql
+rm dump-prod.sql
+psql mam_test -c "UPDATE django_site SET name='MaMweb (test)', domain='mam-test.ks.matfyz.cz' WHERE id=1"
+

make/sync_test_media

@@ -0,0 +1,9 @@
+#!/bin/bash
+
+set -exuo pipefail
+. make/lib.sh
+
+gimli_only
+only_in_directory "$TESTWEB"
+
+rsync -av --delete "$PRODWEB/media/" ./media

make/test

@@ -0,0 +1,9 @@
+#!/bin/bash
+
+set -exuo pipefail
+. make/lib.sh
+
+ensure_web_installed
+
+trap - ERR # Testy nejspíš selžou, ale nechceme kolem toho dělat další chybovou hlášku.
+./manage.py test -v2

mamweb/settings_common.py

@@ -160,6 +160,9 @@ INSTALLED_APPS = (
 #    'admin_tools.menu',
 #    'admin_tools.dashboard',
 	'django.contrib.admin',
+
+	# Nechat na konci (INSTALLED_APPS je uspořádané):
+	'django_cleanup.apps.CleanupConfig',  # Uklízí media/
 )
 
 DEBUG_TOOLBAR_CONFIG = {

mamweb/static/css/mamweb.css

@@ -425,6 +425,13 @@ textarea.feedback {
 	margin: 5px;
 }
 
+/* td obsahující křížek v detailu řešení se nesmí smrštit na 0 */
+/* FIXME až bude firefox příčetný, nahradit td:has(.smazat_hodnoceni) */
+.has_smazat_hodnoceni {
+	min-width: 20px;
+	padding: 3px;
+}
+
 
 /* titulni stranka */
 
@@ -1152,8 +1159,7 @@ div.zadani_termin .datum {
 /* posune kotvu obrázku v galerii o oranžový pruh dolu, aby se pod ním obrázek neschovával */
 /* https://stackoverflow.com/questions/10732690/offsetting-an-html-anchor-to-adjust-for-fixed-header */
 .kotva_obrazku {
-	display: block;
+	position: absolute;
-	position: relative;
 	width: 0;
 	height: 55px; /* viz #title */
 	margin-top: -55px; /* viz #title */
@@ -1210,6 +1216,13 @@ div.gdpr {
 }
 
 /* tabulka odevzdaných a došlých řešení */
+
+/* Roztáhne obsah z containeru na celou šířku obrazovky: */
+.full_width {
+	width: 100vw;
+	margin-left: calc(-50vw + 485px);
+}
+
 .dosla_reseni tr th, .dosla_reseni tr td {
 	padding: 1px 10px 1px 10px;
 	border-collapse: collapse;

odevzdavatko/forms.py

@@ -98,7 +98,7 @@ class JednoHodnoceniForm(forms.ModelForm):
 		fields = ('problem', 'body', 'deadline_body', 'feedback',)
 		widgets = {
 			'problem': autocomplete.ModelSelect2(
-				url='autocomplete_problem_odevzdatelny',   # FIXME: Dovolit i starší?
+				url='autocomplete_problem',
 				),
 			'feedback': forms.Textarea(attrs={'rows': 1, 'cols': 30, 'class': 'feedback'}),
 			}

odevzdavatko/templates/odevzdavatko/detail.html

@@ -56,14 +56,14 @@
 		<td>{{ subform.body }}</td>
 		<td>{{ subform.deadline_body }}</td>
 		<td>{{ subform.feedback }}</td>
-		<td><a href="#" class="smazat_hodnoceni" id="id_{{subform.prefix}}-jsremove"><img src="{% static "odevzdavatko/cross.png" %}" alt="Smazat"></a></td>
+		<td class="has_smazat_hodnoceni"><a href="#" class="smazat_hodnoceni" id="id_{{subform.prefix}}-jsremove" title="Smazat hodnocení"><img src="{% static "odevzdavatko/cross.png" %}" alt="Smazat"></a></td>
 	</tr>
     </tbody>
 {% endfor %}
 </table>
 
 
-<a href="#"> <img src="{% static "odevzdavatko/plus.png" %}" id="pridat_hodnoceni" alt="Přidat hodnocení"></a> <br/>
+<a href="#" title="Přidat hodnocení"> <img src="{% static "odevzdavatko/plus.png" %}" id="pridat_hodnoceni" alt="Přidat hodnocení"></a> <br/>
 <input type=submit value="Uložit"></form>
 
 <table id="empty_form" style="display: none;">
@@ -72,7 +72,7 @@
 		<td>{{ form.empty_form.body }}</td>
 		<td>{{ form.empty_form.deadline_body }}</td>
 		<td>{{ form.empty_form.feedback }}</td>
-		<td><a href="#" class="smazat_hodnoceni" id="id_{{form.empty_form.prefix}}-jsremove"><img src="{% static "odevzdavatko/cross.png" %}" alt="Smazat"></a></td>
+		<td class="has_smazat_hodnoceni"><a href="#" class="smazat_hodnoceni" id="id_{{form.empty_form.prefix}}-jsremove" title="Smazat hodnocení"><img src="{% static "odevzdavatko/cross.png" %}" alt="Smazat"></a></td>
 	</tr>
 </table>
 

odevzdavatko/templates/odevzdavatko/tabulka.html

@@ -4,6 +4,7 @@
 
 {% block content %}
 
+<div class="full_width">
 <form method=get action=.>
 {{ filtr.resitele }}
 {{ filtr.problemy }}
@@ -64,4 +65,5 @@ Do data (včetně): {{ filtr.reseni_do }}
 		location.assign(redirect);
 	}
 </script>
+</div>
 {% endblock %}

requirements.txt

@@ -21,6 +21,7 @@ django-sekizai
 django-countries
 django-solo
 django-ckeditor
+django-cleanup  # Uklízí media/ od smazaných „databázových“ souborů
 django-flat-theme
 django-taggit
 django-autocomplete-light>=3.9.0rc1

seminar/templates/seminar/archiv/cislo.html

@@ -40,7 +40,6 @@
           <li><a href="obalky.pdf">Obálky (PDF)</a></li>
           <li><a href="tituly.tex" download>Tituly (TeX)</a></li>
           <li><a href="vysledkovka.tex" download>Výsledkovka (TeX)</a></li>
-          <li><a href="obalkovani">Obálkování</a></li>
           <li><a href="odmeny/{{prevcislo.rocnik.rocnik}}.{{prevcislo.poradi}}/">Odměny</a></li>
         </ul>
       </div>

seminar/templates/seminar/archiv/cislo_obalkovani.html

@@ -1,33 +0,0 @@
-{% extends "base.html" %}
-
-{% block content %}
-  <h1>
-    {% block nadpis1a %}
-      Obálkování {{ cislo }}
-    {% endblock %}
-  </h1>
-
-  Obálkovat se budou tyto problémy:
-  <ul>
-    {% for p in problemy %}
-
-      <li> {{ p.kod_v_rocniku }} {{ p }}
-    {% endfor %}
-  </ul>
-
-  {% for r in reseni %}
-    {% ifchanged r.resitel %}
-      {% if not forloop.first %}
-        </ul>
-      {% endif %}
-      <h4>{{ r.resitel }}</h4>
-        <ul>
-    {% endifchanged %}
-
-    <li>
-      {{ r.problem.kod_v_rocniku }} {{ r.problem.nazev }} ({{ r.body }})
-
-  {% endfor %}
-  </ul>
-
-{% endblock content %}

seminar/templates/seminar/org/obalkovani.html

@@ -1,30 +0,0 @@
-{% extends "base.html" %}
-
-{% block content %}
-  <h1>
-    {% block nadpis1a %}
-      Obálkování {{ cislo }}
-    {% endblock %}
-  </h1>
-  <ul>
-  {% for reseni in object_list %}
-	{% ifchanged reseni.resitele %}
-		{% if not forloop.first %}
-		 </ul>
-		{% endif %}
-		<h4>{% for resitel in reseni.resitele.all %}{{resitel.osoba}},{% endfor %}</h4>
-	<ul>
-	{% endifchanged %}
-
-  <li>Celkem {{reseni.hodnoceni__body__sum}} bodů z {{reseni.hodnoceni__count}} hodnocení
-	<ul>
-		{% for h in reseni.hodnoceni_set.all %}
-		<li> {{ h.problem }}: {{ h.body }}b </li>
-		{% endfor %}
-	</ul>
-  </li>
-  {% endfor %}
-  </ul>
-
-
-{% endblock content %}

seminar/urls.py

@@ -23,7 +23,6 @@ Soubor sloužící jako „router“, tj. zde se definují url adresy a na co uk
 	- ``cislo/<int:rocnik>.<str:cislo>/obalky.pdf`` (seminar_cislo_obalky) :func:`~seminar.views.views_all.cisloObalkyView`
 	- ``cislo/<int:rocnik>.<str:cislo>/tituly.tex`` (seminar_cislo_titul) :func:`~seminar.views.views_all.TitulyView`
 	- ``stav`` (stav_databaze) :func:`~seminar.views.views_all.StavDatabazeView`
-	- ``cislo/<int:rocnik>.<str:cislo>/obalkovani`` (seminar_cislo_resitel_obalkovani) :class:`~seminar.views.views_all.ObalkovaniView`
 	- ``cislo/<int:trocnik>.<str:tcislo>/odmeny/<int:frocnik>.<str:fcislo>/`` (seminar_archiv_odmeny) :class:`~seminar.views.views_all.OdmenyView`
 - Další
 	- `` `` (titulni_strana) :class:`~seminar.views.views_all.TitulniStranaView`
@@ -102,11 +101,6 @@ urlpatterns = [
 		org_required(views.StavDatabazeView),
 		name='stav_databaze'
 	),
-	path(
-		'cislo/<int:rocnik>.<str:cislo>/obalkovani',
-		org_required(views.ObalkovaniView.as_view()),
-		name='seminar_cislo_resitel_obalkovani'
-	),
 	path(
 		'cislo/<int:trocnik>.<str:tcislo>/odmeny/<int:frocnik>.<str:fcislo>/',
 		org_required(views.OdmenyView.as_view()),

seminar/views/views_all.py

@@ -53,23 +53,6 @@ logger = logging.getLogger(__name__)
 def get_problemy_k_tematu(tema):
 	return Problem.objects.filter(nadproblem = tema)
 
-class ObalkovaniView(generic.ListView):
-	template_name = 'seminar/org/obalkovani.html'
-
-	def get_queryset(self):
-		rocnik = get_object_or_404(Rocnik,rocnik=self.kwargs['rocnik'])
-		cislo = get_object_or_404(Cislo,rocnik=rocnik,poradi=self.kwargs['cislo'])
-		self.cislo = cislo
-		self.hodnoceni = s.Hodnoceni.objects.filter(cislo_body=cislo)
-		self.reseni = Reseni.objects.filter(hodnoceni__in = self.hodnoceni).annotate(Sum('hodnoceni__body')).annotate(Count('hodnoceni')).order_by('resitele__osoba')
-		return self.reseni
-
-	def get_context_data(self, **kwargs):
-		context = super(ObalkovaniView, self).get_context_data(**kwargs)
-		print(self.cislo)
-		context['cislo'] = self.cislo
-		return context
-
 
 # FIXME: Pozor, níž je ještě jeden ProblemView!
 #class ProblemView(generic.DetailView):
@@ -590,28 +573,6 @@ def obalkyView(request, resitele):
 	return response
 
 
-def oldObalkovaniView(request, rocnik, cislo):
-	rocnik = Rocnik.objects.get(rocnik=rocnik)
-	cislo = Cislo.objects.get(rocnik=rocnik, cislo=cislo)
-
-	reseni = (
-		Reseni.objects.filter(cislo_body=cislo)
-		.order_by(
-			'resitel__prijmeni',
-			'resitel__jmeno',
-			'problem__typ',
-			'problem__kod'
-		)
-	)
-
-	problemy = sorted(set(r.problem for r in reseni), key=lambda p: (p.typ, p.kod))
-	return render(
-		request,
-		'seminar/archiv/cislo_obalkovani.html',
-		{'cislo': cislo, 'problemy': problemy, 'reseni': reseni}
-	)
-
-
 ### Tituly
 def TitulyViewRocnik(request, rocnik):
 	return TitulyView(request, rocnik, None)

vysledkovky/templates/vysledkovky/vysledkovka_cisla.html

@@ -1,49 +1,49 @@
 <div style="overflow-x: auto;">
 <table class='vysledkovka'>
     <tr class='border-b'>
-        <th class='border-r'>#
+        <th class='border-r'>#</th>
-        <th class='border-r'>Jméno
+        <th class='border-r'>Jméno</th>
             {% for p in vysledkovka.temata_a_spol%}
-                <th class='border-r' id="problem{{ oznaceni_vysledkovky }}_{{ forloop.counter0 }}">{# <a href="{{ p.verejne_url }}"> #}{{ p.kod_v_rocniku }}{# </a> #}
+                <th class='border-r' id="problem{{ oznaceni_vysledkovky }}_{{ forloop.counter0 }}">{# <a href="{{ p.verejne_url }}"> #}{{ p.kod_v_rocniku }}{# </a> #}</th>
 
                     {# TODELETE #}
                     {% for podproblemy in vysledkovka.podproblemy_iter.next %}
-                        <th class='border-r podproblem{{ oznaceni_vysledkovky }}_{{ forloop.parentloop.counter0 }} podproblem'>{# <a href="{{ podproblemy.verejne_url }}"> #}{{ podproblemy.kod_v_rocniku }}{# </a> #}
+                        <th class='border-r podproblem{{ oznaceni_vysledkovky }}_{{ forloop.parentloop.counter0 }} podproblem'>{# <a href="{{ podproblemy.verejne_url }}"> #}{{ podproblemy.kod_v_rocniku }}{# </a> #}</th>
                     {% endfor %}
                 {# TODELETE #}
 
             {% endfor %}
-        {% if vysledkovka.je_nejake_ostatni %}<th class='border-r' id='problem{{ oznaceni_vysledkovky }}_{{ vysledkovka.temata_a_spol| length }}'>Ostatní {% endif %}
+        {% if vysledkovka.je_nejake_ostatni %}<th class='border-r' id='problem{{ oznaceni_vysledkovky }}_{{ vysledkovka.temata_a_spol| length }}'>Ostatní</th>{% endif %}
 
         {# TODELETE #}
         {% for podproblemy in vysledkovka.podproblemy_iter.next %}
-            <th class='border-r podproblem{{ oznaceni_vysledkovky }}_{{ vysledkovka.temata_a_spol| length }} podproblem'>{# <a href="{{ podproblemy.verejne_url }}"> #}{{ podproblemy.kod_v_rocniku }}{# </a> #}
+            <th class='border-r podproblem{{ oznaceni_vysledkovky }}_{{ vysledkovka.temata_a_spol| length }} podproblem'>{# <a href="{{ podproblemy.verejne_url }}"> #}{{ podproblemy.kod_v_rocniku }}{# </a> #}</th>
         {% endfor %}
         {# TODELETE #}
 
 
-        <th class='border-r'>Za číslo
+        <th class='border-r'>Za číslo</th>
-        <th class='border-r'>Za ročník
+        <th class='border-r'>Za ročník</th>
-        <th class='border-r'>Odjakživa
+        <th class='border-r'>Odjakživa</th>
             {% for rv in vysledkovka.radky_vysledkovky %}
                 <tr>
-                    <td class='border-r'>{% autoescape off %}{{ rv.poradi }}{% endautoescape %}
+                    <td class='border-r'>{% autoescape off %}{{ rv.poradi }}{% endautoescape %}</td>
                     <th class='border-r'>
                         {% if rv.titul %}
                             {{ rv.titul }}<sup>MM</sup>
                         {% endif %}
-                        {{ rv.resitel.osoba.plne_jmeno }}
+                        {{ rv.resitel.osoba.plne_jmeno }}</th>
                         {% for b in rv.body_za_temata_seznam %}
-                            <td class='border-r'>{{ b }}
+                            <td class='border-r'>{{ b }}</td>
 
                                 {% for body_podproblemu in rv.body_podproblemy_iter.next %}
-                                    <td class='border-r podproblem{{ oznaceni_vysledkovky }}_{{ forloop.parentloop.counter0 }} podproblem'>{{ body_podproblemu }}
+                                    <td class='border-r podproblem{{ oznaceni_vysledkovky }}_{{ forloop.parentloop.counter0 }} podproblem'>{{ body_podproblemu }}</td>
                                 {% endfor %}
 
                         {% endfor %}
-                    <td class='border-r'>{{ rv.body_cislo }}
+                    <td class='border-r'>{{ rv.body_cislo }}</td>
-                    <td class='border-r'><b>{{ rv.body_rocnik }}</b>
+                    <td class='border-r'><b>{{ rv.body_rocnik }}</b></td>
-                    <td class='border-r'>{{ rv.body_celkem_odjakziva }}
+                    <td class='border-r'>{{ rv.body_celkem_odjakziva }}</td>
                 </tr>
             {% endfor %}
 </table>

vysledkovky/templates/vysledkovky/vysledkovka_rocnik.html

@@ -1,29 +1,29 @@
 <table class='vysledkovka'>
   <tr class='border-b'>
-    <th class='border-r'>#
+    <th class='border-r'>#</th>
-    <th class='border-r'>Jméno
+    <th class='border-r'>Jméno</th>
-    <th class='border-r'>R.
+    <th class='border-r'>R.</th>
-    <th class='border-r'>Odjakživa
+    <th class='border-r'>Odjakživa</th>
         {% for c in vysledkovka.cisla_rocniku %}
     <th class='border-r'><a href="{{ c.verejne_url }}">
-            {{c.rocnik.rocnik}}.{{ c.poradi }}</a>
+        {{c.rocnik.rocnik}}.{{ c.poradi }}</a></th>
     {% endfor %}
-    <th class='border-r'>Celkem
+    <th class='border-r'>Celkem</th>
 
 {% for rv in vysledkovka.radky_vysledkovky %}
   <tr>
-    <td class='border-r'>{% autoescape off %}{{ rv.poradi }}{% endautoescape %}
+    <td class='border-r'>{% autoescape off %}{{ rv.poradi }}{% endautoescape %}</td>
     <th class='border-r'>
       {% if rv.titul %}
         {{ rv.titul }}<sup>MM</sup>
       {% endif %}
-      {{ rv.resitel.osoba.plne_jmeno }}
+      {{ rv.resitel.osoba.plne_jmeno }}</th>
-    <td class='border-r'>{{ rv.rocnik_resitele }}
+    <td class='border-r'>{{ rv.rocnik_resitele }}</td>
-    <td class='border-r'>{{ rv.body_celkem_odjakziva }}
+    <td class='border-r'>{{ rv.body_celkem_odjakziva }}</td>
     {% for b in rv.body_cisla_seznam %}
-    <td class='border-r'>{{ b }}
+        <td class='border-r'>{{ b }}</td>
     {% endfor %}
-    <td class='border-r'><b>{{ rv.body_rocnik }}</b>
+    <td class='border-r'><b>{{ rv.body_rocnik }}</b></td>
   </tr>
 {% endfor %}
 </table>