From aabce764ace344a93c0876b6a304921b0ca09db6 Mon Sep 17 00:00:00 2001 From: Joscha Date: Tue, 25 May 2021 15:54:01 +0200 Subject: [PATCH] Clean up TODOs --- PFERD/crawl/ilias/kit_ilias_html.py | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/PFERD/crawl/ilias/kit_ilias_html.py b/PFERD/crawl/ilias/kit_ilias_html.py index 032bb27..4bc3161 100644 --- a/PFERD/crawl/ilias/kit_ilias_html.py +++ b/PFERD/crawl/ilias/kit_ilias_html.py @@ -393,8 +393,7 @@ class IliasPage: ) if modification_date_match is None: modification_date = None - # TODO: Figure out if this is expected or *always* an error. - log.explain(f"Element {name} at {url} has no date. Properties: {all_properties_text!r}") + log.explain(f"Element {name} at {url} has no date.") else: modification_date_str = modification_date_match.group(1) modification_date = demangle_date(modification_date_str) @@ -420,9 +419,6 @@ class IliasPage: if "target=file_" in parsed_url.query: return IliasElementType.FILE - # TODO: Match based on CMD_CLASS or icon? The folder_like check looks at the icon, - # but we could also match the command class. I am not sure what's more stable. - # Everything with a ref_id can *probably* be opened to reveal nested things # video groups, directories, exercises, etc if "ref_id=" in parsed_url.query: