Cleanup Tasks and Validators

Emby.Server.Implementations/ScheduledTasks/Tasks/ChapterImagesTask.cs

@@ -17,155 +17,151 @@ using MediaBrowser.Model.IO;
 using MediaBrowser.Model.Tasks;
 using Microsoft.Extensions.Logging;
 
-namespace Emby.Server.Implementations.ScheduledTasks.Tasks
+namespace Emby.Server.Implementations.ScheduledTasks.Tasks;
+
+/// <summary>
+/// Class ChapterImagesTask.
+/// </summary>
+public class ChapterImagesTask : IScheduledTask
 {
+    private readonly ILogger<ChapterImagesTask> _logger;
+    private readonly ILibraryManager _libraryManager;
+    private readonly IApplicationPaths _appPaths;
+    private readonly IChapterManager _chapterManager;
+    private readonly IFileSystem _fileSystem;
+    private readonly ILocalizationManager _localization;
+
     /// <summary>
-    /// Class ChapterImagesTask.
+    /// Initializes a new instance of the <see cref="ChapterImagesTask" /> class.
     /// </summary>
-    public class ChapterImagesTask : IScheduledTask
+    /// <param name="logger">Instance of the <see cref="ILogger"/> interface.</param>
+    /// <param name="libraryManager">Instance of the <see cref="ILibraryManager"/> interface.</param>
+    /// <param name="appPaths">Instance of the <see cref="IApplicationPaths"/> interface.</param>
+    /// <param name="chapterManager">Instance of the <see cref="IChapterManager"/> interface.</param>
+    /// <param name="fileSystem">Instance of the <see cref="IFileSystem"/> interface.</param>
+    /// <param name="localization">Instance of the <see cref="ILocalizationManager"/> interface.</param>
+    public ChapterImagesTask(
+        ILogger<ChapterImagesTask> logger,
+        ILibraryManager libraryManager,
+        IApplicationPaths appPaths,
+        IChapterManager chapterManager,
+        IFileSystem fileSystem,
+        ILocalizationManager localization)
     {
-        private readonly ILogger<ChapterImagesTask> _logger;
-        private readonly ILibraryManager _libraryManager;
-        private readonly IApplicationPaths _appPaths;
-        private readonly IChapterManager _chapterManager;
-        private readonly IFileSystem _fileSystem;
-        private readonly ILocalizationManager _localization;
+        _logger = logger;
+        _libraryManager = libraryManager;
+        _appPaths = appPaths;
+        _chapterManager = chapterManager;
+        _fileSystem = fileSystem;
+        _localization = localization;
+    }
 
-        /// <summary>
-        /// Initializes a new instance of the <see cref="ChapterImagesTask" /> class.
-        /// </summary>
-        /// <param name="logger">Instance of the <see cref="ILogger"/> interface.</param>
-        /// <param name="libraryManager">Instance of the <see cref="ILibraryManager"/> interface.</param>
-        /// <param name="appPaths">Instance of the <see cref="IApplicationPaths"/> interface.</param>
-        /// <param name="chapterManager">Instance of the <see cref="IChapterManager"/> interface.</param>
-        /// <param name="fileSystem">Instance of the <see cref="IFileSystem"/> interface.</param>
-        /// <param name="localization">Instance of the <see cref="ILocalizationManager"/> interface.</param>
-        public ChapterImagesTask(
-            ILogger<ChapterImagesTask> logger,
-            ILibraryManager libraryManager,
-            IApplicationPaths appPaths,
-            IChapterManager chapterManager,
-            IFileSystem fileSystem,
-            ILocalizationManager localization)
+    /// <inheritdoc />
+    public string Name => _localization.GetLocalizedString("TaskRefreshChapterImages");
+
+    /// <inheritdoc />
+    public string Description => _localization.GetLocalizedString("TaskRefreshChapterImagesDescription");
+
+    /// <inheritdoc />
+    public string Category => _localization.GetLocalizedString("TasksLibraryCategory");
+
+    /// <inheritdoc />
+    public string Key => "RefreshChapterImages";
+
+    /// <inheritdoc />
+    public IEnumerable<TaskTriggerInfo> GetDefaultTriggers()
+    {
+        yield return new TaskTriggerInfo
         {
-            _logger = logger;
-            _libraryManager = libraryManager;
-            _appPaths = appPaths;
-            _chapterManager = chapterManager;
-            _fileSystem = fileSystem;
-            _localization = localization;
-        }
+            Type = TaskTriggerInfoType.DailyTrigger,
+            TimeOfDayTicks = TimeSpan.FromHours(2).Ticks,
+            MaxRuntimeTicks = TimeSpan.FromHours(4).Ticks
+        };
+    }
 
-        /// <inheritdoc />
-        public string Name => _localization.GetLocalizedString("TaskRefreshChapterImages");
-
-        /// <inheritdoc />
-        public string Description => _localization.GetLocalizedString("TaskRefreshChapterImagesDescription");
-
-        /// <inheritdoc />
-        public string Category => _localization.GetLocalizedString("TasksLibraryCategory");
-
-        /// <inheritdoc />
-        public string Key => "RefreshChapterImages";
-
-        /// <inheritdoc />
-        public IEnumerable<TaskTriggerInfo> GetDefaultTriggers()
+    /// <inheritdoc />
+    public async Task ExecuteAsync(IProgress<double> progress, CancellationToken cancellationToken)
+    {
+        var videos = _libraryManager.GetItemList(new InternalItemsQuery
         {
-            return
-            [
-                new TaskTriggerInfo
-                {
-                    Type = TaskTriggerInfoType.DailyTrigger,
-                    TimeOfDayTicks = TimeSpan.FromHours(2).Ticks,
-                    MaxRuntimeTicks = TimeSpan.FromHours(4).Ticks
-                }
-            ];
-        }
-
-        /// <inheritdoc />
-        public async Task ExecuteAsync(IProgress<double> progress, CancellationToken cancellationToken)
-        {
-            var videos = _libraryManager.GetItemList(new InternalItemsQuery
+            MediaTypes = [MediaType.Video],
+            IsFolder = false,
+            Recursive = true,
+            DtoOptions = new DtoOptions(false)
             {
-                MediaTypes = [MediaType.Video],
-                IsFolder = false,
-                Recursive = true,
-                DtoOptions = new DtoOptions(false)
-                {
-                    EnableImages = false
-                },
-                SourceTypes = [SourceType.Library],
-                IsVirtualItem = false
-            })
-            .OfType<Video>()
-            .ToList();
+                EnableImages = false
+            },
+            SourceTypes = [SourceType.Library],
+            IsVirtualItem = false
+        })
+        .OfType<Video>()
+        .ToList();
 
-            var numComplete = 0;
+        var numComplete = 0;
 
-            var failHistoryPath = Path.Combine(_appPaths.CachePath, "chapter-failures.txt");
+        var failHistoryPath = Path.Combine(_appPaths.CachePath, "chapter-failures.txt");
 
-            List<string> previouslyFailedImages;
+        List<string> previouslyFailedImages;
 
-            if (File.Exists(failHistoryPath))
+        if (File.Exists(failHistoryPath))
+        {
+            try
             {
-                try
-                {
-                    previouslyFailedImages = (await File.ReadAllTextAsync(failHistoryPath, cancellationToken).ConfigureAwait(false))
-                        .Split('|', StringSplitOptions.RemoveEmptyEntries)
-                        .ToList();
-                }
-                catch (IOException)
-                {
-                    previouslyFailedImages = [];
-                }
+                previouslyFailedImages = (await File.ReadAllTextAsync(failHistoryPath, cancellationToken).ConfigureAwait(false))
+                    .Split('|', StringSplitOptions.RemoveEmptyEntries)
+                    .ToList();
             }
-            else
+            catch (IOException)
             {
                 previouslyFailedImages = [];
             }
+        }
+        else
+        {
+            previouslyFailedImages = [];
+        }
 
-            var directoryService = new DirectoryService(_fileSystem);
+        var directoryService = new DirectoryService(_fileSystem);
 
-            foreach (var video in videos)
+        foreach (var video in videos)
+        {
+            cancellationToken.ThrowIfCancellationRequested();
+
+            var key = video.Path + video.DateModified.Ticks;
+
+            var extract = !previouslyFailedImages.Contains(key, StringComparison.OrdinalIgnoreCase);
+
+            try
             {
-                cancellationToken.ThrowIfCancellationRequested();
+                var chapters = _chapterManager.GetChapters(video.Id);
 
-                var key = video.Path + video.DateModified.Ticks;
+                var success = await _chapterManager.RefreshChapterImages(video, directoryService, chapters, extract, true, cancellationToken).ConfigureAwait(false);
 
-                var extract = !previouslyFailedImages.Contains(key, StringComparison.OrdinalIgnoreCase);
-
-                try
+                if (!success)
                 {
-                    var chapters = _chapterManager.GetChapters(video.Id);
+                    previouslyFailedImages.Add(key);
 
-                    var success = await _chapterManager.RefreshChapterImages(video, directoryService, chapters, extract, true, cancellationToken).ConfigureAwait(false);
-
-                    if (!success)
+                    var parentPath = Path.GetDirectoryName(failHistoryPath);
+                    if (parentPath is not null)
                     {
-                        previouslyFailedImages.Add(key);
-
-                        var parentPath = Path.GetDirectoryName(failHistoryPath);
-                        if (parentPath is not null)
-                        {
-                            Directory.CreateDirectory(parentPath);
-                        }
-
-                        string text = string.Join('|', previouslyFailedImages);
-                        await File.WriteAllTextAsync(failHistoryPath, text, cancellationToken).ConfigureAwait(false);
+                        Directory.CreateDirectory(parentPath);
                     }
 
-                    numComplete++;
-                    double percent = numComplete;
-                    percent /= videos.Count;
+                    string text = string.Join('|', previouslyFailedImages);
+                    await File.WriteAllTextAsync(failHistoryPath, text, cancellationToken).ConfigureAwait(false);
+                }
 
-                    progress.Report(100 * percent);
-                }
-                catch (ObjectDisposedException ex)
-                {
-                    // TODO Investigate and properly fix.
-                    _logger.LogError(ex, "Object Disposed");
-                    break;
-                }
+                numComplete++;
+                double percent = numComplete;
+                percent /= videos.Count;
+
+                progress.Report(100 * percent);
+            }
+            catch (ObjectDisposedException ex)
+            {
+                // TODO Investigate and properly fix.
+                _logger.LogError(ex, "Object Disposed");
+                break;
             }
         }
     }