Add WithSettings<>() method

OrchardCMS · Feb 29, 2024 · 1b1082d · 1b1082d
1 parent 0a396f4
commit 1b1082d
Showing 1 changed file with 13 additions and 4 deletions.
diff --git a/...ore.ContentManagement.Abstractions/Metadata/Builders/ContentPartFieldDefinitionBuilder.cs b/...ore.ContentManagement.Abstractions/Metadata/Builders/ContentPartFieldDefinitionBuilder.cs
@@ -46,28 +46,37 @@ public ContentPartFieldDefinitionBuilder MergeSettings(JsonObject settings)
             // If existing settings do not exist, create.
             if (existingJObject == null)
             {
-                existingJObject = JObject.FromObject(new T(), ContentBuilderSettings.IgnoreDefaultValuesSerializer);
+                existingJObject = ToJsonObject(new T());
                 _settings[typeof(T).Name] = existingJObject;
             }
 
             var settingsToMerge = existingJObject.ToObject<T>();
             setting(settingsToMerge);
-            _settings[typeof(T).Name] = JObject.FromObject(settingsToMerge, ContentBuilderSettings.IgnoreDefaultValuesSerializer);
+            _settings[typeof(T).Name] = ToJsonObject(settingsToMerge);
             return this;
         }
 
         public ContentPartFieldDefinitionBuilder WithSettings<T>(T settings)
         {
             ArgumentNullException.ThrowIfNull(settings);
 
-            var jObject = JObject.FromObject(settings, ContentBuilderSettings.IgnoreDefaultValuesSerializer);
-            _settings[typeof(T).Name] = jObject;
+            _settings[typeof(T).Name] = ToJsonObject(settings);
+
+            return this;
+        }
+
+        public ContentPartFieldDefinitionBuilder WithSettings<T>() where T : class, new()
+        {
+            _settings[typeof(T).Name] = ToJsonObject(new T());
 
             return this;
         }
 
         public abstract ContentPartFieldDefinitionBuilder OfType(ContentFieldDefinition fieldDefinition);
         public abstract ContentPartFieldDefinitionBuilder OfType(string fieldType);
         public abstract ContentPartFieldDefinition Build();
+
+        private static JsonObject ToJsonObject(object obj)
+            => JObject.FromObject(obj, ContentBuilderSettings.IgnoreDefaultValuesSerializer);
     }
 }