From bc7876d50e098950bba8712f0161ef35310248ce Mon Sep 17 00:00:00 2001 From: Jonathan Weth <git@jonathanweth.de> Date: Sun, 27 Dec 2020 17:41:01 +0100 Subject: [PATCH] [Data checks] Simplify way to find out if celery is enabled --- aleksis/core/data_checks.py | 2 ++ aleksis/core/views.py | 3 +-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/aleksis/core/data_checks.py b/aleksis/core/data_checks.py index 1f2e2c5ef..6355983d9 100644 --- a/aleksis/core/data_checks.py +++ b/aleksis/core/data_checks.py @@ -197,6 +197,8 @@ def check_data(): if get_site_preferences()["general__data_checks_send_emails"]: send_emails_for_data_checks() + return True + def send_emails_for_data_checks(): """Notify one or more recipients about new problems with data. diff --git a/aleksis/core/views.py b/aleksis/core/views.py index a84a4a545..c26eb7b36 100644 --- a/aleksis/core/views.py +++ b/aleksis/core/views.py @@ -728,8 +728,7 @@ class RunDataChecks(PermissionRequiredMixin, View): permission_required = "core.run_data_checks" def get(self, request: HttpRequest, *args, **kwargs) -> HttpResponse: - check_data() - if is_celery_enabled(): + if not check_data(): messages.success( request, _( -- GitLab