{% if galerie.zobrazit == 1 %}
-
Vytvořit novou podgalerii ,
upravit galerii v adminu
+
Vytvořit novou podgalerii ,
upravit galerii v adminu
{% else %}
Jestli chceš změnit pořadí podgalerií nebo přidat novou, nastav zobrazení jen pro orgy v
adminu .
{% endif %}
@@ -95,7 +95,7 @@ Galerie popisekgalerie.nazev}}
{% if obrazek.popisek %}
title="{{ obrazek.popisek }}"
{% endif %}
- href="{% url 'galerie_soubor' galerie=galerie.pk soubor=obrazek.pk %}#nahoru" class="galerie_nahled">
+ href="{% url 'galerie_soubor' soustredeni=soustredeni.id galerie=galerie.pk soubor=obrazek.pk %}#nahoru" class="galerie_nahled">
{% zmenseny_nahled obrazek.jako_bazmek %}
{# title=obrazek.popisek (a byl tu if, že se použil jen když existoval…) width=obrazek.obrazek_maly.width height=obrazek.obrazek_maly.height #}
@@ -107,12 +107,12 @@ Galerie popisekgalerie.nazev}}
{% if predchozi %}
{% endif %}
{% if nasledujici %}
{% endif %}
diff --git a/galerie/testfiles/bila.mp4 b/galerie/testfiles/bila.mp4
new file mode 100644
index 00000000..a3ddda9d
Binary files /dev/null and b/galerie/testfiles/bila.mp4 differ
diff --git a/galerie/testfiles/kod_emoji.jpg b/galerie/testfiles/kod_emoji.jpg
new file mode 100644
index 00000000..1703a703
Binary files /dev/null and b/galerie/testfiles/kod_emoji.jpg differ
diff --git a/galerie/testfiles/metal.jpg b/galerie/testfiles/metal.jpg
new file mode 100644
index 00000000..8ea0ebe7
Binary files /dev/null and b/galerie/testfiles/metal.jpg differ
diff --git a/galerie/testfiles/ohno_small.jpg b/galerie/testfiles/ohno_small.jpg
new file mode 100644
index 00000000..607e910a
Binary files /dev/null and b/galerie/testfiles/ohno_small.jpg differ
diff --git a/galerie/testfiles/pf2020_small.jpg b/galerie/testfiles/pf2020_small.jpg
new file mode 100644
index 00000000..4989262a
Binary files /dev/null and b/galerie/testfiles/pf2020_small.jpg differ
diff --git a/galerie/tests.py b/galerie/tests.py
new file mode 100644
index 00000000..2107cb9a
--- /dev/null
+++ b/galerie/tests.py
@@ -0,0 +1,140 @@
+from django.conf import settings
+from django.contrib.contenttypes.models import ContentType
+from django.test import TestCase, Client, override_settings
+from django.urls import reverse
+from galerie.models import Galerie, Soubor
+from soustredeni.models import Soustredeni
+
+import django.contrib.auth.models as auth
+import personalni.models as pers
+import tvorba.models as tv
+
+from contextlib import ExitStack
+from pathlib import Path
+import shutil
+import tempfile
+
+# NOTE: Existují metody, jak si pořídit kontext: https://docs.python.org/3/library/unittest.html#unittest.TestCase.enterClassContext
+# Nepoužíváme je, protože jsem nepřišlo na to, jak je pak použít v `@override_settings(MEDIA_ROOT=media_dir)`
+# Ref: https://stackoverflow.com/a/2269295
+tempdir = tempfile.TemporaryDirectory(prefix='mamweb_test')
+media_dir = Path(tempdir.name) / 'media'
+
+class GalerieTests(TestCase):
+ @classmethod
+ def setUpTestData(cls):
+ org_perm = auth.Permission.objects.get(content_type=ContentType.objects.get_for_model(auth.User), codename='org')
+ resitel_perm = auth.Permission.objects.get(content_type=ContentType.objects.get_for_model(auth.User), codename='resitel')
+
+ r = tv.Rocnik.objects.create(rocnik=42, prvni_rok=1970)
+ cls.sous_a = Soustredeni.objects.create(rocnik=r)
+ cls.sous_b = Soustredeni.objects.create(rocnik=r)
+
+ cls.neucastnik = pers.Resitel.objects.create(osoba=pers.Osoba.objects.create(user=auth.User.objects.create_user(username='ne')))
+ cls.neucastnik.osoba.user.user_permissions.add(resitel_perm)
+
+ cls.ucastnik_a = pers.Resitel.objects.create(osoba=pers.Osoba.objects.create(user=auth.User.objects.create_user(username='ua')))
+ cls.ucastnik_a.osoba.user.user_permissions.add(resitel_perm)
+ cls.sous_a.ucastnici.add(cls.ucastnik_a)
+
+ cls.ucastnik_b = pers.Resitel.objects.create(osoba=pers.Osoba.objects.create(user=auth.User.objects.create_user(username='ub')))
+ cls.ucastnik_b.osoba.user.user_permissions.add(resitel_perm)
+ cls.sous_b.ucastnici.add(cls.ucastnik_b)
+
+ cls.org = pers.Organizator.objects.create(osoba=pers.Osoba.objects.create(user=auth.User.objects.create_user(username='o', is_staff=True)))
+ cls.org.osoba.user.user_permissions.add(org_perm)
+
+ def setUp(self):
+ media_dir.mkdir()
+
+ def tearDown(self):
+ shutil.rmtree(media_dir)
+
+ @classmethod
+ def tearDownClass(cls):
+ super().tearDownClass()
+ tempdir.cleanup()
+
+ # TODO: tohle není moc unit test. Správnější by bylo mít polopřipravené
+ # věci (sous s galerií, sous bez galerie, sous se špatně uspořádanými
+ # galeriemi ap.) a dedikované testy na jednotlivé úkony. Ale teď jsem to
+ # začalo psát takhle a chci to nejdřív dopsat a commitnout, takže pokud to
+ # zapomenu rozstřelit na menší, tak to takhle zůstane…
+ @override_settings(MEDIA_ROOT=media_dir)
+ def test_vyrobeni_galerii_k_sousu(self):
+ """Vyrobí strukturu galerií odpovídající sousové běžné struktuře pomocí příslušných views"""
+ org_client = Client()
+ org_client.force_login(self.org.osoba.user)
+ resp = org_client.get(reverse('galerie_nova', kwargs={'soustredeni': self.sous_a.id, 'galerie': 0}))
+ self.assertEqual(resp.status_code, 200)
+ tf = Path(settings.BASE_DIR) / 'galerie' / 'testfiles'
+ filenames_pa = [
+ tf / 'kod_emoji.jpg',
+ tf / 'metal.jpg',
+ tf / 'ohno_small.jpg',
+ ]
+ filenames_so = [
+ tf / 'pf2020_small.jpg',
+ ]
+ resp = org_client.post(
+ reverse('galerie_nova', kwargs={'soustredeni': self.sous_a.id, 'galerie': 0}),
+ data={'nazev': 'Top galerie k sousu A'},
+ follow=True,
+ )
+ self.assertEqual(resp.status_code, 200)
+ # Teď je kind-of těžké kliknout na „přidat novou podgalerii“, protože ten odkaz je někde v HTML vyrenderovaný templatem.
+ # Ale můžeme vytáhnout pk galerie z kontextu
+ top_gal = resp.context['galerie']
+ self.assertIsInstance(top_gal, Galerie)
+
+ # Záměrně vyrobíme nejdřív sobotní a pak páteční galerii, ať je pak můžeme prohodit
+ with ExitStack() as stack:
+ files = [stack.enter_context(open(fn, 'rb')) for fn in filenames_so]
+ resp = org_client.post(
+ reverse('galerie_nova', kwargs={'soustredeni': self.sous_a.id, 'galerie': top_gal.pk}),
+ data={'nazev': 'Sobota', 'obr': files},
+ follow=True,
+ )
+ self.assertEqual(resp.status_code, 200)
+ so_gal = resp.context['galerie']
+ self.assertIsInstance(so_gal, Galerie)
+
+ with ExitStack() as stack:
+ files = [stack.enter_context(open(fn, 'rb')) for fn in filenames_pa]
+ resp = org_client.post(
+ reverse('galerie_nova', kwargs={'soustredeni': self.sous_a.id, 'galerie': top_gal.pk}),
+ data={'nazev': 'Pátek', 'obr': files},
+ follow=True,
+ )
+ self.assertEqual(resp.status_code, 200)
+ pa_gal = resp.context['galerie']
+ self.assertIsInstance(pa_gal, Galerie)
+
+ self.assertGreater(pa_gal.poradi, so_gal.poradi)
+ # Dokonce i tušíme, jaká ta pořadí jsou
+ # NOTE: pokud tohle začne failovat, tak je asi safe to smazat.
+ self.assertEqual(pa_gal.poradi, 2)
+ self.assertEqual(so_gal.poradi, 1)
+
+ # Teď je prohodíme:
+ # NOTE: použití metody PATCH by tady prošlo, protože nekontrolujeme,
+ # jaká metoda se používá. Ale dokud simulujeme orga na webu s odkazy
+ # (`
`), tak použijeme GET.
+ resp = org_client.get(
+ reverse('galerie_doleva', kwargs={'soustredeni': self.sous_a.id, 'galerie': top_gal.pk, 'subgalerie': pa_gal.pk}),
+ follow=True,
+ )
+ self.assertEqual(resp.status_code, 200)
+ resp = org_client.get(
+ reverse('galerie_doprava', kwargs={'soustredeni': self.sous_a.id, 'galerie': top_gal.pk, 'subgalerie': so_gal.pk}),
+ follow=True,
+ )
+ self.assertEqual(resp.status_code, 200)
+ # Změnil se stav
+ pa_gal.refresh_from_db()
+ so_gal.refresh_from_db()
+
+ self.assertLess(pa_gal.poradi, so_gal.poradi)
+ self.assertEqual(pa_gal.poradi, 1)
+ self.assertEqual(so_gal.poradi, 2)
+
diff --git a/galerie/views.py b/galerie/views.py
index 13cec483..80cb0c75 100644
--- a/galerie/views.py
+++ b/galerie/views.py
@@ -90,6 +90,7 @@ def galerieView(request, soustredeni:int, galerie: int):
'sourozenci': sourozenci,
'predchozi': predchozi,
'nasledujici': nasledujici,
+ 'soustredeni': soustredeni, # Jen kvůli reverse urls…
'object': galerie,
})
@@ -107,6 +108,9 @@ def souborView(request, soustredeni: int, galerie: int, soubor: int):
# Explicitní pořadí ničemu neuškodí :-)
soubory = galerie.soubor_set.all().order_by('poradi', 'nazev')
soubory = list(soubory)
+ # Pokud někdo umí z DB rovnou vytáhnout příslušnou sadu obrázků, tak to
+ # bude (teoreticky a asymptoticky) hezčí. Ale pokud neplánujeme mít tisíce
+ # obrázků v jedné galerii, tak je to asi jedno :-D
index_souboru = soubory.index(soubor)
# Podle mě se nemůže stát, že by volání výš selhalo, kdyžtak shodí web. (původně to byl explicitně ošetřený stav dávající 404)
predchozi_soubory = list(reversed(soubor[:index_souboru]))
@@ -172,7 +176,8 @@ def new_galerie(request, galerie: int, soustredeni: int):
# obsluha formulare umoznujiciho multiple nahravani fotek
if request.method == 'POST':
- form = NewGalerieForm(request.POST, request.FILES)
+ # Tohle je nějaké zbastlené. NewGalerieForm nežere FILES, ručně zpracováváme ``request.FILES.getlist('obr')``. FIXME.
+ form = NewGalerieForm(request.POST)
if form.is_valid():
# vytvoreni nove galerie
gal = Galerie()
@@ -187,7 +192,7 @@ def new_galerie(request, galerie: int, soustredeni: int):
else:
gal.soustredeni = soustredeni
if gal.nadgalerie:
- gal.poradi = int(len(gal.nadgalerie.galerie_set.all())) + 1
+ gal.poradi = int(len(gal.nadgalerie.podgalerie.all())) + 1
gal.save()
# zpracovani souboru v galerii
@@ -201,7 +206,7 @@ def new_galerie(request, galerie: int, soustredeni: int):
o.save()
# presmerovani na prave vzniklou galerii
- return HttpResponseRedirect(reverse('galerie_galerie', kwargs={'galerie': gal.pk}))
+ return HttpResponseRedirect(reverse('galerie_galerie', kwargs={'galerie': gal.pk, 'soustredeni': soustredeni.id}))
else:
form = NewGalerieForm()
@@ -213,22 +218,28 @@ def new_galerie(request, galerie: int, soustredeni: int):
'galerie_text' : galerie_text,
})
+# FIXME: Tohle by neměly být GETy, ale POSTy/PATCHe, protože mění stav
+# (= nejsou safe) a nejsou idempotentní. Ale znamená to mít místo odkazů
+# ``y / JS, což se nikomu nechce…
+# Ref: https://developer.mozilla.org/en-US/docs/Web/HTTP/Reference/Methods
def galerie_doprava(request, soustredeni: int, galerie: int, subgalerie: int):
subgalerie: Galerie = get_object_or_404(Galerie, pk=subgalerie)
+ assert top_galerie(subgalerie).soustredeni.id == soustredeni
assert subgalerie.nadgalerie.pk == galerie
if subgalerie.poradi:
subgalerie.poradi += 1
else:
- subgalerie.poradi = int(len(subgalerie.nadgalerie.galerie_set.all()))
+ subgalerie.poradi = int(len(subgalerie.nadgalerie.podgalerie.all()))
subgalerie.save()
- return HttpResponseRedirect(reverse('galerie_galerie', kwargs={'galerie': galerie}))
+ return HttpResponseRedirect(reverse('galerie_galerie', kwargs={'galerie': galerie, 'soustredeni': soustredeni}))
def galerie_doleva(request, soustredeni: int, galerie: int, subgalerie: int):
subgalerie: Galerie = get_object_or_404(Galerie, pk=subgalerie)
+ assert top_galerie(subgalerie).soustredeni.id == soustredeni
assert subgalerie.nadgalerie.pk == galerie
if subgalerie.poradi:
subgalerie.poradi -= 1
else:
subgalerie.poradi = 1
subgalerie.save()
- return HttpResponseRedirect(reverse('galerie_galerie', kwargs={'galerie': galerie}))
+ return HttpResponseRedirect(reverse('galerie_galerie', kwargs={'galerie': galerie, 'soustredeni': soustredeni}))