From af19a14eee50ae7bd2cd1787e69e5ad36b730050 Mon Sep 17 00:00:00 2001 From: Frisk Date: Wed, 18 Nov 2020 14:58:13 +0100 Subject: [PATCH] Fix critical issues that will happen on 1.2 to 1.3 migration --- src/misc.py | 7 ++++++- src/rc.py | 2 +- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/src/misc.py b/src/misc.py index 25fb039..40ac615 100644 --- a/src/misc.py +++ b/src/misc.py @@ -89,7 +89,12 @@ class DataFile: self.changed = True def __getitem__(self, item): - return self.data[item] + try: + return self.data[item] + except KeyError: # if such value doesn't exist, set to and return none + self.__setitem__(item, None) + self.save_datafile() + return None datafile = DataFile() diff --git a/src/rc.py b/src/rc.py index 1d4b5f6..c380ce2 100644 --- a/src/rc.py +++ b/src/rc.py @@ -242,7 +242,7 @@ class Recent_Changes_Class(object): return else: rc_last_id = self.prepare_rc(rc, amount) - if settings["show_abuselog"]: + if settings.get("show_abuselog", False): try: abuselog = request_json["query"]["abuselog"] # While LYBL approach would be more performant when abuselog is not in request body, I prefer this approach for its clarity except KeyError: