Skip to content
Snippets Groups Projects

O něco blbuvzdornější import účastníků

Merged Jan Prachař requested to merge honza/blbuvzordy-import into master
6 unresolved threads

Garanti FO s tím bojujou. Tohle řeší několik případů ze dneška.

Merge request reports

Merged by Martin MarešMartin Mareš 4 years ago (Mar 4, 2021 10:55pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
162 162
163 163 # Ve snaze zabránit Excelu v interpretování ročníku jako kalendářního data
164 164 # lidé připisují všechny možné i nemožné znaky, které vypadají jako apostrof :)
165 rocnik = re.sub('[\'"\u00b4\u2019]', "", rocnik)
165 rocnik = re.sub('^[^\d]', "", rocnik)
  • 162 162
    163 163 # Ve snaze zabránit Excelu v interpretování ročníku jako kalendářního data
    164 164 # lidé připisují všechny možné i nemožné znaky, které vypadají jako apostrof :)
    165 rocnik = re.sub('[\'"\u00b4\u2019]', "", rocnik)
    165 rocnik = re.sub('^[^\d]', "", rocnik)
    166 166
    167 if (school.is_ss and re.fullmatch(r'\d/\d', rocnik)
    168 or school.is_zs and re.fullmatch(r'\d', rocnik)):
    169 return rocnik
    167 if (school.is_ss and not re.fullmatch(r'\d/\d', rocnik)):
  • Martin Mareš
  • Martin Mareš
  • 94 95 pass
    95 96
    96 97
    98 class UnknownColumnError(RuntimeError):
    99 def __init__(self, unknown_name: str, columns: set):
    100 best_matches = get_close_matches(unknown_name, columns, n=1)
    101 if not best_matches:
    102 self.message = (
    103 "Neznámý sloupec '{}'. Podporovaná množina sloupců je tato: {}".format(
    104 unknown_name, columns))
    105 else:
    106 self.message = (
    107 "Neznámý sloupec '{}', měli jste na mysli '{}'?".format(
    108 unknown_name, best_matches[0]))
    109
  • Jan Prachař added 1 commit

    added 1 commit

    • 2ffefe28 - O něco blbuvzdornější import účastníků

    Compare with previous version

    • Celkově se mi líbí zblbuvzdorňovat import, ale tady mám dojem, že některé věci jsou už trochu přehnané :)

    • Všechno to navazuje na případy z dneška. Matamatičtí garanti jsou asi obratnější :)

      Velký problém byl u hlášky a špatném formátu ročníku. Tam tápali, jestli je špatně škola, nebo ten samotný zápis.

    • Please register or sign in to reply
  • Jan Prachař added 2 commits

    added 2 commits

    • 117d5f4d - import: oprava hlášky pro chybný ročník
    • a294a0c4 - import: unknown columns as warning

    Compare with previous version

  • Jan Prachař added 1 commit

    added 1 commit

    Compare with previous version

  • Martin Mareš
  • Jan Prachař added 2 commits

    added 2 commits

    • eb31cd8e - ještě srozumitelnější hláška
    • 32d2de15 - revert regexp na apostrofy

    Compare with previous version

  • Jan Prachař added 1 commit

    added 1 commit

    • b9bd99e6 - varování při importu bodů

    Compare with previous version

  • Jan Prachař added 3 commits

    added 3 commits

    Compare with previous version

  • Martin Mareš
    Martin Mareš @mj started a thread on commit ed29dd9c
  • 69 69
    70 70 def __init__(self):
    71 71 self.errors = []
    72 self.warnings = []
  • Jan Prachař added 1 commit

    added 1 commit

    • 19cc463e - O něco blbuvzdornější import účastníků

    Compare with previous version

  • merged

  • Martin Mareš mentioned in commit 6058cac8

    mentioned in commit 6058cac8

  • Please register or sign in to reply
    Loading