From b9260dbdb1c5c872cd2bb6aefdbd21cbaee496b5 Mon Sep 17 00:00:00 2001
From: MattMo <matt@montoyatech.com>
Date: Sun, 24 Sep 2023 19:34:26 -0700
Subject: [PATCH] Fixed a bug where a double empty constructor could be
 generated for a C# client if there is no fields/properties. Bumped package
 version to 1.7.3

---
 Rest.Net/Rest.Net.csproj              |  2 +-
 Rest.Net/RestCSharpClientGenerator.cs | 19 +++++++++++--------
 2 files changed, 12 insertions(+), 9 deletions(-)

diff --git a/Rest.Net/Rest.Net.csproj b/Rest.Net/Rest.Net.csproj
index 7a50d05..7e06527 100644
--- a/Rest.Net/Rest.Net.csproj
+++ b/Rest.Net/Rest.Net.csproj
@@ -17,7 +17,7 @@
     <AssemblyName>MontoyaTech.Rest.Net</AssemblyName>
     <RootNamespace>MontoyaTech.Rest.Net</RootNamespace>
     <GenerateDocumentationFile>True</GenerateDocumentationFile>
-    <Version>1.7.2</Version>
+    <Version>1.7.3</Version>
     <PackageReleaseNotes></PackageReleaseNotes>
     <PackageIcon>Logo_Symbol_Black_Outline.png</PackageIcon>
   </PropertyGroup>
diff --git a/Rest.Net/RestCSharpClientGenerator.cs b/Rest.Net/RestCSharpClientGenerator.cs
index 5873498..6b071ca 100644
--- a/Rest.Net/RestCSharpClientGenerator.cs
+++ b/Rest.Net/RestCSharpClientGenerator.cs
@@ -250,17 +250,20 @@ namespace MontoyaTech.Rest.Net
                 writer.Outdent().WriteLine('}');
 
                 //Generate a constructor to set all the fields/properties with optional default values
-                writer.WriteBreak();
-                writer.Write($"public {(newName != null ? newName.Name : type.Name)}(");
+                if (fields.Length > 0 || properties.Length > 0)
+                {
+                    writer.WriteBreak();
+                    writer.Write($"public {(newName != null ? newName.Name : type.Name)}(");
 
-                foreach (var field in fields)
-                    writer.WriteSeparator().Write($"{this.GetTypeFullyResolvedName(field.FieldType)} {field.Name} = {this.GetTypeDefaultValue(field.FieldType)}");
+                    foreach (var field in fields)
+                        writer.WriteSeparator().Write($"{this.GetTypeFullyResolvedName(field.FieldType)} {field.Name} = {this.GetTypeDefaultValue(field.FieldType)}");
 
-                foreach (var property in properties)
-                    writer.WriteSeparator().Write($"{this.GetTypeFullyResolvedName(property.PropertyType)} {property.Name} = {this.GetTypeDefaultValue(property.PropertyType)}");
+                    foreach (var property in properties)
+                        writer.WriteSeparator().Write($"{this.GetTypeFullyResolvedName(property.PropertyType)} {property.Name} = {this.GetTypeDefaultValue(property.PropertyType)}");
 
-                writer.WriteLine(")");
-                writer.WriteLine('{').Indent();
+                    writer.WriteLine(")");
+                    writer.WriteLine('{').Indent();
+                }
 
                 foreach (var field in fields)
                     writer.WriteLine($"this.{field.Name} = {field.Name};");