diff --git a/plugins/OptionalManager/ContentDbPlugin.py b/plugins/OptionalManager/ContentDbPlugin.py index ccfd6637..21193fa0 100644 --- a/plugins/OptionalManager/ContentDbPlugin.py +++ b/plugins/OptionalManager/ContentDbPlugin.py @@ -25,7 +25,7 @@ class ContentDbPlugin(object): self.my_optional_files = {} # Last 50 site_address/inner_path called by fileWrite (auto-pinning these files) self.optional_files = collections.defaultdict(dict) self.optional_files_loading = False - helper.timer(60 * 5, self.checkOptionalLimit) + self.timer_check_optional = helper.timer(60 * 5, self.checkOptionalLimit) super(ContentDbPlugin, self).__init__(*args, **kwargs) def getSchema(self): diff --git a/plugins/OptionalManager/OptionalManagerPlugin.py b/plugins/OptionalManager/OptionalManagerPlugin.py index c33063dc..fbb8158c 100644 --- a/plugins/OptionalManager/OptionalManagerPlugin.py +++ b/plugins/OptionalManager/OptionalManagerPlugin.py @@ -19,6 +19,8 @@ def importPluginnedClasses(): def processAccessLog(): if access_log: content_db = ContentDbPlugin.content_db + if not content_db.conn: + return False now = int(time.time()) num = 0 for site_id in access_log: @@ -33,6 +35,8 @@ def processAccessLog(): def processRequestLog(): if request_log: content_db = ContentDbPlugin.content_db + if not content_db.conn: + return False cur = content_db.getCursor() num = 0 for site_id in request_log: diff --git a/src/Test/conftest.py b/src/Test/conftest.py index 8155cdc2..b1944cd7 100644 --- a/src/Test/conftest.py +++ b/src/Test/conftest.py @@ -186,7 +186,8 @@ def site(request): def cleanup(): site.delete() - site.content_manager.contents.db.close() + site.content_manager.contents.db.close("Test cleanup") + site.content_manager.contents.db.timer_check_optional.kill() SiteManager.site_manager.sites.clear() db_path = "%s/content.db" % config.data_dir os.unlink(db_path) @@ -213,7 +214,8 @@ def site_temp(request): def cleanup(): site_temp.delete() - site_temp.content_manager.contents.db.close() + site_temp.content_manager.contents.db.close("Test cleanup") + site_temp.content_manager.contents.db.timer_check_optional.kill() db_path = "%s-temp/content.db" % config.data_dir os.unlink(db_path) del ContentDb.content_dbs[db_path]