Fix logic of getNullableType of csharp server and client.

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AbstractCSharpCodegen.java

@@ -160,6 +160,7 @@ public AbstractCSharpCodegen() {
  "Int64",
  "Float",
  "Guid?",
+ "Guid",
  "System.IO.Stream", // not really a primitive, we include it to avoid model import
  "Object")
  );

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AspNetCoreServerCodegen.java

@@ -22,6 +22,7 @@
 import io.swagger.v3.oas.models.media.Schema;
 import io.swagger.v3.parser.util.SchemaTypeUtil;
 import org.openapitools.codegen.*;
+import org.openapitools.codegen.utils.ModelUtils;
 import org.openapitools.codegen.utils.URLPathUtils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -353,7 +354,7 @@ public void processOpts() {
  supportingFiles.add(new SupportingFile("Filters" + File.separator + "GeneratePathParamsValidationFilter.mustache",
  packageFolder + File.separator + "Filters", "GeneratePathParamsValidationFilter.cs"));
  }
-
+
  supportingFiles.add(new SupportingFile("Authentication" + File.separator + "ApiAuthentication.mustache",packageFolder + File.separator + "Authentication", "ApiAuthentication.cs"));
  }
 
@@ -407,12 +408,12 @@ public String toRegularExpression(String pattern) {
 
  @Override
  public String getNullableType(Schema p, String type) {
-boolean isNullableExpected = p.getNullable() == null || (p.getNullable() != null && p.getNullable());
-
-if (isNullableExpected && languageSpecificPrimitives.contains(type + "?")) {
-return type + "?";
-} else if (languageSpecificPrimitives.contains(type)) {
-return type;
+if (languageSpecificPrimitives.contains(type)) {
+ if (isSupportNullable() && ModelUtils.isNullable(p) && nullableType.contains(type)) {
+ return type + "?";
+} else {
+ return type;
+}
  } else {
  return null;
  }

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/CSharpClientCodegen.java

@@ -879,15 +879,15 @@ public String toInstantiationType(Schema schema) {
  return null;
  }
  }
-
+
  @Override
  public String getNullableType(Schema p, String type) {
-boolean isNullableExpected = p.getNullable() == null || (p.getNullable() != null && p.getNullable());
-
-if (isNullableExpected && languageSpecificPrimitives.contains(type + "?")) {
-return type + "?";
-} else if (languageSpecificPrimitives.contains(type)) {
-return type;
+if (languageSpecificPrimitives.contains(type)) {
+ if (isSupportNullable() && ModelUtils.isNullable(p) && nullableType.contains(type)) {
+ return type + "?";
+} else {
+ return type;
+}
  } else {
  return null;
  }

modules/openapi-generator/src/main/resources/csharp/api.mustache

@@ -32,7 +32,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>{{#returnType}}{{returnType}}{{/returnType}}</returns>
- {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
 
  /// <summary>
  /// {{summary}}
@@ -43,7 +43,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>ApiResponse of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}Object(void){{/returnType}}</returns>
- ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  {{/operation}}
  #endregion Synchronous Operations
  {{#supportsAsync}}
@@ -58,7 +58,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}void{{/returnType}}</returns>
- {{#returnType}}System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ {{#returnType}}System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
 
  /// <summary>
  /// {{summary}}
@@ -69,7 +69,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of ApiResponse{{#returnType}} ({{returnType}}){{/returnType}}</returns>
- System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  {{/operation}}
  #endregion Asynchronous Operations
  {{/supportsAsync}}
@@ -190,7 +190,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>{{#returnType}}{{returnType}}{{/returnType}}</returns>
- public {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#returnType}}ApiResponse<{{{returnType}}}> localVarResponse = {{operationId}}WithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  return localVarResponse.Data;{{/returnType}}{{^returnType}}{{operationId}}WithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});{{/returnType}}
@@ -202,7 +202,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>ApiResponse of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}Object(void){{/returnType}}</returns>
- public ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#allParams}}
  {{#required}}
@@ -325,7 +325,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}void{{/returnType}}</returns>
- {{#returnType}}public async System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}public async System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ {{#returnType}}public async System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}public async System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#returnType}}ApiResponse<{{{returnType}}}> localVarResponse = await {{operationId}}AsyncWithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  return localVarResponse.Data;{{/returnType}}{{^returnType}}await {{operationId}}AsyncWithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});{{/returnType}}
@@ -338,7 +338,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of ApiResponse{{#returnType}} ({{returnType}}){{/returnType}}</returns>
- public async System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public async System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#allParams}}
  {{#required}}

modules/openapi-generator/src/test/java/org/openapitools/codegen/csharp/CSharpModelTest.java

@@ -141,10 +141,10 @@ public void simpleModelTest() {
 
  final CodegenProperty property1 = cm.vars.get(0);
  Assert.assertEquals(property1.baseName, "id");
- Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.dataType, "long");
  Assert.assertEquals(property1.name, "Id");
  Assert.assertNull(property1.defaultValue);
- Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertEquals(property1.baseType, "long");
  Assert.assertTrue(property1.hasMore);
  Assert.assertTrue(property1.required);
  Assert.assertTrue(property1.isPrimitiveType);
@@ -161,10 +161,10 @@ public void simpleModelTest() {
 
  final CodegenProperty property3 = cm.vars.get(2);
  Assert.assertEquals(property3.baseName, "createdAt");
- Assert.assertEquals(property3.dataType, "DateTime?");
+ Assert.assertEquals(property3.dataType, "DateTime");
  Assert.assertEquals(property3.name, "CreatedAt");
  Assert.assertNull(property3.defaultValue);
- Assert.assertEquals(property3.baseType, "DateTime?");
+ Assert.assertEquals(property3.baseType, "DateTime");
  Assert.assertFalse(property3.hasMore);
  Assert.assertFalse(property3.required);
  }
@@ -209,7 +209,59 @@ public void nonNullablePropertyTest() {
  Assert.assertFalse(property2.required);
  Assert.assertTrue(property2.isPrimitiveType);
  Assert.assertTrue(property2.isContainer);
-
+
+ final CodegenProperty property3 = cm.vars.get(2);
+ Assert.assertEquals(property3.baseName, "name");
+ Assert.assertEquals(property3.dataType, "string");
+ Assert.assertEquals(property3.name, "Name");
+ Assert.assertNull(property3.defaultValue);
+ Assert.assertEquals(property3.baseType, "string");
+ Assert.assertFalse(property3.hasMore);
+ Assert.assertFalse(property3.required);
+ Assert.assertTrue(property3.isPrimitiveType);
+ }
+
+ @Test(description = "convert a model with a nullable property")
+ public void nullablePropertyTest() {
+ final Schema model = new Schema()
+ .description("a sample model")
+ .addProperties("id", new IntegerSchema().format(SchemaTypeUtil.INTEGER64_FORMAT).nullable(true))
+ .addProperties("urls", new ArraySchema()
+ .items(new StringSchema()))
+ .addProperties("name", new StringSchema().nullable(true))
+ .addRequiredItem("id");
+ final DefaultCodegen codegen = new CSharpClientCodegen();
+ OpenAPI openAPI = TestUtils.createOpenAPIWithOneSchema("sample", model);
+ codegen.setOpenAPI(openAPI);
+ final CodegenModel cm = codegen.fromModel("sample", model);
+
+ Assert.assertEquals(cm.name, "sample");
+ Assert.assertEquals(cm.classname, "Sample");
+ Assert.assertEquals(cm.description, "a sample model");
+ Assert.assertEquals(cm.vars.size(), 3);
+
+ final CodegenProperty property1 = cm.vars.get(0);
+ Assert.assertEquals(property1.baseName, "id");
+ Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.name, "Id");
+ Assert.assertNull(property1.defaultValue);
+ Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertTrue(property1.hasMore);
+ Assert.assertTrue(property1.required);
+ Assert.assertTrue(property1.isPrimitiveType);
+
+ final CodegenProperty property2 = cm.vars.get(1);
+ Assert.assertEquals(property2.baseName, "urls");
+ Assert.assertEquals(property2.dataType, "List<string>");
+ Assert.assertEquals(property2.name, "Urls");
+ Assert.assertNull(property2.defaultValue);
+ Assert.assertEquals(property2.baseType, "List");
+ Assert.assertTrue(property2.hasMore);
+ Assert.assertEquals(property2.containerType, "array");
+ Assert.assertFalse(property2.required);
+ Assert.assertTrue(property2.isPrimitiveType);
+ Assert.assertTrue(property2.isContainer);
+
  final CodegenProperty property3 = cm.vars.get(2);
  Assert.assertEquals(property3.baseName, "name");
  Assert.assertEquals(property3.dataType, "string");
@@ -241,10 +293,10 @@ public void listPropertyTest() {
 
  final CodegenProperty property1 = cm.vars.get(0);
  Assert.assertEquals(property1.baseName, "id");
- Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.dataType, "long");
  Assert.assertEquals(property1.name, "Id");
  Assert.assertNull(property1.defaultValue);
- Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertEquals(property1.baseType, "long");
  Assert.assertTrue(property1.hasMore);
  Assert.assertTrue(property1.required);
  Assert.assertTrue(property1.isPrimitiveType);

samples/client/petstore/csharp-dotnet2/OpenAPIClientTest/Lib/OpenAPIClient/.openapi-generator/VERSION

@@ -1 +1 @@
-4.0.0-SNAPSHOT
+4.1.0-SNAPSHOT

samples/client/petstore/csharp-dotnet2/OpenAPIClientTest/Lib/OpenAPIClient/docs/UserApi.md

@@ -117,7 +117,7 @@ namespace Example
 
 Name | Type | Description | Notes
 ------------- | ------------- | ------------- | -------------
- **body** | [**List<User>**](List.md)| List of user object | 
+ **body** | [**List<User>**](User.md)| List of user object | 
 
 ### Return type
 
@@ -176,7 +176,7 @@ namespace Example
 
 Name | Type | Description | Notes
 ------------- | ------------- | ------------- | -------------
- **body** | [**List<User>**](List.md)| List of user object | 
+ **body** | [**List<User>**](User.md)| List of user object | 
 
 ### Return type
 

samples/client/petstore/csharp-netcore/OpenAPIClient/docs/FormatTest.md

@@ -14,7 +14,7 @@ Name | Type | Description | Notes
 **Binary** | **System.IO.Stream** | | [optional] 
 **Date** | **DateTime** | | 
 **DateTime** | **DateTime** | | [optional] 
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **Password** | **string** | | 
 
 [[Back to Model list]](../README.md#documentation-for-models) [[Back to API list]](../README.md#documentation-for-api-endpoints) [[Back to README]](../README.md)

samples/client/petstore/csharp-netcore/OpenAPIClient/docs/MixedPropertiesAndAdditionalPropertiesClass.md

@@ -3,7 +3,7 @@
 
 Name | Type | Description | Notes
 ------------ | ------------- | ------------- | -------------
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **DateTime** | **DateTime** | | [optional] 
 **Map** | [**Dictionary<string, Animal>**](Animal.md) | | [optional] 
 

samples/client/petstore/csharp-netcore/OpenAPIClientCore/docs/FormatTest.md

@@ -14,7 +14,7 @@ Name | Type | Description | Notes
 **Binary** | **System.IO.Stream** | | [optional] 
 **Date** | **DateTime** | | 
 **DateTime** | **DateTime** | | [optional] 
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **Password** | **string** | | 
 
 [[Back to Model list]](../README.md#documentation-for-models) [[Back to API list]](../README.md#documentation-for-api-endpoints) [[Back to README]](../README.md)

samples/client/petstore/csharp-netcore/OpenAPIClientCore/docs/MixedPropertiesAndAdditionalPropertiesClass.md

@@ -3,7 +3,7 @@
 
 Name | Type | Description | Notes
 ------------ | ------------- | ------------- | -------------
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **DateTime** | **DateTime** | | [optional] 
 **Map** | [**Dictionary<string, Animal>**](Animal.md) | | [optional] 
 

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AbstractCSharpCodegen.java

@@ -160,6 +160,7 @@ public AbstractCSharpCodegen() {
  "Int64",
  "Float",
  "Guid?",
+ "Guid",
  "System.IO.Stream", // not really a primitive, we include it to avoid model import
  "Object")
  );

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AspNetCoreServerCodegen.java

@@ -22,6 +22,7 @@
 import io.swagger.v3.oas.models.media.Schema;
 import io.swagger.v3.parser.util.SchemaTypeUtil;
 import org.openapitools.codegen.*;
+import org.openapitools.codegen.utils.ModelUtils;
 import org.openapitools.codegen.utils.URLPathUtils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -353,7 +354,7 @@ public void processOpts() {
  supportingFiles.add(new SupportingFile("Filters" + File.separator + "GeneratePathParamsValidationFilter.mustache",
  packageFolder + File.separator + "Filters", "GeneratePathParamsValidationFilter.cs"));
  }
-
+
  supportingFiles.add(new SupportingFile("Authentication" + File.separator + "ApiAuthentication.mustache",packageFolder + File.separator + "Authentication", "ApiAuthentication.cs"));
  }
 
@@ -407,12 +408,12 @@ public String toRegularExpression(String pattern) {
 
  @Override
  public String getNullableType(Schema p, String type) {
-boolean isNullableExpected = p.getNullable() == null || (p.getNullable() != null && p.getNullable());
-
-if (isNullableExpected && languageSpecificPrimitives.contains(type + "?")) {
-return type + "?";
-} else if (languageSpecificPrimitives.contains(type)) {
-return type;
+if (languageSpecificPrimitives.contains(type)) {
+ if (isSupportNullable() && ModelUtils.isNullable(p) && nullableType.contains(type)) {
+ return type + "?";
+} else {
+ return type;
+}
  } else {
  return null;
  }

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/CSharpClientCodegen.java

@@ -879,15 +879,15 @@ public String toInstantiationType(Schema schema) {
  return null;
  }
  }
-
+
  @Override
  public String getNullableType(Schema p, String type) {
-boolean isNullableExpected = p.getNullable() == null || (p.getNullable() != null && p.getNullable());
-
-if (isNullableExpected && languageSpecificPrimitives.contains(type + "?")) {
-return type + "?";
-} else if (languageSpecificPrimitives.contains(type)) {
-return type;
+if (languageSpecificPrimitives.contains(type)) {
+ if (isSupportNullable() && ModelUtils.isNullable(p) && nullableType.contains(type)) {
+ return type + "?";
+} else {
+ return type;
+}
  } else {
  return null;
  }

modules/openapi-generator/src/main/resources/csharp/api.mustache

@@ -32,7 +32,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>{{#returnType}}{{returnType}}{{/returnType}}</returns>
- {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
 
  /// <summary>
  /// {{summary}}
@@ -43,7 +43,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>ApiResponse of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}Object(void){{/returnType}}</returns>
- ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  {{/operation}}
  #endregion Synchronous Operations
  {{#supportsAsync}}
@@ -58,7 +58,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}void{{/returnType}}</returns>
- {{#returnType}}System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ {{#returnType}}System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
 
  /// <summary>
  /// {{summary}}
@@ -69,7 +69,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of ApiResponse{{#returnType}} ({{returnType}}){{/returnType}}</returns>
- System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  {{/operation}}
  #endregion Asynchronous Operations
  {{/supportsAsync}}
@@ -190,7 +190,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>{{#returnType}}{{returnType}}{{/returnType}}</returns>
- public {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#returnType}}ApiResponse<{{{returnType}}}> localVarResponse = {{operationId}}WithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  return localVarResponse.Data;{{/returnType}}{{^returnType}}{{operationId}}WithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});{{/returnType}}
@@ -202,7 +202,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>ApiResponse of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}Object(void){{/returnType}}</returns>
- public ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#allParams}}
  {{#required}}
@@ -325,7 +325,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}void{{/returnType}}</returns>
- {{#returnType}}public async System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}public async System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ {{#returnType}}public async System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}public async System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#returnType}}ApiResponse<{{{returnType}}}> localVarResponse = await {{operationId}}AsyncWithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  return localVarResponse.Data;{{/returnType}}{{^returnType}}await {{operationId}}AsyncWithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});{{/returnType}}
@@ -338,7 +338,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of ApiResponse{{#returnType}} ({{returnType}}){{/returnType}}</returns>
- public async System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public async System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#allParams}}
  {{#required}}

modules/openapi-generator/src/test/java/org/openapitools/codegen/csharp/CSharpModelTest.java

@@ -141,10 +141,10 @@ public void simpleModelTest() {
 
  final CodegenProperty property1 = cm.vars.get(0);
  Assert.assertEquals(property1.baseName, "id");
- Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.dataType, "long");
  Assert.assertEquals(property1.name, "Id");
  Assert.assertNull(property1.defaultValue);
- Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertEquals(property1.baseType, "long");
  Assert.assertTrue(property1.hasMore);
  Assert.assertTrue(property1.required);
  Assert.assertTrue(property1.isPrimitiveType);
@@ -161,10 +161,10 @@ public void simpleModelTest() {
 
  final CodegenProperty property3 = cm.vars.get(2);
  Assert.assertEquals(property3.baseName, "createdAt");
- Assert.assertEquals(property3.dataType, "DateTime?");
+ Assert.assertEquals(property3.dataType, "DateTime");
  Assert.assertEquals(property3.name, "CreatedAt");
  Assert.assertNull(property3.defaultValue);
- Assert.assertEquals(property3.baseType, "DateTime?");
+ Assert.assertEquals(property3.baseType, "DateTime");
  Assert.assertFalse(property3.hasMore);
  Assert.assertFalse(property3.required);
  }
@@ -209,7 +209,59 @@ public void nonNullablePropertyTest() {
  Assert.assertFalse(property2.required);
  Assert.assertTrue(property2.isPrimitiveType);
  Assert.assertTrue(property2.isContainer);
-
+
+ final CodegenProperty property3 = cm.vars.get(2);
+ Assert.assertEquals(property3.baseName, "name");
+ Assert.assertEquals(property3.dataType, "string");
+ Assert.assertEquals(property3.name, "Name");
+ Assert.assertNull(property3.defaultValue);
+ Assert.assertEquals(property3.baseType, "string");
+ Assert.assertFalse(property3.hasMore);
+ Assert.assertFalse(property3.required);
+ Assert.assertTrue(property3.isPrimitiveType);
+ }
+
+ @Test(description = "convert a model with a nullable property")
+ public void nullablePropertyTest() {
+ final Schema model = new Schema()
+ .description("a sample model")
+ .addProperties("id", new IntegerSchema().format(SchemaTypeUtil.INTEGER64_FORMAT).nullable(true))
+ .addProperties("urls", new ArraySchema()
+ .items(new StringSchema()))
+ .addProperties("name", new StringSchema().nullable(true))
+ .addRequiredItem("id");
+ final DefaultCodegen codegen = new CSharpClientCodegen();
+ OpenAPI openAPI = TestUtils.createOpenAPIWithOneSchema("sample", model);
+ codegen.setOpenAPI(openAPI);
+ final CodegenModel cm = codegen.fromModel("sample", model);
+
+ Assert.assertEquals(cm.name, "sample");
+ Assert.assertEquals(cm.classname, "Sample");
+ Assert.assertEquals(cm.description, "a sample model");
+ Assert.assertEquals(cm.vars.size(), 3);
+
+ final CodegenProperty property1 = cm.vars.get(0);
+ Assert.assertEquals(property1.baseName, "id");
+ Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.name, "Id");
+ Assert.assertNull(property1.defaultValue);
+ Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertTrue(property1.hasMore);
+ Assert.assertTrue(property1.required);
+ Assert.assertTrue(property1.isPrimitiveType);
+
+ final CodegenProperty property2 = cm.vars.get(1);
+ Assert.assertEquals(property2.baseName, "urls");
+ Assert.assertEquals(property2.dataType, "List<string>");
+ Assert.assertEquals(property2.name, "Urls");
+ Assert.assertNull(property2.defaultValue);
+ Assert.assertEquals(property2.baseType, "List");
+ Assert.assertTrue(property2.hasMore);
+ Assert.assertEquals(property2.containerType, "array");
+ Assert.assertFalse(property2.required);
+ Assert.assertTrue(property2.isPrimitiveType);
+ Assert.assertTrue(property2.isContainer);
+
  final CodegenProperty property3 = cm.vars.get(2);
  Assert.assertEquals(property3.baseName, "name");
  Assert.assertEquals(property3.dataType, "string");
@@ -241,10 +293,10 @@ public void listPropertyTest() {
 
  final CodegenProperty property1 = cm.vars.get(0);
  Assert.assertEquals(property1.baseName, "id");
- Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.dataType, "long");
  Assert.assertEquals(property1.name, "Id");
  Assert.assertNull(property1.defaultValue);
- Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertEquals(property1.baseType, "long");
  Assert.assertTrue(property1.hasMore);
  Assert.assertTrue(property1.required);
  Assert.assertTrue(property1.isPrimitiveType);

samples/client/petstore/csharp-dotnet2/OpenAPIClientTest/Lib/OpenAPIClient/.openapi-generator/VERSION

@@ -1 +1 @@
-4.0.0-SNAPSHOT
+4.1.0-SNAPSHOT

samples/client/petstore/csharp-dotnet2/OpenAPIClientTest/Lib/OpenAPIClient/docs/UserApi.md

@@ -117,7 +117,7 @@ namespace Example
 
 Name | Type | Description | Notes
 ------------- | ------------- | ------------- | -------------
- **body** | [**List<User>**](List.md)| List of user object | 
+ **body** | [**List<User>**](User.md)| List of user object | 
 
 ### Return type
 
@@ -176,7 +176,7 @@ namespace Example
 
 Name | Type | Description | Notes
 ------------- | ------------- | ------------- | -------------
- **body** | [**List<User>**](List.md)| List of user object | 
+ **body** | [**List<User>**](User.md)| List of user object | 
 
 ### Return type
 

samples/client/petstore/csharp-netcore/OpenAPIClient/docs/FormatTest.md

@@ -14,7 +14,7 @@ Name | Type | Description | Notes
 **Binary** | **System.IO.Stream** | | [optional] 
 **Date** | **DateTime** | | 
 **DateTime** | **DateTime** | | [optional] 
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **Password** | **string** | | 
 
 [[Back to Model list]](../README.md#documentation-for-models) [[Back to API list]](../README.md#documentation-for-api-endpoints) [[Back to README]](../README.md)

samples/client/petstore/csharp-netcore/OpenAPIClient/docs/MixedPropertiesAndAdditionalPropertiesClass.md

@@ -3,7 +3,7 @@
 
 Name | Type | Description | Notes
 ------------ | ------------- | ------------- | -------------
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **DateTime** | **DateTime** | | [optional] 
 **Map** | [**Dictionary<string, Animal>**](Animal.md) | | [optional] 
 

samples/client/petstore/csharp-netcore/OpenAPIClientCore/docs/FormatTest.md

@@ -14,7 +14,7 @@ Name | Type | Description | Notes
 **Binary** | **System.IO.Stream** | | [optional] 
 **Date** | **DateTime** | | 
 **DateTime** | **DateTime** | | [optional] 
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **Password** | **string** | | 
 
 [[Back to Model list]](../README.md#documentation-for-models) [[Back to API list]](../README.md#documentation-for-api-endpoints) [[Back to README]](../README.md)

samples/client/petstore/csharp-netcore/OpenAPIClientCore/docs/MixedPropertiesAndAdditionalPropertiesClass.md

@@ -3,7 +3,7 @@
 
 Name | Type | Description | Notes
 ------------ | ------------- | ------------- | -------------
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **DateTime** | **DateTime** | | [optional] 
 **Map** | [**Dictionary<string, Animal>**](Animal.md) | | [optional] 
 

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AbstractCSharpCodegen.java

@@ -160,6 +160,7 @@ public AbstractCSharpCodegen() {
  "Int64",
  "Float",
  "Guid?",
+ "Guid",
  "System.IO.Stream", // not really a primitive, we include it to avoid model import
  "Object")
  );

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AspNetCoreServerCodegen.java

@@ -22,6 +22,7 @@
 import io.swagger.v3.oas.models.media.Schema;
 import io.swagger.v3.parser.util.SchemaTypeUtil;
 import org.openapitools.codegen.*;
+import org.openapitools.codegen.utils.ModelUtils;
 import org.openapitools.codegen.utils.URLPathUtils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -353,7 +354,7 @@ public void processOpts() {
  supportingFiles.add(new SupportingFile("Filters" + File.separator + "GeneratePathParamsValidationFilter.mustache",
  packageFolder + File.separator + "Filters", "GeneratePathParamsValidationFilter.cs"));
  }
-
+
  supportingFiles.add(new SupportingFile("Authentication" + File.separator + "ApiAuthentication.mustache",packageFolder + File.separator + "Authentication", "ApiAuthentication.cs"));
  }
 
@@ -407,12 +408,12 @@ public String toRegularExpression(String pattern) {
 
  @Override
  public String getNullableType(Schema p, String type) {
-boolean isNullableExpected = p.getNullable() == null || (p.getNullable() != null && p.getNullable());
-
-if (isNullableExpected && languageSpecificPrimitives.contains(type + "?")) {
-return type + "?";
-} else if (languageSpecificPrimitives.contains(type)) {
-return type;
+if (languageSpecificPrimitives.contains(type)) {
+ if (isSupportNullable() && ModelUtils.isNullable(p) && nullableType.contains(type)) {
+ return type + "?";
+} else {
+ return type;
+}
  } else {
  return null;
  }

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/CSharpClientCodegen.java

@@ -879,15 +879,15 @@ public String toInstantiationType(Schema schema) {
  return null;
  }
  }
-
+
  @Override
  public String getNullableType(Schema p, String type) {
-boolean isNullableExpected = p.getNullable() == null || (p.getNullable() != null && p.getNullable());
-
-if (isNullableExpected && languageSpecificPrimitives.contains(type + "?")) {
-return type + "?";
-} else if (languageSpecificPrimitives.contains(type)) {
-return type;
+if (languageSpecificPrimitives.contains(type)) {
+ if (isSupportNullable() && ModelUtils.isNullable(p) && nullableType.contains(type)) {
+ return type + "?";
+} else {
+ return type;
+}
  } else {
  return null;
  }

modules/openapi-generator/src/main/resources/csharp/api.mustache

@@ -32,7 +32,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>{{#returnType}}{{returnType}}{{/returnType}}</returns>
- {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
 
  /// <summary>
  /// {{summary}}
@@ -43,7 +43,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>ApiResponse of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}Object(void){{/returnType}}</returns>
- ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  {{/operation}}
  #endregion Synchronous Operations
  {{#supportsAsync}}
@@ -58,7 +58,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}void{{/returnType}}</returns>
- {{#returnType}}System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ {{#returnType}}System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
 
  /// <summary>
  /// {{summary}}
@@ -69,7 +69,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of ApiResponse{{#returnType}} ({{returnType}}){{/returnType}}</returns>
- System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  {{/operation}}
  #endregion Asynchronous Operations
  {{/supportsAsync}}
@@ -190,7 +190,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>{{#returnType}}{{returnType}}{{/returnType}}</returns>
- public {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#returnType}}ApiResponse<{{{returnType}}}> localVarResponse = {{operationId}}WithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  return localVarResponse.Data;{{/returnType}}{{^returnType}}{{operationId}}WithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});{{/returnType}}
@@ -202,7 +202,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>ApiResponse of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}Object(void){{/returnType}}</returns>
- public ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#allParams}}
  {{#required}}
@@ -325,7 +325,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}void{{/returnType}}</returns>
- {{#returnType}}public async System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}public async System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ {{#returnType}}public async System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}public async System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#returnType}}ApiResponse<{{{returnType}}}> localVarResponse = await {{operationId}}AsyncWithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  return localVarResponse.Data;{{/returnType}}{{^returnType}}await {{operationId}}AsyncWithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});{{/returnType}}
@@ -338,7 +338,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of ApiResponse{{#returnType}} ({{returnType}}){{/returnType}}</returns>
- public async System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public async System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#allParams}}
  {{#required}}

modules/openapi-generator/src/test/java/org/openapitools/codegen/csharp/CSharpModelTest.java

@@ -141,10 +141,10 @@ public void simpleModelTest() {
 
  final CodegenProperty property1 = cm.vars.get(0);
  Assert.assertEquals(property1.baseName, "id");
- Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.dataType, "long");
  Assert.assertEquals(property1.name, "Id");
  Assert.assertNull(property1.defaultValue);
- Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertEquals(property1.baseType, "long");
  Assert.assertTrue(property1.hasMore);
  Assert.assertTrue(property1.required);
  Assert.assertTrue(property1.isPrimitiveType);
@@ -161,10 +161,10 @@ public void simpleModelTest() {
 
  final CodegenProperty property3 = cm.vars.get(2);
  Assert.assertEquals(property3.baseName, "createdAt");
- Assert.assertEquals(property3.dataType, "DateTime?");
+ Assert.assertEquals(property3.dataType, "DateTime");
  Assert.assertEquals(property3.name, "CreatedAt");
  Assert.assertNull(property3.defaultValue);
- Assert.assertEquals(property3.baseType, "DateTime?");
+ Assert.assertEquals(property3.baseType, "DateTime");
  Assert.assertFalse(property3.hasMore);
  Assert.assertFalse(property3.required);
  }
@@ -209,7 +209,59 @@ public void nonNullablePropertyTest() {
  Assert.assertFalse(property2.required);
  Assert.assertTrue(property2.isPrimitiveType);
  Assert.assertTrue(property2.isContainer);
-
+
+ final CodegenProperty property3 = cm.vars.get(2);
+ Assert.assertEquals(property3.baseName, "name");
+ Assert.assertEquals(property3.dataType, "string");
+ Assert.assertEquals(property3.name, "Name");
+ Assert.assertNull(property3.defaultValue);
+ Assert.assertEquals(property3.baseType, "string");
+ Assert.assertFalse(property3.hasMore);
+ Assert.assertFalse(property3.required);
+ Assert.assertTrue(property3.isPrimitiveType);
+ }
+
+ @Test(description = "convert a model with a nullable property")
+ public void nullablePropertyTest() {
+ final Schema model = new Schema()
+ .description("a sample model")
+ .addProperties("id", new IntegerSchema().format(SchemaTypeUtil.INTEGER64_FORMAT).nullable(true))
+ .addProperties("urls", new ArraySchema()
+ .items(new StringSchema()))
+ .addProperties("name", new StringSchema().nullable(true))
+ .addRequiredItem("id");
+ final DefaultCodegen codegen = new CSharpClientCodegen();
+ OpenAPI openAPI = TestUtils.createOpenAPIWithOneSchema("sample", model);
+ codegen.setOpenAPI(openAPI);
+ final CodegenModel cm = codegen.fromModel("sample", model);
+
+ Assert.assertEquals(cm.name, "sample");
+ Assert.assertEquals(cm.classname, "Sample");
+ Assert.assertEquals(cm.description, "a sample model");
+ Assert.assertEquals(cm.vars.size(), 3);
+
+ final CodegenProperty property1 = cm.vars.get(0);
+ Assert.assertEquals(property1.baseName, "id");
+ Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.name, "Id");
+ Assert.assertNull(property1.defaultValue);
+ Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertTrue(property1.hasMore);
+ Assert.assertTrue(property1.required);
+ Assert.assertTrue(property1.isPrimitiveType);
+
+ final CodegenProperty property2 = cm.vars.get(1);
+ Assert.assertEquals(property2.baseName, "urls");
+ Assert.assertEquals(property2.dataType, "List<string>");
+ Assert.assertEquals(property2.name, "Urls");
+ Assert.assertNull(property2.defaultValue);
+ Assert.assertEquals(property2.baseType, "List");
+ Assert.assertTrue(property2.hasMore);
+ Assert.assertEquals(property2.containerType, "array");
+ Assert.assertFalse(property2.required);
+ Assert.assertTrue(property2.isPrimitiveType);
+ Assert.assertTrue(property2.isContainer);
+
  final CodegenProperty property3 = cm.vars.get(2);
  Assert.assertEquals(property3.baseName, "name");
  Assert.assertEquals(property3.dataType, "string");
@@ -241,10 +293,10 @@ public void listPropertyTest() {
 
  final CodegenProperty property1 = cm.vars.get(0);
  Assert.assertEquals(property1.baseName, "id");
- Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.dataType, "long");
  Assert.assertEquals(property1.name, "Id");
  Assert.assertNull(property1.defaultValue);
- Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertEquals(property1.baseType, "long");
  Assert.assertTrue(property1.hasMore);
  Assert.assertTrue(property1.required);
  Assert.assertTrue(property1.isPrimitiveType);

samples/client/petstore/csharp-dotnet2/OpenAPIClientTest/Lib/OpenAPIClient/.openapi-generator/VERSION

@@ -1 +1 @@
-4.0.0-SNAPSHOT
+4.1.0-SNAPSHOT

samples/client/petstore/csharp-dotnet2/OpenAPIClientTest/Lib/OpenAPIClient/docs/UserApi.md

@@ -117,7 +117,7 @@ namespace Example
 
 Name | Type | Description | Notes
 ------------- | ------------- | ------------- | -------------
- **body** | [**List<User>**](List.md)| List of user object | 
+ **body** | [**List<User>**](User.md)| List of user object | 
 
 ### Return type
 
@@ -176,7 +176,7 @@ namespace Example
 
 Name | Type | Description | Notes
 ------------- | ------------- | ------------- | -------------
- **body** | [**List<User>**](List.md)| List of user object | 
+ **body** | [**List<User>**](User.md)| List of user object | 
 
 ### Return type
 

samples/client/petstore/csharp-netcore/OpenAPIClient/docs/FormatTest.md

@@ -14,7 +14,7 @@ Name | Type | Description | Notes
 **Binary** | **System.IO.Stream** | | [optional] 
 **Date** | **DateTime** | | 
 **DateTime** | **DateTime** | | [optional] 
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **Password** | **string** | | 
 
 [[Back to Model list]](../README.md#documentation-for-models) [[Back to API list]](../README.md#documentation-for-api-endpoints) [[Back to README]](../README.md)

samples/client/petstore/csharp-netcore/OpenAPIClient/docs/MixedPropertiesAndAdditionalPropertiesClass.md

@@ -3,7 +3,7 @@
 
 Name | Type | Description | Notes
 ------------ | ------------- | ------------- | -------------
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **DateTime** | **DateTime** | | [optional] 
 **Map** | [**Dictionary<string, Animal>**](Animal.md) | | [optional] 
 

samples/client/petstore/csharp-netcore/OpenAPIClientCore/docs/FormatTest.md

@@ -14,7 +14,7 @@ Name | Type | Description | Notes
 **Binary** | **System.IO.Stream** | | [optional] 
 **Date** | **DateTime** | | 
 **DateTime** | **DateTime** | | [optional] 
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **Password** | **string** | | 
 
 [[Back to Model list]](../README.md#documentation-for-models) [[Back to API list]](../README.md#documentation-for-api-endpoints) [[Back to README]](../README.md)

samples/client/petstore/csharp-netcore/OpenAPIClientCore/docs/MixedPropertiesAndAdditionalPropertiesClass.md

@@ -3,7 +3,7 @@
 
 Name | Type | Description | Notes
 ------------ | ------------- | ------------- | -------------
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **DateTime** | **DateTime** | | [optional] 
 **Map** | [**Dictionary<string, Animal>**](Animal.md) | | [optional] 
 

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AbstractCSharpCodegen.java

@@ -160,6 +160,7 @@ public AbstractCSharpCodegen() {
  "Int64",
  "Float",
  "Guid?",
+ "Guid",
  "System.IO.Stream", // not really a primitive, we include it to avoid model import
  "Object")
  );

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AspNetCoreServerCodegen.java

@@ -22,6 +22,7 @@
 import io.swagger.v3.oas.models.media.Schema;
 import io.swagger.v3.parser.util.SchemaTypeUtil;
 import org.openapitools.codegen.*;
+import org.openapitools.codegen.utils.ModelUtils;
 import org.openapitools.codegen.utils.URLPathUtils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -353,7 +354,7 @@ public void processOpts() {
  supportingFiles.add(new SupportingFile("Filters" + File.separator + "GeneratePathParamsValidationFilter.mustache",
  packageFolder + File.separator + "Filters", "GeneratePathParamsValidationFilter.cs"));
  }
-
+
  supportingFiles.add(new SupportingFile("Authentication" + File.separator + "ApiAuthentication.mustache",packageFolder + File.separator + "Authentication", "ApiAuthentication.cs"));
  }
 
@@ -407,12 +408,12 @@ public String toRegularExpression(String pattern) {
 
  @Override
  public String getNullableType(Schema p, String type) {
-boolean isNullableExpected = p.getNullable() == null || (p.getNullable() != null && p.getNullable());
-
-if (isNullableExpected && languageSpecificPrimitives.contains(type + "?")) {
-return type + "?";
-} else if (languageSpecificPrimitives.contains(type)) {
-return type;
+if (languageSpecificPrimitives.contains(type)) {
+ if (isSupportNullable() && ModelUtils.isNullable(p) && nullableType.contains(type)) {
+ return type + "?";
+} else {
+ return type;
+}
  } else {
  return null;
  }

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/CSharpClientCodegen.java

@@ -879,15 +879,15 @@ public String toInstantiationType(Schema schema) {
  return null;
  }
  }
-
+
  @Override
  public String getNullableType(Schema p, String type) {
-boolean isNullableExpected = p.getNullable() == null || (p.getNullable() != null && p.getNullable());
-
-if (isNullableExpected && languageSpecificPrimitives.contains(type + "?")) {
-return type + "?";
-} else if (languageSpecificPrimitives.contains(type)) {
-return type;
+if (languageSpecificPrimitives.contains(type)) {
+ if (isSupportNullable() && ModelUtils.isNullable(p) && nullableType.contains(type)) {
+ return type + "?";
+} else {
+ return type;
+}
  } else {
  return null;
  }

modules/openapi-generator/src/main/resources/csharp/api.mustache

@@ -32,7 +32,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>{{#returnType}}{{returnType}}{{/returnType}}</returns>
- {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
 
  /// <summary>
  /// {{summary}}
@@ -43,7 +43,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>ApiResponse of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}Object(void){{/returnType}}</returns>
- ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  {{/operation}}
  #endregion Synchronous Operations
  {{#supportsAsync}}
@@ -58,7 +58,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}void{{/returnType}}</returns>
- {{#returnType}}System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ {{#returnType}}System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
 
  /// <summary>
  /// {{summary}}
@@ -69,7 +69,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of ApiResponse{{#returnType}} ({{returnType}}){{/returnType}}</returns>
- System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  {{/operation}}
  #endregion Asynchronous Operations
  {{/supportsAsync}}
@@ -190,7 +190,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>{{#returnType}}{{returnType}}{{/returnType}}</returns>
- public {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#returnType}}ApiResponse<{{{returnType}}}> localVarResponse = {{operationId}}WithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  return localVarResponse.Data;{{/returnType}}{{^returnType}}{{operationId}}WithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});{{/returnType}}
@@ -202,7 +202,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>ApiResponse of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}Object(void){{/returnType}}</returns>
- public ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#allParams}}
  {{#required}}
@@ -325,7 +325,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}void{{/returnType}}</returns>
- {{#returnType}}public async System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}public async System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ {{#returnType}}public async System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}public async System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#returnType}}ApiResponse<{{{returnType}}}> localVarResponse = await {{operationId}}AsyncWithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  return localVarResponse.Data;{{/returnType}}{{^returnType}}await {{operationId}}AsyncWithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});{{/returnType}}
@@ -338,7 +338,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of ApiResponse{{#returnType}} ({{returnType}}){{/returnType}}</returns>
- public async System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public async System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#allParams}}
  {{#required}}

modules/openapi-generator/src/test/java/org/openapitools/codegen/csharp/CSharpModelTest.java

@@ -141,10 +141,10 @@ public void simpleModelTest() {
 
  final CodegenProperty property1 = cm.vars.get(0);
  Assert.assertEquals(property1.baseName, "id");
- Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.dataType, "long");
  Assert.assertEquals(property1.name, "Id");
  Assert.assertNull(property1.defaultValue);
- Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertEquals(property1.baseType, "long");
  Assert.assertTrue(property1.hasMore);
  Assert.assertTrue(property1.required);
  Assert.assertTrue(property1.isPrimitiveType);
@@ -161,10 +161,10 @@ public void simpleModelTest() {
 
  final CodegenProperty property3 = cm.vars.get(2);
  Assert.assertEquals(property3.baseName, "createdAt");
- Assert.assertEquals(property3.dataType, "DateTime?");
+ Assert.assertEquals(property3.dataType, "DateTime");
  Assert.assertEquals(property3.name, "CreatedAt");
  Assert.assertNull(property3.defaultValue);
- Assert.assertEquals(property3.baseType, "DateTime?");
+ Assert.assertEquals(property3.baseType, "DateTime");
  Assert.assertFalse(property3.hasMore);
  Assert.assertFalse(property3.required);
  }
@@ -209,7 +209,59 @@ public void nonNullablePropertyTest() {
  Assert.assertFalse(property2.required);
  Assert.assertTrue(property2.isPrimitiveType);
  Assert.assertTrue(property2.isContainer);
-
+
+ final CodegenProperty property3 = cm.vars.get(2);
+ Assert.assertEquals(property3.baseName, "name");
+ Assert.assertEquals(property3.dataType, "string");
+ Assert.assertEquals(property3.name, "Name");
+ Assert.assertNull(property3.defaultValue);
+ Assert.assertEquals(property3.baseType, "string");
+ Assert.assertFalse(property3.hasMore);
+ Assert.assertFalse(property3.required);
+ Assert.assertTrue(property3.isPrimitiveType);
+ }
+
+ @Test(description = "convert a model with a nullable property")
+ public void nullablePropertyTest() {
+ final Schema model = new Schema()
+ .description("a sample model")
+ .addProperties("id", new IntegerSchema().format(SchemaTypeUtil.INTEGER64_FORMAT).nullable(true))
+ .addProperties("urls", new ArraySchema()
+ .items(new StringSchema()))
+ .addProperties("name", new StringSchema().nullable(true))
+ .addRequiredItem("id");
+ final DefaultCodegen codegen = new CSharpClientCodegen();
+ OpenAPI openAPI = TestUtils.createOpenAPIWithOneSchema("sample", model);
+ codegen.setOpenAPI(openAPI);
+ final CodegenModel cm = codegen.fromModel("sample", model);
+
+ Assert.assertEquals(cm.name, "sample");
+ Assert.assertEquals(cm.classname, "Sample");
+ Assert.assertEquals(cm.description, "a sample model");
+ Assert.assertEquals(cm.vars.size(), 3);
+
+ final CodegenProperty property1 = cm.vars.get(0);
+ Assert.assertEquals(property1.baseName, "id");
+ Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.name, "Id");
+ Assert.assertNull(property1.defaultValue);
+ Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertTrue(property1.hasMore);
+ Assert.assertTrue(property1.required);
+ Assert.assertTrue(property1.isPrimitiveType);
+
+ final CodegenProperty property2 = cm.vars.get(1);
+ Assert.assertEquals(property2.baseName, "urls");
+ Assert.assertEquals(property2.dataType, "List<string>");
+ Assert.assertEquals(property2.name, "Urls");
+ Assert.assertNull(property2.defaultValue);
+ Assert.assertEquals(property2.baseType, "List");
+ Assert.assertTrue(property2.hasMore);
+ Assert.assertEquals(property2.containerType, "array");
+ Assert.assertFalse(property2.required);
+ Assert.assertTrue(property2.isPrimitiveType);
+ Assert.assertTrue(property2.isContainer);
+
  final CodegenProperty property3 = cm.vars.get(2);
  Assert.assertEquals(property3.baseName, "name");
  Assert.assertEquals(property3.dataType, "string");
@@ -241,10 +293,10 @@ public void listPropertyTest() {
 
  final CodegenProperty property1 = cm.vars.get(0);
  Assert.assertEquals(property1.baseName, "id");
- Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.dataType, "long");
  Assert.assertEquals(property1.name, "Id");
  Assert.assertNull(property1.defaultValue);
- Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertEquals(property1.baseType, "long");
  Assert.assertTrue(property1.hasMore);
  Assert.assertTrue(property1.required);
  Assert.assertTrue(property1.isPrimitiveType);

samples/client/petstore/csharp-dotnet2/OpenAPIClientTest/Lib/OpenAPIClient/.openapi-generator/VERSION

@@ -1 +1 @@
-4.0.0-SNAPSHOT
+4.1.0-SNAPSHOT

samples/client/petstore/csharp-dotnet2/OpenAPIClientTest/Lib/OpenAPIClient/docs/UserApi.md

@@ -117,7 +117,7 @@ namespace Example
 
 Name | Type | Description | Notes
 ------------- | ------------- | ------------- | -------------
- **body** | [**List<User>**](List.md)| List of user object | 
+ **body** | [**List<User>**](User.md)| List of user object | 
 
 ### Return type
 
@@ -176,7 +176,7 @@ namespace Example
 
 Name | Type | Description | Notes
 ------------- | ------------- | ------------- | -------------
- **body** | [**List<User>**](List.md)| List of user object | 
+ **body** | [**List<User>**](User.md)| List of user object | 
 
 ### Return type
 

samples/client/petstore/csharp-netcore/OpenAPIClient/docs/FormatTest.md

@@ -14,7 +14,7 @@ Name | Type | Description | Notes
 **Binary** | **System.IO.Stream** | | [optional] 
 **Date** | **DateTime** | | 
 **DateTime** | **DateTime** | | [optional] 
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **Password** | **string** | | 
 
 [[Back to Model list]](../README.md#documentation-for-models) [[Back to API list]](../README.md#documentation-for-api-endpoints) [[Back to README]](../README.md)

samples/client/petstore/csharp-netcore/OpenAPIClient/docs/MixedPropertiesAndAdditionalPropertiesClass.md

@@ -3,7 +3,7 @@
 
 Name | Type | Description | Notes
 ------------ | ------------- | ------------- | -------------
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **DateTime** | **DateTime** | | [optional] 
 **Map** | [**Dictionary<string, Animal>**](Animal.md) | | [optional] 
 

samples/client/petstore/csharp-netcore/OpenAPIClientCore/docs/FormatTest.md

@@ -14,7 +14,7 @@ Name | Type | Description | Notes
 **Binary** | **System.IO.Stream** | | [optional] 
 **Date** | **DateTime** | | 
 **DateTime** | **DateTime** | | [optional] 
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **Password** | **string** | | 
 
 [[Back to Model list]](../README.md#documentation-for-models) [[Back to API list]](../README.md#documentation-for-api-endpoints) [[Back to README]](../README.md)

samples/client/petstore/csharp-netcore/OpenAPIClientCore/docs/MixedPropertiesAndAdditionalPropertiesClass.md

@@ -3,7 +3,7 @@
 
 Name | Type | Description | Notes
 ------------ | ------------- | ------------- | -------------
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **DateTime** | **DateTime** | | [optional] 
 **Map** | [**Dictionary<string, Animal>**](Animal.md) | | [optional] 
 

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AbstractCSharpCodegen.java

@@ -160,6 +160,7 @@ public AbstractCSharpCodegen() {
  "Int64",
  "Float",
  "Guid?",
+ "Guid",
  "System.IO.Stream", // not really a primitive, we include it to avoid model import
  "Object")
  );

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AspNetCoreServerCodegen.java

@@ -22,6 +22,7 @@
 import io.swagger.v3.oas.models.media.Schema;
 import io.swagger.v3.parser.util.SchemaTypeUtil;
 import org.openapitools.codegen.*;
+import org.openapitools.codegen.utils.ModelUtils;
 import org.openapitools.codegen.utils.URLPathUtils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -353,7 +354,7 @@ public void processOpts() {
  supportingFiles.add(new SupportingFile("Filters" + File.separator + "GeneratePathParamsValidationFilter.mustache",
  packageFolder + File.separator + "Filters", "GeneratePathParamsValidationFilter.cs"));
  }
-
+
  supportingFiles.add(new SupportingFile("Authentication" + File.separator + "ApiAuthentication.mustache",packageFolder + File.separator + "Authentication", "ApiAuthentication.cs"));
  }
 
@@ -407,12 +408,12 @@ public String toRegularExpression(String pattern) {
 
  @Override
  public String getNullableType(Schema p, String type) {
-boolean isNullableExpected = p.getNullable() == null || (p.getNullable() != null && p.getNullable());
-
-if (isNullableExpected && languageSpecificPrimitives.contains(type + "?")) {
-return type + "?";
-} else if (languageSpecificPrimitives.contains(type)) {
-return type;
+if (languageSpecificPrimitives.contains(type)) {
+ if (isSupportNullable() && ModelUtils.isNullable(p) && nullableType.contains(type)) {
+ return type + "?";
+} else {
+ return type;
+}
  } else {
  return null;
  }

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/CSharpClientCodegen.java

@@ -879,15 +879,15 @@ public String toInstantiationType(Schema schema) {
  return null;
  }
  }
-
+
  @Override
  public String getNullableType(Schema p, String type) {
-boolean isNullableExpected = p.getNullable() == null || (p.getNullable() != null && p.getNullable());
-
-if (isNullableExpected && languageSpecificPrimitives.contains(type + "?")) {
-return type + "?";
-} else if (languageSpecificPrimitives.contains(type)) {
-return type;
+if (languageSpecificPrimitives.contains(type)) {
+ if (isSupportNullable() && ModelUtils.isNullable(p) && nullableType.contains(type)) {
+ return type + "?";
+} else {
+ return type;
+}
  } else {
  return null;
  }

modules/openapi-generator/src/main/resources/csharp/api.mustache

@@ -32,7 +32,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>{{#returnType}}{{returnType}}{{/returnType}}</returns>
- {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
 
  /// <summary>
  /// {{summary}}
@@ -43,7 +43,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>ApiResponse of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}Object(void){{/returnType}}</returns>
- ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  {{/operation}}
  #endregion Synchronous Operations
  {{#supportsAsync}}
@@ -58,7 +58,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}void{{/returnType}}</returns>
- {{#returnType}}System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ {{#returnType}}System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
 
  /// <summary>
  /// {{summary}}
@@ -69,7 +69,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of ApiResponse{{#returnType}} ({{returnType}}){{/returnType}}</returns>
- System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  {{/operation}}
  #endregion Asynchronous Operations
  {{/supportsAsync}}
@@ -190,7 +190,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>{{#returnType}}{{returnType}}{{/returnType}}</returns>
- public {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#returnType}}ApiResponse<{{{returnType}}}> localVarResponse = {{operationId}}WithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  return localVarResponse.Data;{{/returnType}}{{^returnType}}{{operationId}}WithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});{{/returnType}}
@@ -202,7 +202,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>ApiResponse of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}Object(void){{/returnType}}</returns>
- public ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#allParams}}
  {{#required}}
@@ -325,7 +325,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}void{{/returnType}}</returns>
- {{#returnType}}public async System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}public async System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ {{#returnType}}public async System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}public async System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#returnType}}ApiResponse<{{{returnType}}}> localVarResponse = await {{operationId}}AsyncWithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  return localVarResponse.Data;{{/returnType}}{{^returnType}}await {{operationId}}AsyncWithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});{{/returnType}}
@@ -338,7 +338,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of ApiResponse{{#returnType}} ({{returnType}}){{/returnType}}</returns>
- public async System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public async System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#allParams}}
  {{#required}}

modules/openapi-generator/src/test/java/org/openapitools/codegen/csharp/CSharpModelTest.java

@@ -141,10 +141,10 @@ public void simpleModelTest() {
 
  final CodegenProperty property1 = cm.vars.get(0);
  Assert.assertEquals(property1.baseName, "id");
- Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.dataType, "long");
  Assert.assertEquals(property1.name, "Id");
  Assert.assertNull(property1.defaultValue);
- Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertEquals(property1.baseType, "long");
  Assert.assertTrue(property1.hasMore);
  Assert.assertTrue(property1.required);
  Assert.assertTrue(property1.isPrimitiveType);
@@ -161,10 +161,10 @@ public void simpleModelTest() {
 
  final CodegenProperty property3 = cm.vars.get(2);
  Assert.assertEquals(property3.baseName, "createdAt");
- Assert.assertEquals(property3.dataType, "DateTime?");
+ Assert.assertEquals(property3.dataType, "DateTime");
  Assert.assertEquals(property3.name, "CreatedAt");
  Assert.assertNull(property3.defaultValue);
- Assert.assertEquals(property3.baseType, "DateTime?");
+ Assert.assertEquals(property3.baseType, "DateTime");
  Assert.assertFalse(property3.hasMore);
  Assert.assertFalse(property3.required);
  }
@@ -209,7 +209,59 @@ public void nonNullablePropertyTest() {
  Assert.assertFalse(property2.required);
  Assert.assertTrue(property2.isPrimitiveType);
  Assert.assertTrue(property2.isContainer);
-
+
+ final CodegenProperty property3 = cm.vars.get(2);
+ Assert.assertEquals(property3.baseName, "name");
+ Assert.assertEquals(property3.dataType, "string");
+ Assert.assertEquals(property3.name, "Name");
+ Assert.assertNull(property3.defaultValue);
+ Assert.assertEquals(property3.baseType, "string");
+ Assert.assertFalse(property3.hasMore);
+ Assert.assertFalse(property3.required);
+ Assert.assertTrue(property3.isPrimitiveType);
+ }
+
+ @Test(description = "convert a model with a nullable property")
+ public void nullablePropertyTest() {
+ final Schema model = new Schema()
+ .description("a sample model")
+ .addProperties("id", new IntegerSchema().format(SchemaTypeUtil.INTEGER64_FORMAT).nullable(true))
+ .addProperties("urls", new ArraySchema()
+ .items(new StringSchema()))
+ .addProperties("name", new StringSchema().nullable(true))
+ .addRequiredItem("id");
+ final DefaultCodegen codegen = new CSharpClientCodegen();
+ OpenAPI openAPI = TestUtils.createOpenAPIWithOneSchema("sample", model);
+ codegen.setOpenAPI(openAPI);
+ final CodegenModel cm = codegen.fromModel("sample", model);
+
+ Assert.assertEquals(cm.name, "sample");
+ Assert.assertEquals(cm.classname, "Sample");
+ Assert.assertEquals(cm.description, "a sample model");
+ Assert.assertEquals(cm.vars.size(), 3);
+
+ final CodegenProperty property1 = cm.vars.get(0);
+ Assert.assertEquals(property1.baseName, "id");
+ Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.name, "Id");
+ Assert.assertNull(property1.defaultValue);
+ Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertTrue(property1.hasMore);
+ Assert.assertTrue(property1.required);
+ Assert.assertTrue(property1.isPrimitiveType);
+
+ final CodegenProperty property2 = cm.vars.get(1);
+ Assert.assertEquals(property2.baseName, "urls");
+ Assert.assertEquals(property2.dataType, "List<string>");
+ Assert.assertEquals(property2.name, "Urls");
+ Assert.assertNull(property2.defaultValue);
+ Assert.assertEquals(property2.baseType, "List");
+ Assert.assertTrue(property2.hasMore);
+ Assert.assertEquals(property2.containerType, "array");
+ Assert.assertFalse(property2.required);
+ Assert.assertTrue(property2.isPrimitiveType);
+ Assert.assertTrue(property2.isContainer);
+
  final CodegenProperty property3 = cm.vars.get(2);
  Assert.assertEquals(property3.baseName, "name");
  Assert.assertEquals(property3.dataType, "string");
@@ -241,10 +293,10 @@ public void listPropertyTest() {
 
  final CodegenProperty property1 = cm.vars.get(0);
  Assert.assertEquals(property1.baseName, "id");
- Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.dataType, "long");
  Assert.assertEquals(property1.name, "Id");
  Assert.assertNull(property1.defaultValue);
- Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertEquals(property1.baseType, "long");
  Assert.assertTrue(property1.hasMore);
  Assert.assertTrue(property1.required);
  Assert.assertTrue(property1.isPrimitiveType);

samples/client/petstore/csharp-dotnet2/OpenAPIClientTest/Lib/OpenAPIClient/.openapi-generator/VERSION

@@ -1 +1 @@
-4.0.0-SNAPSHOT
+4.1.0-SNAPSHOT

samples/client/petstore/csharp-dotnet2/OpenAPIClientTest/Lib/OpenAPIClient/docs/UserApi.md

@@ -117,7 +117,7 @@ namespace Example
 
 Name | Type | Description | Notes
 ------------- | ------------- | ------------- | -------------
- **body** | [**List<User>**](List.md)| List of user object | 
+ **body** | [**List<User>**](User.md)| List of user object | 
 
 ### Return type
 
@@ -176,7 +176,7 @@ namespace Example
 
 Name | Type | Description | Notes
 ------------- | ------------- | ------------- | -------------
- **body** | [**List<User>**](List.md)| List of user object | 
+ **body** | [**List<User>**](User.md)| List of user object | 
 
 ### Return type
 

samples/client/petstore/csharp-netcore/OpenAPIClient/docs/FormatTest.md

@@ -14,7 +14,7 @@ Name | Type | Description | Notes
 **Binary** | **System.IO.Stream** | | [optional] 
 **Date** | **DateTime** | | 
 **DateTime** | **DateTime** | | [optional] 
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **Password** | **string** | | 
 
 [[Back to Model list]](../README.md#documentation-for-models) [[Back to API list]](../README.md#documentation-for-api-endpoints) [[Back to README]](../README.md)

samples/client/petstore/csharp-netcore/OpenAPIClient/docs/MixedPropertiesAndAdditionalPropertiesClass.md

@@ -3,7 +3,7 @@
 
 Name | Type | Description | Notes
 ------------ | ------------- | ------------- | -------------
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **DateTime** | **DateTime** | | [optional] 
 **Map** | [**Dictionary<string, Animal>**](Animal.md) | | [optional] 
 

samples/client/petstore/csharp-netcore/OpenAPIClientCore/docs/FormatTest.md

@@ -14,7 +14,7 @@ Name | Type | Description | Notes
 **Binary** | **System.IO.Stream** | | [optional] 
 **Date** | **DateTime** | | 
 **DateTime** | **DateTime** | | [optional] 
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **Password** | **string** | | 
 
 [[Back to Model list]](../README.md#documentation-for-models) [[Back to API list]](../README.md#documentation-for-api-endpoints) [[Back to README]](../README.md)

samples/client/petstore/csharp-netcore/OpenAPIClientCore/docs/MixedPropertiesAndAdditionalPropertiesClass.md

@@ -3,7 +3,7 @@
 
 Name | Type | Description | Notes
 ------------ | ------------- | ------------- | -------------
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **DateTime** | **DateTime** | | [optional] 
 **Map** | [**Dictionary<string, Animal>**](Animal.md) | | [optional] 
 

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AbstractCSharpCodegen.java

@@ -160,6 +160,7 @@ public AbstractCSharpCodegen() {
  "Int64",
  "Float",
  "Guid?",
+ "Guid",
  "System.IO.Stream", // not really a primitive, we include it to avoid model import
  "Object")
  );

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AspNetCoreServerCodegen.java

@@ -22,6 +22,7 @@
 import io.swagger.v3.oas.models.media.Schema;
 import io.swagger.v3.parser.util.SchemaTypeUtil;
 import org.openapitools.codegen.*;
+import org.openapitools.codegen.utils.ModelUtils;
 import org.openapitools.codegen.utils.URLPathUtils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -353,7 +354,7 @@ public void processOpts() {
  supportingFiles.add(new SupportingFile("Filters" + File.separator + "GeneratePathParamsValidationFilter.mustache",
  packageFolder + File.separator + "Filters", "GeneratePathParamsValidationFilter.cs"));
  }
-
+
  supportingFiles.add(new SupportingFile("Authentication" + File.separator + "ApiAuthentication.mustache",packageFolder + File.separator + "Authentication", "ApiAuthentication.cs"));
  }
 
@@ -407,12 +408,12 @@ public String toRegularExpression(String pattern) {
 
  @Override
  public String getNullableType(Schema p, String type) {
-boolean isNullableExpected = p.getNullable() == null || (p.getNullable() != null && p.getNullable());
-
-if (isNullableExpected && languageSpecificPrimitives.contains(type + "?")) {
-return type + "?";
-} else if (languageSpecificPrimitives.contains(type)) {
-return type;
+if (languageSpecificPrimitives.contains(type)) {
+ if (isSupportNullable() && ModelUtils.isNullable(p) && nullableType.contains(type)) {
+ return type + "?";
+} else {
+ return type;
+}
  } else {
  return null;
  }

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/CSharpClientCodegen.java

@@ -879,15 +879,15 @@ public String toInstantiationType(Schema schema) {
  return null;
  }
  }
-
+
  @Override
  public String getNullableType(Schema p, String type) {
-boolean isNullableExpected = p.getNullable() == null || (p.getNullable() != null && p.getNullable());
-
-if (isNullableExpected && languageSpecificPrimitives.contains(type + "?")) {
-return type + "?";
-} else if (languageSpecificPrimitives.contains(type)) {
-return type;
+if (languageSpecificPrimitives.contains(type)) {
+ if (isSupportNullable() && ModelUtils.isNullable(p) && nullableType.contains(type)) {
+ return type + "?";
+} else {
+ return type;
+}
  } else {
  return null;
  }

modules/openapi-generator/src/main/resources/csharp/api.mustache

@@ -32,7 +32,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>{{#returnType}}{{returnType}}{{/returnType}}</returns>
- {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
 
  /// <summary>
  /// {{summary}}
@@ -43,7 +43,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>ApiResponse of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}Object(void){{/returnType}}</returns>
- ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  {{/operation}}
  #endregion Synchronous Operations
  {{#supportsAsync}}
@@ -58,7 +58,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}void{{/returnType}}</returns>
- {{#returnType}}System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ {{#returnType}}System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
 
  /// <summary>
  /// {{summary}}
@@ -69,7 +69,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of ApiResponse{{#returnType}} ({{returnType}}){{/returnType}}</returns>
- System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  {{/operation}}
  #endregion Asynchronous Operations
  {{/supportsAsync}}
@@ -190,7 +190,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>{{#returnType}}{{returnType}}{{/returnType}}</returns>
- public {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#returnType}}ApiResponse<{{{returnType}}}> localVarResponse = {{operationId}}WithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  return localVarResponse.Data;{{/returnType}}{{^returnType}}{{operationId}}WithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});{{/returnType}}
@@ -202,7 +202,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>ApiResponse of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}Object(void){{/returnType}}</returns>
- public ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#allParams}}
  {{#required}}
@@ -325,7 +325,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}void{{/returnType}}</returns>
- {{#returnType}}public async System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}public async System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ {{#returnType}}public async System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}public async System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#returnType}}ApiResponse<{{{returnType}}}> localVarResponse = await {{operationId}}AsyncWithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  return localVarResponse.Data;{{/returnType}}{{^returnType}}await {{operationId}}AsyncWithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});{{/returnType}}
@@ -338,7 +338,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of ApiResponse{{#returnType}} ({{returnType}}){{/returnType}}</returns>
- public async System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public async System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#allParams}}
  {{#required}}

modules/openapi-generator/src/test/java/org/openapitools/codegen/csharp/CSharpModelTest.java

@@ -141,10 +141,10 @@ public void simpleModelTest() {
 
  final CodegenProperty property1 = cm.vars.get(0);
  Assert.assertEquals(property1.baseName, "id");
- Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.dataType, "long");
  Assert.assertEquals(property1.name, "Id");
  Assert.assertNull(property1.defaultValue);
- Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertEquals(property1.baseType, "long");
  Assert.assertTrue(property1.hasMore);
  Assert.assertTrue(property1.required);
  Assert.assertTrue(property1.isPrimitiveType);
@@ -161,10 +161,10 @@ public void simpleModelTest() {
 
  final CodegenProperty property3 = cm.vars.get(2);
  Assert.assertEquals(property3.baseName, "createdAt");
- Assert.assertEquals(property3.dataType, "DateTime?");
+ Assert.assertEquals(property3.dataType, "DateTime");
  Assert.assertEquals(property3.name, "CreatedAt");
  Assert.assertNull(property3.defaultValue);
- Assert.assertEquals(property3.baseType, "DateTime?");
+ Assert.assertEquals(property3.baseType, "DateTime");
  Assert.assertFalse(property3.hasMore);
  Assert.assertFalse(property3.required);
  }
@@ -209,7 +209,59 @@ public void nonNullablePropertyTest() {
  Assert.assertFalse(property2.required);
  Assert.assertTrue(property2.isPrimitiveType);
  Assert.assertTrue(property2.isContainer);
-
+
+ final CodegenProperty property3 = cm.vars.get(2);
+ Assert.assertEquals(property3.baseName, "name");
+ Assert.assertEquals(property3.dataType, "string");
+ Assert.assertEquals(property3.name, "Name");
+ Assert.assertNull(property3.defaultValue);
+ Assert.assertEquals(property3.baseType, "string");
+ Assert.assertFalse(property3.hasMore);
+ Assert.assertFalse(property3.required);
+ Assert.assertTrue(property3.isPrimitiveType);
+ }
+
+ @Test(description = "convert a model with a nullable property")
+ public void nullablePropertyTest() {
+ final Schema model = new Schema()
+ .description("a sample model")
+ .addProperties("id", new IntegerSchema().format(SchemaTypeUtil.INTEGER64_FORMAT).nullable(true))
+ .addProperties("urls", new ArraySchema()
+ .items(new StringSchema()))
+ .addProperties("name", new StringSchema().nullable(true))
+ .addRequiredItem("id");
+ final DefaultCodegen codegen = new CSharpClientCodegen();
+ OpenAPI openAPI = TestUtils.createOpenAPIWithOneSchema("sample", model);
+ codegen.setOpenAPI(openAPI);
+ final CodegenModel cm = codegen.fromModel("sample", model);
+
+ Assert.assertEquals(cm.name, "sample");
+ Assert.assertEquals(cm.classname, "Sample");
+ Assert.assertEquals(cm.description, "a sample model");
+ Assert.assertEquals(cm.vars.size(), 3);
+
+ final CodegenProperty property1 = cm.vars.get(0);
+ Assert.assertEquals(property1.baseName, "id");
+ Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.name, "Id");
+ Assert.assertNull(property1.defaultValue);
+ Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertTrue(property1.hasMore);
+ Assert.assertTrue(property1.required);
+ Assert.assertTrue(property1.isPrimitiveType);
+
+ final CodegenProperty property2 = cm.vars.get(1);
+ Assert.assertEquals(property2.baseName, "urls");
+ Assert.assertEquals(property2.dataType, "List<string>");
+ Assert.assertEquals(property2.name, "Urls");
+ Assert.assertNull(property2.defaultValue);
+ Assert.assertEquals(property2.baseType, "List");
+ Assert.assertTrue(property2.hasMore);
+ Assert.assertEquals(property2.containerType, "array");
+ Assert.assertFalse(property2.required);
+ Assert.assertTrue(property2.isPrimitiveType);
+ Assert.assertTrue(property2.isContainer);
+
  final CodegenProperty property3 = cm.vars.get(2);
  Assert.assertEquals(property3.baseName, "name");
  Assert.assertEquals(property3.dataType, "string");
@@ -241,10 +293,10 @@ public void listPropertyTest() {
 
  final CodegenProperty property1 = cm.vars.get(0);
  Assert.assertEquals(property1.baseName, "id");
- Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.dataType, "long");
  Assert.assertEquals(property1.name, "Id");
  Assert.assertNull(property1.defaultValue);
- Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertEquals(property1.baseType, "long");
  Assert.assertTrue(property1.hasMore);
  Assert.assertTrue(property1.required);
  Assert.assertTrue(property1.isPrimitiveType);

samples/client/petstore/csharp-dotnet2/OpenAPIClientTest/Lib/OpenAPIClient/.openapi-generator/VERSION

@@ -1 +1 @@
-4.0.0-SNAPSHOT
+4.1.0-SNAPSHOT

samples/client/petstore/csharp-dotnet2/OpenAPIClientTest/Lib/OpenAPIClient/docs/UserApi.md

@@ -117,7 +117,7 @@ namespace Example
 
 Name | Type | Description | Notes
 ------------- | ------------- | ------------- | -------------
- **body** | [**List<User>**](List.md)| List of user object | 
+ **body** | [**List<User>**](User.md)| List of user object | 
 
 ### Return type
 
@@ -176,7 +176,7 @@ namespace Example
 
 Name | Type | Description | Notes
 ------------- | ------------- | ------------- | -------------
- **body** | [**List<User>**](List.md)| List of user object | 
+ **body** | [**List<User>**](User.md)| List of user object | 
 
 ### Return type
 

samples/client/petstore/csharp-netcore/OpenAPIClient/docs/FormatTest.md

@@ -14,7 +14,7 @@ Name | Type | Description | Notes
 **Binary** | **System.IO.Stream** | | [optional] 
 **Date** | **DateTime** | | 
 **DateTime** | **DateTime** | | [optional] 
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **Password** | **string** | | 
 
 [[Back to Model list]](../README.md#documentation-for-models) [[Back to API list]](../README.md#documentation-for-api-endpoints) [[Back to README]](../README.md)

samples/client/petstore/csharp-netcore/OpenAPIClient/docs/MixedPropertiesAndAdditionalPropertiesClass.md

@@ -3,7 +3,7 @@
 
 Name | Type | Description | Notes
 ------------ | ------------- | ------------- | -------------
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **DateTime** | **DateTime** | | [optional] 
 **Map** | [**Dictionary<string, Animal>**](Animal.md) | | [optional] 
 

samples/client/petstore/csharp-netcore/OpenAPIClientCore/docs/FormatTest.md

@@ -14,7 +14,7 @@ Name | Type | Description | Notes
 **Binary** | **System.IO.Stream** | | [optional] 
 **Date** | **DateTime** | | 
 **DateTime** | **DateTime** | | [optional] 
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **Password** | **string** | | 
 
 [[Back to Model list]](../README.md#documentation-for-models) [[Back to API list]](../README.md#documentation-for-api-endpoints) [[Back to README]](../README.md)

samples/client/petstore/csharp-netcore/OpenAPIClientCore/docs/MixedPropertiesAndAdditionalPropertiesClass.md

@@ -3,7 +3,7 @@
 
 Name | Type | Description | Notes
 ------------ | ------------- | ------------- | -------------
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **DateTime** | **DateTime** | | [optional] 
 **Map** | [**Dictionary<string, Animal>**](Animal.md) | | [optional] 
 

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AbstractCSharpCodegen.java

@@ -160,6 +160,7 @@ public AbstractCSharpCodegen() {
  "Int64",
  "Float",
  "Guid?",
+ "Guid",
  "System.IO.Stream", // not really a primitive, we include it to avoid model import
  "Object")
  );

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AspNetCoreServerCodegen.java

@@ -22,6 +22,7 @@
 import io.swagger.v3.oas.models.media.Schema;
 import io.swagger.v3.parser.util.SchemaTypeUtil;
 import org.openapitools.codegen.*;
+import org.openapitools.codegen.utils.ModelUtils;
 import org.openapitools.codegen.utils.URLPathUtils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -353,7 +354,7 @@ public void processOpts() {
  supportingFiles.add(new SupportingFile("Filters" + File.separator + "GeneratePathParamsValidationFilter.mustache",
  packageFolder + File.separator + "Filters", "GeneratePathParamsValidationFilter.cs"));
  }
-
+
  supportingFiles.add(new SupportingFile("Authentication" + File.separator + "ApiAuthentication.mustache",packageFolder + File.separator + "Authentication", "ApiAuthentication.cs"));
  }
 
@@ -407,12 +408,12 @@ public String toRegularExpression(String pattern) {
 
  @Override
  public String getNullableType(Schema p, String type) {
-boolean isNullableExpected = p.getNullable() == null || (p.getNullable() != null && p.getNullable());
-
-if (isNullableExpected && languageSpecificPrimitives.contains(type + "?")) {
-return type + "?";
-} else if (languageSpecificPrimitives.contains(type)) {
-return type;
+if (languageSpecificPrimitives.contains(type)) {
+ if (isSupportNullable() && ModelUtils.isNullable(p) && nullableType.contains(type)) {
+ return type + "?";
+} else {
+ return type;
+}
  } else {
  return null;
  }

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/CSharpClientCodegen.java

@@ -879,15 +879,15 @@ public String toInstantiationType(Schema schema) {
  return null;
  }
  }
-
+
  @Override
  public String getNullableType(Schema p, String type) {
-boolean isNullableExpected = p.getNullable() == null || (p.getNullable() != null && p.getNullable());
-
-if (isNullableExpected && languageSpecificPrimitives.contains(type + "?")) {
-return type + "?";
-} else if (languageSpecificPrimitives.contains(type)) {
-return type;
+if (languageSpecificPrimitives.contains(type)) {
+ if (isSupportNullable() && ModelUtils.isNullable(p) && nullableType.contains(type)) {
+ return type + "?";
+} else {
+ return type;
+}
  } else {
  return null;
  }

modules/openapi-generator/src/main/resources/csharp/api.mustache

@@ -32,7 +32,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>{{#returnType}}{{returnType}}{{/returnType}}</returns>
- {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
 
  /// <summary>
  /// {{summary}}
@@ -43,7 +43,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>ApiResponse of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}Object(void){{/returnType}}</returns>
- ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  {{/operation}}
  #endregion Synchronous Operations
  {{#supportsAsync}}
@@ -58,7 +58,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}void{{/returnType}}</returns>
- {{#returnType}}System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ {{#returnType}}System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
 
  /// <summary>
  /// {{summary}}
@@ -69,7 +69,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of ApiResponse{{#returnType}} ({{returnType}}){{/returnType}}</returns>
- System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  {{/operation}}
  #endregion Asynchronous Operations
  {{/supportsAsync}}
@@ -190,7 +190,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>{{#returnType}}{{returnType}}{{/returnType}}</returns>
- public {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#returnType}}ApiResponse<{{{returnType}}}> localVarResponse = {{operationId}}WithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  return localVarResponse.Data;{{/returnType}}{{^returnType}}{{operationId}}WithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});{{/returnType}}
@@ -202,7 +202,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>ApiResponse of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}Object(void){{/returnType}}</returns>
- public ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#allParams}}
  {{#required}}
@@ -325,7 +325,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}void{{/returnType}}</returns>
- {{#returnType}}public async System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}public async System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ {{#returnType}}public async System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}public async System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#returnType}}ApiResponse<{{{returnType}}}> localVarResponse = await {{operationId}}AsyncWithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  return localVarResponse.Data;{{/returnType}}{{^returnType}}await {{operationId}}AsyncWithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});{{/returnType}}
@@ -338,7 +338,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of ApiResponse{{#returnType}} ({{returnType}}){{/returnType}}</returns>
- public async System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public async System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#allParams}}
  {{#required}}

modules/openapi-generator/src/test/java/org/openapitools/codegen/csharp/CSharpModelTest.java

@@ -141,10 +141,10 @@ public void simpleModelTest() {
 
  final CodegenProperty property1 = cm.vars.get(0);
  Assert.assertEquals(property1.baseName, "id");
- Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.dataType, "long");
  Assert.assertEquals(property1.name, "Id");
  Assert.assertNull(property1.defaultValue);
- Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertEquals(property1.baseType, "long");
  Assert.assertTrue(property1.hasMore);
  Assert.assertTrue(property1.required);
  Assert.assertTrue(property1.isPrimitiveType);
@@ -161,10 +161,10 @@ public void simpleModelTest() {
 
  final CodegenProperty property3 = cm.vars.get(2);
  Assert.assertEquals(property3.baseName, "createdAt");
- Assert.assertEquals(property3.dataType, "DateTime?");
+ Assert.assertEquals(property3.dataType, "DateTime");
  Assert.assertEquals(property3.name, "CreatedAt");
  Assert.assertNull(property3.defaultValue);
- Assert.assertEquals(property3.baseType, "DateTime?");
+ Assert.assertEquals(property3.baseType, "DateTime");
  Assert.assertFalse(property3.hasMore);
  Assert.assertFalse(property3.required);
  }
@@ -209,7 +209,59 @@ public void nonNullablePropertyTest() {
  Assert.assertFalse(property2.required);
  Assert.assertTrue(property2.isPrimitiveType);
  Assert.assertTrue(property2.isContainer);
-
+
+ final CodegenProperty property3 = cm.vars.get(2);
+ Assert.assertEquals(property3.baseName, "name");
+ Assert.assertEquals(property3.dataType, "string");
+ Assert.assertEquals(property3.name, "Name");
+ Assert.assertNull(property3.defaultValue);
+ Assert.assertEquals(property3.baseType, "string");
+ Assert.assertFalse(property3.hasMore);
+ Assert.assertFalse(property3.required);
+ Assert.assertTrue(property3.isPrimitiveType);
+ }
+
+ @Test(description = "convert a model with a nullable property")
+ public void nullablePropertyTest() {
+ final Schema model = new Schema()
+ .description("a sample model")
+ .addProperties("id", new IntegerSchema().format(SchemaTypeUtil.INTEGER64_FORMAT).nullable(true))
+ .addProperties("urls", new ArraySchema()
+ .items(new StringSchema()))
+ .addProperties("name", new StringSchema().nullable(true))
+ .addRequiredItem("id");
+ final DefaultCodegen codegen = new CSharpClientCodegen();
+ OpenAPI openAPI = TestUtils.createOpenAPIWithOneSchema("sample", model);
+ codegen.setOpenAPI(openAPI);
+ final CodegenModel cm = codegen.fromModel("sample", model);
+
+ Assert.assertEquals(cm.name, "sample");
+ Assert.assertEquals(cm.classname, "Sample");
+ Assert.assertEquals(cm.description, "a sample model");
+ Assert.assertEquals(cm.vars.size(), 3);
+
+ final CodegenProperty property1 = cm.vars.get(0);
+ Assert.assertEquals(property1.baseName, "id");
+ Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.name, "Id");
+ Assert.assertNull(property1.defaultValue);
+ Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertTrue(property1.hasMore);
+ Assert.assertTrue(property1.required);
+ Assert.assertTrue(property1.isPrimitiveType);
+
+ final CodegenProperty property2 = cm.vars.get(1);
+ Assert.assertEquals(property2.baseName, "urls");
+ Assert.assertEquals(property2.dataType, "List<string>");
+ Assert.assertEquals(property2.name, "Urls");
+ Assert.assertNull(property2.defaultValue);
+ Assert.assertEquals(property2.baseType, "List");
+ Assert.assertTrue(property2.hasMore);
+ Assert.assertEquals(property2.containerType, "array");
+ Assert.assertFalse(property2.required);
+ Assert.assertTrue(property2.isPrimitiveType);
+ Assert.assertTrue(property2.isContainer);
+
  final CodegenProperty property3 = cm.vars.get(2);
  Assert.assertEquals(property3.baseName, "name");
  Assert.assertEquals(property3.dataType, "string");
@@ -241,10 +293,10 @@ public void listPropertyTest() {
 
  final CodegenProperty property1 = cm.vars.get(0);
  Assert.assertEquals(property1.baseName, "id");
- Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.dataType, "long");
  Assert.assertEquals(property1.name, "Id");
  Assert.assertNull(property1.defaultValue);
- Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertEquals(property1.baseType, "long");
  Assert.assertTrue(property1.hasMore);
  Assert.assertTrue(property1.required);
  Assert.assertTrue(property1.isPrimitiveType);

samples/client/petstore/csharp-dotnet2/OpenAPIClientTest/Lib/OpenAPIClient/.openapi-generator/VERSION

@@ -1 +1 @@
-4.0.0-SNAPSHOT
+4.1.0-SNAPSHOT

samples/client/petstore/csharp-dotnet2/OpenAPIClientTest/Lib/OpenAPIClient/docs/UserApi.md

@@ -117,7 +117,7 @@ namespace Example
 
 Name | Type | Description | Notes
 ------------- | ------------- | ------------- | -------------
- **body** | [**List<User>**](List.md)| List of user object | 
+ **body** | [**List<User>**](User.md)| List of user object | 
 
 ### Return type
 
@@ -176,7 +176,7 @@ namespace Example
 
 Name | Type | Description | Notes
 ------------- | ------------- | ------------- | -------------
- **body** | [**List<User>**](List.md)| List of user object | 
+ **body** | [**List<User>**](User.md)| List of user object | 
 
 ### Return type
 

samples/client/petstore/csharp-netcore/OpenAPIClient/docs/FormatTest.md

@@ -14,7 +14,7 @@ Name | Type | Description | Notes
 **Binary** | **System.IO.Stream** | | [optional] 
 **Date** | **DateTime** | | 
 **DateTime** | **DateTime** | | [optional] 
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **Password** | **string** | | 
 
 [[Back to Model list]](../README.md#documentation-for-models) [[Back to API list]](../README.md#documentation-for-api-endpoints) [[Back to README]](../README.md)

samples/client/petstore/csharp-netcore/OpenAPIClient/docs/MixedPropertiesAndAdditionalPropertiesClass.md

@@ -3,7 +3,7 @@
 
 Name | Type | Description | Notes
 ------------ | ------------- | ------------- | -------------
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **DateTime** | **DateTime** | | [optional] 
 **Map** | [**Dictionary<string, Animal>**](Animal.md) | | [optional] 
 

samples/client/petstore/csharp-netcore/OpenAPIClientCore/docs/FormatTest.md

@@ -14,7 +14,7 @@ Name | Type | Description | Notes
 **Binary** | **System.IO.Stream** | | [optional] 
 **Date** | **DateTime** | | 
 **DateTime** | **DateTime** | | [optional] 
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **Password** | **string** | | 
 
 [[Back to Model list]](../README.md#documentation-for-models) [[Back to API list]](../README.md#documentation-for-api-endpoints) [[Back to README]](../README.md)

samples/client/petstore/csharp-netcore/OpenAPIClientCore/docs/MixedPropertiesAndAdditionalPropertiesClass.md

@@ -3,7 +3,7 @@
 
 Name | Type | Description | Notes
 ------------ | ------------- | ------------- | -------------
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **DateTime** | **DateTime** | | [optional] 
 **Map** | [**Dictionary<string, Animal>**](Animal.md) | | [optional] 
 

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AbstractCSharpCodegen.java

@@ -160,6 +160,7 @@ public AbstractCSharpCodegen() {
  "Int64",
  "Float",
  "Guid?",
+ "Guid",
  "System.IO.Stream", // not really a primitive, we include it to avoid model import
  "Object")
  );

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AspNetCoreServerCodegen.java

@@ -22,6 +22,7 @@
 import io.swagger.v3.oas.models.media.Schema;
 import io.swagger.v3.parser.util.SchemaTypeUtil;
 import org.openapitools.codegen.*;
+import org.openapitools.codegen.utils.ModelUtils;
 import org.openapitools.codegen.utils.URLPathUtils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -353,7 +354,7 @@ public void processOpts() {
  supportingFiles.add(new SupportingFile("Filters" + File.separator + "GeneratePathParamsValidationFilter.mustache",
  packageFolder + File.separator + "Filters", "GeneratePathParamsValidationFilter.cs"));
  }
-
+
  supportingFiles.add(new SupportingFile("Authentication" + File.separator + "ApiAuthentication.mustache",packageFolder + File.separator + "Authentication", "ApiAuthentication.cs"));
  }
 
@@ -407,12 +408,12 @@ public String toRegularExpression(String pattern) {
 
  @Override
  public String getNullableType(Schema p, String type) {
-boolean isNullableExpected = p.getNullable() == null || (p.getNullable() != null && p.getNullable());
-
-if (isNullableExpected && languageSpecificPrimitives.contains(type + "?")) {
-return type + "?";
-} else if (languageSpecificPrimitives.contains(type)) {
-return type;
+if (languageSpecificPrimitives.contains(type)) {
+ if (isSupportNullable() && ModelUtils.isNullable(p) && nullableType.contains(type)) {
+ return type + "?";
+} else {
+ return type;
+}
  } else {
  return null;
  }

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/CSharpClientCodegen.java

@@ -879,15 +879,15 @@ public String toInstantiationType(Schema schema) {
  return null;
  }
  }
-
+
  @Override
  public String getNullableType(Schema p, String type) {
-boolean isNullableExpected = p.getNullable() == null || (p.getNullable() != null && p.getNullable());
-
-if (isNullableExpected && languageSpecificPrimitives.contains(type + "?")) {
-return type + "?";
-} else if (languageSpecificPrimitives.contains(type)) {
-return type;
+if (languageSpecificPrimitives.contains(type)) {
+ if (isSupportNullable() && ModelUtils.isNullable(p) && nullableType.contains(type)) {
+ return type + "?";
+} else {
+ return type;
+}
  } else {
  return null;
  }

modules/openapi-generator/src/main/resources/csharp/api.mustache

@@ -32,7 +32,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>{{#returnType}}{{returnType}}{{/returnType}}</returns>
- {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
 
  /// <summary>
  /// {{summary}}
@@ -43,7 +43,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>ApiResponse of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}Object(void){{/returnType}}</returns>
- ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  {{/operation}}
  #endregion Synchronous Operations
  {{#supportsAsync}}
@@ -58,7 +58,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}void{{/returnType}}</returns>
- {{#returnType}}System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ {{#returnType}}System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
 
  /// <summary>
  /// {{summary}}
@@ -69,7 +69,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of ApiResponse{{#returnType}} ({{returnType}}){{/returnType}}</returns>
- System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
+ System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  {{/operation}}
  #endregion Asynchronous Operations
  {{/supportsAsync}}
@@ -190,7 +190,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>{{#returnType}}{{returnType}}{{/returnType}}</returns>
- public {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public {{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}void{{/returnType}} {{operationId}} ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#returnType}}ApiResponse<{{{returnType}}}> localVarResponse = {{operationId}}WithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  return localVarResponse.Data;{{/returnType}}{{^returnType}}{{operationId}}WithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});{{/returnType}}
@@ -202,7 +202,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>ApiResponse of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}Object(void){{/returnType}}</returns>
- public ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}> {{operationId}}WithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#allParams}}
  {{#required}}
@@ -325,7 +325,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of {{#returnType}}{{returnType}}{{/returnType}}{{^returnType}}void{{/returnType}}</returns>
- {{#returnType}}public async System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}public async System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ {{#returnType}}public async System.Threading.Tasks.Task<{{{returnType}}}>{{/returnType}}{{^returnType}}public async System.Threading.Tasks.Task{{/returnType}} {{operationId}}Async ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#returnType}}ApiResponse<{{{returnType}}}> localVarResponse = await {{operationId}}AsyncWithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});
  return localVarResponse.Data;{{/returnType}}{{^returnType}}await {{operationId}}AsyncWithHttpInfo({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}});{{/returnType}}
@@ -338,7 +338,7 @@ namespace {{packageName}}.{{apiPackage}}
  /// <exception cref="{{packageName}}.Client.ApiException">Thrown when fails to make API call</exception>
  {{#allParams}}/// <param name="{{paramName}}">{{description}}{{^required}} (optional{{#defaultValue}}, default to {{.}}{{/defaultValue}}){{/required}}</param>
  {{/allParams}}/// <returns>Task of ApiResponse{{#returnType}} ({{returnType}}){{/returnType}}</returns>
- public async System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = null{{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
+ public async System.Threading.Tasks.Task<ApiResponse<{{#returnType}}{{{returnType}}}{{/returnType}}{{^returnType}}Object{{/returnType}}>> {{operationId}}AsyncWithHttpInfo ({{#allParams}}{{{dataType}}} {{paramName}}{{^required}}{{#optionalMethodArgument}} = default({{{dataType}}}){{/optionalMethodArgument}}{{/required}}{{#hasMore}}, {{/hasMore}}{{/allParams}})
  {
  {{#allParams}}
  {{#required}}

modules/openapi-generator/src/test/java/org/openapitools/codegen/csharp/CSharpModelTest.java

@@ -141,10 +141,10 @@ public void simpleModelTest() {
 
  final CodegenProperty property1 = cm.vars.get(0);
  Assert.assertEquals(property1.baseName, "id");
- Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.dataType, "long");
  Assert.assertEquals(property1.name, "Id");
  Assert.assertNull(property1.defaultValue);
- Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertEquals(property1.baseType, "long");
  Assert.assertTrue(property1.hasMore);
  Assert.assertTrue(property1.required);
  Assert.assertTrue(property1.isPrimitiveType);
@@ -161,10 +161,10 @@ public void simpleModelTest() {
 
  final CodegenProperty property3 = cm.vars.get(2);
  Assert.assertEquals(property3.baseName, "createdAt");
- Assert.assertEquals(property3.dataType, "DateTime?");
+ Assert.assertEquals(property3.dataType, "DateTime");
  Assert.assertEquals(property3.name, "CreatedAt");
  Assert.assertNull(property3.defaultValue);
- Assert.assertEquals(property3.baseType, "DateTime?");
+ Assert.assertEquals(property3.baseType, "DateTime");
  Assert.assertFalse(property3.hasMore);
  Assert.assertFalse(property3.required);
  }
@@ -209,7 +209,59 @@ public void nonNullablePropertyTest() {
  Assert.assertFalse(property2.required);
  Assert.assertTrue(property2.isPrimitiveType);
  Assert.assertTrue(property2.isContainer);
-
+
+ final CodegenProperty property3 = cm.vars.get(2);
+ Assert.assertEquals(property3.baseName, "name");
+ Assert.assertEquals(property3.dataType, "string");
+ Assert.assertEquals(property3.name, "Name");
+ Assert.assertNull(property3.defaultValue);
+ Assert.assertEquals(property3.baseType, "string");
+ Assert.assertFalse(property3.hasMore);
+ Assert.assertFalse(property3.required);
+ Assert.assertTrue(property3.isPrimitiveType);
+ }
+
+ @Test(description = "convert a model with a nullable property")
+ public void nullablePropertyTest() {
+ final Schema model = new Schema()
+ .description("a sample model")
+ .addProperties("id", new IntegerSchema().format(SchemaTypeUtil.INTEGER64_FORMAT).nullable(true))
+ .addProperties("urls", new ArraySchema()
+ .items(new StringSchema()))
+ .addProperties("name", new StringSchema().nullable(true))
+ .addRequiredItem("id");
+ final DefaultCodegen codegen = new CSharpClientCodegen();
+ OpenAPI openAPI = TestUtils.createOpenAPIWithOneSchema("sample", model);
+ codegen.setOpenAPI(openAPI);
+ final CodegenModel cm = codegen.fromModel("sample", model);
+
+ Assert.assertEquals(cm.name, "sample");
+ Assert.assertEquals(cm.classname, "Sample");
+ Assert.assertEquals(cm.description, "a sample model");
+ Assert.assertEquals(cm.vars.size(), 3);
+
+ final CodegenProperty property1 = cm.vars.get(0);
+ Assert.assertEquals(property1.baseName, "id");
+ Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.name, "Id");
+ Assert.assertNull(property1.defaultValue);
+ Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertTrue(property1.hasMore);
+ Assert.assertTrue(property1.required);
+ Assert.assertTrue(property1.isPrimitiveType);
+
+ final CodegenProperty property2 = cm.vars.get(1);
+ Assert.assertEquals(property2.baseName, "urls");
+ Assert.assertEquals(property2.dataType, "List<string>");
+ Assert.assertEquals(property2.name, "Urls");
+ Assert.assertNull(property2.defaultValue);
+ Assert.assertEquals(property2.baseType, "List");
+ Assert.assertTrue(property2.hasMore);
+ Assert.assertEquals(property2.containerType, "array");
+ Assert.assertFalse(property2.required);
+ Assert.assertTrue(property2.isPrimitiveType);
+ Assert.assertTrue(property2.isContainer);
+
  final CodegenProperty property3 = cm.vars.get(2);
  Assert.assertEquals(property3.baseName, "name");
  Assert.assertEquals(property3.dataType, "string");
@@ -241,10 +293,10 @@ public void listPropertyTest() {
 
  final CodegenProperty property1 = cm.vars.get(0);
  Assert.assertEquals(property1.baseName, "id");
- Assert.assertEquals(property1.dataType, "long?");
+ Assert.assertEquals(property1.dataType, "long");
  Assert.assertEquals(property1.name, "Id");
  Assert.assertNull(property1.defaultValue);
- Assert.assertEquals(property1.baseType, "long?");
+ Assert.assertEquals(property1.baseType, "long");
  Assert.assertTrue(property1.hasMore);
  Assert.assertTrue(property1.required);
  Assert.assertTrue(property1.isPrimitiveType);

samples/client/petstore/csharp-dotnet2/OpenAPIClientTest/Lib/OpenAPIClient/.openapi-generator/VERSION

@@ -1 +1 @@
-4.0.0-SNAPSHOT
+4.1.0-SNAPSHOT

samples/client/petstore/csharp-dotnet2/OpenAPIClientTest/Lib/OpenAPIClient/docs/UserApi.md

@@ -117,7 +117,7 @@ namespace Example
 
 Name | Type | Description | Notes
 ------------- | ------------- | ------------- | -------------
- **body** | [**List<User>**](List.md)| List of user object | 
+ **body** | [**List<User>**](User.md)| List of user object | 
 
 ### Return type
 
@@ -176,7 +176,7 @@ namespace Example
 
 Name | Type | Description | Notes
 ------------- | ------------- | ------------- | -------------
- **body** | [**List<User>**](List.md)| List of user object | 
+ **body** | [**List<User>**](User.md)| List of user object | 
 
 ### Return type
 

samples/client/petstore/csharp-netcore/OpenAPIClient/docs/FormatTest.md

@@ -14,7 +14,7 @@ Name | Type | Description | Notes
 **Binary** | **System.IO.Stream** | | [optional] 
 **Date** | **DateTime** | | 
 **DateTime** | **DateTime** | | [optional] 
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **Password** | **string** | | 
 
 [[Back to Model list]](../README.md#documentation-for-models) [[Back to API list]](../README.md#documentation-for-api-endpoints) [[Back to README]](../README.md)

samples/client/petstore/csharp-netcore/OpenAPIClient/docs/MixedPropertiesAndAdditionalPropertiesClass.md

@@ -3,7 +3,7 @@
 
 Name | Type | Description | Notes
 ------------ | ------------- | ------------- | -------------
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **DateTime** | **DateTime** | | [optional] 
 **Map** | [**Dictionary<string, Animal>**](Animal.md) | | [optional] 
 

samples/client/petstore/csharp-netcore/OpenAPIClientCore/docs/FormatTest.md

@@ -14,7 +14,7 @@ Name | Type | Description | Notes
 **Binary** | **System.IO.Stream** | | [optional] 
 **Date** | **DateTime** | | 
 **DateTime** | **DateTime** | | [optional] 
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **Password** | **string** | | 
 
 [[Back to Model list]](../README.md#documentation-for-models) [[Back to API list]](../README.md#documentation-for-api-endpoints) [[Back to README]](../README.md)

samples/client/petstore/csharp-netcore/OpenAPIClientCore/docs/MixedPropertiesAndAdditionalPropertiesClass.md

@@ -3,7 +3,7 @@
 
 Name | Type | Description | Notes
 ------------ | ------------- | ------------- | -------------
-**Uuid** | [**Guid**](Guid.md) | | [optional] 
+**Uuid** | **Guid** | | [optional] 
 **DateTime** | **DateTime** | | [optional] 
 **Map** | [**Dictionary<string, Animal>**](Animal.md) | | [optional]