Update to Deploy 15.0.0-rc1 and fix breaking changes

Author: ronaldbarendse

Directory.Packages.props

@@ -5,14 +5,14 @@
   </PropertyGroup>
   <!-- Global packages (private, build-time packages for all projects) -->
   <ItemGroup>
-    <GlobalPackageReference Include="Nerdbank.GitVersioning" Version="3.6.133" />
+    <GlobalPackageReference Include="Nerdbank.GitVersioning" Version="3.6.143" />
     <GlobalPackageReference Include="StyleCop.Analyzers" Version="1.2.0-beta.556" />
     <GlobalPackageReference Include="Umbraco.Code" Version="2.2.0" />
     <GlobalPackageReference Include="Umbraco.GitVersioning.Extensions" Version="0.2.0" />
   </ItemGroup>
   <!-- Umbraco packages -->
   <ItemGroup>
-    <PackageVersion Include="Umbraco.Cms.Web.Common" Version="[15.0.0-rc1, 15)" />
-    <PackageVersion Include="Umbraco.Deploy.Infrastructure" Version="[15.0.0--rc1.preview.14.gf9ce468, 15)" />
+    <PackageVersion Include="Umbraco.Cms.Web.Common" Version="[15.0.0-rc1, 16)" />
+    <PackageVersion Include="Umbraco.Deploy.Infrastructure" Version="[15.0.0-rc1, 16)" />
   </ItemGroup>
 </Project>

src/Umbraco.Deploy.Contrib/Migrators/DocTypeGridEditorPropertyTypeMigrator.cs

@@ -4,6 +4,8 @@
 using System.Text.Json;
 using System.Text.Json.Nodes;
 using System.Text.Json.Serialization;
+using System.Threading;
+using System.Threading.Tasks;
 using Microsoft.Extensions.Logging;
 using Newtonsoft.Json;
 using Umbraco.Cms.Core;
@@ -40,14 +42,14 @@ public DocTypeGridEditorPropertyTypeMigrator(ILogger<GridPropertyTypeMigrator> l
         => _jsonSerializer = jsonSerializer;
 
     /// <inheritdoc />
-    protected override BlockItemData? MigrateGridControl(GridValue.GridControl gridControl, BlockGridConfiguration configuration, IContextCache contextCache)
+    protected override async Task<BlockItemData?> MigrateGridControlAsync(GridValue.GridControl gridControl, BlockGridConfiguration configuration, IContextCache contextCache, CancellationToken cancellationToken = default)
     {
         if (TryDeserialize(gridControl.Value, out DocTypeGridEditorValue? value))
         {
-            return MigrateGridControl(value, configuration, contextCache);
+            return await MigrateGridControlAsync(value, configuration, contextCache).ConfigureAwait(false);
         }
 
-        return base.MigrateGridControl(gridControl, configuration, contextCache);
+        return await base.MigrateGridControlAsync(gridControl, configuration, contextCache, cancellationToken).ConfigureAwait(false);
     }
 
     /// <summary>
@@ -57,11 +59,11 @@ public DocTypeGridEditorPropertyTypeMigrator(ILogger<GridPropertyTypeMigrator> l
     /// <param name="configuration">The configuration.</param>
     /// <param name="contextCache">The context cache.</param>
     /// <returns>
-    /// The block item data, or <c>null</c> if migration should be skipped.
+    /// A task that represents the asynchronous operation. The task result contains the block item data, or <c>null</c> if migration should be skipped.
     /// </returns>
-    protected virtual BlockItemData? MigrateGridControl(DocTypeGridEditorValue value, BlockGridConfiguration configuration, IContextCache contextCache)
+    protected virtual async Task<BlockItemData?> MigrateGridControlAsync(DocTypeGridEditorValue value, BlockGridConfiguration configuration, IContextCache contextCache)
     {
-        IContentType contentType = GetContentType(value.ContentTypeAlias, configuration, contextCache)
+        IContentType contentType = await GetContentTypeAsync(value.ContentTypeAlias, configuration, contextCache).ConfigureAwait(false)
             ?? throw new InvalidOperationException($"Migrating legacy grid failed, because content type with alias '{value.ContentTypeAlias}' could not be found (in the Block Grid configuration).");
 
         return new BlockItemData()

src/Umbraco.Deploy.Contrib/Migrators/Legacy/Content/CheckBoxListPropertyValueArtifactMigrator.cs

@@ -1,5 +1,7 @@
 using System;
 using System.Collections.Generic;
+using System.Threading;
+using System.Threading.Tasks;
 using Umbraco.Cms.Core.Deploy;
 using Umbraco.Cms.Core.Models;
 using Umbraco.Cms.Core.Serialization;
@@ -8,7 +10,7 @@
 namespace Umbraco.Deploy.Contrib.Migrators.Legacy;
 
 /// <summary>
-/// Migrates the property value containing prevalues (seperated by <see cref="Delimiter" />) from Umbraco 7 to a single value or JSON array.
+/// Migrates the property value containing pre-values (separated by <see cref="Delimiter" />) from Umbraco 7 to a single value or JSON array.
 /// </summary>
 public abstract class PrevaluePropertyTypeMigratorBase : PropertyTypeMigratorBase
 {
@@ -18,10 +20,10 @@ public abstract class PrevaluePropertyTypeMigratorBase : PropertyTypeMigratorBas
     private readonly IJsonSerializer _jsonSerializer;
 
     /// <summary>
-    /// Gets a value indicating whether the property type stores multiple prevalues as a JSON array or single value.
+    /// Gets a value indicating whether the property type stores multiple pre-values as a JSON array or single value.
     /// </summary>
     /// <value>
-    ///   <c>true</c> if multiple prevalues are stored as a JSON array; otherwise, <c>false</c>.
+    ///   <c>true</c> if multiple pre-values are stored as a JSON array; otherwise, <c>false</c>.
     /// </value>
     protected abstract bool Multiple { get; }
 
@@ -45,7 +47,10 @@ protected PrevaluePropertyTypeMigratorBase(string fromEditorAlias, string toEdit
         => _jsonSerializer = jsonSerializer;
 
     /// <inheritdoc />
-    public override object? Migrate(IPropertyType propertyType, object? value, IDictionary<string, string> propertyEditorAliases, IContextCache contextCache)
+    public override Task<object?> MigrateAsync(IPropertyType propertyType, object? value, IDictionary<string, string> propertyEditorAliases, IContextCache contextCache, CancellationToken cancellationToken = default)
+        => Task.FromResult(Migrate(propertyType));
+
+    private object? Migrate(object? value)
     {
         if (value is not string stringValue)
         {

src/Umbraco.Deploy.Contrib/Migrators/Legacy/Content/DropDownListFlexiblePropertyValueArtifactMigrator.cs

@@ -1,8 +1,5 @@
-using System;
 using System.Collections.Generic;
-using Microsoft.Extensions.DependencyInjection;
 using Umbraco.Cms.Core;
-using Umbraco.Cms.Core.DependencyInjection;
 using Umbraco.Cms.Core.PropertyEditors;
 using Umbraco.Cms.Core.Semver;
 using Umbraco.Cms.Core.Serialization;
@@ -23,21 +20,6 @@ public class MultiNodeTreePicker2DataTypeArtifactMigrator : LegacyReplaceDataTyp
     private readonly IMediaTypeService _mediaTypeService;
     private readonly IMemberTypeService _memberTypeService;
 
-    /// <summary>
-    /// Initializes a new instance of the <see cref="MultiNodeTreePicker2DataTypeArtifactMigrator" /> class.
-    /// </summary>
-    /// <param name="propertyEditors">The property editors.</param>
-    /// <param name="configurationEditorJsonSerializer">The configuration editor JSON serializer.</param>
-    [Obsolete("Please use the constructor taking all parameters. This constructor will be removed in a future version.")]
-    public MultiNodeTreePicker2DataTypeArtifactMigrator(PropertyEditorCollection propertyEditors, IConfigurationEditorJsonSerializer configurationEditorJsonSerializer)
-        : this(
-              propertyEditors,
-              configurationEditorJsonSerializer,
-              StaticServiceProvider.Instance.GetRequiredService<IContentTypeService>(),
-              StaticServiceProvider.Instance.GetRequiredService<IMediaTypeService>(),
-              StaticServiceProvider.Instance.GetRequiredService<IMemberTypeService>())
-    { }
-
     /// <summary>
     /// Initializes a new instance of the <see cref="MultiNodeTreePicker2DataTypeArtifactMigrator" /> class.
     /// </summary>