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

Vylepšení hodnotítka fix #1354 fix #1237 #20

Merged
zelvuska merged 25 commits from vylepseni_odevzdavatka into master 2023-06-12 22:17:08 +02:00
Conversation 3 Commits 25 Files changed 7 +2 -1
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 6e1b1ef4e8 - Show all commits

3
odevzdavatko/views.py
View file

@ -315,7 +315,8 @@ def hodnoceniReseniView(request, pk, *args, **kwargs):
zmeny_bodu = [it for it in form.changed_data if it.startswith("body")] zmeny_bodu = [it for it in form.changed_data if it.startswith("body")]
if len(zmeny_bodu) == 1: if len(zmeny_bodu) == 1:
hodnoceni.__setattr__(zmeny_bodu[0], data_for_body[zmeny_bodu[0]]) hodnoceni.__setattr__(zmeny_bodu[0], data_for_body[zmeny_bodu[0]])
if len(zmeny_bodu) != 1 and len(zmeny_bodu) != 4: # > jedna změna je špatně, ale 4 "změny" znamenají že nebylo nic zadáno
zelvuska marked this conversation as resolved Outdated
ledoian commented 2023-02-25 14:19:28 +01:00
Outdated
Review
Copy link

Proč že můžou být 4?

Proč že můžou být 4?
ledoian commented 2023-02-25 14:21:15 +01:00
Outdated
Review
Copy link

Už to vidím: je to 4 když se nic nezmění. Prosím, napišme to k tomu do komentáře, rozhodně mi to nepřijde intuitivní.

Už to vidím: je to 4 když se nic nezmění. Prosím, napišme to k tomu do komentáře, rozhodně mi to nepřijde intuitivní.
if len(zmeny_bodu) > 1 and len(zmeny_bodu) != 4:
zelvuska marked this conversation as resolved Outdated
ledoian commented 2023-02-25 14:17:54 +01:00
Outdated
Review
Copy link

Tenhle print je k něčemu dobrý? Nechal bych jen warning o řádek níž, je o dost víc konfigurovatelný…

Tenhle print je k něčemu dobrý? Nechal bych jen warning o řádek níž, je o dost víc konfigurovatelný…
logger.warning(f"Hodnocení {hodnoceni} mělo mít nastavené víc různých bodů: {zmeny_bodu}. Nastavuji -0.1.") logger.warning(f"Hodnocení {hodnoceni} mělo mít nastavené víc různých bodů: {zmeny_bodu}. Nastavuji -0.1.")
hodnoceni.body = -0.1 hodnoceni.body = -0.1
hodnoceni.save() hodnoceni.save()