[spidermila]

re #217
tohle jsem udelal uz pred mesicem, ale zapomnel jsem na to udelat PR. prosim, mrkni, jestli to je dost dobre.

[frenzymadness]

Testy padají, protože není dosaženo 100% pokrytí kódu, takže budeme muset přidat ještě i testy pro tu novou funkcionalitu.

Můj pohled:

• Pozor na nesouvisející změny: Na jednom místě mažeš prázdný řádek a na jiném přidáváš individuální typovou anotaci. Typové anotace v projektu mají velký smysl, ale jen pokud je ten projekt kontroluje. Jinak je to kousek kódu, se kterým Python nijak nepracuje a neověřuje jej a mohou zastarat stejně snadno jako neudržované komentáře.
• Moc se mi nelíbí, že se volá flash na dvou místech. Navíc ta funkce je velmi obecná a je možné ji volat i úplně mimo kontext webové apky - třeba z konzole při nějakém hromadném zpracování dat a odesílání e-mailů. A v takovém případě by flash uvnitř ani nefungoval. Možná řešení:
  • Můžeme tu funkci buď nechat vrátit tuple – něco jako (success: bool, error_msg: str) a podle toho se pak chovat ve view.
  • Můžeme nechat tu vyjímku probublat až do view a chytit jí tam.
  • Můžeme vrátit string s chybou – pokud nastane – nebo nechat implicitní None, pokud bude vše v pořádku a reagovat na to flash zprávami ve view.

Co myslíš?