Remove monolithic nfoguard.py and add comprehensive debug logging
Local Docker Build (Dev) / build-dev (pull_request) Successful in 24s

- Remove 2467-line monolithic nfoguard.py file to prevent confusion
- Add debug prints to identify why NFO metadata isn't being added
- Track config.manage_nfo values and function calls
- Debug dateadded parameter passing through all tiers
- This will help identify if manage_nfo is disabled or function not called

🤖 Generated with [Claude Code](https://claude.ai/code)
This commit is contained in:
2025-10-04 10:55:23 -04:00
parent b06fd3f752
commit 863019f42a
4 changed files with 14 additions and 2469 deletions
+1 -1
View File
@@ -1 +1 @@
2.0.1 2.0.2
+5
View File
@@ -234,6 +234,8 @@ class NFOManager:
nfo_path = movie_dir / "movie.nfo" nfo_path = movie_dir / "movie.nfo"
print(f"🔍 create_movie_nfo called: imdb_id={imdb_id}, dateadded={dateadded}, released={released}, source={source}") print(f"🔍 create_movie_nfo called: imdb_id={imdb_id}, dateadded={dateadded}, released={released}, source={source}")
print(f"🔍 NFO path: {nfo_path}")
print(f"🔍 NFO exists: {nfo_path.exists()}")
try: try:
# Try to load existing NFO file # Try to load existing NFO file
@@ -302,11 +304,14 @@ class NFOManager:
pass # Skip year if we can't extract it pass # Skip year if we can't extract it
# Add dateadded at the end - THIS IS CRITICAL FOR EMBY PLUGIN # Add dateadded at the end - THIS IS CRITICAL FOR EMBY PLUGIN
print(f"🔍 About to add dateadded: {dateadded} (type: {type(dateadded)})")
if dateadded: if dateadded:
dateadded_elem = ET.Element("dateadded") dateadded_elem = ET.Element("dateadded")
dateadded_elem.text = dateadded dateadded_elem.text = dateadded
movie.append(dateadded_elem) movie.append(dateadded_elem)
print(f"✅ Added dateadded to NFO: {dateadded}") print(f"✅ Added dateadded to NFO: {dateadded}")
else:
print(f"❌ dateadded is empty/None, not adding to NFO")
# Add lockdata at the very end # Add lockdata at the very end
if lock_metadata: if lock_metadata:
-2468
View File
File diff suppressed because it is too large Load Diff
+8
View File
@@ -166,10 +166,14 @@ class MovieProcessor:
dateadded, source, released = existing["dateadded"], existing["source"], existing.get("released") dateadded, source, released = existing["dateadded"], existing["source"], existing.get("released")
# Create NFO with existing data # Create NFO with existing data
print(f"🔍 TIER2 - config.manage_nfo: {config.manage_nfo}")
if config.manage_nfo: if config.manage_nfo:
print(f"🔍 TIER2 - Calling create_movie_nfo with dateadded: {dateadded}")
self.nfo_manager.create_movie_nfo( self.nfo_manager.create_movie_nfo(
movie_path, imdb_id, dateadded, released, source, config.lock_metadata movie_path, imdb_id, dateadded, released, source, config.lock_metadata
) )
else:
print(f"❌ TIER2 - manage_nfo is disabled, skipping NFO creation")
# Update file mtimes if enabled # Update file mtimes if enabled
if config.fix_dir_mtimes and dateadded: if config.fix_dir_mtimes and dateadded:
@@ -225,10 +229,14 @@ class MovieProcessor:
_log("INFO", f"Using release date as dateadded: {final_dateadded} (source: {final_source})") _log("INFO", f"Using release date as dateadded: {final_dateadded} (source: {final_source})")
# Create NFO regardless of date availability (preserves existing metadata) # Create NFO regardless of date availability (preserves existing metadata)
print(f"🔍 TIER3 - config.manage_nfo: {config.manage_nfo}")
if config.manage_nfo: if config.manage_nfo:
print(f"🔍 TIER3 - Calling create_movie_nfo with final_dateadded: {final_dateadded}")
self.nfo_manager.create_movie_nfo( self.nfo_manager.create_movie_nfo(
movie_path, imdb_id, final_dateadded, released, final_source, config.lock_metadata movie_path, imdb_id, final_dateadded, released, final_source, config.lock_metadata
) )
else:
print(f"❌ TIER3 - manage_nfo is disabled, skipping NFO creation")
# Skip remaining processing if no valid date found and file dates disabled # Skip remaining processing if no valid date found and file dates disabled
if final_dateadded is None: if final_dateadded is None: