fix order
This commit is contained in:
@@ -167,7 +167,7 @@ class RadarrDbClient:
|
|||||||
|
|
||||||
def get_earliest_import_date(self, movie_id: int) -> Tuple[Optional[str], str]:
|
def get_earliest_import_date(self, movie_id: int) -> Tuple[Optional[str], str]:
|
||||||
"""
|
"""
|
||||||
Get earliest import date from History table
|
Get earliest import date from History table, accounting for upgrade scenarios
|
||||||
|
|
||||||
Args:
|
Args:
|
||||||
movie_id: Radarr movie ID
|
movie_id: Radarr movie ID
|
||||||
@@ -175,6 +175,11 @@ class RadarrDbClient:
|
|||||||
Returns:
|
Returns:
|
||||||
(date_iso, source_description)
|
(date_iso, source_description)
|
||||||
"""
|
"""
|
||||||
|
# If first event is rename, all subsequent imports are upgrades - skip them
|
||||||
|
if self.is_first_event_rename_based(movie_id):
|
||||||
|
_log("INFO", f"Movie {movie_id} has rename-first history - all imports are upgrades, skipping")
|
||||||
|
return None, "radarr:db.upgrade_imports_skipped"
|
||||||
|
|
||||||
# Query for earliest import event - PostgreSQL uses INTEGER EventType (3 = import)
|
# Query for earliest import event - PostgreSQL uses INTEGER EventType (3 = import)
|
||||||
import_query = """
|
import_query = """
|
||||||
SELECT
|
SELECT
|
||||||
@@ -346,14 +351,14 @@ class RadarrDbClient:
|
|||||||
Returns:
|
Returns:
|
||||||
(date_iso, source_description)
|
(date_iso, source_description)
|
||||||
"""
|
"""
|
||||||
# Try history first - always check for true import dates
|
# Try history first - this handles upgrade detection internally
|
||||||
date_iso, source = self.get_earliest_import_date(movie_id)
|
date_iso, source = self.get_earliest_import_date(movie_id)
|
||||||
if date_iso:
|
if date_iso:
|
||||||
return date_iso, source
|
return date_iso, source
|
||||||
|
|
||||||
# If no true import found, check if first event is rename-based (upgrade scenario)
|
# Check if we skipped upgrades and should prefer release dates
|
||||||
if self.is_first_event_rename_based(movie_id):
|
if source == "radarr:db.upgrade_imports_skipped":
|
||||||
_log("INFO", f"Movie {movie_id} has rename-first history with no true import - signaling to prefer release dates")
|
_log("INFO", f"Movie {movie_id} upgrade scenario detected - signaling to prefer release dates")
|
||||||
return None, "radarr:db.prefer_release_dates"
|
return None, "radarr:db.prefer_release_dates"
|
||||||
|
|
||||||
# Fallback to file date if requested
|
# Fallback to file date if requested
|
||||||
|
|||||||
Reference in New Issue
Block a user