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

Upgrade odevzdavatka #30

Merged
zelvuska merged 37 commits from upgrade_odevzdavatka into master 2023-06-19 20:48:30 +02:00
Conversation 3 Commits 37 Files changed 15 +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 18f55153be - Show all commits

2
api/views/autocomplete.py
View file

@ -80,7 +80,7 @@ class OdevzdatelnyProblemAutocomplete(autocomplete.Select2QuerySetView):
# Seřadíme tak, aby ty s nadproblem==None byly dole (větší motivace tam naklikat konkrétní úlohy) a pak nějak rozumně. # Seřadíme tak, aby ty s nadproblem==None byly dole (větší motivace tam naklikat konkrétní úlohy) a pak nějak rozumně.
# Tohle je řazení pro odevzdávátko, kde je definován nadproblém, proto je to v tomto ifu. (Jinde si to netroufám řadit) # Tohle je řazení pro odevzdávátko, kde je definován nadproblém, proto je to v tomto ifu. (Jinde si to netroufám řadit)
qs = qs.order_by("nadproblem", "kod", "nazev") qs = qs.order_by("nadproblem", "kod", "nazev")
qs = [problem for problem in qs if problem.hlavni_problem.id == nadproblem_id] qs = list(filter(lambda problem: problem.hlavni_problem.id == nadproblem_id, qs))
zelvuska marked this conversation as resolved Outdated
ledoian commented 2023-06-19 18:51:06 +02:00
Outdated
Review
Copy link

Tohle je možná lepší psát nějakým filterem (ať už QuerySetovým, nebo čistě pythonovým list(filter(lambda: ..., qs)) – není to úprava dat, jen filtrování, tak ať je to názornější.

(A přirozeně, pokud by to šlo tím QuerySetovým, tak to může být rychlejší, protože databáze není napsaná v Pythonu :-))

Tohle je možná lepší psát nějakým filterem (ať už QuerySetovým, nebo čistě pythonovým `list(filter(lambda: ..., qs))` – není to úprava dat, jen filtrování, tak ať je to názornější. (A přirozeně, pokud by to šlo tím QuerySetovým, tak to může být rychlejší, protože databáze není napsaná v Pythonu :-))
zelvuska commented 2023-06-19 18:54:04 +02:00
Outdated
Review
Copy link

No QuerySetovým to nejde, protože hlavni_problem není databázová věc… Nebo to jde nějak obejít?

No QuerySetovým to nejde, protože hlavni_problem není databázová věc… Nebo to jde nějak obejít?
ledoian commented 2023-06-19 19:53:07 +02:00
Outdated
Review
Copy link

Já si úplně nepamatuju, jestli to je nebo není DB věc :-) Takže holt klasickým pythoním filterem.

Já si úplně nepamatuju, jestli to je nebo není DB věc :-) Takže holt klasickým pythoním filterem.
return qs return qs
class ProblemAutocomplete(autocomplete.Select2QuerySetView): class ProblemAutocomplete(autocomplete.Select2QuerySetView):