OSOE-818: Upgrade to Orchard Core 2.0

Lombiq.HelpfulExtensions.Tests.UI/Constants/GeneratedMigrationCodes.cs

@@ -1,39 +1,48 @@
-using System;
+using Shouldly;
+using System.Diagnostics.CodeAnalysis;
 
 namespace Lombiq.HelpfulExtensions.Tests.UI.Constants;
 
 internal static class GeneratedMigrationCodes
 {
-    // Environment.NewLine is used for this to work regardless of the mismatch of line ending style between the code
-    // file, the platform where it was compiled, and where it was executed.
-    public static string Page =
-        string.Join(
-            Environment.NewLine,
-            "_contentDefinitionManager.AlterTypeDefinition(\"Page\", type => type",
-            "    .DisplayedAs(\"Page\")",
-            "    .Creatable()",
-            "    .Listable()",
-            "    .Draftable()",
-            "    .Versionable()",
-            "    .Securable()",
-            "    .WithPart(\"TitlePart\", part => part",
-            "        .WithPosition(\"0\")",
-            "    )",
-            "    .WithPart(\"AutoroutePart\", part => part",
-            "        .WithPosition(\"1\")",
-            "        .WithSettings(new AutoroutePartSettings",
-            "        {",
-            "            AllowCustomPath = true,",
-            "            ShowHomepageOption = true,",
-            "            Pattern = \"{{ Model.ContentItem | display_text | slugify }}\",",
-            "        })",
-            "    )",
-            "    .WithPart(\"FlowPart\", part => part",
-            "        .WithPosition(\"2\")",
-            "    )",
-            "    .WithPart(\"Page\", part => part",
-            "        .WithPosition(\"3\")",
-            "    )",
-            ");",
-            string.Empty);
+    [SuppressMessage(
+        "Usage",
+        "MA0136:Raw String contains an implicit end of line character",
+        Justification = "It's wrapped to prevent issues related to that.")]
+    private const string Page =
+        """
+        _contentDefinitionManager.AlterTypeDefinition("Page", type => type
+            .DisplayedAs("Page")
+            .Creatable()
+            .Listable()
+            .Draftable()
+            .Versionable()
+            .Securable()
+            .WithPart("TitlePart", part => part
+                .WithPosition("0")
+            )
+            .WithPart("AutoroutePart", part => part
+                .WithPosition("1")
+                .WithSettings(new AutoroutePartSettings
+                {
+                    AllowCustomPath = true,
+                    Pattern = {{ Model.ContentItem | display_text | slugify }},
+                    ShowHomepageOption = true,
+                    AllowUpdatePath = false,
+                    AllowDisabled = false,
+                    AllowRouteContainedItems = false,
+                    ManageContainedItemRoutes = false,
+                    AllowAbsolutePath = false,
+                })
+            )
+            .WithPart("FlowPart", part => part
+                .WithPosition("2")
+            )
+            .WithPart("Page", part => part
+                .WithPosition("3")
+            )
+        );
+        """;
+
+    public static void ShouldBePage(string actual) => actual.ShouldBeByLine(Page);
 }

Lombiq.HelpfulExtensions.Tests.UI/Extensions/TestCaseUITestContextExtensions.cs

@@ -116,7 +116,8 @@ await context.RetryIfStaleOrFailAsync(async () =>
             {
                 await context.ClickReliablyOnAsync(By.ClassName("toggle-showing-generated-migration-code"));
 
-                context.Get(By.Id("generated-migration-code").OfAnyVisibility()).GetValue().ShouldBe(GeneratedMigrationCodes.Page);
+                GeneratedMigrationCodes.ShouldBePage(
+                    context.Get(By.Id("generated-migration-code").OfAnyVisibility()).GetValue());
 
                 // Making sure that the collapsible area is open.
                 context.Get(By.CssSelector("#generated-migration-code-container.collapse.show"));

Lombiq.HelpfulExtensions.Tests.UI/Lombiq.HelpfulExtensions.Tests.UI.csproj

@@ -36,7 +36,7 @@
   </ItemGroup>
 
   <ItemGroup Condition="'$(NuGetBuild)' == 'true'">
-    <PackageReference Include="Lombiq.Tests.UI" Version="10.0.1" />
+    <PackageReference Include="Lombiq.Tests.UI" Version="10.0.2-alpha.5.osoe-818" />
   </ItemGroup>
 
 </Project>

Lombiq.HelpfulExtensions/Extensions/CodeGeneration/CodeGenerationDisplayDriver.cs

@@ -1,25 +1,29 @@
 using Microsoft.Extensions.Localization;
-using Newtonsoft.Json.Linq;
 using OrchardCore.ContentManagement.Metadata.Models;
 using OrchardCore.ContentManagement.Metadata.Settings;
 using OrchardCore.ContentTypes.Editors;
+using OrchardCore.DisplayManagement.Handlers;
 using OrchardCore.DisplayManagement.Views;
 using System;
 using System.Collections.Generic;
 using System.Globalization;
 using System.Linq;
 using System.Text;
+using System.Text.Json.Nodes;
 
 namespace Lombiq.HelpfulExtensions.Extensions.CodeGeneration;
 
 public class CodeGenerationDisplayDriver : ContentTypeDefinitionDisplayDriver
 {
+    private const int IndentationDepth = 4;
+    private const string EmptyString = "\"\"";
+
     private readonly IStringLocalizer T;
 
     public CodeGenerationDisplayDriver(IStringLocalizer<CodeGenerationDisplayDriver> stringLocalizer) =>
         T = stringLocalizer;
 
-    public override IDisplayResult Edit(ContentTypeDefinition model) =>
+    public override IDisplayResult Edit(ContentTypeDefinition model, BuildEditorContext context) =>
         Initialize<ContentTypeMigrationsViewModel>(
             "ContentTypeMigrations_Edit",
             viewModel => viewModel.MigrationCodeLazy = new Lazy<string>(() =>
@@ -37,7 +41,7 @@ public override IDisplayResult Edit(ContentTypeDefinition model) =>
                 codeBuilder.AppendLine(CultureInfo.InvariantCulture, $"    .DisplayedAs(\"{model.DisplayName}\")");
 
                 GenerateCodeForSettings(codeBuilder, model.GetSettings<ContentTypeSettings>());
-                AddSettingsWithout<ContentTypeSettings>(codeBuilder, model.Settings, 4);
+                AddSettingsWithout<ContentTypeSettings>(codeBuilder, model.Settings);
                 GenerateCodeForParts(codeBuilder, model.Parts);
                 codeBuilder.AppendLine(");");
 
@@ -63,7 +67,7 @@ private void GenerateCodeForParts(StringBuilder codeBuilder, IEnumerable<Content
             AddWithLine(codeBuilder, nameof(partSettings.DisplayMode), partSettings.DisplayMode);
             AddWithLine(codeBuilder, nameof(partSettings.Editor), partSettings.Editor);
 
-            AddSettingsWithout<ContentTypePartSettings>(codeBuilder, part.Settings, 8);
+            AddSettingsWithout<ContentTypePartSettings>(codeBuilder, part.Settings, 2 * IndentationDepth);
 
             // Checking if anything was added to the part's settings.
             if (codeBuilder.Length == partStartingLength)
@@ -102,7 +106,7 @@ private void GenerateCodeForPartsWithFields(
             AddWithLine(codeBuilder, nameof(partSettings.Description), partSettings.Description);
             AddWithLine(codeBuilder, nameof(partSettings.DefaultPosition), partSettings.DefaultPosition);
 
-            AddSettingsWithout<ContentPartSettings>(codeBuilder, part.Settings, 4);
+            AddSettingsWithout<ContentPartSettings>(codeBuilder, part.Settings);
 
             foreach (var field in part.Fields)
             {
@@ -116,7 +120,7 @@ private void GenerateCodeForPartsWithFields(
                 AddWithLine(codeBuilder, nameof(fieldSettings.DisplayMode), fieldSettings.DisplayMode);
                 AddWithLine(codeBuilder, nameof(fieldSettings.Position), fieldSettings.Position);
 
-                AddSettingsWithout<ContentPartFieldSettings>(codeBuilder, field.Settings, 8);
+                AddSettingsWithout<ContentPartFieldSettings>(codeBuilder, field.Settings, 2 * IndentationDepth);
 
                 codeBuilder.AppendLine("    )");
             }
@@ -125,32 +129,20 @@ private void GenerateCodeForPartsWithFields(
         }
     }
 
-    private string ConvertJToken(JToken jToken, int indentationDepth)
-    {
-        switch (jToken)
+    private string ConvertNode(JsonNode node, int indentationDepth) =>
+        node switch
         {
-            case JValue jValue:
-                var value = jValue.Value;
-                return value switch
-                {
-                    bool boolValue => boolValue ? "true" : "false",
-                    string => $"\"{value}\"",
-                    _ => value?.ToString()?.Replace(',', '.'), // Replace decimal commas.
-                };
-            case JArray jArray:
-                return ConvertJArray(jArray, indentationDepth);
-            case JObject jObject:
-                return ConvertJObject(jObject, indentationDepth);
-            default:
-                throw new NotSupportedException($"Settings values of type {jToken.GetType()} are not supported.");
-        }
-    }
+            JsonValue jsonValue => jsonValue.ToString(),
+            JsonArray jsonArray => ConvertJsonArray(jsonArray, indentationDepth),
+            JsonObject jsonObject => ConvertJsonObject(jsonObject, indentationDepth),
+            _ => throw new NotSupportedException($"Settings values of type {node.GetType()} are not supported."),
+        };
 
-    private string ConvertJArray(JArray jArray, int indentationDepth)
+    private string ConvertJsonArray(JsonArray jArray, int indentationDepth)
     {
-        var indentation = new string(' ', indentationDepth + 4);
+        var indentation = new string(' ', indentationDepth + IndentationDepth);
 
-        var items = jArray.Select(item => ConvertJToken(item, indentationDepth + 8)).ToList();
+        var items = jArray.Select(item => ConvertNode(item, indentationDepth + (2 * IndentationDepth))).ToList();
 
         // If the items are formatted (for ListValueOption) then don't inject line-by-line formatting.
         if (items.Exists(item => item.ContainsOrdinalIgnoreCase(Environment.NewLine)))
@@ -164,7 +156,7 @@ private string ConvertJArray(JArray jArray, int indentationDepth)
         stringArrayCodeBuilder.AppendLine();
         stringArrayCodeBuilder.AppendLine(CultureInfo.InvariantCulture, $"{indentation}{{");
 
-        var itemIndentation = new string(' ', indentationDepth + 8);
+        var itemIndentation = new string(' ', indentationDepth + (2 * IndentationDepth));
 
         foreach (var item in items)
         {
@@ -176,53 +168,46 @@ private string ConvertJArray(JArray jArray, int indentationDepth)
         return stringArrayCodeBuilder.ToString();
     }
 
-    private string ConvertJObject(JObject jObject, int indentationDepth)
+    private string ConvertJsonObject(JsonObject jsonObject, int indentationDepth)
     {
         var braceIndentation = new string(' ', indentationDepth);
-        var propertyIndentation = new string(' ', indentationDepth + 4);
-        if (jObject["name"] != null && jObject["value"] != null)
+        var propertyIndentation = new string(' ', indentationDepth + IndentationDepth);
+        if (jsonObject["name"] is { } name && jsonObject["value"] is { } value)
         {
             var objectCodeBuilder = new StringBuilder();
             objectCodeBuilder.AppendLine(CultureInfo.InvariantCulture, $"{braceIndentation}new ListValueOption");
             objectCodeBuilder.AppendLine(CultureInfo.InvariantCulture, $"{braceIndentation}{{");
-            objectCodeBuilder.AppendLine(CultureInfo.InvariantCulture, $"{propertyIndentation}Name = \"{jObject["name"]}\",");
-            objectCodeBuilder.AppendLine(CultureInfo.InvariantCulture, $"{propertyIndentation}Value = \"{jObject["value"]}\",");
+            objectCodeBuilder.AppendLine(CultureInfo.InvariantCulture, $"{propertyIndentation}Name = \"{name}\",");
+            objectCodeBuilder.AppendLine(CultureInfo.InvariantCulture, $"{propertyIndentation}Value = \"{value}\",");
             objectCodeBuilder.AppendLine(CultureInfo.InvariantCulture, $"{braceIndentation}}},");
 
             return objectCodeBuilder.ToString();
         }
 
         // Using a quoted string so it doesn't mess up the syntax highlighting of the rest of the code.
-        return T["\"FIX ME! Couldn't determine the actual type to instantiate.\" {0}", jObject.ToString()];
+        return T["\"FIX ME! Couldn't determine the actual type to instantiate.\" {0}", jsonObject.ToString()];
     }
 
-    private void AddSettingsWithout<T>(StringBuilder codeBuilder, JObject settings, int indentationDepth)
+    private void AddSettingsWithout<T>(StringBuilder codeBuilder, JsonObject settings, int indentationDepth = IndentationDepth)
     {
         var indentation = new string(' ', indentationDepth);
 
-        var filteredSettings = ((IEnumerable<KeyValuePair<string, JToken>>)settings)
-            .Where(setting => setting.Key != typeof(T).Name);
+        var filteredSettings = settings
+            .Where(pair => pair.Key != typeof(T).Name && pair.Value is JsonObject)
+            .Select(pair => (pair.Key, (JsonObject)pair.Value));
 
-        foreach (var setting in filteredSettings)
+        foreach (var (typeName, properties) in filteredSettings)
         {
-            var properties = setting.Value.Where(property => property is JProperty).Cast<JProperty>().ToArray();
+            if (properties.Count == 0) continue;
 
-            if (properties.Length == 0) continue;
-
-            codeBuilder.AppendLine(CultureInfo.InvariantCulture, $"{indentation}.WithSettings(new {setting.Key}");
+            codeBuilder.AppendLine(CultureInfo.InvariantCulture, $"{indentation}.WithSettings(new {typeName}");
             codeBuilder.AppendLine(indentation + "{");
 
-            // This doesn't support multi-level object hierarchies for settings but come on, who uses complex settings
-            // objects?
-            for (int i = 0; i < properties.Length; i++)
+            foreach (var (name, value) in properties)
             {
-                var property = properties[i];
-
-                var propertyValue = ConvertJToken(property.Value, indentationDepth);
-
-                propertyValue ??= "\"\"";
-
-                codeBuilder.AppendLine(CultureInfo.InvariantCulture, $"{indentation}    {property.Name} = {propertyValue},");
+                codeBuilder.AppendLine(
+                    CultureInfo.InvariantCulture,
+                    $"{indentation}    {name} = {ConvertNode(value, indentationDepth) ?? EmptyString},");
             }
 
             codeBuilder.AppendLine(indentation + "})");

Lombiq.HelpfulExtensions/Extensions/ContentSets/Drivers/ContentSetPartDisplayDriver.cs

@@ -6,7 +6,7 @@
 using Microsoft.Extensions.Localization;
 using OrchardCore.ContentManagement.Display.ContentDisplay;
 using OrchardCore.ContentManagement.Display.Models;
-using OrchardCore.DisplayManagement.ModelBinding;
+using OrchardCore.DisplayManagement.Handlers;
 using OrchardCore.DisplayManagement.Views;
 using OrchardCore.Entities;
 using System.Collections.Generic;
@@ -64,23 +64,17 @@ public override IDisplayResult Edit(ContentSetPart part, BuildPartEditorContext
                 context.IsNew))
             .Location($"Parts:0%{context.TypePartDefinition.Name};0");
 
-    public override async Task<IDisplayResult> UpdateAsync(
-        ContentSetPart part,
-        IUpdateModel updater,
-        UpdatePartEditorContext context)
+    public override async Task<IDisplayResult> UpdateAsync(ContentSetPart part, UpdatePartEditorContext context)
     {
-        var viewModel = new ContentSetPartViewModel();
+        var viewModel = await context.CreateModelAsync<ContentSetPartViewModel>(Prefix);
 
-        if (await updater.TryUpdateModelAsync(viewModel, Prefix))
-        {
-            part.Key = viewModel.Key;
+        part.Key = viewModel.Key;
 
-            // Need to do this here to support displaying the message to save before adding when the
-            // item has not been saved yet.
-            if (string.IsNullOrEmpty(part.ContentSet))
-            {
-                part.ContentSet = _idGenerator.GenerateUniqueId();
-            }
+        // Need to do this here to support displaying the message to save before adding when the item has not been saved
+        // yet.
+        if (string.IsNullOrEmpty(part.ContentSet))
+        {
+            part.ContentSet = _idGenerator.GenerateUniqueId();
         }
 
         return await EditAsync(part, context);

Lombiq.HelpfulExtensions/Extensions/ContentSets/ViewModels/ContentSetPartViewModel.cs

@@ -8,6 +8,7 @@
 using OrchardCore.ContentManagement.Metadata.Settings;
 using System.Collections.Generic;
 using System.Linq;
+using System.Text.Json.Nodes;
 using System.Threading.Tasks;
 
 namespace Lombiq.HelpfulExtensions.Extensions.ContentSets.ViewModels;
@@ -33,8 +34,7 @@ public class ContentSetPartViewModel
     public string DisplayName =>
         Definition?
             .Settings?
-            .Property(nameof(ContentTypePartSettings))?
-            .Value
+            .GetMaybe(nameof(ContentTypePartSettings))?
             .ToObject<ContentTypePartSettings>()?
             .DisplayName ?? Definition?.Name;
 

Lombiq.HelpfulExtensions/Extensions/ContentSets/Workflows/Handlers/WorkflowContentSetEventHandler.cs

@@ -4,15 +4,15 @@
 using Lombiq.HelpfulExtensions.Extensions.ContentSets.Workflows.Activities;
 using Lombiq.HelpfulExtensions.Extensions.ContentSets.Workflows.Models;
 using Lombiq.HelpfulLibraries.OrchardCore.Workflow;
-using Newtonsoft.Json;
-using Newtonsoft.Json.Linq;
 using OrchardCore.ContentManagement;
 using OrchardCore.ContentManagement.Metadata.Models;
 using OrchardCore.Workflows.Models;
 using OrchardCore.Workflows.Services;
 using System.Collections.Generic;
 using System.Dynamic;
 using System.Linq;
+using System.Text.Json;
+using System.Text.Json.Nodes;
 using System.Threading.Tasks;
 
 namespace Lombiq.HelpfulExtensions.Extensions.ContentSets.Workflows.Handlers;
@@ -54,14 +54,16 @@ public async Task<IEnumerable<ContentSetLinkViewModel>> GetSupportedOptionsAsync
                 case ContentSetLinkViewModel viewModel:
                     links.Add(viewModel);
                     break;
-                case IEnumerable<object> collection when collection.CastWhere<ExpandoObject>() is { } objects && objects.Any():
-                    links.AddRange(JToken.FromObject(objects).ToObject<IEnumerable<ContentSetLinkViewModel>>());
-                    break;
                 case ExpandoObject expandoObject:
-                    links.Add(JToken.FromObject(expandoObject).ToObject<ContentSetLinkViewModel>());
+                    links.Add(SerializeAndDeserialize<ContentSetLinkViewModel>(expandoObject));
+                    break;
+                case IEnumerable<object> collection when
+                    collection.CastWhere<ExpandoObject>().ToList() is { } objects &&
+                    objects.Count != 0:
+                    links.AddRange(SerializeAndDeserialize<IEnumerable<ContentSetLinkViewModel>>(objects));
                     break;
                 case string json when !string.IsNullOrWhiteSpace(json):
-                    links.AddRange(JsonConvert.DeserializeObject<List<ContentSetLinkViewModel>>(json));
+                    links.AddRange(JsonSerializer.Deserialize<List<ContentSetLinkViewModel>>(json));
                     break;
                 default: continue;
             }
@@ -79,4 +81,7 @@ public Task CreatingAsync(
             new CreatingContext(content, definition, contentSet, newKey),
             $"{nameof(WorkflowContentSetEventHandler)}.{nameof(CreatingAsync)}" +
             $"({content.ContentItemId}, {definition.Name}, {contentSet}, {newKey})");
+
+    private static T SerializeAndDeserialize<T>(object source) =>
+        JNode.FromObject(source).ToObject<T>();
 }