From 20f7ea202b07de6bc997da62879c9f86a9c74399 Mon Sep 17 00:00:00 2001 From: why-not-try-calmer Date: Tue, 14 Nov 2023 16:50:21 +0100 Subject: [PATCH] push_info cannot be called from its parent django app, so let us fallback on exceptions --- comptages/core/report.py | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/comptages/core/report.py b/comptages/core/report.py index e74f0941..bb0597a4 100644 --- a/comptages/core/report.py +++ b/comptages/core/report.py @@ -2,7 +2,6 @@ from datetime import timedelta, datetime from openpyxl import load_workbook -from comptages.core.utils import push_info from comptages.datamodel import models from comptages.core import statistics @@ -74,13 +73,12 @@ def _prepare_yearly_report( count_nb = count_qs.count() if count_nb < lower_bound: - push_info( + raise ValueError( f""" - Only {count_nb} Count objects were found! Please add {lower_bound - count_nb} objects before retrying. - No report will be generated until then. + Only {count_nb} Count objects were found! Please add {lower_bound - count_nb} objects before retrying. + No report will be generated until then. """ ) - return count = count_qs[0] workbook = load_workbook(filename=template_path)