mam/mamweb
13
1
Fork 0
Code Issues Pull requests 12 Projects Wiki Activity Actions

Předělání sousových views #57

Merged
zelvuska merged 11 commits from predelani_sousovych_view into master 2024-11-12 19:33:11 +01:00
Conversation 2 Commits 11 Files changed 4 +134 -54
zelvuska commented 2024-08-03 00:14:57 +02:00
Owner
Copy link

Hrál jsem si se sousovými views a vznikly z toho nějaké myšlenky, které bych tu rozebíral samostatně:

  • mixin KonkretniSoustredeniMixin, který všem templatům konkrétního sousu dodává atribut self.soustredeni (aby se nemusel vždy tahat ze self.kwargs)
    • trošku zmenšuje čitelnost, ale je pohodlný (nemusí se u každého view psát mašinérie na získání toho Soustředění)
    • (aktuálně vlastně skoro zbytečný, protože ke konkrétnímu sous máme jen účastnické „seznamy“, ale do budoucna se může hodit víc)
  • view NeprazdnyListView, který ListView přidává handlování prázdného seznamu (inteligentnější než vyhodit 404)
    • ListView je bohužel moc zadrátovaný, takže v třídě NeprazdnyListView je nějaký copy-paste kód (potenciálně lze nahradit něčím jednodušším, ale méně obecným)
    • myslím, že se hodí na spoustu míst, protože často dává větší smysl vyhodit hlášku stylu „je to prázdné, naplň si to v adminu“, než vygenerovat prázdnou věc (CSV, PDF, ale i html stránku, speciálně, pokud je pro tisk)
    • zase ale přidává nějakou vrstvu, ve které se musí člověk vyznat
  • mixin TeXResponseMixin, který generalizuje generování PDF z TeXu
    • asi nejméně problematická změna
  • z funkcionálních Views se stali třídní Views (aby mohli dědit od věcí výše)
    • to zhoršilo čitelnost, na druhou stranu class based Views jsou doporučované oproti function based Views

Prefix WIP: tu je proto, že tento pull request koliduje na #56 (speciálně to mám aktuálně jako pull request do split_sous, aby byly dobře vidět změny).

Hrál jsem si se sousovými views a vznikly z toho nějaké myšlenky, které bych tu rozebíral samostatně: - mixin `KonkretniSoustredeniMixin`, který všem templatům konkrétního sousu dodává atribut `self.soustredeni` (aby se nemusel vždy tahat ze `self.kwargs`) - trošku zmenšuje čitelnost, ale je pohodlný (nemusí se u každého view psát mašinérie na získání toho Soustředění) - (aktuálně vlastně skoro zbytečný, protože ke konkrétnímu sous máme jen účastnické „seznamy“, ale do budoucna se může hodit víc) - view `NeprazdnyListView`, který `ListView` přidává handlování prázdného seznamu (inteligentnější než vyhodit 404) - `ListView` je bohužel moc zadrátovaný, takže v třídě `NeprazdnyListView` je nějaký copy-paste kód (potenciálně lze nahradit něčím jednodušším, ale méně obecným) - myslím, že se hodí na spoustu míst, protože často dává větší smysl vyhodit hlášku stylu „je to prázdné, naplň si to v adminu“, než vygenerovat prázdnou věc (CSV, PDF, ale i html stránku, speciálně, pokud je pro tisk) - zase ale přidává nějakou vrstvu, ve které se musí člověk vyznat - mixin `TeXResponseMixin`, který generalizuje generování PDF z TeXu - asi nejméně problematická změna - z funkcionálních Views se stali třídní Views (aby mohli dědit od věcí výše) - to zhoršilo čitelnost, na druhou stranu class based Views jsou doporučované oproti function based Views Prefix `WIP:` tu je proto, že tento pull request koliduje na [#56](https://gitea.ks.matfyz.cz/mam/mamweb/pulls/56) (speciálně to mám aktuálně jako pull request do `split_sous`, aby byly dobře vidět změny).
zelvuska added 13 commits 2024-08-03 00:14:58 +02:00 
Vím, že je toho tady trochu moc najednou, ale napadalo mě to tak propleteně…
zelvuska changed target branch from master to split_sous 2024-08-03 00:15:43 +02:00
zelvuska commented 2024-08-03 00:19:09 +02:00
Author
Owner
Copy link

Možná někam napsat, že object_list je list objektů třídy Soustredeni_Ucastnici, protože na sobě mají ještě navíc poznámku (k danému účastníkovi na daný sous)?

Možná někam napsat, že `object_list` je list objektů třídy `Soustredeni_Ucastnici`, protože na sobě mají ještě navíc poznámku (k danému účastníkovi na daný sous)?
zelvuska added 2 commits 2024-08-04 17:51:42 +02:00
zelvuska added 16 commits 2024-08-06 02:40:23 +02:00 
Reviewed-on: #58

Není tam nic sporného ani kritického[^1], takže rovnou mergeuji…

[^1]: a máme docela málo kapacity na pročítání pull requestů.
zelvuska added 1 commit 2024-08-06 02:41:32 +02:00
zelvuska closed this pull request 2024-10-22 22:17:36 +02:00
zelvuska reopened this pull request 2024-10-24 11:34:08 +02:00
zelvuska changed target branch from split_sous to master 2024-10-24 11:34:16 +02:00
zelvuska changed title from WIP: Předělání sousových views to Předělání sousových views 2024-10-24 11:37:57 +02:00
zelvuska added 1 commit 2024-10-24 11:46:06 +02:00 
# Conflicts:
#	soustredeni/views.py
#	various/views/generic.py
zelvuska added 1 commit 2024-10-24 11:51:41 +02:00
zelvuska added 1 commit 2024-11-05 20:36:45 +01:00
ledoian reviewed 2024-11-06 00:42:15 +01:00
soustredeni/views.py
@ -92,0 +77,4 @@
response = HttpResponse(content_type='text/csv')
response['Content-Disposition'] = 'attachment; filename="ucastnici.csv"'
writer = csv.writer(response)
ledoian commented 2024-11-06 00:42:15 +01:00
Owner
Copy link

Nemám to teď nacachované, ale pročže nepoužijeme generický export řešitelů, který máme už u výsledkovek?

Nemám to teď nacachované, ale pročže nepoužijeme generický export řešitelů, který máme už u výsledkovek?
zelvuska commented 2024-11-12 19:27:09 +01:00
Author
Owner
Copy link

Protože to takhle už bylo (tj. není to předmětem tohodle pullrequestu). Teď bych to nechal a můžem upravit časem.

Protože to takhle už bylo (tj. není to předmětem tohodle pullrequestu). Teď bych to nechal a můžem upravit časem.
zelvuska marked this conversation as resolved
ledoian reviewed 2024-11-06 00:45:32 +01:00
various/views/generic.py Outdated
@ -30,0 +52,4 @@
s titlem `self.if_prazdny_title` a textem `self.if_prazdny_text`
a způsob renderování (např. CSV) lze změnit přepsáním metody render.
"""
allow_empty = False
ledoian commented 2024-11-06 00:45:32 +01:00
Owner
Copy link

Tohle je naše věc, nebo něco co využívá ListView interně?

Tohle je naše věc, nebo něco co využívá `ListView` interně?
zelvuska marked this conversation as resolved
ledoian reviewed 2024-11-06 00:47:09 +01:00
various/views/generic.py Outdated
@ -30,0 +89,4 @@
nebo jiný seznam absolutních cest).
"""
dalsi_potrebne_soubory = []
tex = "pdflatex"
ledoian commented 2024-11-06 00:47:09 +01:00
Owner
Copy link

Navrhuju spíš pojmenovat tex_command nebo i česky tex_prikaz.

Navrhuju spíš pojmenovat `tex_command` nebo i česky `tex_prikaz`.
zelvuska marked this conversation as resolved
ledoian commented 2024-11-06 00:58:55 +01:00
Owner
Copy link

Asi OK, i guess. Principiálně to věci podle mě nezhoršuje, za sebe tam zas tak moc zesložitění nevidím (ale nemám optiku nových webařů…). Neříkám, že vidím úplně moc užitečnost (která další PDF kromě obálek a stvrzenek web vůbec výhledově bude generovat?), ale třeba nás to aspoň motivuje implementovat věci ve webu a ne úplně ad hoc jinde (kontaktníček, pozvánky pro sous, …) :-D

Asi OK, i guess. Principiálně to věci podle mě nezhoršuje, za sebe tam zas tak moc zesložitění nevidím (ale nemám optiku nových webařů…). Neříkám, že vidím úplně moc užitečnost (která další PDF kromě obálek a stvrzenek web vůbec výhledově bude generovat?), ale třeba nás to aspoň motivuje implementovat věci ve webu a ne úplně ad hoc jinde (kontaktníček, pozvánky pro sous, …) :-D
zelvuska added 1 commit 2024-11-12 19:29:24 +01:00
zelvuska added 1 commit 2024-11-12 19:32:27 +01:00
zelvuska merged commit 5883a5cd28 into master 2024-11-12 19:33:11 +01:00
zelvuska deleted branch predelani_sousovych_view 2024-11-12 19:33:11 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: mam/mamweb#57
No description provided.
Delete branch "predelani_sousovych_view"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?