Skip to content

Commit 897a9df

Browse files
Fix deepsource concerns
Excepted those unrelated to the change and most of those in the tests.
1 parent 75b9f9d commit 897a9df

File tree

3 files changed

+7
-5
lines changed

3 files changed

+7
-5
lines changed

src/NHibernate.Test/Async/Linq/DateTimeTests.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,7 @@ private void AssertDateTimeOffsetSupported()
111111
using var session = OpenSession();
112112
using var trans = session.BeginTransaction();
113113
var entity1 = await (session.GetAsync<DateTimeTestsClass>(_referenceEntities[0].Id, cancellationToken));
114+
await (trans.CommitAsync(cancellationToken));
114115
if (entity1.DateTimeValueWithScale != entity1.DateTimeValue + FractionalSecondsAdded)
115116
{
116117
Assert.Ignore("Current setup doesn't support DateTime with scale (2)");

src/NHibernate.Test/Linq/DateTimeTests.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,7 @@ private void AssertDateTimeWithFractionalSecondsSupported()
9999
using var session = OpenSession();
100100
using var trans = session.BeginTransaction();
101101
var entity1 = session.Get<DateTimeTestsClass>(_referenceEntities[0].Id);
102+
trans.Commit();
102103
if (entity1.DateTimeValueWithScale != entity1.DateTimeValue + FractionalSecondsAdded)
103104
{
104105
Assert.Ignore("Current setup doesn't support DateTime with scale (2)");

src/NHibernate/Dialect/Function/SQLFunctionRegistry.cs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,8 @@ public class SQLFunctionRegistry
77
{
88
private readonly Dialect dialect;
99
private readonly IDictionary<string, ISQLFunction> userFunctions;
10-
//Temporary alias support
11-
private static Dictionary<string, string> _functionAliases = new Dictionary<string, string>(StringComparer.OrdinalIgnoreCase) { { "secondtruncated", "second" } };
10+
// Temporary alias support
11+
private readonly static Dictionary<string, string> _functionAliases = new(StringComparer.OrdinalIgnoreCase) { { "secondtruncated", "second" } };
1212

1313
public SQLFunctionRegistry(Dialect dialect, IDictionary<string, ISQLFunction> userFunctions)
1414
{
@@ -22,10 +22,10 @@ public SQLFunctionRegistry(Dialect dialect, IDictionary<string, ISQLFunction> us
2222
/// </summary>
2323
public ISQLFunction FindSQLFunction(string functionName)
2424
{
25-
if (!userFunctions.TryGetValue(functionName, out ISQLFunction result) && !dialect.Functions.TryGetValue(functionName, out result))
25+
if (!userFunctions.TryGetValue(functionName, out ISQLFunction result) && !dialect.Functions.TryGetValue(functionName, out result)
26+
&& _functionAliases.TryGetValue(functionName, out var sqlFunction) && !_functionAliases.ContainsKey(sqlFunction))
2627
{
27-
if (_functionAliases.TryGetValue(functionName, out var sqlFunction) && !_functionAliases.ContainsKey(sqlFunction))
28-
return FindSQLFunction(sqlFunction);
28+
return FindSQLFunction(sqlFunction);
2929
}
3030
return result;
3131
}

0 commit comments

Comments
 (0)