Fix string usage and remove extra usings

Directory.Build.props

@@ -72,32 +72,21 @@
       [IDE0290] Use primary constructor
       [IDE0330] Use 'System.Threading.Lock'
       [CA1200] Avoid using cref tags with a prefix
-      [CA1304] The behavior of 'string.ToUpper()' could vary based on the current user's locale settings
-      [CA1305] The behavior of 'int.ToString()' could vary based on the current user's locale settings
-      [CA1310] The behavior of 'string.StartsWith(string)' could vary based on the current user's locale settings
-      [CA1311] The behavior of 'string.StartsWith(string)' could vary based on the current user's locale settings
-      [CA1507] Use nameof in place of string literal
       [CA1510] Use 'ArgumentNullException.ThrowIfNull' instead of explicitly throwing a new exception instance
       [CA1514] 'System.ReadOnlySpan<char>.Slice(int, int)' uses a redundant length calculation that can be removed
       [CA1710] Rename to end in either 'Dictionary' or 'Collection'
       [CA1716] rename parameter property so that it no longer conflicts with the reserved language keyword
       [CA1720] Identifier 'xxx' contains type name
       [CA1725] Overriden parameter name mismatch
-      [CA1834] Use 'StringBuilder.Append(char)' instead of 'StringBuilder.Append(string)' when the input is a constant unit string
       [CA1845] Use span-based 'string.Concat' and 'AsSpan' instead of 'Substring'
-      [CA1847] Use 'string.Contains(char)' instead of 'string.Contains(string)' - needs polyfill
       [CA1861] Prefer 'static readonly' fields over constant array arguments
-      [CA1862] Prefer using 'string.Equals(string, StringComparison)' to perform a case-insensitive comparison
-      [CA1865] Use 'string.StartsWith(char)' instead of 'string.StartsWith(string)'  - needs polyfill
-      [CA1866] Use 'string.StartsWith(char)' instead of 'string.StartsWith(string)'  - needs polyfill
       [CA1870] Use a cached 'SearchValues' instance for improved searching performance
-      [CA2249] Use 'string.Contains' instead of 'string.IndexOf' to improve readability -  needs polyfill
       [CA2263] Prefer the generic overload 'System.Enum.GetValues<TEnum>()'
 
       [SYSLIB0012] 'Assembly.CodeBase' is obsolete
     -->
     <NoWarn>$(NoWarn);IDE0005;IDE0008;IDE0017;IDE0021;IDE0022;IDE0025;IDE0027;IDE0029;IDE0032;IDE0039;IDE0045;IDE0046;IDE0055;IDE0056;IDE0057;IDE0060;IDE0061;IDE0090;IDE0130;IDE0160;IDE0200;IDE0270;IDE0290;IDE0330</NoWarn>
-    <NoWarn>$(NoWarn);CA1200;CA1304;CA1305;CA1310;CA1311;CA1507;CA1510;CA1514;CA1710;CA1716;CA1720;CA1725;CA1834;CA1845;CA1847;CA1861;CA1862;CA1865;CA1866;CA1870;CA2249;CA2263;SYSLIB0012</NoWarn>
+    <NoWarn>$(NoWarn);CA1200;CA1510;CA1514;CA1710;CA1716;CA1720;CA1725;CA1845;CA1861;CA1870;CA2263;SYSLIB0012</NoWarn>
   </PropertyGroup>
 
 </Project>

src/NSwag.Annotations/ResponseTypeAttribute.cs

@@ -6,6 +6,7 @@
 // <author>Rico Suter, [email protected]</author>
 //-----------------------------------------------------------------------
 
+using System.Globalization;
 using System.Net;
 
 namespace NSwag.Annotations
@@ -37,7 +38,7 @@ public ResponseTypeAttribute(string httpStatusCode, Type responseType)
         /// <param name="responseType">The JSON result type of the MVC or Web API action method.</param>
         public ResponseTypeAttribute(int httpStatusCode, Type responseType)
         {
-            HttpStatusCode = httpStatusCode.ToString();
+            HttpStatusCode = httpStatusCode.ToString(CultureInfo.InvariantCulture);
             ResponseType = responseType;
         }
 
@@ -46,7 +47,7 @@ public ResponseTypeAttribute(int httpStatusCode, Type responseType)
         /// <param name="responseType">The JSON result type of the MVC or Web API action method.</param>
         public ResponseTypeAttribute(HttpStatusCode httpStatusCode, Type responseType)
         {
-            HttpStatusCode = ((int)httpStatusCode).ToString();
+            HttpStatusCode = ((int)httpStatusCode).ToString(CultureInfo.InvariantCulture);
             ResponseType = responseType;
         }
 

src/NSwag.Annotations/SwaggerResponseAttribute.cs

@@ -6,6 +6,7 @@
 // <author>Rico Suter, [email protected]</author>
 //-----------------------------------------------------------------------
 
+using System.Globalization;
 using System.Net;
 
 namespace NSwag.Annotations
@@ -37,7 +38,7 @@ public SwaggerResponseAttribute(string httpStatusCode, Type responseType)
         /// <param name="responseType">The JSON result type of the MVC or Web API action method.</param>
         public SwaggerResponseAttribute(int httpStatusCode, Type responseType)
         {
-            StatusCode = httpStatusCode.ToString();
+            StatusCode = httpStatusCode.ToString(CultureInfo.InvariantCulture);
             Type = responseType;
         }
 
@@ -46,7 +47,7 @@ public SwaggerResponseAttribute(int httpStatusCode, Type responseType)
         /// <param name="responseType">The JSON result type of the MVC or Web API action method.</param>
         public SwaggerResponseAttribute(HttpStatusCode httpStatusCode, Type responseType)
         {
-            StatusCode = ((int)httpStatusCode).ToString();
+            StatusCode = ((int)httpStatusCode).ToString(CultureInfo.InvariantCulture);
             Type = responseType;
         }
 

src/NSwag.AspNet.Owin/Middlewares/OpenApiDocumentMiddleware.cs

@@ -31,7 +31,7 @@ public class OpenApiDocumentMiddleware : OwinMiddleware
         public OpenApiDocumentMiddleware(OwinMiddleware next, string path, IEnumerable<Type> controllerTypes, SwaggerSettings<WebApiOpenApiDocumentGeneratorSettings> settings)
             : base(next)
         {
-            _path = path.StartsWith("/") ? path : '/' + path;
+            _path = path.StartsWith('/') ? path : '/' + path;
             _controllerTypes = controllerTypes;
             _settings = settings;
         }

src/NSwag.AspNet.Owin/NSwag.AspNet.Owin.csproj

@@ -17,6 +17,7 @@
     <Compile Include="..\NSwag.AspNetCore\SwaggerSettings.cs" Link="SwaggerSettings.cs" />
     <Compile Include="..\NSwag.AspNetCore\SwaggerUiSettings.cs" Link="SwaggerUiSettings.cs" />
     <Compile Include="..\NSwag.AspNetCore\SwaggerUiSettingsBase.cs" Link="SwaggerUiSettingsBase.cs" />
+    <Compile Include="..\NSwag.Core\Polyfills.cs" Link="Polyfills.cs" />
   </ItemGroup>
 
   <ItemGroup>

src/NSwag.AspNetCore.Launcher/Program.cs

@@ -1,7 +1,4 @@
-using System;
-using System.Collections.Generic;
-using System.IO;
-using System.Reflection;
+using System.Reflection;
 
 namespace NSwag.AspNetCore.Launcher
 {

src/NSwag.AspNetCore/Middlewares/OpenApiDocumentMiddleware.cs

@@ -39,7 +39,7 @@ public OpenApiDocumentMiddleware(RequestDelegate nextDelegate, IServiceProvider
             _nextDelegate = nextDelegate;
 
             _documentName = documentName;
-            _path = path.StartsWith("/") ? path : '/' + path;
+            _path = path.StartsWith('/') ? path : '/' + path;
 
             _apiDescriptionGroupCollectionProvider = serviceProvider.GetService<IApiDescriptionGroupCollectionProvider>() ??
                 throw new InvalidOperationException("API Explorer not registered in DI.");
@@ -58,7 +58,7 @@ public async Task Invoke(HttpContext context)
             {
                 var schemaJson = await GetDocumentAsync(context);
                 context.Response.StatusCode = 200;
-                context.Response.Headers["Content-Type"] = _path.IndexOf(".yaml", StringComparison.OrdinalIgnoreCase) >= 0 ?
+                context.Response.Headers["Content-Type"] = _path.Contains(".yaml", StringComparison.OrdinalIgnoreCase) ?
                     "application/yaml; charset=utf-8" :
                     "application/json; charset=utf-8";
 
@@ -99,7 +99,7 @@ protected virtual async Task<string> GetDocumentAsync(HttpContext context)
             try
             {
                 var openApiDocument = await GenerateDocumentAsync(context);
-                var data = _path.IndexOf(".yaml", StringComparison.OrdinalIgnoreCase) >= 0 ?
+                var data = _path.Contains(".yaml", StringComparison.OrdinalIgnoreCase) ?
                     OpenApiYamlDocument.ToYaml(openApiDocument) :
                     openApiDocument.ToJson();
 

src/NSwag.AspNetCore/NSwag.AspNetCore.csproj

@@ -54,6 +54,10 @@
     <ProjectReference Include="..\NSwag.Generation.AspNetCore\NSwag.Generation.AspNetCore.csproj" />
   </ItemGroup>
 
+  <ItemGroup>
+    <Compile Include="..\NSwag.Core\Polyfills.cs" Link="Polyfills.cs" />
+  </ItemGroup>
+
   <Target Name="PopulateNuspec">
     <PropertyGroup>
       <NuspecProperties>

src/NSwag.AspNetCore/SwaggerUiSettingsBase.cs

@@ -7,6 +7,7 @@
 //-----------------------------------------------------------------------
 
 #pragma warning disable IDE0005
+#pragma warning disable CA1305
 
 using NSwag.Generation;
 using Newtonsoft.Json;

src/NSwag.CodeGeneration.CSharp.Tests/AllowNullableBodyParametersTests.cs

@@ -1,10 +1,8 @@
 using Microsoft.AspNetCore.Mvc;
-using NJsonSchema.Generation;
 using NJsonSchema.NewtonsoftJson.Generation;
 using NSwag.CodeGeneration.OperationNameGenerators;
 using NSwag.Generation.WebApi;
 using System.ComponentModel.DataAnnotations;
-using System.Threading.Tasks;
 using Xunit;
 
 namespace NSwag.CodeGeneration.CSharp.Tests

src/NSwag.CodeGeneration.CSharp.Tests/ArrayParameterTests.cs

@@ -1,5 +1,4 @@
-using System.Threading.Tasks;
-using Xunit;
+using Xunit;
 
 namespace NSwag.CodeGeneration.CSharp.Tests
 {

src/NSwag.CodeGeneration.CSharp.Tests/BinaryTests.cs

@@ -1,5 +1,4 @@
 using NSwag.CodeGeneration.CSharp.Models;
-using System.Threading.Tasks;
 using Xunit;
 
 namespace NSwag.CodeGeneration.CSharp.Tests

src/NSwag.CodeGeneration.CSharp.Tests/CSharpClientSettingsTests.cs

@@ -1,6 +1,4 @@
-using System.Threading.Tasks;
 using Microsoft.AspNetCore.Mvc;
-using NJsonSchema.Generation;
 using NJsonSchema.NewtonsoftJson.Generation;
 using NSwag.Generation.WebApi;
 using Xunit;

src/NSwag.CodeGeneration.CSharp.Tests/ControllerGenerationFormatTests.cs

@@ -1,5 +1,4 @@
-using System.Collections.Generic;
-using System.Text.RegularExpressions;
+using System.Text.RegularExpressions;
 using NJsonSchema;
 using NJsonSchema.NewtonsoftJson.Generation;
 using NSwag.CodeGeneration.CSharp.Models;

src/NSwag.CodeGeneration.CSharp.Tests/Controllers/ControllerGenerationBasePathTests.cs

@@ -1,5 +1,4 @@
-using System.Threading.Tasks;
-using Xunit;
+using Xunit;
 
 namespace NSwag.CodeGeneration.CSharp.Tests
 {

src/NSwag.CodeGeneration.CSharp.Tests/FileDownloadTests.cs

@@ -1,8 +1,4 @@
-using System;
-using System.Net.Http;
-using System.Threading.Tasks;
-using Microsoft.AspNetCore.Mvc;
-using NJsonSchema.Generation;
+using Microsoft.AspNetCore.Mvc;
 using NJsonSchema.NewtonsoftJson.Generation;
 using NSwag.Generation.WebApi;
 using Xunit;

src/NSwag.CodeGeneration.CSharp.Tests/FileTests.cs

@@ -1,7 +1,4 @@
-using System;
-using System.Net.Http;
-using System.Threading.Tasks;
-using Microsoft.AspNetCore.Mvc;
+using Microsoft.AspNetCore.Mvc;
 using NJsonSchema.NewtonsoftJson.Generation;
 using NSwag.Generation.WebApi;
 using Xunit;

src/NSwag.CodeGeneration.CSharp.Tests/FileUploadTests.cs

@@ -1,5 +1,4 @@
-using System.Threading.Tasks;
-using NJsonSchema;
+using NJsonSchema;
 using Xunit;
 
 namespace NSwag.CodeGeneration.CSharp.Tests

src/NSwag.CodeGeneration.CSharp.Tests/FormParameterTests.cs

@@ -1,7 +1,5 @@
-using System.Threading.Tasks;
-using Microsoft.AspNetCore.Mvc;
+using Microsoft.AspNetCore.Mvc;
 using NJsonSchema;
-using NJsonSchema.Generation;
 using NJsonSchema.NewtonsoftJson.Generation;
 using NSwag.Generation.WebApi;
 using Xunit;

src/NSwag.CodeGeneration.CSharp.Tests/HeadRequestTests.cs

@@ -1,6 +1,4 @@
-using System.Threading.Tasks;
-using Microsoft.AspNetCore.Mvc;
-using NJsonSchema.Generation;
+using Microsoft.AspNetCore.Mvc;
 using NJsonSchema.NewtonsoftJson.Generation;
 using NSwag.Generation.WebApi;
 using Xunit;

src/NSwag.CodeGeneration.CSharp.Tests/InheritanceTests.cs

@@ -1,5 +1,4 @@
 using NSwag.CodeGeneration.CSharp;
-using System.Threading.Tasks;
 using Xunit;
 
 namespace NSwag.CodeGeneration.TypeScript.Tests

src/NSwag.CodeGeneration.CSharp.Tests/ObjectParameterTests.cs

@@ -1,4 +1,3 @@
-using System.Threading.Tasks;
 using Xunit;
 
 namespace NSwag.CodeGeneration.CSharp.Tests

src/NSwag.CodeGeneration.CSharp.Tests/OptionalParameterTests.cs

@@ -1,9 +1,5 @@
-using System;
-using System.Linq;
-using System.Threading.Tasks;
-using Microsoft.AspNetCore.Mvc;
+using Microsoft.AspNetCore.Mvc;
 using NJsonSchema;
-using NJsonSchema.Generation;
 using NJsonSchema.NewtonsoftJson.Generation;
 using NSwag.Generation.WebApi;
 using Xunit;

src/NSwag.CodeGeneration.CSharp.Tests/ParameterTests.cs

@@ -1,6 +1,4 @@
-using System;
-using System.Threading.Tasks;
-using NJsonSchema;
+using NJsonSchema;
 using Xunit;
 
 namespace NSwag.CodeGeneration.CSharp.Tests

src/NSwag.CodeGeneration.CSharp.Tests/PlainResponseTests.cs

@@ -1,9 +1,4 @@
-using System;
-using System.Net.Http;
-using System.Threading.Tasks;
-using Microsoft.AspNetCore.Mvc;
-using NSwag.Generation.WebApi;
-using Xunit;
+using Xunit;
 
 namespace NSwag.CodeGeneration.CSharp.Tests
 {

src/NSwag.CodeGeneration.CSharp.Tests/PlainTextBodyTests.cs

@@ -1,5 +1,4 @@
-using System.Threading.Tasks;
-using Xunit;
+using Xunit;
 
 namespace NSwag.CodeGeneration.CSharp.Tests
 {

src/NSwag.CodeGeneration.CSharp.Tests/QueryParameterTests.cs

@@ -1,5 +1,4 @@
-using System.Threading.Tasks;
-using Xunit;
+using Xunit;
 
 namespace NSwag.CodeGeneration.CSharp.Tests
 {

src/NSwag.CodeGeneration.CSharp.Tests/ResponseTests.cs

@@ -1,5 +1,4 @@
-using System.Threading.Tasks;
-using Xunit;
+using Xunit;
 
 namespace NSwag.CodeGeneration.CSharp.Tests
 {

src/NSwag.CodeGeneration.CSharp.Tests/UseCancellationTokenTests.cs

@@ -1,7 +1,4 @@
-using System;
-using System.Threading.Tasks;
-using Microsoft.AspNetCore.Mvc;
-using NJsonSchema.Generation;
+using Microsoft.AspNetCore.Mvc;
 using NJsonSchema.NewtonsoftJson.Generation;
 using NSwag.CodeGeneration.CSharp.Models;
 using NSwag.Generation.WebApi;

src/NSwag.CodeGeneration.CSharp.Tests/WrapResponsesTests.cs

@@ -1,7 +1,4 @@
-using System;
-using System.Threading.Tasks;
 using Microsoft.AspNetCore.Mvc;
-using NJsonSchema.Generation;
 using NJsonSchema.NewtonsoftJson.Generation;
 using NSwag.Generation.WebApi;
 using Xunit;

src/NSwag.CodeGeneration.CSharp/Models/CSharpParameterModel.cs

@@ -40,10 +40,10 @@ public CSharpParameterModel(
         }
 
         /// <summary>Gets a value indicating whether the type is a Nullable&lt;&gt;.</summary>
-        public bool IsSystemNullable => Type.EndsWith("?");
+        public bool IsSystemNullable => Type.EndsWith('?');
 
         /// <summary>Gets the type of the parameter when used in a controller interface where we can set default values before calling.</summary>
-        public string TypeInControllerInterface => HasDefault ? Type.EndsWith("?") ? Type.Substring(0, Type.Length - 1) : Type : Type;
+        public string TypeInControllerInterface => HasDefault ? Type.EndsWith('?') ? Type.Substring(0, Type.Length - 1) : Type : Type;
 
         /// <summary>Gets a value indicating whether the parameter name is a valid CSharp identifier.</summary>
         public bool IsValidIdentifier => Name.Equals(VariableName, StringComparison.OrdinalIgnoreCase);

src/NSwag.CodeGeneration.CSharp/NSwag.CodeGeneration.CSharp.csproj

@@ -6,6 +6,7 @@
 
   <ItemGroup>
     <EmbeddedResource Include="Templates\*.liquid" />
+    <Compile Include="..\NSwag.Core\Polyfills.cs" Link="Polyfills.cs" />
   </ItemGroup>
 
   <ItemGroup>
@@ -15,4 +16,5 @@
   <ItemGroup>
     <PackageReference Include="NJsonSchema.CodeGeneration.CSharp" Version="11.1.0" />
   </ItemGroup>
+
 </Project>

src/NSwag.CodeGeneration.Tests/CodeGenerationTests.cs

@@ -1,9 +1,5 @@
-using System;
-using System.Collections.Generic;
-using System.ComponentModel.DataAnnotations;
-using System.Linq;
+using System.ComponentModel.DataAnnotations;
 using System.Text.RegularExpressions;
-using System.Threading.Tasks;
 using NJsonSchema;
 using NJsonSchema.CodeGeneration.TypeScript;
 using NJsonSchema.Generation;

src/NSwag.CodeGeneration.TypeScript.Tests/AngularJSTests.cs

@@ -1,8 +1,6 @@
-using System.Threading.Tasks;
-using Xunit;
+using Xunit;
 using NSwag.Generation.WebApi;
 using Microsoft.AspNetCore.Mvc;
-using NJsonSchema.Generation;
 using NJsonSchema;
 using NJsonSchema.NewtonsoftJson.Generation;
 

src/NSwag.CodeGeneration.TypeScript.Tests/AngularTests.cs

@@ -1,9 +1,7 @@
-using System.Threading.Tasks;
-using Xunit;
+using Xunit;
 using NSwag.Generation.WebApi;
 using Microsoft.AspNetCore.Mvc;
 using System.ComponentModel.DataAnnotations;
-using NJsonSchema.Generation;
 using NJsonSchema;
 using NJsonSchema.NewtonsoftJson.Generation;
 

src/NSwag.CodeGeneration.TypeScript.Tests/ArrayParameterTests.cs

@@ -1,5 +1,4 @@
-using System.Threading.Tasks;
-using Xunit;
+using Xunit;
 
 namespace NSwag.CodeGeneration.TypeScript.Tests
 {