From a1101e912616758a7f70eba222092f2217d82e68 Mon Sep 17 00:00:00 2001 From: PinieP <59698589+PinieP@users.noreply.github.com> Date: Tue, 5 Nov 2024 00:01:15 +0100 Subject: [PATCH] Fix CONFIG.md and change elif to if --- CONFIG.md | 4 ++-- PFERD/crawl/ilias/ilias_web_crawler.py | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/CONFIG.md b/CONFIG.md index 62ab596..9b79be8 100644 --- a/CONFIG.md +++ b/CONFIG.md @@ -184,8 +184,8 @@ If the values work, feel free to submit a PR and add them to the table above. - `login_type`: How you authenticate. (Required) - `local`: Use `client_id` for authentication. - `shibboleth`: Use shibboleth for authentication. -- `client_id`: An ID used for authentication if `client_type` is `local`. - Is ignored if `client_type` is `shibboleth`. +- `client_id`: An ID used for authentication if `login_type` is `local`. Is + ignored if `login_type` is `shibboleth`. - `target`: The ILIAS element to crawl. (Required) - `desktop`: Crawl your personal desktop / dashboard - ``: Crawl the course with the given id diff --git a/PFERD/crawl/ilias/ilias_web_crawler.py b/PFERD/crawl/ilias/ilias_web_crawler.py index 66b6f54..a391c2f 100644 --- a/PFERD/crawl/ilias/ilias_web_crawler.py +++ b/PFERD/crawl/ilias/ilias_web_crawler.py @@ -47,7 +47,7 @@ class IliasWebCrawlerSection(HttpCrawlerSection): self.missing_value("login_type") if login_type == "shibboleth": return "shibboleth" - elif login_type == "local": + if login_type == "local": client_id = self.s.get("client_id") if not client_id: self.missing_value("client_id")