Fixed stale episode logic.

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);
+        if (Configuration.DebugMode)
-        _logger.LogInformation("-------------------------------------------------");
+        {
+            _logger.LogInformation("End of scanning for series: {Series}", item);
+            _logger.LogInformation("-------------------------------------------------");
+        }
     }
 
 
@@ -122,11 +125,11 @@ public class LoggingHelper
         }
     }
 
-    public void PrintDebugSeasonNumber(int seasonNumber)
+    public void PrintDebugSeasonInfo()
     {
         if (Configuration.DebugMode)
         {
-            _logger.LogInformation("Season {SeasonNumber} debug information:", [seasonNumber]);
+            _logger.LogInformation("Season debug information:");
         }
     }
 
@@ -146,12 +149,12 @@ public class LoggingHelper
         }
     }
 
-    public void PrintDebugNoUserDataAndOutsideCutoffEpisodeInfo(IReadOnlyCollection<BaseItem> episodes, int seasonNumber)
+    public void PrintDebugNoUserDataAndOutsideCutoffEpisodeInfo(IReadOnlyCollection<BaseItem> episodes)
     {
         ArgumentNullException.ThrowIfNull(episodes);
 
         if(Configuration.DebugMode){
-            _logger.LogInformation("No user data, and creation date is outside of media cutoff, Season {SeasonNumber} is stale.", seasonNumber);
+            _logger.LogInformation("No user data, and creation date is outside of media cutoff, Season is stale.");
 
             _logger.LogInformation("-------------------------------------------------");
             _logger.LogInformation("Episode creation dates:");

Jellyfin.Plugin.MediaCleaner/Helpers/MovieHelper.cs

@@ -22,9 +22,12 @@ public class MovieHelper
 
     public bool IsMovieStale(BaseItem movie)
     {
-        _logger.LogInformation("-------------------------------------------------");
+        if (Configuration.DebugMode)
-        _logger.LogInformation("Start of scanning for movie: {Movie}", movie);
+        {
-        _logger.LogInformation("-------------------------------------------------");
+            _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;
         }
 
-
+        if (Configuration.DebugMode)
-        _logger.LogInformation("-------------------------------------------------");
+        {
-        _logger.LogInformation("End of scanning for movie: {Movie}", movie);
+            _logger.LogInformation("-------------------------------------------------");
-        _logger.LogInformation("-------------------------------------------------");
+            _logger.LogInformation("End of scanning for movie: {Movie}", movie);
+            _logger.LogInformation("-------------------------------------------------");
+        }
 
         return movieIsStale;
     }

Jellyfin.Plugin.MediaCleaner/Helpers/SeriesHelper.cs

@@ -25,17 +25,24 @@ public class SeriesHelper
     private static PluginConfiguration Configuration =>
         Plugin.Instance!.Configuration;
 
-    public bool IsSeasonUserDataStale(IReadOnlyList<BaseItem> episodes, int seasonNumber)
+    public bool IsSeasonUserDataStale(IReadOnlyList<BaseItem> episodes)
     {
         bool seasonIsStale = false;
 
+        List<BaseItem> staleEpisodes = [];
+
         var episodesWithUserData = episodes.Where(episode => episode.UserData.Where(data => data.LastPlayedDate != null).ToList().Count > 0).ToList();
 
         _loggingHelper.PrintDebugEpisodesWithUserData(episodesWithUserData);
 
         foreach (var episode in episodesWithUserData)
         {
-            var mostRecentUserData = episode.UserData.OrderByDescending(data => data.LastPlayedDate).Where(data => data.LastPlayedDate != null).First();
+            bool episodeIsStale = false;
+
+            var mostRecentUserData = episode.UserData.OrderByDescending(data => data.LastPlayedDate).First(data => data.LastPlayedDate != null);
+            var staleLastPlayedDate = mostRecentUserData.LastPlayedDate < DateTime.Now.AddDays(-Configuration.StaleMediaCutoff);
+            var staleCreationDate = episode.DateCreated < DateTime.Now.AddDays(-Configuration.StaleMediaCutoff);
+
             if(Configuration.DebugMode){
                 _logger.LogInformation("User data for episode: {Episode}", episode);
                 _logger.LogInformation("-------------------------------------------------");
@@ -45,15 +52,30 @@ public class SeriesHelper
                 }
                 _logger.LogInformation("-------------------------------------------------");
             }
-            if (mostRecentUserData.LastPlayedDate < DateTime.Now.AddDays(-Configuration.StaleMediaCutoff))
-            {
-                if(Configuration.DebugMode){
-                    _logger.LogInformation("Most recent user data has a last played date of: {LastPlayedDate}. Therefore all episodes are stale. Adding Season {SeasonNumber} to stale list.", [mostRecentUserData.LastPlayedDate, seasonNumber]);
-                    _logger.LogInformation("-------------------------------------------------");
-                }
 
-                seasonIsStale = true;
+            if (staleLastPlayedDate && staleCreationDate)
+            {
+                episodeIsStale = true;
+                if(Configuration.DebugMode){
+                    _logger.LogInformation("Most recent user data has a last played date of: {LastPlayedDate}.", [mostRecentUserData.LastPlayedDate]);
+                    _logger.LogInformation("And episode created {DateCreated}.", episode.DateCreated);
+                }
             }
+
+            if (episodeIsStale)
+            {
+                staleEpisodes.Add(episode);
+                if(Configuration.DebugMode){
+                    _logger.LogInformation("Episode is stale.");
+                }
+            }
+        }
+
+        if(staleEpisodes.Count == episodes.Count)
+        {
+            seasonIsStale = true;
+            _logger.LogInformation("Stale episodes count matches all episode count. Season is stale.");
+            _logger.LogInformation("-------------------------------------------------");
         }
 
         return seasonIsStale;

Jellyfin.Plugin.MediaCleaner/ScheduledTasks/StaleMediaTask.cs

@@ -112,8 +112,6 @@ public sealed class StaleMediaTask : IScheduledTask
 
         _loggingHelper.PrintDebugDataForSeries(item);
 
-        int seasonNumber = 1;
-
         foreach (var season in seasons)
         {
             // Gets each episode, to access user data.
@@ -125,7 +123,7 @@ public sealed class StaleMediaTask : IScheduledTask
 
             bool seasonCreatedOutsideCutoff = episodes.All(episode => episode.DateCreated < DateTime.Now.AddDays(-Configuration.StaleMediaCutoff));
 
-            _loggingHelper.PrintDebugSeasonNumber(seasonNumber);
+            _loggingHelper.PrintDebugSeasonInfo();
 
             if (seasonCreatedOutsideCutoff)
             {
@@ -133,22 +131,21 @@ public sealed class StaleMediaTask : IScheduledTask
             }
 
             bool seasonHasUserData = episodes.Any(episode => episode.UserData.Count > 0);
-            bool seasonIsStale = (seasonHasUserData && _seriesHelper.IsSeasonUserDataStale(episodes, seasonNumber)) || seasonCreatedOutsideCutoff;
+            bool seasonIsStale = (seasonHasUserData && _seriesHelper.IsSeasonUserDataStale(episodes)) || seasonCreatedOutsideCutoff;
             bool noUserDataAndOutsideCutoff = !seasonHasUserData && seasonCreatedOutsideCutoff;
 
             if (seasonIsStale)
             {
                 if (noUserDataAndOutsideCutoff)
                 {
-                    _loggingHelper.PrintDebugNoUserDataAndOutsideCutoffEpisodeInfo(episodes, seasonNumber);
+                    _loggingHelper.PrintDebugNoUserDataAndOutsideCutoffEpisodeInfo(episodes);
                 }
 
                 staleEpisodes.AddRange(episodes);
             }
-            seasonNumber++;
         }
 
-        _loggingHelper.EndOfScanningForSeries(item);
+        _loggingHelper.PrintDebugEndOfScanningForSeries(item);
 
         return staleEpisodes;
     }