From 30107010b11381b24fe24096635bf5e6f2153981 Mon Sep 17 00:00:00 2001 From: Thomas Gander Date: Sun, 18 Jan 2026 19:23:59 -0700 Subject: [PATCH] Simplified some linq queries and fixed some debug logging getting through --- .../Helpers/LoggingHelper.cs | 9 +++++--- .../Helpers/MovieHelper.cs | 21 ++++++++++++------- .../Helpers/SeriesHelper.cs | 2 +- .../ScheduledTasks/StaleMediaTask.cs | 2 +- 4 files changed, 21 insertions(+), 13 deletions(-) diff --git a/Jellyfin.Plugin.MediaCleaner/Helpers/LoggingHelper.cs b/Jellyfin.Plugin.MediaCleaner/Helpers/LoggingHelper.cs index 6fe517f..06be3e7 100644 --- a/Jellyfin.Plugin.MediaCleaner/Helpers/LoggingHelper.cs +++ b/Jellyfin.Plugin.MediaCleaner/Helpers/LoggingHelper.cs @@ -39,10 +39,13 @@ public class LoggingHelper _logger.LogInformation("-------------------------------------------------"); } - public void EndOfScanningForSeries(BaseItem item) + public void PrintDebugEndOfScanningForSeries(BaseItem item) { - _logger.LogInformation("End of scanning for series: {Series}", item); - _logger.LogInformation("-------------------------------------------------"); + if (Configuration.DebugMode) + { + _logger.LogInformation("End of scanning for series: {Series}", item); + _logger.LogInformation("-------------------------------------------------"); + } } diff --git a/Jellyfin.Plugin.MediaCleaner/Helpers/MovieHelper.cs b/Jellyfin.Plugin.MediaCleaner/Helpers/MovieHelper.cs index 80fa38b..3793ad1 100644 --- a/Jellyfin.Plugin.MediaCleaner/Helpers/MovieHelper.cs +++ b/Jellyfin.Plugin.MediaCleaner/Helpers/MovieHelper.cs @@ -22,9 +22,12 @@ public class MovieHelper public bool IsMovieStale(BaseItem movie) { - _logger.LogInformation("-------------------------------------------------"); - _logger.LogInformation("Start of scanning for movie: {Movie}", movie); - _logger.LogInformation("-------------------------------------------------"); + if (Configuration.DebugMode) + { + _logger.LogInformation("-------------------------------------------------"); + _logger.LogInformation("Start of scanning for movie: {Movie}", movie); + _logger.LogInformation("-------------------------------------------------"); + } bool movieIsStale = false; @@ -33,7 +36,7 @@ public class MovieHelper if (hasUserData) { - var mostRecentUserData = movie.UserData.OrderByDescending(data => data.LastPlayedDate).Where(data => data.LastPlayedDate != null).First(); + var mostRecentUserData = movie.UserData.OrderByDescending(data => data.LastPlayedDate).First(data => data.LastPlayedDate != null); if (Configuration.DebugMode){ _logger.LogInformation("Most recent user data: {Movie}", movie); @@ -63,10 +66,12 @@ public class MovieHelper movieIsStale = true; } - - _logger.LogInformation("-------------------------------------------------"); - _logger.LogInformation("End of scanning for movie: {Movie}", movie); - _logger.LogInformation("-------------------------------------------------"); + if (Configuration.DebugMode) + { + _logger.LogInformation("-------------------------------------------------"); + _logger.LogInformation("End of scanning for movie: {Movie}", movie); + _logger.LogInformation("-------------------------------------------------"); + } return movieIsStale; } diff --git a/Jellyfin.Plugin.MediaCleaner/Helpers/SeriesHelper.cs b/Jellyfin.Plugin.MediaCleaner/Helpers/SeriesHelper.cs index b5c3f84..5d83dbc 100644 --- a/Jellyfin.Plugin.MediaCleaner/Helpers/SeriesHelper.cs +++ b/Jellyfin.Plugin.MediaCleaner/Helpers/SeriesHelper.cs @@ -35,7 +35,7 @@ public class SeriesHelper foreach (var episode in episodesWithUserData) { - var mostRecentUserData = episode.UserData.OrderByDescending(data => data.LastPlayedDate).Where(data => data.LastPlayedDate != null).First(); + var mostRecentUserData = episode.UserData.OrderByDescending(data => data.LastPlayedDate).First(data => data.LastPlayedDate != null); if(Configuration.DebugMode){ _logger.LogInformation("User data for episode: {Episode}", episode); _logger.LogInformation("-------------------------------------------------"); diff --git a/Jellyfin.Plugin.MediaCleaner/ScheduledTasks/StaleMediaTask.cs b/Jellyfin.Plugin.MediaCleaner/ScheduledTasks/StaleMediaTask.cs index b221fb0..99c2269 100644 --- a/Jellyfin.Plugin.MediaCleaner/ScheduledTasks/StaleMediaTask.cs +++ b/Jellyfin.Plugin.MediaCleaner/ScheduledTasks/StaleMediaTask.cs @@ -148,7 +148,7 @@ public sealed class StaleMediaTask : IScheduledTask seasonNumber++; } - _loggingHelper.EndOfScanningForSeries(item); + _loggingHelper.PrintDebugEndOfScanningForSeries(item); return staleEpisodes; }