From 3f4b1ec4fc9a4722e4505def23ad326deea94e98 Mon Sep 17 00:00:00 2001
From: ChuckBuilds <33324927+ChuckBuilds@users.noreply.github.com>
Date: Wed, 23 Apr 2025 15:38:40 -0500
Subject: [PATCH] refactor: reduce debug logging output in calendar and display
 managers

---
 src/calendar_manager.py | 19 +++++--------------
 src/display_manager.py  |  6 ------
 2 files changed, 5 insertions(+), 20 deletions(-)

diff --git a/src/calendar_manager.py b/src/calendar_manager.py
index 2eec6b5a..ad3db24a 100644
--- a/src/calendar_manager.py
+++ b/src/calendar_manager.py
@@ -33,13 +33,9 @@ class CalendarManager:
         self.events = []
         self.service = None
         
-        # Log font information during initialization
-        logger.debug(f"Display Manager fonts:")
-        logger.debug(f"  Small font: {self.display_manager.small_font}")
-        logger.debug(f"  Calendar font: {self.display_manager.calendar_font}")
-        logger.debug(f"  Font types - Small: {type(self.display_manager.small_font)}, Calendar: {type(self.display_manager.calendar_font)}")
-        
-        logger.info(f"Calendar configuration: enabled={self.enabled}, update_interval={self.update_interval}, max_events={self.max_events}, calendars={self.calendars}")
+        # Log font information during initialization only
+        if self.enabled:
+            logger.info(f"Calendar configuration: enabled={self.enabled}, update_interval={self.update_interval}, max_events={self.max_events}, calendars={self.calendars}")
         
         # Get timezone from config
         self.config_manager = ConfigManager()
@@ -122,7 +118,7 @@ class CalendarManager:
         try:
             # Get date, time, and summary
             date_text = self._format_event_date(event)
-            time_text = self._format_event_time(event).replace(" ", "")  # Remove space between time and AM/PM
+            time_text = self._format_event_time(event)
             datetime_text = f"{date_text} {time_text}"
             summary = event.get('summary', 'No Title')
             
@@ -242,15 +238,12 @@ class CalendarManager:
             if 'T' in start:
                 # The datetime string already includes timezone info (-05:00)
                 dt = datetime.fromisoformat(start)
-                logger.debug(f"Parsed datetime from event: {dt}")
             else:
                 dt = datetime.strptime(start, '%Y-%m-%d')
                 # Make date object timezone-aware (assume UTC if no tz info)
                 dt = pytz.utc.localize(dt)
-                logger.debug(f"Parsed date from event: {dt}")
             
             # No need to convert timezone since it's already in the correct one
-            logger.debug(f"Using event timezone: {dt}")
             return dt.strftime("%a %-m/%-d") # e.g., "Mon 4/21"
         except ValueError as e:
             logging.error(f"Could not parse date string: {start} - {e}")
@@ -265,10 +258,8 @@ class CalendarManager:
         try:
             # The datetime string already includes timezone info (-05:00)
             dt = datetime.fromisoformat(start)
-            logger.debug(f"Parsed time from event: {dt}")
             # No need to convert timezone since it's already in the correct one
-            logger.debug(f"Using event timezone: {dt}")
-            return dt.strftime("%I:%M %p")
+            return dt.strftime("%I:%M%p")
         except ValueError as e:
             logging.error(f"Could not parse time string: {start} - {e}")
             return "Invalid Time"
diff --git a/src/display_manager.py b/src/display_manager.py
index 5bb71734..d62f618b 100644
--- a/src/display_manager.py
+++ b/src/display_manager.py
@@ -243,14 +243,8 @@ class DisplayManager:
             # Select font based on parameters
             if font:
                 current_font = font
-                if not hasattr(self, '_last_font') or self._last_font != font:
-                    logger.debug(f"Using custom font: {font}")
-                    self._last_font = font
             else:
                 current_font = self.small_font if small_font else self.regular_font
-                if not hasattr(self, '_last_font') or self._last_font != current_font:
-                    logger.debug(f"Using {'small' if small_font else 'regular'} font: {current_font}")
-                    self._last_font = current_font
             
             # Calculate x position if not provided (center text)
             if x is None: