Merge pull request #295 from Kentico/fix/issue-294

Fix properties migration in case of properties null
Kentico · Nov 12, 2024 · ab27b8f · ab27b8f
2 parents fd66e32 + 620c23b
commit ab27b8f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/KVA/Migration.Tool.Source/Services/PageBuilderPatcher.cs b/KVA/Migration.Tool.Source/Services/PageBuilderPatcher.cs
@@ -92,8 +92,12 @@ private async Task<bool> WalkSections(int siteId, List<SectionConfiguration> sec
             logger.LogTrace("Walk section {TypeIdentifier}|{Identifier}", section.TypeIdentifier, section.Identifier);
 
             var sectionFcs = sourceInstanceContext.GetSectionFormComponents(siteId, section.TypeIdentifier);
-            bool ndp1 = await MigrateProperties(siteId, section.Properties, sectionFcs, new Dictionary<string, IWidgetPropertyMigration>());
-            needsDeferredPatch = ndp1 || needsDeferredPatch;
+
+            if (section.Properties is { Count: > 0 } properties)
+            {
+                bool ndp1 = await MigrateProperties(siteId, properties, sectionFcs, new Dictionary<string, IWidgetPropertyMigration>());
+                needsDeferredPatch = ndp1 || needsDeferredPatch;
+            }
 
             if (section.Zones is { Count: > 0 })
             {

diff --git a/Migration.Tool.Common/Model/EditableAreasConfiguration.cs b/Migration.Tool.Common/Model/EditableAreasConfiguration.cs
@@ -89,7 +89,7 @@ public sealed class SectionConfiguration
     [DataMember]
     [JsonProperty("properties")]
     // public ISectionProperties Properties { get; set; }
-    public JObject Properties { get; set; }
+    public JObject? Properties { get; set; }
 
     /// <summary>Zones within the section.</summary>
     [DataMember]