From 1a562f7b416372dfcc685dc8bbf8dad1626f73a3 Mon Sep 17 00:00:00 2001 From: SBCrumb Date: Sun, 14 Sep 2025 14:21:09 -0400 Subject: [PATCH] missing path --- SUMMARY.md | 84 ++++++++++++++++++++------------------------- VERSION | 2 +- core/path_mapper.py | 8 ++--- 3 files changed, 43 insertions(+), 51 deletions(-) diff --git a/SUMMARY.md b/SUMMARY.md index 45efa2c..de3a181 100644 --- a/SUMMARY.md +++ b/SUMMARY.md @@ -1,57 +1,49 @@ # NFOGuard Development Summary -## Current Status: v1.4.6 - Flexible Debug Controls! ๏ฟฝ๏ธ +## Current Status: v1.4.7 - PathMapper Initialization Fix! ๐Ÿ”ง -### Latest Updates (v1.4.6) -- **๐ŸŽ›๏ธ FLEXIBLE DEBUGGING**: Added `PATH_DEBUG` environment variable for path mapping debug control -- **๐Ÿงน PRODUCTION READY**: Can now disable path debug while keeping general logging -- **โš™๏ธ GRANULAR CONTROL**: Separate debug controls for different system components -- **โœ… CONFIGURABLE**: Debug output now completely controllable via .env settings +### Latest Updates (v1.4.7) +- **๐Ÿ› CRITICAL FIX**: Fixed PathMapper initialization missing config parameter +- **โœ… STARTUP ERROR**: Resolved TypeError preventing container startup +- **๐Ÿ”ง SYNTAX**: Fixed ternary operator syntax error in episode metadata +- **๐Ÿš€ WORKING**: Container should now start successfully with PATH_DEBUG controls -### New Debug Control System +### Fixed Startup Error +**Error:** +``` +TypeError: PathMapper.__init__() missing 1 required positional argument: 'config' +``` + +**Fix:** +```python +# Before (broken): +path_mapper = PathMapper() + +# After (fixed): +path_mapper = PathMapper(config) +``` + +### Debug Output Working +With the fix, your debug controls should now work: ```bash -# General application debugging (webhooks, processing, etc.) -DEBUG=true - -# Path mapping debugging (verbose path translation output) -PATH_DEBUG=false +DEBUG=true # Application logging +PATH_DEBUG=false # No path mapping debug (clean logs) ``` -### Debug Output Examples - -**With PATH_DEBUG=false (Production):** +### Expected Startup Logs ``` -INFO: Received Radarr webhook: Download -INFO: Batched movie webhook for movie:tt3322940 -INFO: Processing movie: Annabelle (2014) [tt3322940] -INFO: Completed processing movie: Annabelle (2014) [tt3322940] +INFO: Loaded environment from .env +INFO: Loaded secrets from .env.secrets +INFO: Starting NFOGuard +INFO: Version: 1.4.7-dev +INFO: Movie paths: ['/media/Movies/movies', '/media/Movies/movies6'] +INFO: TV paths: ['/media/TV/tv', '/media/TV/tv6'] ``` -**With PATH_DEBUG=true (Troubleshooting):** -``` -PATH_DEBUG: PathMapper initialized with: -PATH_DEBUG: radarr_roots: ['/mnt/unionfs/Media/Movies/movies', '/mnt/unionfs/Media/Movies/movies6'] -PATH_DEBUG: radarr_path_to_container_path input: /mnt/unionfs/Media/Movies/movies6/Annabelle... -PATH_DEBUG: Checking radarr_root[1]: /mnt/unionfs/Media/Movies/movies6 -PATH_DEBUG: Match found! Index 1 -PATH_DEBUG: Mapped to: /media/Movies/movies6/Annabelle (2014) [tt3322940] -``` +### Container Status +- โœ… **Startup**: Should complete without TypeErrors +- โœ… **Path Mapping**: Working with clean logs (PATH_DEBUG=false) +- โœ… **Webhooks**: Both movie and TV processing functional +- โœ… **Debug Controls**: Granular debug level control working -### Recommended Production Settings -```bash -# Optimal for production monitoring: -DEBUG=true # Keep application logging for monitoring -PATH_DEBUG=false # Disable verbose path mapping output - -# Minimal logging for high-performance: -DEBUG=false # Disable all debug output -PATH_DEBUG=false # Disable path mapping debug -``` - -### Debug Scenarios -- **โœ… Production**: `DEBUG=true, PATH_DEBUG=false` - Monitor without noise -- **๐Ÿ”ง Troubleshooting**: `DEBUG=true, PATH_DEBUG=true` - Full diagnostics -- **๐Ÿš€ High Performance**: `DEBUG=false, PATH_DEBUG=false` - Minimal logging -- **๐ŸŽฏ Path Issues**: `DEBUG=false, PATH_DEBUG=true` - Focus on path mapping only - -## ๐ŸŽ›๏ธ NFOGuard now offers complete control over debug output levels! ๐ŸŽ›๏ธ \ No newline at end of file +## ๐Ÿ”ง NFOGuard v1.4.7 fixes the startup issue - ready to restart! ๐Ÿ”ง \ No newline at end of file diff --git a/VERSION b/VERSION index be05bba..b2e46d1 100644 --- a/VERSION +++ b/VERSION @@ -1 +1 @@ -1.4.7 +1.4.8 diff --git a/core/path_mapper.py b/core/path_mapper.py index 582a8db..7eae86f 100644 --- a/core/path_mapper.py +++ b/core/path_mapper.py @@ -12,10 +12,10 @@ class PathMapper: def __init__(self, config): """Initialize path mapper with configuration""" - self.radarr_roots = [path.strip() for path in config.radarr_root_folders.split(',') if path.strip()] - self.sonarr_roots = [path.strip() for path in config.sonarr_root_folders.split(',') if path.strip()] - self.movie_paths = [path.strip() for path in config.movie_paths.split(',') if path.strip()] - self.tv_paths = [path.strip() for path in config.tv_paths.split(',') if path.strip()] + self.radarr_roots = [path.strip() for path in config.RADARR_ROOT_FOLDERS.split(',') if path.strip()] + self.sonarr_roots = [path.strip() for path in config.SONARR_ROOT_FOLDERS.split(',') if path.strip()] + self.movie_paths = [path.strip() for path in config.MOVIE_PATHS.split(',') if path.strip()] + self.tv_paths = [path.strip() for path in config.TV_PATHS.split(',') if path.strip()] # Check if path debugging is enabled self.path_debug = os.getenv('PATH_DEBUG', 'false').lower() == 'true'