Merge pull request #1612 from json-api-dotnet/formatter-primary-ctor-newline

Update formatting rules to place call to base in primary constructor on separate line

Author: bkoelman

JsonApiDotNetCore.sln.DotSettings

@@ -137,6 +137,7 @@ JsonApiDotNetCore.ArgumentGuard.NotNull($EXPR$);</s:String>
 	<s:String x:Key="/Default/CodeStyle/CodeFormatting/CSharpFormat/PLACE_ACCESSORHOLDER_ATTRIBUTE_ON_SAME_LINE_EX/@EntryValue">NEVER</s:String>
 	<s:Boolean x:Key="/Default/CodeStyle/CodeFormatting/CSharpFormat/PLACE_CONSTRUCTOR_INITIALIZER_ON_SAME_LINE/@EntryValue">False</s:Boolean>
 	<s:String x:Key="/Default/CodeStyle/CodeFormatting/CSharpFormat/PLACE_FIELD_ATTRIBUTE_ON_SAME_LINE_EX/@EntryValue">NEVER</s:String>
+	<s:Boolean x:Key="/Default/CodeStyle/CodeFormatting/CSharpFormat/PLACE_PRIMARY_CONSTRUCTOR_INITIALIZER_ON_SAME_LINE/@EntryValue">False</s:Boolean>
 	<s:Boolean x:Key="/Default/CodeStyle/CodeFormatting/CSharpFormat/PLACE_SIMPLE_ACCESSOR_ON_SINGLE_LINE/@EntryValue">False</s:Boolean>
 	<s:Boolean x:Key="/Default/CodeStyle/CodeFormatting/CSharpFormat/PLACE_SIMPLE_ANONYMOUSMETHOD_ON_SINGLE_LINE/@EntryValue">False</s:Boolean>
 	<s:String x:Key="/Default/CodeStyle/CodeFormatting/CSharpFormat/PLACE_SIMPLE_EMBEDDED_STATEMENT_ON_SAME_LINE/@EntryValue">NEVER</s:String>

src/Examples/DapperExample/Controllers/OperationsController.cs

@@ -8,5 +8,5 @@ namespace DapperExample.Controllers;
 
 public sealed class OperationsController(
     IJsonApiOptions options, IResourceGraph resourceGraph, ILoggerFactory loggerFactory, IOperationsProcessor processor, IJsonApiRequest request,
-    ITargetedFields targetedFields, IAtomicOperationFilter operationFilter) : JsonApiOperationsController(options, resourceGraph, loggerFactory, processor,
-    request, targetedFields, operationFilter);
+    ITargetedFields targetedFields, IAtomicOperationFilter operationFilter)
+    : JsonApiOperationsController(options, resourceGraph, loggerFactory, processor, request, targetedFields, operationFilter);

src/Examples/DapperExample/TranslationToSql/Builders/DeleteOneToOneStatementBuilder.cs

@@ -6,7 +6,8 @@
 
 namespace DapperExample.TranslationToSql.Builders;
 
-internal sealed class DeleteOneToOneStatementBuilder(IDataModelService dataModelService) : StatementBuilder(dataModelService)
+internal sealed class DeleteOneToOneStatementBuilder(IDataModelService dataModelService)
+    : StatementBuilder(dataModelService)
 {
     public DeleteNode Build(ResourceType resourceType, string whereColumnName, object? whereValue)
     {

src/Examples/DapperExample/TranslationToSql/Builders/DeleteResourceStatementBuilder.cs

@@ -7,7 +7,8 @@
 
 namespace DapperExample.TranslationToSql.Builders;
 
-internal sealed class DeleteResourceStatementBuilder(IDataModelService dataModelService) : StatementBuilder(dataModelService)
+internal sealed class DeleteResourceStatementBuilder(IDataModelService dataModelService)
+    : StatementBuilder(dataModelService)
 {
     public DeleteNode Build(ResourceType resourceType, params object[] idValues)
     {

src/Examples/DapperExample/TranslationToSql/Builders/InsertStatementBuilder.cs

@@ -7,7 +7,8 @@
 
 namespace DapperExample.TranslationToSql.Builders;
 
-internal sealed class InsertStatementBuilder(IDataModelService dataModelService) : StatementBuilder(dataModelService)
+internal sealed class InsertStatementBuilder(IDataModelService dataModelService)
+    : StatementBuilder(dataModelService)
 {
     public InsertNode Build(ResourceType resourceType, IReadOnlyDictionary<string, object?> columnsToSet)
     {

src/Examples/DapperExample/TranslationToSql/Builders/UpdateClearOneToOneStatementBuilder.cs

@@ -6,7 +6,8 @@
 
 namespace DapperExample.TranslationToSql.Builders;
 
-internal sealed class UpdateClearOneToOneStatementBuilder(IDataModelService dataModelService) : StatementBuilder(dataModelService)
+internal sealed class UpdateClearOneToOneStatementBuilder(IDataModelService dataModelService)
+    : StatementBuilder(dataModelService)
 {
     public UpdateNode Build(ResourceType resourceType, string setColumnName, string whereColumnName, object? whereValue)
     {

src/Examples/DapperExample/TranslationToSql/Builders/UpdateResourceStatementBuilder.cs

@@ -7,7 +7,8 @@
 
 namespace DapperExample.TranslationToSql.Builders;
 
-internal sealed class UpdateResourceStatementBuilder(IDataModelService dataModelService) : StatementBuilder(dataModelService)
+internal sealed class UpdateResourceStatementBuilder(IDataModelService dataModelService)
+    : StatementBuilder(dataModelService)
 {
     public UpdateNode Build(ResourceType resourceType, IReadOnlyDictionary<string, object?> columnsToUpdate, params object[] idValues)
     {

src/Examples/DapperExample/TranslationToSql/DataModel/FromEntitiesDataModelService.cs

@@ -12,7 +12,8 @@ namespace DapperExample.TranslationToSql.DataModel;
 /// <summary>
 /// Derives foreign keys and connection strings from an existing Entity Framework Core model.
 /// </summary>
-public sealed class FromEntitiesDataModelService(IResourceGraph resourceGraph) : BaseDataModelService(resourceGraph)
+public sealed class FromEntitiesDataModelService(IResourceGraph resourceGraph)
+    : BaseDataModelService(resourceGraph)
 {
     private readonly Dictionary<RelationshipAttribute, RelationshipForeignKey> _foreignKeysByRelationship = [];
     private readonly Dictionary<AttrAttribute, bool> _columnNullabilityPerAttribute = [];

src/Examples/DapperExample/TranslationToSql/Generators/ParameterGenerator.cs

@@ -20,5 +20,6 @@ public void Reset()
         _nameGenerator.Reset();
     }
 
-    private sealed class ParameterNameGenerator() : UniqueNameGenerator("@p");
+    private sealed class ParameterNameGenerator()
+        : UniqueNameGenerator("@p");
 }

src/Examples/DapperExample/TranslationToSql/Generators/TableAliasGenerator.cs

@@ -3,4 +3,5 @@ namespace DapperExample.TranslationToSql.Generators;
 /// <summary>
 /// Generates a SQL table alias with a unique name.
 /// </summary>
-internal sealed class TableAliasGenerator() : UniqueNameGenerator("t");
+internal sealed class TableAliasGenerator()
+    : UniqueNameGenerator("t");