Připsání mailu do chybové hlášky při importu.
2 unresolved threads
2 unresolved threads
Closes #190 (closed)
Merge request reports
Activity
84 85 85 86 def error(self, msg: str) -> Any: 86 87 if self.line_number > 0: 87 msg = f"Řádek {self.line_number}: {msg}" 88 try: 89 msg = f"Řádek {self.line_number} ({self.row.email}): {msg}" # type: ignore 90 # (snaha čtení prvku potomka) 91 except: 92 msg = f"Řádek {self.line_number}: {msg}" Odchytávat úplně všechny výjimky mi přijde dost nešikovné: může to zamaskovat skutečné chyby v programu. Spíš bych použil
if self.row and hasattr(self.row, 'email')
.Ale obecně přemýšlím, jestli vůbec je rozumné udržovat odkaz na aktuální řádek -- běžné funkce na něj nemají co sahat. Nebylo by lepší zavést
self.row_name
a upravit tu smyčku, která voláimport_row
, aby zavolala nějakou metoduget_row_name(row)
? Ta by byla v obecné tříděImport
definovaná tak, že by zkusila najít email, a podtřídy by si ji mohly předefinovat (třeba import bodů by to mohl poskládat ze jména a příjmení). Dává to smysl?
added 1 commit
- 99df2111 - Připsání mailu do chybové hlášky při importu.
376 380 except UnicodeDecodeError: 377 381 return False 378 382 383 def get_row_name(self, row: mo.csv.Row) -> Optional[str]: 384 if row is not None and hasattr(row, 'email'): changed this line in version 3 of the diff
added 1 commit
- e8013730 - Připsání mailu do chybové hlášky při importu.
mentioned in commit 2801e5a3
Please register or sign in to reply