Cleanup Tasks and Validators

1 files changed, 50 insertions, 51 deletions

diff --git a/Emby.Server.Implementations/ScheduledTasks/Tasks/CleanActivityLogTask.cs b/Emby.Server.Implementations/ScheduledTasks/Tasks/CleanActivityLogTask.cs
index fe1832165..1621bbaa1 100644
--- a/Emby.Server.Implementations/ScheduledTasks/Tasks/CleanActivityLogTask.cs
+++ b/Emby.Server.Implementations/ScheduledTasks/Tasks/CleanActivityLogTask.cs
@@ -7,71 +7,70 @@ using MediaBrowser.Model.Activity;
 using MediaBrowser.Model.Globalization;
 using MediaBrowser.Model.Tasks;
 
-namespace Emby.Server.Implementations.ScheduledTasks.Tasks
+namespace Emby.Server.Implementations.ScheduledTasks.Tasks;
+
+/// <summary>
+/// Deletes old activity log entries.
+/// </summary>
+public class CleanActivityLogTask : IScheduledTask, IConfigurableScheduledTask
 {
+    private readonly ILocalizationManager _localization;
+    private readonly IActivityManager _activityManager;
+    private readonly IServerConfigurationManager _serverConfigurationManager;
+
     /// <summary>
-    /// Deletes old activity log entries.
+    /// Initializes a new instance of the <see cref="CleanActivityLogTask"/> class.
     /// </summary>
-    public class CleanActivityLogTask : IScheduledTask, IConfigurableScheduledTask
+    /// <param name="localization">Instance of the <see cref="ILocalizationManager"/> interface.</param>
+    /// <param name="activityManager">Instance of the <see cref="IActivityManager"/> interface.</param>
+    /// <param name="serverConfigurationManager">Instance of the <see cref="IServerConfigurationManager"/> interface.</param>
+    public CleanActivityLogTask(
+        ILocalizationManager localization,
+        IActivityManager activityManager,
+        IServerConfigurationManager serverConfigurationManager)
     {
-        private readonly ILocalizationManager _localization;
-        private readonly IActivityManager _activityManager;
-        private readonly IServerConfigurationManager _serverConfigurationManager;
-
-        /// <summary>
-        /// Initializes a new instance of the <see cref="CleanActivityLogTask"/> class.
-        /// </summary>
-        /// <param name="localization">Instance of the <see cref="ILocalizationManager"/> interface.</param>
-        /// <param name="activityManager">Instance of the <see cref="IActivityManager"/> interface.</param>
-        /// <param name="serverConfigurationManager">Instance of the <see cref="IServerConfigurationManager"/> interface.</param>
-        public CleanActivityLogTask(
-            ILocalizationManager localization,
-            IActivityManager activityManager,
-            IServerConfigurationManager serverConfigurationManager)
-        {
-            _localization = localization;
-            _activityManager = activityManager;
-            _serverConfigurationManager = serverConfigurationManager;
-        }
+        _localization = localization;
+        _activityManager = activityManager;
+        _serverConfigurationManager = serverConfigurationManager;
+    }
 
-        /// <inheritdoc />
-        public string Name => _localization.GetLocalizedString("TaskCleanActivityLog");
+    /// <inheritdoc />
+    public string Name => _localization.GetLocalizedString("TaskCleanActivityLog");
 
-        /// <inheritdoc />
-        public string Key => "CleanActivityLog";
+    /// <inheritdoc />
+    public string Key => "CleanActivityLog";
 
-        /// <inheritdoc />
-        public string Description => _localization.GetLocalizedString("TaskCleanActivityLogDescription");
+    /// <inheritdoc />
+    public string Description => _localization.GetLocalizedString("TaskCleanActivityLogDescription");
 
-        /// <inheritdoc />
-        public string Category => _localization.GetLocalizedString("TasksMaintenanceCategory");
+    /// <inheritdoc />
+    public string Category => _localization.GetLocalizedString("TasksMaintenanceCategory");
 
-        /// <inheritdoc />
-        public bool IsHidden => false;
+    /// <inheritdoc />
+    public bool IsHidden => false;
 
-        /// <inheritdoc />
-        public bool IsEnabled => true;
+    /// <inheritdoc />
+    public bool IsEnabled => true;
 
-        /// <inheritdoc />
-        public bool IsLogged => true;
+    /// <inheritdoc />
+    public bool IsLogged => true;
 
-        /// <inheritdoc />
-        public Task ExecuteAsync(IProgress<double> progress, CancellationToken cancellationToken)
+    /// <inheritdoc />
+    public Task ExecuteAsync(IProgress<double> progress, CancellationToken cancellationToken)
+    {
+        var retentionDays = _serverConfigurationManager.Configuration.ActivityLogRetentionDays;
+        if (!retentionDays.HasValue || retentionDays < 0)
         {
-            var retentionDays = _serverConfigurationManager.Configuration.ActivityLogRetentionDays;
-            if (!retentionDays.HasValue || retentionDays < 0)
-            {
-                throw new InvalidOperationException($"Activity Log Retention days must be at least 0. Currently: {retentionDays}");
-            }
-
-            var startDate = DateTime.UtcNow.AddDays(-retentionDays.Value);
-            return _activityManager.CleanAsync(startDate);
+            throw new InvalidOperationException($"Activity Log Retention days must be at least 0. Currently: {retentionDays}");
         }
 
-        /// <inheritdoc />
-        public IEnumerable<TaskTriggerInfo> GetDefaultTriggers()
-        {
-            return [];
-        }
+        var startDate = DateTime.UtcNow.AddDays(-retentionDays.Value);
+        return _activityManager.CleanAsync(startDate);
+    }
+
+    /// <inheritdoc />
+    public IEnumerable<TaskTriggerInfo> GetDefaultTriggers()
+    {
+        return [];
     }
 }