akce na sjednoceni cisel do +420 123 456 789 #53
Merged
zelvuska
merged 3 commits from sjednoceni_telefonu into master 4 months ago
zelvuska
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'sjednoceni_telefonu'
Deleting a branch is permanent. It CANNOT be undone. Continue?
Sjednocení telefonů do formátu "+420 123 456 789"
Netestováno!
akce na sjednoceni cisel do +420 123 456 78to akce na sjednoceni cisel do +420 123 456 789 4 months agoSkvělý začátek, ale chybová hláška není pro průměrné orgy (takové ty, co musí mít vše na 3 kliknutí).
else:raise ValueErrorexcept:print("Hmm má divné číslo - " + o.telefon + " - " + str(o.id))Hmm má divné číslo - +42077123881237 (jsem-li v česku); jinak +213893246791425 - 50
Představoval bych si spíše
Jonáš Havelka (50) má divné číslo "+42077123881237 (jsem-li v česku); jinak +213893246791425"
Super :)
40ca76d414into master 4 months agoNo, pořád bych chtěl protestovat proti tomu obecnému exceptu. Mimo jiné to znamená, že když v nešikovnou chvíli zkusím zabít web pomocí Ctrl+C, tak to v Adminu vypíše hlášku o blbosti a web to nezabije.
Celkově je docela dobrý nápad nemít moc dlouhé
trybloky a hlídat jen chyby, které umíme řešit (podobně irelevantní je problém s databází, ten taky může nastat kdykoliv…)Já vím, já si to třeba jednou fixnu, spíš pro příště :-)
Reviewers
40ca76d414.