mam/mamweb
11
0
Fork 0
Code Issues Pull requests 10 Projects Wiki Activity Actions

Vylepšení odevzdávátka #13

Merged
zelvuska merged 17 commits from vylepseni_odevzdavatka into master 2023-01-02 20:25:01 +01:00
Conversation 5 Commits 17 Files changed 17 +1 -1
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 3b6e238323 - Show all commits

2
api/views/autocomplete.py
View file

@ -56,7 +56,7 @@ class PublicResitelAutocomplete(LoginRequiredAjaxMixin, autocomplete.Select2Quer
).exclude(
prezdivka_resitele=""
zelvuska marked this conversation as resolved Outdated
ledoian commented 2022-12-19 22:33:29 +01:00
Outdated
Review
Copy link

Nebylo by lepší ty řádky spojit do něčeho jako prezdivka__len__gt=0?

Nebylo by lepší ty řádky spojit do něčeho jako `prezdivka__len__gt=0`?
zelvuska commented 2022-12-19 22:35:44 +01:00
Outdated
Review
Copy link

Mě to takhle asi přijde popisnější a jsem si jistější, že to funguje…

Mě to takhle asi přijde popisnější a jsem si jistější, že to funguje…
).filter(
prezdivka_resitele__startswith=self.q
prezdivka_resitele__icontains=self.q
zelvuska marked this conversation as resolved Outdated
auburn commented 2022-12-19 20:38:27 +01:00
Outdated
Review
Copy link

Možná to chceme mít aspoň case-insensitive.

Možná to chceme mít aspoň case-insensitive.
zelvuska commented 2022-12-19 20:40:10 +01:00
Outdated
Review
Copy link

asi …__icontains=self.q

asi `…__icontains=self.q`
).all()
zelvuska marked this conversation as resolved
ledoian commented 2022-12-19 22:36:54 +01:00
Review
Copy link

Tenhle způsob zalámání nejspíš nikde nemáme – trochu mi to rozbíjí čtení kódu. Ale možná nikde nemáme takovýhle kód, takže by to pokus o fluent kód (vizte níž) udělal možná taky…

Pod fluent kódem si představuji cca toto:

qs = (m.Resitel.objects
    .filter(…)
    .exclude(...)
    .whatever()
    .all())

(Je to o něco kratší a neskáče tam odsazení jak na pile, což mi přijde trošičku lepší pro čtení, ale je to jen malá preference…)

Tenhle způsob zalámání nejspíš nikde nemáme – trochu mi to rozbíjí čtení kódu. Ale možná nikde nemáme takovýhle kód, takže by to pokus o fluent kód (vizte níž) udělal možná taky… Pod fluent kódem si představuji cca toto: ```python3 qs = (m.Resitel.objects .filter(…) .exclude(...) .whatever() .all()) ``` (Je to o něco kratší a neskáče tam odsazení jak na pile, což mi přijde trošičku lepší pro čtení, ale je to jen malá preference…)
zelvuska commented 2022-12-19 22:39:42 +01:00
Review
Copy link

Tak takhle mi zase chvíli trvá, než mi dojde, k čemu ta tečka na začátku řádku patří…

Tak takhle mi zase chvíli trvá, než mi dojde, k čemu ta tečka na začátku řádku patří…
return qs