a8b975d4b9
Add code to add a "MatchSimple" annotation to a foreign key
2 years ago
e04ee43a29
UNIONing projection with a numeric column to another projection that specifically projects NULL the the same name as a numeric field, the JOINING that to another table can leave that numeric field as another data type (NULL is not a numeric data type). Hence when using that supposedly numeric field in the join predicat against another numeric field we get a data type mismatch.
...
We try to detect this and when visiting the column (but only if we are within a binary expression and not a projection) we quickly wrap it in a convert function to make it properly numeric
The top/outer projection of this field can still be in a non numeric format, but JetDataReader is able to handle that. Would be better to produce the correctly converted data output anyway
2 years ago
ffef910dae
Add Check_all_tests_overridden to more test classes and ensure that it passes
2 years ago
11f406955d
Fix some conversion errors when getting a numeric value. See detailed notes in the GetInt32 function
2 years ago
92ea73548d
DISTINCT AND TOP can't be in the same stateement with Jet. Push DISTINCT into a subquery
2 years ago
f1878a8950
Fix generating sets (union, intersect). The code was originally executing and returning before it could handle adding the SQL for the Alias in (including wrapping the section in brackets). This realigns with how the parent VisitSelect works
2 years ago
2230f06f13
With InlineTopParameters it didn't handle the case of a sum of two constant valuees
2 years ago
8ba710ed47
Fix the formatting for a string literal of a byte
2 years ago
70e4de26e3
Make sure the start index for Instr is set. Normally is optional but it seems some sections seem to prefer it to be defined
2 years ago
a93c205194
When using FromSql, parameters can end up named without the @ at the beginning. This leads to the failure to match it to the Sql where the string is generated with the @.
...
If we can't find the parameter using the normal name, try to pick it up when excluding the @
2 years ago
db7a513b64
Remove the DeepSkip tag on FirstOrDefault. When generating the sql it was preventing the TOP expression from being generated
2 years ago
fbf0f794a7
use .GetMAppStoreObjects to try get the correct table (this is especially needed for properties in split tables). Default to the original storeObject if can't find anything
2 years ago
2dbd0fde32
Rewrite some INNER JOIN's. According to the docs Jet doesn't support an INNER JOIN being nested under a LEFT JOIN. We try to figure out which one's they are and rewrite it. As an INNER JOIN is a join where both sides have a value, we can rewrite it as a LEFT JOIN and then filter out the rows where the columns the join is on is not NULL
2 years ago
c46f980c09
Start work on some support for sequence value generation. Still under investigation if possible
2 years ago
944843bfad
Some datetime parameters don't work unless they have been properly converted
2 years ago
3ef1503bb9
Translate the Atan2 function
3 years ago
5e32650f6d
Dispose of any new JetCommands created when splitting commands properly. This were not being disposed of and still holding a link to the database thereby preventing some file access e.g. Deleting a file
3 years ago
3f5c1b6005
Make sure a DateTime in a constant has a DateTime Type Mapping
3 years ago
429c67ca3a
Some skip...take optimization and fix using multiple take after a skip
3 years ago
8c9dff5d36
Initial support for Skip....Take. Take...Skip and Skip currently do not work properly
3 years ago
fd57f4a3bd
Rename some extension methods so that it doesn't interfere with other providers (namely sql server)
3 years ago
fd6c2aec01
Revert ca0feb49 . Same name extensions when using sql server and jet
3 years ago
84e40f5085
Fix some string resources
3 years ago
cbb3f94440
Remove unnecessary references in `EntityFrameworkCore.Jet.Data`
...
Now that `EntityFrameworkCore.Jet.Data` targets `net6.0` those references are not required anymore.
Note that the `Microsoft.Win32.Registry` package reference was the root cause of the failure to publish the `EntityFrameworkCore.Jet.Data` 7.0.0 package because of [warning NU5104][1] (treated as error):
> A stable release of a package should not have a prerelease dependency. Either modify the version spec of dependency "Microsoft.Win32.Registry [6.0.0-preview.5.21301.5, )" or update the version field in the nuspec.
[1]: https://bubibubi.visualstudio.com/EntityFrameworkCore.Jet/_build/results?buildId=280&view=logs&j=67ea8d94-b5a7-504b-2d4b-07baddfad3f1&t=1e07b7f6-b7a1-55f6-9cca-c04683790a41&l=37
3 years ago
a9c2f36da5
Jet conversion functions like Cdbl/Csng/CLng don't accept or propogate null values. Normally we can just use an IIF to check if the expression is null or not. That does work fine but here we also check if the base expression is nullable. If the base expression will never return null we can simplify and just use the base expression rather than test for null
3 years ago
6963a165e5
The result of the Round function is never null
3 years ago
c8e109fb94
Jet can't candle a left join straight after a cross join. We have to push the cross join specifically down into its own subquery and then do a left join on that
3 years ago
83c29e4c43
Introduce a new typed JetConnectionStringBuilder
...
This makes it easier to use than the extensions (EntityFrameworkCore.Jet.Data.DbConnectionStringBuilderExtensions) with Get/Set methods.
With the new `JetConnectionStringBuilder` class:
```csharp
var csb = new JetConnectionStringBuilder(DataAccessProviderType.OleDb)
{
Provider = "Microsoft.ACE.OLEDB.12.0",
DataSource = @"C:\myFolder\myAccessFile.accdb",
DatabasePassword = "hunter2",
};
var connectionString = csb.ConnectionString;
```
Without the new `JetConnectionStringBuilder` class:
```csharp
var csb = new OleDbConnectionStringBuilder();
csb.SetProvider("Microsoft.ACE.OLEDB.12.0");
csb.SetDataSource(@"C:\myFolder\myAccessFile.accdb");
csb.SetDatabasePassword("hunter2");
var connectionString = csb.ConnectionString;
```
3 years ago
c795ce6228
Make EntityFrameworkCore.Jet available on `net6.0`
...
By using `[SupportedOSPlatform("windows")]` at assembly level instead of targeting `net6.0-windows`.
This will enable taking a dependency on the EntityFrameworkCore.Jet* NuGet packages on Linux and macOS. The consumer of EntityFrameworkCore.Jet can then decide how to handle [CA1416][1] either by adding `[SupportedOSPlatform("windows")]`, by targeting `net6.0-windows` or by testing `OperatingSystem.IsWindows()` at runtime.
[1]: https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca1416
3 years ago
12aca42198
Add option to use the normal short text type when mapping against an unbounded System.String instead of long text/memo. Jet has limitations when using memo (e.g. joins are not supported)
3 years ago
950089a113
Set the precision to be maximum of 28 (Jet limit).
...
When using property.HasConversion<decimal>() the default for EF Core is with precision 38 and scale 17 which is passed Jet's limit
3 years ago
acc3fd5223
Revert "Set the precision to be maximum of 28 (Jet limit)."
...
This reverts commit a1904bf113
3 years ago
a1904bf113
Set the precision to be maximum of 28 (Jet limit).
...
When using property.HasConversion<decimal>() the default for EF Core is with precision 38 and scale 17 which is passed Jet's limit
3 years ago
dc049d804d
Do the same clr type check when mapping "text" as what we do for other store type mappings. This allows conversions do work better (e.g. originally a mappingInfo with a System.Char type was passed in with store type of "text" and a variable length or unbounded string type was returned but with clr type of System.String. Going with the extra checked returns null and gets EF Core to find a converter and then do the mapping again on the converted type
3 years ago
ee54f1fac0
Use MidB/InstrB for the byte versions. (still doesnt produce correct results yet)
3 years ago
ee30910fa9
Update the ValueGeneratorSelector to current ef core style. Don't rely on sequential guid's as Jet doesn't order them properly (treats them as string as sorts them numerically/alphabetically)
3 years ago
0eb8baefa9
Add a clause/filter to the index to ignore nulls
3 years ago
306585af13
When dealing with convert, if we don't have a specific conversion function, try just send the original operand and let Jet work with the types. Seems to be fairly lenient
3 years ago
6cb1748237
Remove sbyte from clr mappings. Not needed as EF Core will use automatic conversions for those types. EF Core SQL Server behaves the same
3 years ago
9489bc4564
Jet seems to output bytes only as an array. so a single byte becomes a byte[1] array. Return the first byte of that array in this instance
3 years ago
3b0bc7d5ad
Fix the GetValueGenerated function. Was not being called and as such for an identity the ValueGenerated.OnAdd was not being set
3 years ago
78f3dbae15
Use our own Byte type mapping for generating the correct string literal
3 years ago
bc5b51fa4a
Fix check against typeof JetBoolTypeMapping. For the tests Everything is Bytes/Strings all type mappings are the same which makes the check not behave correctly
3 years ago
f15d60aaab
Fix escaping wild chars. Use the correct form to escape. Also the LIKE clause does nt have an ESCAPE clause to set the escape char so set that to null
3 years ago
73ef68188f
Ordering of boolean: Rather order by the NOT expression rather than change the ascending or descending order. NULL values are handled in the correct order now
...
Division: If the result of the binary is meant to be integer, use the Jet binary divide operatior '\'
3 years ago
87aa0dbe93
Fix incorrect check for whether ordering expression is boolean typeexpression if it is a boolean type.
3 years ago
03579d380c
Add a JetGuidTypeMapping class. We need to override the template for the SQL literal. Jet uses the format with th curly braces at the beginning and end. Default (and same as sql server) is the format without curly braces
3 years ago
d7033125be
Properly read a TimeSpan value from the database.
...
Add member translator for timespan so SQL works
3 years ago
096a0955e2
Add JetQueryableMethodTranslatingExpressionVisitorFactory if needed to override anything from it
3 years ago
e01686d4b5
MID doesn't allow a null value for the length parameter. Add a IIF wrapper around it if that argument is marked as nullable.
...
Added here instead of stringmethodtranslator as we need the nullability optimizations on the sqlfunctionexpression. When wrapped in a caseexpression we get some different optimizations. Produces the correct result on the test case but is different to SQL Server. This way we keep it closer to SQL Server
3 years ago
Christopher Jolly
Cédric Luthi