Fix default value generation (DNET-873)

Cataurus · cincuranet · commit 3b007d318032 · 2019-03-19T21:25:09.000+01:00
diff --git a/Provider/src/FirebirdSql.EntityFrameworkCore.Firebird/Migrations/FbMigrationsSqlGenerator.cs b/Provider/src/FirebirdSql.EntityFrameworkCore.Firebird/Migrations/FbMigrationsSqlGenerator.cs
@@ -311,12 +311,12 @@ protected override void ColumnDefinition(string schema, string table, string nam
 				builder.Append(" GENERATED BY DEFAULT AS IDENTITY");
 			}
 
+			DefaultValue(defaultValue, defaultValueSql, builder);
+
 			if (!nullable)
 			{
 				builder.Append(" NOT NULL");
 			}
-
-			DefaultValue(defaultValue, defaultValueSql, builder);
 		}
 
 		protected override void ForeignKeyConstraint(AddForeignKeyOperation operation, IModel model, MigrationCommandListBuilder builder)

Original file line number	Diff line number	Diff line change
`@@ -311,12 +311,12 @@ protected override void ColumnDefinition(string schema, string table, string nam`
`311`	`311`	`builder.Append(" GENERATED BY DEFAULT AS IDENTITY");`
`312`	`312`	`}`
`313`	`313`
	`314`	`+ DefaultValue(defaultValue, defaultValueSql, builder);`
	`315`	`+`
`314`	`316`	`if (!nullable)`
`315`	`317`	`{`
`316`	`318`	`builder.Append(" NOT NULL");`
`317`	`319`	`}`
`318`		`-`
`319`		`- DefaultValue(defaultValue, defaultValueSql, builder);`
`320`	`320`	`}`
`321`	`321`
`322`	`322`	`protected override void ForeignKeyConstraint(AddForeignKeyOperation operation, IModel model, MigrationCommandListBuilder builder)`