OSOE-818: Upgrade to Orchard Core 2.0

.gitignore

@@ -8,3 +8,4 @@ wwwroot/
 node_modules/
 *.user
 .pnpm-debug.log
+*.orig

Lombiq.DataTables.Samples/Lombiq.DataTables.Samples.csproj

@@ -33,8 +33,8 @@
   </ItemGroup>
 
   <ItemGroup>
-    <PackageReference Include="OrchardCore.ContentFields" Version="1.8.0" />
-    <PackageReference Include="OrchardCore.Module.Targets" Version="1.8.0" />
+    <PackageReference Include="OrchardCore.ContentFields" Version="2.0.0" />
+    <PackageReference Include="OrchardCore.Module.Targets" Version="2.0.0" />
   </ItemGroup>
 
   <ItemGroup>

Lombiq.DataTables/Controllers/Api/RowsController.cs

@@ -1,9 +1,9 @@
 using Lombiq.DataTables.Models;
 using Lombiq.DataTables.Services;
+using Lombiq.HelpfulLibraries.AspNetCore.Mvc;
 using Microsoft.AspNetCore.Authorization;
 using Microsoft.AspNetCore.Mvc;
 using Microsoft.Extensions.Localization;
-using Newtonsoft.Json;
 using System.Collections.Generic;
 using System.Linq;
 using System.Threading.Tasks;
@@ -32,7 +32,7 @@ public RowsController(
     /// <summary>
     /// Gets the current table view's rows.
     /// </summary>
-    /// <param name="requestJson">The request to fulfill serialized as JSON.</param>
+    /// <param name="request">The request to fulfill serialized as JSON.</param>
     /// <returns>The response for this API call.</returns>
     /// <remarks>
     ///   <list type="bullet">
@@ -53,12 +53,11 @@ public RowsController(
     /// </remarks>
     [IgnoreAntiforgeryToken]
     [HttpGet]
-    public async Task<ActionResult<DataTableDataResponse>> Get(string requestJson)
+    public async Task<ActionResult<DataTableDataResponse>> Get([FromJsonQueryString(Name = "requestJson")] DataTableDataRequest request)
     {
-        if (string.IsNullOrEmpty(requestJson)) ModelState.AddModelError(nameof(requestJson), "Controller parameter is missing.");
+        if (request == null) return BadRequest();
         if (!ModelState.IsValid) return BadRequest(ModelState);
 
-        var request = JsonConvert.DeserializeObject<DataTableDataRequest>(requestJson);
         var dataProvider = _dataTableDataProviderAccessor.GetDataProvider(request.DataProvider);
         if (dataProvider == null)
         {
@@ -87,13 +86,12 @@ public async Task<ActionResult<DataTableDataResponse>> Get(string requestJson)
     }
 
     public async Task<ActionResult<DataTableDataResponse>> Export(
-        string requestJson,
+        [FromJsonQueryString(Name = "requestJson")] DataTableDataRequest request,
         string name = null,
         bool exportAll = true)
     {
         if (!ModelState.IsValid) return BadRequest(ModelState);
 
-        var request = JsonConvert.DeserializeObject<DataTableDataRequest>(requestJson);
         if (exportAll)
         {
             request.Start = 0;

Lombiq.DataTables/Controllers/TableController.cs

@@ -1,31 +1,31 @@
 using Lombiq.DataTables.Services;
 using Lombiq.DataTables.ViewModels;
-using Lombiq.HelpfulLibraries.OrchardCore.Mvc;
 using Microsoft.AspNetCore.Mvc;
-using Newtonsoft.Json.Linq;
 using OrchardCore.Admin;
 using System.Collections.Generic;
 using System.Linq;
+using System.Text.Json.Nodes;
 using System.Threading.Tasks;
 
 namespace Lombiq.DataTables.Controllers;
 
-[Admin]
 public class TableController : Controller
 {
     private readonly IEnumerable<IDataTableDataProvider> _dataTableDataProviders;
 
     public TableController(IEnumerable<IDataTableDataProvider> dataTableDataProviders) =>
         _dataTableDataProviders = dataTableDataProviders;
 
-    [AdminRoute("DataTable/{providerName}/{queryId?}")]
+    [Admin("DataTable/{providerName}/{queryId?}")]
     public async Task<IActionResult> Get(string providerName, string queryId = null, bool paging = true, bool viewAction = false)
     {
         if (!ModelState.IsValid) return BadRequest(ModelState);
 
         var provider = _dataTableDataProviders.Single(provider => provider.Name == providerName);
         if (string.IsNullOrEmpty(queryId)) queryId = providerName;
-        var definition = new DataTableDefinitionViewModel(JObject.FromObject(new { paging, viewAction }))
+
+        var additionalDatatableOptions = JObject.FromObject(new { paging, viewAction })!.AsObject();
+        var definition = new DataTableDefinitionViewModel(additionalDatatableOptions)
         {
             DataProvider = providerName,
             QueryId = queryId,

Lombiq.DataTables/Extensions/DataTableDataProviderExtensions.cs

@@ -5,13 +5,13 @@
 using Microsoft.AspNetCore.Http;
 using Microsoft.AspNetCore.Routing;
 using Microsoft.Extensions.Localization;
-using Newtonsoft.Json;
 using OrchardCore.Mvc.Core.Utilities;
 using OrchardCore.Security.Permissions;
 using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Security.Claims;
+using System.Text.Json;
 using System.Threading.Tasks;
 using static Lombiq.DataTables.Constants.SortingDirection;
 
@@ -167,7 +167,7 @@ public static string GetCustomActionsJson(
         IHttpContextAccessor hca,
         LinkGenerator linkGenerator,
         IStringLocalizer<ActionsDescriptor> actionsStringLocalizer) =>
-        JsonConvert.SerializeObject(GetCustomActions(
+        JsonSerializer.Serialize(GetCustomActions(
             dataProvider,
             contentItemId,
             canDelete,

Lombiq.DataTables/Extensions/JsonNodeExtensions.cs

@@ -0,0 +1,11 @@
+namespace System.Text.Json.Nodes;
+
+public static class JsonNodeExtensions
+{
+    /// <summary>
+    /// Checks if the provided <paramref name="node"/> is object and has a <c>Type</c> property, and if its value
+    /// matches <typeparamref name="T"/>.
+    /// </summary>
+    public static bool HasMatchingTypeProperty<T>(this JsonNode node) =>
+        node is JsonObject jsonObject && jsonObject["Type"]?.ToString() == typeof(T).Name;
+}

Lombiq.DataTables/Liquid/ActionsLiquidFilter.cs

@@ -5,13 +5,13 @@
 using Microsoft.AspNetCore.Http;
 using Microsoft.AspNetCore.Routing;
 using Microsoft.Extensions.Localization;
-using Newtonsoft.Json.Linq;
 using OrchardCore.ContentManagement;
 using OrchardCore.DisplayManagement;
 using OrchardCore.Liquid;
 using System;
 using System.IO;
 using System.Text.Encodings.Web;
+using System.Text.Json.Nodes;
 using System.Threading.Tasks;
 
 namespace Lombiq.DataTables.Liquid;
@@ -71,7 +71,7 @@ public ValueTask<FluidValue> ProcessAsync(FluidValue input, FilterArguments argu
             FluidValues.Object => input!.ToObjectValue() switch
             {
                 ActionsDescriptor model => FromObjectAsync(model, title, returnUrl),
-                JToken jToken => FromObjectAsync(jToken.ToObject<ActionsDescriptor>(), title, returnUrl),
+                JsonNode jsonNode => FromObjectAsync(jsonNode.ToObject<ActionsDescriptor>(), title, returnUrl),
                 { } unknown => throw GetException(unknown),
                 _ => throw new ArgumentNullException(nameof(input)),
             },

Lombiq.DataTables/Lombiq.DataTables.csproj

@@ -38,11 +38,10 @@
 
   <ItemGroup>
     <PackageReference Include="ClosedXML" Version="0.102.1" />
-    <PackageReference Include="Newtonsoft.Json" Version="13.0.3" />
     <PackageReference Include="Nito.AsyncEx.Tasks" Version="5.1.2" />
-    <PackageReference Include="OrchardCore.Contents" Version="1.8.0" />
-    <PackageReference Include="OrchardCore.Module.Targets" Version="1.8.0" />
-    <PackageReference Include="OrchardCore.Search.Lucene" Version="1.8.0" />
+    <PackageReference Include="OrchardCore.Contents" Version="2.0.0" />
+    <PackageReference Include="OrchardCore.Module.Targets" Version="2.0.0" />
+    <PackageReference Include="OrchardCore.Search.Lucene" Version="2.0.0" />
   </ItemGroup>
 
   <ItemGroup Condition="'$(NuGetBuild)' != 'true'">
@@ -52,8 +51,8 @@
   </ItemGroup>
 
   <ItemGroup Condition="'$(NuGetBuild)' == 'true'">
-    <PackageReference Include="Lombiq.HelpfulLibraries.OrchardCore" Version="10.0.0" />
-    <PackageReference Include="Lombiq.HelpfulLibraries.LinqToDb" Version="10.0.0" />
+    <PackageReference Include="Lombiq.HelpfulLibraries.OrchardCore" Version="10.0.1-alpha.8.osoe-818" />
+    <PackageReference Include="Lombiq.HelpfulLibraries.LinqToDb" Version="10.0.1-alpha.8.osoe-818" />
     <PackageReference Include="Lombiq.NodeJs.Extensions" Version="2.1.0" />
   </ItemGroup>
 

Lombiq.DataTables/Models/DataTableChildRowResponse.cs

@@ -1,12 +1,13 @@
-using Newtonsoft.Json;
-using Newtonsoft.Json.Serialization;
+using System.Text.Json.Serialization;
 
 namespace Lombiq.DataTables.Models;
 
-[JsonObject(NamingStrategyType = typeof(CamelCaseNamingStrategy))]
 public class DataTableChildRowResponse
 {
+    [JsonPropertyName("error")]
     public string Error { get; set; }
+
+    [JsonPropertyName("content")]
     public string Content { get; set; }
 
     public static DataTableChildRowResponse ErrorResult(string errorText) => new() { Error = errorText };

Lombiq.DataTables/Models/DataTableColumn.cs

@@ -1,14 +1,21 @@
-using Newtonsoft.Json;
-using Newtonsoft.Json.Serialization;
+using System.Text.Json.Serialization;
 
 namespace Lombiq.DataTables.Models;
 
-[JsonObject(NamingStrategyType = typeof(CamelCaseNamingStrategy))]
 public class DataTableColumn
 {
+    [JsonPropertyName("data")]
     public string Data { get; set; }
+
+    [JsonPropertyName("name")]
     public string Name { get; set; }
+
+    [JsonPropertyName("searchable")]
     public bool Searchable { get; set; }
+
+    [JsonPropertyName("orderable")]
     public bool Orderable { get; set; }
+
+    [JsonPropertyName("search")]
     public DataTableSearchParameters Search { get; set; }
 }

Lombiq.DataTables/Models/DataTableDataRequest.cs

@@ -1,20 +1,28 @@
-using Newtonsoft.Json;
-using Newtonsoft.Json.Serialization;
 using System.Collections.Generic;
 using System.Linq;
+using System.Text.Json.Serialization;
 
 namespace Lombiq.DataTables.Models;
 
-[JsonObject(NamingStrategyType = typeof(CamelCaseNamingStrategy))]
 public class DataTableDataRequest
 {
     public string QueryId { get; set; }
+
     public string DataProvider { get; set; }
+
+    [JsonRequired]
     public int Draw { get; set; }
+
+    [JsonRequired]
     public int Start { get; set; }
+
+    [JsonRequired]
     public int Length { get; set; }
+
     public IEnumerable<DataTableColumn> ColumnFilters { get; set; }
+
     public DataTableSearchParameters Search { get; set; }
+
     public IEnumerable<DataTableOrder> Order { get; set; }
 
     public bool HasSearch => !string.IsNullOrWhiteSpace(Search?.Value);

Lombiq.DataTables/Models/DataTableDataResponse.cs

@@ -1,10 +1,8 @@
-using Newtonsoft.Json;
-using Newtonsoft.Json.Serialization;
 using System.Collections.Generic;
+using System.Text.Json.Serialization;
 
 namespace Lombiq.DataTables.Models;
 
-[JsonObject(NamingStrategyType = typeof(CamelCaseNamingStrategy))]
 public class DataTableDataResponse
 {
     /// <summary>
@@ -14,28 +12,33 @@ public class DataTableDataResponse
     /// <remarks>
     /// <para>For internal use only. It's overwritten during normal use.</para>
     /// </remarks>
-    [JsonProperty]
+    [JsonPropertyName("draw")]
+    [JsonInclude]
     internal int Draw { get; set; }
 
     /// <summary>
     /// Gets or sets the extra informational field that shows the actual total if filtering (such as keyword
     /// search) is used. When not filtering it must be the same as <see cref="RecordsFiltered"/>.
     /// </summary>
+    [JsonPropertyName("recordsTotal")]
     public int RecordsTotal { get; set; }
 
     /// <summary>
     /// Gets or sets the total number of results; used for paging.
     /// </summary>
+    [JsonPropertyName("recordsFiltered")]
     public int RecordsFiltered { get; set; }
 
     /// <summary>
     /// Gets or sets the table contents of the current page.
     /// </summary>
+    [JsonPropertyName("data")]
     public IEnumerable<DataTableRow> Data { get; set; }
 
     /// <summary>
     /// Gets or sets the user-facing error message in case something went wrong.
     /// </summary>
+    [JsonPropertyName("error")]
     public string Error { get; set; }
 
     /// <summary>

Lombiq.DataTables/Models/DataTableOrder.cs

@@ -1,15 +1,25 @@
 using Lombiq.DataTables.Constants;
-using Newtonsoft.Json;
-using Newtonsoft.Json.Serialization;
+using System.Diagnostics.CodeAnalysis;
+using System.Text.Json.Serialization;
 
 namespace Lombiq.DataTables.Models;
 
-[JsonObject(NamingStrategyType = typeof(CamelCaseNamingStrategy))]
 public class DataTableOrder
 {
     public string Column { get; set; }
+
+    [JsonIgnore]
     public SortingDirection Direction { get; set; }
 
+    [JsonPropertyName("direction")]
+    [JsonInclude]
+    [SuppressMessage("CodeQuality", "IDE0051:Remove unused private members", Justification = "It's used for JSON conversion.")]
+    private string DirectionString
+    {
+        get => IsAscending ? "ascending" : "descending";
+        set => Direction = value == "descending" ? SortingDirection.Descending : SortingDirection.Ascending;
+    }
+
     [JsonIgnore]
     public bool IsAscending => Direction == SortingDirection.Ascending;
 }