mam/mamweb
13
1
Fork 0
Code Issues Pull requests 13 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 -4
1 changed files with 1 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 98ac3f63bf - Show all commits

5
personalni/forms.py
View file

@ -208,11 +208,8 @@ class ProfileEditForm(forms.Form):
prezdivka_resitele = self.cleaned_data.get('prezdivka_resitele') prezdivka_resitele = self.cleaned_data.get('prezdivka_resitele')
if prezdivka_resitele == '': if prezdivka_resitele == '':
return prezdivka_resitele return prezdivka_resitele
try: if Resitel.objects.filter(prezdivka_resitele=prezdivka_resitele).exclude(osoba__user__username=self.username).count() > 0:
Resitel.objects.get(prezdivka_resitele=prezdivka_resitele)
raise forms.ValidationError('Přezdívka je již použita') raise forms.ValidationError('Přezdívka je již použita')
zelvuska marked this conversation as resolved Outdated
ledoian commented 2022-12-19 23:10:04 +01:00
Outdated
Review
Copy link

Tohle velmi zní, jako že ta přezdívka prostě nejde editovat vůbec – pokud už v databázi jsem, tak tohle matchne.

Tohle velmi zní, jako že ta přezdívka prostě nejde editovat vůbec – pokud už v databázi jsem, tak tohle matchne.
zelvuska commented 2022-12-19 23:14:25 +01:00
Outdated
Review
Copy link

?

?
ledoian commented 2022-12-19 23:17:40 +01:00
Outdated
Review
Copy link

Pokud si Alice nastaví přezdívku „Axolotl“, tak ji má v databázi. Dotaz Restitel.objects.get(prezdivka=Axolotl) tedy tuto přezdívku najde a celý ResitelEditForm kvůli tomu nepůjde Alici odeslat, přestože je to její přezdívka.

Pokud si Alice nastaví přezdívku „Axolotl“, tak ji má v databázi. Dotaz `Restitel.objects.get(prezdivka=Axolotl)` tedy tuto přezdívku najde a celý `ResitelEditForm` kvůli tomu nepůjde Alici odeslat, přestože je to její přezdívka.
zelvuska commented 2022-12-19 23:19:24 +01:00
Outdated
Review
Copy link

Promiň, tvůj komentář jsem pochopil jinak. Samozřejmě tohle je špatně… Dík

Promiň, tvůj komentář jsem pochopil jinak. Samozřejmě tohle je špatně… Dík
zelvuska commented 2022-12-19 23:28:41 +01:00
Outdated
Review
Copy link

Ha, já nevím, podle čeho určit, který řešitel to edituje…

Ha, já nevím, podle čeho určit, který řešitel to edituje…
ledoian commented 2022-12-19 23:32:10 +01:00
Outdated
Review
Copy link

A nestačí to prostě uložit a zjistit, jestli to vyšlo? (Dost možná nestačí, nemám nastudovanou interakci formů a modelů, ale kdyby to stačilo, tak je to zdaleka nejjednodušší…)

A nestačí to prostě uložit a zjistit, jestli to vyšlo? (Dost možná nestačí, nemám nastudovanou interakci formů a modelů, ale kdyby to stačilo, tak je to zdaleka nejjednodušší…)
zelvuska commented 2022-12-19 23:33:00 +01:00
Outdated
Review
Copy link

A poznáš, co selhalo? A přijde mi, že tohle by mělo být ještě v clean_…

A poznáš, co selhalo? A přijde mi, že tohle by mělo být ještě v clean_…
zelvuska commented 2022-12-19 23:53:01 +01:00
Outdated
Review
Copy link

Vyřešeno. Ten objekt má při použití nastavený username…

Vyřešeno. Ten objekt má při použití nastavený username…
except ObjectDoesNotExist:
pass
return prezdivka_resitele return prezdivka_resitele
def clean_email(self): def clean_email(self):
zelvuska marked this conversation as resolved Outdated
ledoian commented 2022-12-19 23:11:54 +01:00
Outdated
Review
Copy link

Reálně se celý try-block snaží jen imitovat podmínku „pokud přezdívka není zabraná“ – je přehlednější to vyřešit spíš něčím jako if m.Resitel.objects.….count() > 0: raise Exception() (ne nutně na jednom řádku a s určitě s příslušnými třídami a doplněními).

Reálně se celý try-block snaží jen imitovat podmínku „pokud přezdívka není zabraná“ – je přehlednější to vyřešit spíš něčím jako `if m.Resitel.objects.….count() > 0: raise Exception()` (ne nutně na jednom řádku a s určitě s příslušnými třídami a doplněními).