Update to System.Data.SqlClient v4.5.1

Author: mythz

NuGet/ServiceStack.OrmLite.Core/servicestack.ormlite.core.nuspec

@@ -25,7 +25,7 @@
         <dependency id="System.Collections.NonGeneric" version="[4.3.0, )" />
         <dependency id="System.Xml.XmlDocument" version="[4.3.0, )" />
         <dependency id="System.ComponentModel.Primitives" version="[4.3.0, )" />
-        <dependency id="System.Data.SqlClient" version="[4.4.3, )" />
+        <dependency id="System.Data.SqlClient" version="[4.5.1, )" />
         <dependency id="System.ComponentModel.Annotations" version="[4.3.0, )" />
         <dependency id="ServiceStack.Common.Core" version="5.0.0" />
       </group>

NuGet/ServiceStack.OrmLite.SqlServer.Core/servicestack.ormlite.sqlserver.core.nuspec

@@ -18,7 +18,7 @@
     <copyright>ServiceStack and contributors</copyright>
     <dependencies>
       <group targetFramework=".netstandard2.0">
-        <dependency id="System.Data.SqlClient" version="[4.4.3, )" />
+        <dependency id="System.Data.SqlClient" version="[4.5.1, )" />
         <dependency id="ServiceStack.OrmLite.Core" version="5.0.0" />
       </group>
     </dependencies>

NuGet/ServiceStack.OrmLite.SqlServer/servicestack.ormlite.sqlserver.nuspec

@@ -22,7 +22,7 @@
         <dependency id="ServiceStack.OrmLite" version="5.0.0" />
       </group>
       <group targetFramework=".netstandard2.0">
-        <dependency id="System.Data.SqlClient" version="[4.4.3, )" />
+        <dependency id="System.Data.SqlClient" version="[4.5.1, )" />
         <dependency id="ServiceStack.OrmLite" version="5.0.0" />
       </group>
     </dependencies>

NuGet/ServiceStack.OrmLite/servicestack.ormlite.nuspec

@@ -30,7 +30,7 @@
         <dependency id="System.Collections.NonGeneric" version="[4.3.0, )" />
         <dependency id="System.Xml.XmlDocument" version="[4.3.0, )" />
         <dependency id="System.ComponentModel.Primitives" version="[4.3.0, )" />
-        <dependency id="System.Data.SqlClient" version="[4.4.3, )" />
+        <dependency id="System.Data.SqlClient" version="[4.5.1, )" />
         <dependency id="System.ComponentModel.Annotations" version="[4.3.0, )" />
         <dependency id="ServiceStack.Common" version="5.0.0" />
       </group>

src/ServiceStack.OrmLite.SqlServer/ServiceStack.OrmLite.SqlServer.csproj

@@ -41,7 +41,7 @@
   </PropertyGroup>
 
   <ItemGroup Condition=" '$(TargetFramework)' == 'netstandard2.0' ">
-    <PackageReference Include="System.Data.SqlClient" Version="4.4.3" />
+    <PackageReference Include="System.Data.SqlClient" Version="4.5.1" />
     <ProjectReference Include="..\ServiceStack.OrmLite\ServiceStack.OrmLite.csproj" />
 
     <Reference Include="..\..\lib\netstandard2.0\ServiceStack.Interfaces.dll" />

src/ServiceStack.OrmLite.SqlServer/SqlServerOrmLiteDialectProvider.cs

@@ -341,14 +341,17 @@ public override void PrepareParameterizedInsertStatement<T>(IDbCommand cmd, ICol
 
             foreach (var fieldDef in modelDef.FieldDefinitionsArray)
             {
-                if (ShouldReturnOnInsert(modelDef, fieldDef))
+                //insertFields contains Property "Name" of fields to insert
+                var includeField = insertFields == null || insertFields.Contains(fieldDef.Name, StringComparer.OrdinalIgnoreCase);
+
+                if (ShouldReturnOnInsert(modelDef, fieldDef) && (!fieldDef.AutoId || !includeField))
                 {
                     if (sbReturningColumns.Length > 0)
                         sbReturningColumns.Append(",");
                     sbReturningColumns.Append("INSERTED." + GetQuotedColumnName(fieldDef.FieldName));
                 }
 
-                if (ShouldSkipInsert(fieldDef))
+                if (ShouldSkipInsert(fieldDef) && (!fieldDef.AutoId || !includeField))
                     continue;
 
                 //insertFields contains Property "Name" of fields to insert ( that's how expressions work )

src/ServiceStack.OrmLite/ServiceStack.OrmLite.csproj

@@ -48,7 +48,7 @@
     <PackageReference Include="System.Collections.NonGeneric" Version="4.3.0" />
     <PackageReference Include="System.Xml.XmlDocument" Version="4.3.0" />
     <PackageReference Include="System.ComponentModel.Primitives" Version="4.3.0" />
-    <PackageReference Include="System.Data.SqlClient" Version="4.4.0" />
+    <PackageReference Include="System.Data.SqlClient" Version="4.5.1" />
     <PackageReference Include="System.ComponentModel.Annotations" Version="4.3.0" />
 
     <Reference Include="..\..\lib\netstandard2.0\ServiceStack.Interfaces.dll" />

tests/ServiceStack.OrmLite.Tests/OrmLiteSelectTests.cs

@@ -527,7 +527,7 @@ public void Does_project_Sql_columns()
 
                 var q = db.From<Rockstar>()
                     .Join<RockstarAlbum>()
-                    .GroupBy(r => new { r.Id })
+                    .GroupBy(r => new { r.Id, r.FirstName, r.LastName })
                     .Select<Rockstar, RockstarAlbum>((r,a) => new {
                         r.Id,
                         Name = r.FirstName + " " + r.LastName,