Implemented ContentVersionCleanup scheduled task.

Note: adding ref to Microsoft.NET.Test.Sdk fixes AutoFixture AutoDataAttribute (and sub classes)

Author: Paul Johnson

src/Umbraco.Core/Composing/CompositionExtensions/Repositories.cs

@@ -49,6 +49,7 @@ public static Composition ComposeRepositories(this Composition composition)
             composition.RegisterUnique<IContentTypeCommonRepository, ContentTypeCommonRepository>();
             composition.RegisterUnique<IInstallationRepository, InstallationRepository>();
             composition.RegisterUnique<IUpgradeCheckRepository, UpgradeCheckRepository>();
+            composition.RegisterUnique<IDocumentVersionRepository, DocumentVersionRepository>();
 
             return composition;
         }

src/Umbraco.Core/Composing/CompositionExtensions/Services.cs

@@ -31,7 +31,12 @@ public static Composition ComposeServices(this Composition composition)
             composition.RegisterUnique<IDomainService, DomainService>();
             composition.RegisterUnique<IAuditService, AuditService>();
             composition.RegisterUnique<ITagService, TagService>();
-            composition.RegisterUnique<IContentService, ContentService>();
+
+            composition.RegisterUnique<ContentService>();
+            composition.RegisterUnique<IContentService>(factory => factory.GetInstance<ContentService>());
+            composition.RegisterUnique<IContentVersionCleanupService>(factory => factory.GetInstance<ContentService>());
+            composition.RegisterUnique<IContentVersionCleanupPolicy, DefaultContentVersionCleanupPolicy>();
+
             composition.RegisterUnique<IUserService, UserService>();
             composition.RegisterUnique<IMemberService, MemberService>();
             composition.RegisterUnique<IMediaService, MediaService>();

src/Umbraco.Core/Configuration/UmbracoSettings/ContentElement.cs

@@ -22,8 +22,8 @@ internal class ContentElement : UmbracoConfigurationElement, IContentSection
         [ConfigurationProperty("notifications", IsRequired = true)]
         internal NotificationsElement Notifications => (NotificationsElement) base["notifications"];
 
-        [ConfigurationProperty("contentVersionCleanupPolicy", IsRequired = false)]
-        internal ContentVersionCleanupPolicyElement ContentVersionCleanupPolicy => (ContentVersionCleanupPolicyElement) this["contentVersionCleanupPolicy"];
+        [ConfigurationProperty("contentVersionCleanupPolicyGlobalSettings", IsRequired = false)]
+        internal ContentVersionCleanupPolicyGlobalSettingsElement ContentVersionCleanupPolicyGlobalSettingsElement => (ContentVersionCleanupPolicyGlobalSettingsElement) this["contentVersionCleanupPolicyGlobalSettings"];
 
         [ConfigurationProperty("PreviewBadge")]
         internal InnerTextConfigurationElement<string> PreviewBadge => GetOptionalTextElement("PreviewBadge", DefaultPreviewBadge);
@@ -64,7 +64,7 @@ internal class ContentElement : UmbracoConfigurationElement, IContentSection
 
         IEnumerable<string> IContentSection.AllowedUploadFiles => AllowedUploadFiles;
 
-        IContentVersionCleanupPolicySettings IContentSection.ContentVersionCleanupPolicySettings => ContentVersionCleanupPolicy;
+        IContentVersionCleanupPolicyGlobalSettings IContentSection.ContentVersionCleanupPolicyGlobalSettings => ContentVersionCleanupPolicyGlobalSettingsElement;
 
         bool IContentSection.ShowDeprecatedPropertyEditors => ShowDeprecatedPropertyEditors;
 

src/Umbraco.Core/Configuration/UmbracoSettings/ContentVersionCleanupPolicyElement.cs renamed to src/Umbraco.Core/Configuration/UmbracoSettings/ContentVersionCleanupPolicyGlobalSettingsElement.cs

@@ -2,7 +2,7 @@
 
 namespace Umbraco.Core.Configuration.UmbracoSettings
 {
-    internal class ContentVersionCleanupPolicyElement : UmbracoConfigurationElement, IContentVersionCleanupPolicySettings
+    internal class ContentVersionCleanupPolicyGlobalSettingsElement : UmbracoConfigurationElement, IContentVersionCleanupPolicyGlobalSettings
     {
         [ConfigurationProperty("enable", DefaultValue = false)]
         public bool EnableCleanup => (bool)this["enable"];

src/Umbraco.Core/Configuration/UmbracoSettings/IContentSection.cs

@@ -25,7 +25,7 @@ public interface IContentSection : IUmbracoConfigurationSection
 
         IEnumerable<string> AllowedUploadFiles { get; }
 
-        IContentVersionCleanupPolicySettings ContentVersionCleanupPolicySettings { get; }
+        IContentVersionCleanupPolicyGlobalSettings ContentVersionCleanupPolicyGlobalSettings { get; }
 
         /// <summary>
         /// Gets a value indicating whether to show deprecated property editors in

src/Umbraco.Core/Configuration/UmbracoSettings/IContentVersionCleanupPolicySettings.cs renamed to src/Umbraco.Core/Configuration/UmbracoSettings/IContentVersionCleanupPolicyGlobalSettings.cs

@@ -1,6 +1,6 @@
 namespace Umbraco.Core.Configuration.UmbracoSettings
 {
-    public interface IContentVersionCleanupPolicySettings
+    public interface IContentVersionCleanupPolicyGlobalSettings
     {
         bool EnableCleanup { get; }
         int KeepAllVersionsNewerThanDays { get; }

src/Umbraco.Core/Models/ContentVersionCleanupPolicySettings.cs

@@ -0,0 +1,13 @@
+using System;
+
+namespace Umbraco.Core.Models
+{
+    public class ContentVersionCleanupPolicySettings
+    {
+        public int ContentTypeId { get; set; }
+        public int? KeepAllVersionsNewerThanDays { get; set; }
+        public int? KeepLatestVersionPerDayForDays { get; set; }
+        public bool PreventCleanup { get; set; }
+        public DateTime Updated { get; set; }
+    }
+}

src/Umbraco.Core/Models/HistoricContentVersionMeta.cs

@@ -0,0 +1,24 @@
+using System;
+
+namespace Umbraco.Core.Models
+{
+    public class HistoricContentVersionMeta
+    {
+        public int ContentId { get; }
+        public int ContentTypeId { get; }
+        public int VersionId { get; }
+        public DateTime VersionDate { get; }
+
+        public HistoricContentVersionMeta() { }
+
+        public HistoricContentVersionMeta(int contentId, int contentTypeId, int versionId, DateTime versionDate)
+        {
+            ContentId = contentId;
+            ContentTypeId = contentTypeId;
+            VersionId = versionId;
+            VersionDate = versionDate;
+        }
+
+        public override string ToString() => $"HistoricContentVersionMeta(versionId: {VersionId}, versionDate: {VersionDate:s}";
+    }
+}

src/Umbraco.Core/Persistence/Dtos/ContentVersionCleanupPolicyDto.cs

@@ -6,14 +6,13 @@
 namespace Umbraco.Core.Persistence.Dtos
 {
     [TableName(TableName)]
-    [PrimaryKey("contentTypeId")]
+    [PrimaryKey("contentTypeId", AutoIncrement = false)]
     [ExplicitColumns]
     internal class ContentVersionCleanupPolicyDto
     {
         public const string TableName = Constants.DatabaseSchema.Tables.ContentVersionCleanupPolicy;
 
         [Column("contentTypeId")]
-        [PrimaryKeyColumn(AutoIncrement = false)]
         [ForeignKey(typeof(ContentTypeDto), Column = "nodeId", OnDelete = Rule.Cascade)]
         public int ContentTypeId { get; set; }
 

src/Umbraco.Core/Persistence/Repositories/IDocumentVersionRepository.cs

@@ -0,0 +1,23 @@
+using System.Collections.Generic;
+using Umbraco.Core.Models;
+
+namespace Umbraco.Core.Persistence.Repositories
+{
+    public interface IDocumentVersionRepository : IRepository
+    {
+        /// <summary>
+        /// Gets a list of all historic content versions.
+        /// </summary>
+        public IReadOnlyCollection<HistoricContentVersionMeta> GetDocumentVersionsEligibleForCleanup();
+
+        /// <summary>
+        /// Gets cleanup policy override settings per content type.
+        /// </summary>
+        public IReadOnlyCollection<ContentVersionCleanupPolicySettings> GetCleanupPolicies();
+
+        /// <summary>
+        /// Deletes multiple content versions by ID.
+        /// </summary>
+        void DeleteVersions(IEnumerable<int> versionIds);
+    }
+}