Skip to content

Commit 51037ec

Browse files
ChrisJollyAUChrisJollyAU
committed
Use GenGUID function for DefaultValueSql instead of the undefined newid
1 parent ae722d9 commit 51037ec

3 files changed

Lines changed: 19 additions & 19 deletions

File tree

‎test/EFCore.Jet.FunctionalTests/BatchingTest.cs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -231,7 +231,7 @@ protected override void OnModelCreating(ModelBuilder modelBuilder)
231231
modelBuilder.Entity<Blog>(
232232
b =>
233233
{
234-
b.Property(e => e.Id).HasDefaultValueSql("NEWID()");
234+
b.Property(e => e.Id).HasDefaultValueSql("GenGUID()");
235235
b.Property(e => e.Version).IsConcurrencyToken().ValueGeneratedOnAddOrUpdate();
236236
});
237237
}

‎test/EFCore.Jet.FunctionalTests/Query/NorthwindFunctionsQueryJetTest.Functions.cs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1173,7 +1173,7 @@ public override async Task Where_guid_newguid(bool isAsync)
11731173
AssertSql(
11741174
$@"SELECT `o`.`OrderID`, `o`.`ProductID`, `o`.`Discount`, `o`.`Quantity`, `o`.`UnitPrice`
11751175
FROM `Order Details` AS `o`
1176-
WHERE (NEWID() <> '00000000-0000-0000-0000-000000000000') OR NEWID() IS NULL");
1176+
WHERE (GenGUID() <> '00000000-0000-0000-0000-000000000000') OR GenGUID() IS NULL");
11771177
}
11781178

11791179
public override async Task Where_string_to_upper(bool isAsync)

‎test/EFCore.Jet.FunctionalTests/StoreGeneratedFixupJetTest.cs‎

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -83,117 +83,117 @@ protected override void OnModelCreating(ModelBuilder modelBuilder, DbContext con
8383
b =>
8484
{
8585
b.Property(e => e.Id1).ValueGeneratedOnAdd();
86-
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("newid()");
86+
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("GenGUID()");
8787
});
8888

8989
modelBuilder.Entity<Child>(
9090
b =>
9191
{
9292
b.Property(e => e.Id1).ValueGeneratedOnAdd();
93-
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("newid()");
93+
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("GenGUID()");
9494
});
9595

9696
modelBuilder.Entity<ParentPN>(
9797
b =>
9898
{
9999
b.Property(e => e.Id1).ValueGeneratedOnAdd();
100-
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("newid()");
100+
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("GenGUID()");
101101
});
102102

103103
modelBuilder.Entity<ChildPN>(
104104
b =>
105105
{
106106
b.Property(e => e.Id1).ValueGeneratedOnAdd();
107-
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("newid()");
107+
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("GenGUID()");
108108
});
109109

110110
modelBuilder.Entity<ParentDN>(
111111
b =>
112112
{
113113
b.Property(e => e.Id1).ValueGeneratedOnAdd();
114-
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("newid()");
114+
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("GenGUID()");
115115
});
116116

117117
modelBuilder.Entity<ChildDN>(
118118
b =>
119119
{
120120
b.Property(e => e.Id1).ValueGeneratedOnAdd();
121-
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("newid()");
121+
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("GenGUID()");
122122
});
123123

124124
modelBuilder.Entity<ParentNN>(
125125
b =>
126126
{
127127
b.Property(e => e.Id1).ValueGeneratedOnAdd();
128-
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("newid()");
128+
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("GenGUID()");
129129
});
130130

131131
modelBuilder.Entity<ChildNN>(
132132
b =>
133133
{
134134
b.Property(e => e.Id1).ValueGeneratedOnAdd();
135-
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("newid()");
135+
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("GenGUID()");
136136
});
137137

138138
modelBuilder.Entity<CategoryDN>(
139139
b =>
140140
{
141141
b.Property(e => e.Id1).ValueGeneratedOnAdd();
142-
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("newid()");
142+
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("GenGUID()");
143143
});
144144

145145
modelBuilder.Entity<ProductDN>(
146146
b =>
147147
{
148148
b.Property(e => e.Id1).ValueGeneratedOnAdd();
149-
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("newid()");
149+
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("GenGUID()");
150150
});
151151

152152
modelBuilder.Entity<CategoryPN>(
153153
b =>
154154
{
155155
b.Property(e => e.Id1).ValueGeneratedOnAdd();
156-
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("newid()");
156+
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("GenGUID()");
157157
});
158158

159159
modelBuilder.Entity<ProductPN>(
160160
b =>
161161
{
162162
b.Property(e => e.Id1).ValueGeneratedOnAdd();
163-
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("newid()");
163+
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("GenGUID()");
164164
});
165165

166166
modelBuilder.Entity<CategoryNN>(
167167
b =>
168168
{
169169
b.Property(e => e.Id1).ValueGeneratedOnAdd();
170-
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("newid()");
170+
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("GenGUID()");
171171
});
172172

173173
modelBuilder.Entity<ProductNN>(
174174
b =>
175175
{
176176
b.Property(e => e.Id1).ValueGeneratedOnAdd();
177-
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("newid()");
177+
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("GenGUID()");
178178
});
179179

180180
modelBuilder.Entity<Category>(
181181
b =>
182182
{
183183
b.Property(e => e.Id1).ValueGeneratedOnAdd();
184-
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("newid()");
184+
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("GenGUID()");
185185
});
186186

187187
modelBuilder.Entity<Product>(
188188
b =>
189189
{
190190
b.Property(e => e.Id1).ValueGeneratedOnAdd();
191-
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("newid()");
191+
b.Property(e => e.Id2).ValueGeneratedOnAdd().HasDefaultValueSql("GenGUID()");
192192
});
193193

194194
modelBuilder.Entity<Item>(b => b.Property(e => e.Id).ValueGeneratedOnAdd());
195195

196-
modelBuilder.Entity<Game>(b => b.Property(e => e.Id).ValueGeneratedOnAdd().HasDefaultValueSql("newid()"));
196+
modelBuilder.Entity<Game>(b => b.Property(e => e.Id).ValueGeneratedOnAdd().HasDefaultValueSql("GenGUID()"));
197197
}
198198
}
199199
}

0 commit comments

Comments
 (0)