Merge branch 'hotfix/2.0.1'

Author: roji

.appveyor.yml

@@ -1,5 +1,5 @@
 image: Visual Studio 2017 Preview
-version: 2.0.0-{build}
+version: 2.0.1-{build}
 services:
   - postgresql
 environment:
@@ -23,12 +23,11 @@ build_script:
   - msbuild /p:Configuration=Release
   - dotnet pack src\EFCore.PG\EFCore.PG.csproj -c Release --version-suffix ci-%PADDED_BUILD_NUMBER%
 test_script:
-  - cd test\EFCore.PG.Tests && dotnet xunit && cd ..\..\
-  - cd test\EFCore.PG.FunctionalTests && dotnet xunit && cd ..\..\
-  - cd test\EFCore.PG.Design.FunctionalTests && dotnet xunit && cd ..\..\
+  - dotnet test test\EFCore.PG.Tests\EFCore.PG.Tests.csproj
+  - dotnet test test\EFCore.PG.FunctionalTests\EFCore.PG.FunctionalTests.csproj
+  - dotnet test test\EFCore.PG.Design.FunctionalTests\EFCore.PG.Design.FunctionalTests.csproj
 artifacts:
   - path: 'src\EFCore.PG\bin\**\*.nupkg'
-  - path: 'src\EFCore.PG.Design\bin\**\*.nupkg'
 deploy:
   - provider: NuGet
     server: https://www.myget.org/F/npgsql-unstable/api/v2/package

.gitignore

@@ -6,6 +6,7 @@
 *.userprefs
 /*.nupkg
 .nuget/
+.vs/
 [Bb]in/
 [Bb]uild/
 [Oo]bj/

.travis.yml

@@ -16,9 +16,9 @@ env:
 before_script:
   - dotnet restore -v Minimal
 script:
-  - cd test/EFCore.PG.Tests && dotnet xunit -framework netcoreapp2.0 && cd ../../
-  - cd test/EFCore.PG.FunctionalTests && dotnet xunit -framework netcoreapp2.0 && cd ../../
-  - cd test/EFCore.PG.Design.FunctionalTests && dotnet xunit -framework netcoreapp2.0 && cd ../../
+  - dotnet test test/EFCore.PG.Tests/EFCore.PG.Tests.csproj
+  - dotnet test test/EFCore.PG.FunctionalTests/EFCore.PG.FunctionalTests.csproj
+  - dotnet test test/EFCore.PG.Design.FunctionalTests/EFCore.PG.Design.FunctionalTests.csproj
 
 cache:
   directories:

src/EFCore.PG/EFCore.PG.csproj

@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <VersionPrefix>2.0.0</VersionPrefix>
+    <VersionPrefix>2.0.1</VersionPrefix>
     <TargetFrameworks>netstandard2.0</TargetFrameworks>
     <AssemblyName>Npgsql.EntityFrameworkCore.PostgreSQL</AssemblyName>
     <RootNamespace>Microsoft.EntityFrameworkCore</RootNamespace>
@@ -25,11 +25,11 @@
   </ItemGroup>
 
   <ItemGroup>
-    <PackageReference Include="Microsoft.EntityFrameworkCore" Version="2.0.0" />
-    <PackageReference Include="Microsoft.EntityFrameworkCore.Relational" Version="2.0.0" />
+    <PackageReference Include="Microsoft.EntityFrameworkCore" Version="2.0.1" />
+    <PackageReference Include="Microsoft.EntityFrameworkCore.Relational" Version="2.0.1" />
     <PackageReference Include="Microsoft.Extensions.DependencyInjection.Abstractions" Version="2.0.0" />
     <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="2.0.0" />
-    <PackageReference Include="Npgsql" Version="3.2.5" />
+    <PackageReference Include="Npgsql" Version="3.2.6" />
   </ItemGroup>
 
 </Project>

src/EFCore.PG/Migrations/NpgsqlMigrationsSqlGenerator.cs

@@ -787,20 +787,6 @@ public virtual void Transfer(
                 .Append(Dependencies.SqlGenerationHelper.DelimitIdentifier(newSchema));
         }
 
-        protected override void ForeignKeyAction(ReferentialAction referentialAction, MigrationCommandListBuilder builder)
-        {
-            Check.NotNull(builder, nameof(builder));
-
-            if (referentialAction == ReferentialAction.Restrict)
-            {
-                builder.Append("NO ACTION");
-            }
-            else
-            {
-                base.ForeignKeyAction(referentialAction, builder);
-            }
-        }
-
         #endregion Utilities
 
         #region System column utilities

src/EFCore.PG/NpgsqlDbFunctionsExtensions.cs

@@ -20,7 +20,7 @@ public static bool ILike(
             [CanBeNull] this DbFunctions _,
             [CanBeNull] string matchExpression,
             [CanBeNull] string pattern)
-            => LikeCore(matchExpression, pattern, escapeCharacter: null);
+            => ILikeCore(matchExpression, pattern, escapeCharacter: null);
 
         /// <summary>
         ///     An implementation of the PostgreSQL ILIKE operation, which is an insensitive LIKE.
@@ -38,7 +38,7 @@ public static bool ILike(
             [CanBeNull] string matchExpression,
             [CanBeNull] string pattern,
             [CanBeNull] string escapeCharacter)
-            => LikeCore(matchExpression, pattern, escapeCharacter);
+            => ILikeCore(matchExpression, pattern, escapeCharacter);
 
         // Regex special chars defined here:
         // https://msdn.microsoft.com/en-us/library/4edbef7e(v=vs.110).aspx
@@ -56,7 +56,7 @@ private static string BuildEscapeRegexCharsPattern(IEnumerable<char> regexSpecia
             return string.Join("|", regexSpecialChars.Select(c => @"\" + c));
         }
 
-        private static bool LikeCore(string matchExpression, string pattern, string escapeCharacter)
+        private static bool ILikeCore(string matchExpression, string pattern, string escapeCharacter)
         {
             //TODO: this fixes https://github.com/aspnet/EntityFramework/issues/8656 by insisting that
             // the "escape character" is a string but just using the first character of that string,
@@ -133,7 +133,7 @@ var regexPattern
             return Regex.IsMatch(
                 matchExpression,
                 @"\A" + regexPattern + @"\s*\z",
-                RegexOptions.Singleline,
+                RegexOptions.IgnoreCase | RegexOptions.Singleline,
                 _regexTimeout);
         }
     }

src/EFCore.PG/Query/ExpressionTranslators/Internal/NpgsqlObjectToStringTranslator.cs

@@ -34,7 +34,12 @@ public virtual Expression Translate(MethodCallExpression methodCallExpression)
             => methodCallExpression.Method.Name == nameof(ToString) &&
                methodCallExpression.Arguments.Count == 0 &&
                methodCallExpression.Object != null &&
-               SupportedTypes.Contains(methodCallExpression.Object.Type.UnwrapNullableType().UnwrapEnumType())
+               SupportedTypes.Contains(
+                   AppContext.TryGetSwitch("Microsoft.EntityFrameworkCore.Issue9894", out var enabled)
+                   && enabled
+                       ? methodCallExpression.Object.Type.UnwrapNullableType().UnwrapEnumType()
+                       : methodCallExpression.Object.Type.UnwrapNullableType()
+               )
                 ? new ExplicitCastExpression(methodCallExpression.Object, typeof(string))
                 : null;
     }

src/EFCore.PG/Query/ExpressionVisitors/NpgsqlSqlTranslatingExpressionVisitor.cs

@@ -40,7 +40,7 @@ protected override Expression VisitSubQuery(SubQueryExpression expression)
             if (properties.Count == 0)
                 return null;
             var lastPropertyType = properties[properties.Count - 1].ClrType;
-            if (lastPropertyType.IsArray && lastPropertyType.GetArrayRank() == 1)
+            if (lastPropertyType.IsArray && lastPropertyType.GetArrayRank() == 1 && subQueryModel.ResultOperators.Count > 0)
             {
                 // Translate someArray.Length
                 if (subQueryModel.ResultOperators.First() is CountResultOperator)

src/EFCore.PG/Scaffolding/Internal/NpgsqlDatabaseModelFactory.cs

@@ -163,7 +163,7 @@ void GetTables()
 
         const string GetColumnsQuery = @"
 SELECT
-    nspname, relname, attisdropped, attname, typ.typname, atttypmod, description,
+    nspname, relname, attisdropped, attname, typ.typname, atttypmod, description, basetyp.typname AS domtypname,
     CASE WHEN pg_proc.proname='array_recv' THEN 'a' ELSE typ.typtype END AS typtype,
     CASE
       WHEN pg_proc.proname='array_recv' THEN elemtyp.typname
@@ -177,6 +177,7 @@ FROM pg_class AS cls
 LEFT OUTER JOIN pg_type AS typ ON attr.atttypid = typ.oid
 LEFT OUTER JOIN pg_proc ON pg_proc.oid = typ.typreceive
 LEFT OUTER JOIN pg_type AS elemtyp ON (elemtyp.oid = typ.typelem)
+LEFT OUTER JOIN pg_type AS basetyp ON (basetyp.oid = typ.typbasetype)
 LEFT OUTER JOIN pg_description AS des ON des.objoid = cls.oid AND des.objsubid = attnum
 WHERE
     relkind = 'r' AND
@@ -261,6 +262,9 @@ void GetColumns()
                     case 'e':
                         column[NpgsqlAnnotationNames.PostgresTypeType] = PostgresTypeType.Enum;
                         break;
+                    case 'd':
+                        column.StoreType = GetStoreType(reader.GetValueOrDefault<string>("domtypname"), typeModifier);
+                        break;
                     default:
                         Logger.Logger.LogWarning($"Can't scaffold column '{columnName}' of type '{dataType}': unknown type char '{typeChar}'");
                         continue;
@@ -535,7 +539,10 @@ void GetSequences()
                             Debug.Assert(sequence.MinValue.HasValue);
                             defaultStart = sequence.MinValue.Value;
                         } else {
-                            defaultMin = long.MinValue + 1;
+                            // PostgreSQL 10 changed the default minvalue for a descending sequence, see #264
+                            defaultMin = _connection.PostgreSqlVersion >= new Version(10,0)
+                                ? long.MinValue
+                                : long.MinValue + 1;
                             defaultMax = -1;
                             Debug.Assert(sequence.MaxValue.HasValue);
                             defaultStart = sequence.MaxValue.Value;

src/EFCore.PG/Storage/Internal/Mapping/NpgsqlDateTimeOffsetTypeMapping.cs

@@ -0,0 +1,22 @@
+using System.Data;
+using JetBrains.Annotations;
+
+namespace Microsoft.EntityFrameworkCore.Storage.Internal
+{
+    public class NpgsqlDateTimeOffsetTypeMapping : DateTimeOffsetTypeMapping
+    {
+        private const string DateTimeOffsetFormatConst = "{0:yyyy-MM-ddTHH:mm:ss.fffzzz}";
+
+        public NpgsqlDateTimeOffsetTypeMapping(
+            [NotNull] string storeType,
+            [NotNull] DbType? dbType = System.Data.DbType.DateTimeOffset)
+            : base(storeType, dbType: dbType)
+        {
+        }
+
+        public override RelationalTypeMapping Clone(string storeType, int? size)
+            => new NpgsqlDateTimeOffsetTypeMapping(storeType, DbType);
+
+        protected override string SqlLiteralFormatString => $"'{DateTimeOffsetFormatConst}'";
+    }
+}