Format record braces consistently with class (#313)

* Format record braces consistently with class closes #233 * introduce variable to make code easier to understand
belav · Jun 19, 2021 · 4c179a1 · 4c179a1
1 parent d8956d7
commit 4c179a1
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/Src/CSharpier.Tests/TestFiles/RecordDeclaration/RecordDeclarations.cst b/Src/CSharpier.Tests/TestFiles/RecordDeclaration/RecordDeclarations.cst
@@ -15,6 +15,11 @@ record MostlyWeeklyTemperature(
 
 record PC(string x) : PrimaryConstructor(x) { }
 
+public record RecordWithProperty
+{
+    public string Property { get; set; }
+}
+
 record RecordWithoutBody(string property);
 
 record LongerRecordNameWhatHappens_________________________________________(string x)

diff --git a/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/BaseTypeDeclaration.cs b/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/BaseTypeDeclaration.cs
@@ -104,8 +104,11 @@ public static Doc Print(BaseTypeDeclarationSyntax node)
             {
                 DocUtilities.RemoveInitialDoubleHardLine(members);
 
+                var useSpaceBraceWithParameterList = groupId != null && parameterList != null;
                 docs.Add(
-                    groupId != null ? Doc.IfBreak(" ", Doc.Line, groupId) : Doc.HardLine,
+                    useSpaceBraceWithParameterList
+                        ? Doc.IfBreak(" ", Doc.Line, groupId)
+                        : Doc.HardLine,
                     Token.Print(node.OpenBraceToken),
                     members,
                     Doc.HardLine,