diff --git a/expenses_manager/expenses/views.py b/expenses_manager/expenses/views.py index bbb139e..dcd384f 100644 --- a/expenses_manager/expenses/views.py +++ b/expenses_manager/expenses/views.py @@ -1,3 +1,4 @@ +import logging from operator import truediv from datetime import date, datetime from django.contrib import messages @@ -39,6 +40,7 @@ MONTHS = { 12: "DICIEMBRE", } +logger = logging.getLogger(__name__) def _get_int(value): try: @@ -127,8 +129,11 @@ def expense_list(request): category = _get_int(request.GET.get("category")) account_id = _get_int(request.GET.get("account")) - tag_ids = request.GET.getlist("tag") - tag_ids = [int(t) for t in tag_ids] + tag_ids = [ + t_id + for t in request.GET.getlist("tag") + if (t_id:= _get_int(t)) is not None + ] if year: expenses = expenses.filter(date__year=year) @@ -449,7 +454,11 @@ def dashboard(request): try: monthly_data = acc.monthly_balance(selected_year) m_balance = [float(row["balance"]) for row in monthly_data] - except: + except Exception: + logger.exception( + "Error calculando monthly_balance para la cuenta %s (año %s)", + acc.id, selected_year + ) m_balance = [0] * 12 if selected_year == today.year: