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

Open
zelvuska wants to merge 8 commits from predelani_sousovych_view into master
Owner

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 3 months ago
zelvuska changed target branch from master to split_sous 3 months ago
Poster
Owner

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 3 months ago
zelvuska added 16 commits 3 months ago
zelvuska added 1 commit 3 months ago
zelvuska closed this pull request 1 week ago
zelvuska reopened this pull request 1 week ago
zelvuska changed target branch from split_sous to master 1 week ago
zelvuska changed title from WIP: Předělání sousových views to Předělání sousových views 1 week ago
zelvuska added 1 commit 1 week ago
446515a52e Merge branch 'master' into predelani_sousovych_view
zelvuska added 1 commit 1 week ago
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.