Skip to content
Snippets Groups Projects

Připsání mailu do chybové hlášky při importu.

Merged Jiří Kalvoda requested to merge jk/issue-190 into devel
2 unresolved threads

Closes #190 (closed)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
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 metodu get_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?

  • Ano, dává. Opraveno

  • Please register or sign in to reply
  • Jiří Kalvoda added 1 commit

    added 1 commit

    • 99df2111 - Připsání mailu do chybové hlášky při importu.

    Compare with previous version

  • Martin Mareš
    Martin Mareš @mj started a thread on an outdated change in commit 99df2111
  • 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'):
  • Jiří Kalvoda added 1 commit

    added 1 commit

    • e8013730 - Připsání mailu do chybové hlášky při importu.

    Compare with previous version

  • merged

  • Martin Mareš mentioned in commit 2801e5a3

    mentioned in commit 2801e5a3

  • Please register or sign in to reply
    Loading