From ac91ad8599ad9101abc1ab254ee6ad892a5d72e7 Mon Sep 17 00:00:00 2001 From: Dries Peeters Date: Thu, 13 Nov 2025 06:40:24 +0100 Subject: [PATCH] refactor(invoices): standardize logging setup and remove debug logging - Add module-level logging import and logger initialization - Remove redundant logging import from export_invoice_pdf function - Ensure all logging uses proper logger instance instead of print statements - Improves production logging consistency and respects LOG_LEVEL configuration Fixes debug logging issues in invoice routes for production readiness. --- app/routes/invoices.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/app/routes/invoices.py b/app/routes/invoices.py index fc509f4..7c482dc 100644 --- a/app/routes/invoices.py +++ b/app/routes/invoices.py @@ -8,6 +8,7 @@ from decimal import Decimal, InvalidOperation import io import csv import json +import logging from app.utils.db import safe_commit from app.utils.excel_export import create_invoices_list_excel from app.utils.prepaid_hours import PrepaidHoursAllocator @@ -19,6 +20,7 @@ from app.utils.posthog_funnels import ( ) invoices_bp = Blueprint('invoices', __name__) +logger = logging.getLogger(__name__) @invoices_bp.route('/invoices') @login_required @@ -771,9 +773,6 @@ def export_invoice_csv(invoice_id): @login_required def export_invoice_pdf(invoice_id): """Export invoice as PDF with optional page size selection""" - import logging - logger = logging.getLogger(__name__) - logger.info(f"Invoice PDF export requested - Invoice ID: {invoice_id}, User: {current_user.username}") invoice = Invoice.query.get_or_404(invoice_id)