Skip to content

Commit 59da1f3

Browse files
committed
NH-3807 - Replace use of InvariantCultureIgnoreCase with OrdinalIgnoreCase.
StringComparison.InvariantCultureIgnoreCase and StringComparer.InvariantCultureIgnoreCase has been removed from CoreClr.
1 parent dfc8a1b commit 59da1f3

File tree

15 files changed

+50
-50
lines changed

15 files changed

+50
-50
lines changed

src/NHibernate.Test/NHSpecificTest/NH732/Fixture.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -70,12 +70,12 @@ public System.Type ReturnedType
7070

7171
public new bool Equals(object x, object y)
7272
{
73-
return StringComparer.InvariantCultureIgnoreCase.Equals((string)x, (string)y);
73+
return StringComparer.OrdinalIgnoreCase.Equals((string)x, (string)y);
7474
}
7575

7676
public int GetHashCode(object x)
7777
{
78-
return StringComparer.InvariantCultureIgnoreCase.GetHashCode((string)x);
78+
return StringComparer.OrdinalIgnoreCase.GetHashCode((string)x);
7979
}
8080

8181
public object NullSafeGet(DbDataReader rs, string[] names, object owner)

src/NHibernate/Cfg/Mappings.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -538,7 +538,7 @@ public void AddColumnBinding(string logicalName, Column finalColumn, Table table
538538
if (oldFinalName != null &&
539539
!(finalColumn.IsQuoted
540540
? oldFinalName.Equals(finalColumn.GetQuotedName())
541-
: oldFinalName.Equals(finalColumn.GetQuotedName(), StringComparison.InvariantCultureIgnoreCase)))
541+
: oldFinalName.Equals(finalColumn.GetQuotedName(), StringComparison.OrdinalIgnoreCase)))
542542
{
543543
//TODO possibly relax that
544544
throw new MappingException("Same logical column name referenced by different physical ones: " + table.Name + "."

src/NHibernate/Dialect/Function/CastFunction.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ protected virtual bool CastingIsRequired(string sqlType)
9393

9494
bool IFunctionGrammar.IsSeparator(string token)
9595
{
96-
return "as".Equals(token, StringComparison.InvariantCultureIgnoreCase);
96+
return "as".Equals(token, StringComparison.OrdinalIgnoreCase);
9797
}
9898

9999
bool IFunctionGrammar.IsKnownArgument(string token)

src/NHibernate/Dialect/Function/SQLFunctionRegistry.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ public SQLFunctionRegistry(Dialect dialect, IDictionary<string, ISQLFunction> us
1212
{
1313
this.dialect = dialect;
1414
this.userFunctions = new Dictionary<string, ISQLFunction>(userFunctions,
15-
StringComparer.InvariantCultureIgnoreCase);
15+
StringComparer.OrdinalIgnoreCase);
1616
}
1717

1818
public ISQLFunction FindSQLFunction(string functionName)

src/NHibernate/Hql/Ast/ANTLR/Tree/FromClause.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -325,7 +325,7 @@ private FromElement FindIntendedAliasedFromElementBasedOnCrazyJPARequirements(st
325325
foreach (var entry in _fromElementByClassAlias)
326326
{
327327
string alias = entry.Key;
328-
if (string.Equals(alias, specifiedAlias, StringComparison.InvariantCultureIgnoreCase))
328+
if (string.Equals(alias, specifiedAlias, StringComparison.OrdinalIgnoreCase))
329329
{
330330
return entry.Value;
331331
}

src/NHibernate/Mapping/ByCode/SimpleModelInspector.cs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -353,10 +353,10 @@ protected bool CanReadCantWriteInBaseType(PropertyInfo property)
353353
protected bool MatchPoIdPattern(MemberInfo subject)
354354
{
355355
var name = subject.Name;
356-
return name.Equals("id", StringComparison.InvariantCultureIgnoreCase)
357-
|| name.Equals("poid", StringComparison.InvariantCultureIgnoreCase)
358-
|| name.Equals("oid", StringComparison.InvariantCultureIgnoreCase)
359-
|| (name.StartsWith(subject.DeclaringType.Name) && name.Equals(subject.DeclaringType.Name + "id", StringComparison.InvariantCultureIgnoreCase));
356+
return name.Equals("id", StringComparison.OrdinalIgnoreCase)
357+
|| name.Equals("poid", StringComparison.OrdinalIgnoreCase)
358+
|| name.Equals("oid", StringComparison.OrdinalIgnoreCase)
359+
|| (name.StartsWith(subject.DeclaringType.Name) && name.Equals(subject.DeclaringType.Name + "id", StringComparison.OrdinalIgnoreCase));
360360
}
361361

362362
protected bool MatchComponentPattern(System.Type subject)
@@ -1083,4 +1083,4 @@ public void IsDynamicComponent(Func<MemberInfo, bool, bool> match)
10831083
isDynamicComponent = match;
10841084
}
10851085
}
1086-
}
1086+
}

src/NHibernate/Mapping/ManyToOne.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ public void CreatePropertyRefConstraints(IDictionary<string, PersistentClass> pe
7373
if (property == null)
7474
throw new MappingException("Could not find property " + ReferencedPropertyName + " on " + ReferencedEntityName);
7575

76-
if (!HasFormula && !"none".Equals(ForeignKeyName, StringComparison.InvariantCultureIgnoreCase))
76+
if (!HasFormula && !"none".Equals(ForeignKeyName, StringComparison.OrdinalIgnoreCase))
7777
{
7878

7979
IEnumerable<Column> ce = new SafetyEnumerable<Column>(property.ColumnIterator);
@@ -92,4 +92,4 @@ public void CreatePropertyRefConstraints(IDictionary<string, PersistentClass> pe
9292
}
9393
}
9494
}
95-
}
95+
}

src/NHibernate/Mapping/ReferenceDependantValue.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ public IEnumerable<Column> ReferenceColumns
2424

2525
public override void CreateForeignKeyOfEntity(string entityName)
2626
{
27-
if (!HasFormula && !string.Equals("none", ForeignKeyName, StringComparison.InvariantCultureIgnoreCase))
27+
if (!HasFormula && !string.Equals("none", ForeignKeyName, StringComparison.OrdinalIgnoreCase))
2828
{
2929
var referencedColumns = new List<Column>(_prototype.ColumnSpan);
3030
foreach (Column column in _prototype.ColumnIterator)

src/NHibernate/Mapping/SimpleValue.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ public virtual bool IsComposite
7575

7676
public virtual void CreateForeignKeyOfEntity(string entityName)
7777
{
78-
if (!HasFormula && ! "none".Equals(ForeignKeyName, StringComparison.InvariantCultureIgnoreCase))
78+
if (!HasFormula && ! "none".Equals(ForeignKeyName, StringComparison.OrdinalIgnoreCase))
7979
{
8080
ForeignKey fk = table.CreateForeignKey(ForeignKeyName, ConstraintColumns, entityName);
8181
fk.CascadeDeleteEnabled = cascadeDeleteEnabled;

src/NHibernate/SqlCommand/Parser/MsSqlSelectParser.cs

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -134,15 +134,15 @@ private IEnumerable<ColumnDefinition> ParseColumnDefinitions(IEnumerator<SqlToke
134134
case SqlTokenType.Text:
135135
if (blockLevel != 0) break;
136136

137-
if (token.Equals(",", StringComparison.InvariantCultureIgnoreCase))
137+
if (token.Equals(",", StringComparison.OrdinalIgnoreCase))
138138
{
139139
if (columnAliasToken != null)
140140
{
141141
yield return ParseSelectColumnDefinition(columnBeginToken, columnEndToken ?? columnAliasToken, columnAliasToken);
142142
}
143143
}
144144

145-
if (token.Equals("from", StringComparison.InvariantCultureIgnoreCase))
145+
if (token.Equals("from", StringComparison.OrdinalIgnoreCase))
146146
{
147147
if (columnAliasToken != null)
148148
{
@@ -151,7 +151,7 @@ private IEnumerable<ColumnDefinition> ParseColumnDefinitions(IEnumerator<SqlToke
151151
yield break;
152152
}
153153

154-
if (token.Equals("as", StringComparison.InvariantCultureIgnoreCase))
154+
if (token.Equals("as", StringComparison.OrdinalIgnoreCase))
155155
{
156156
columnEndToken = prevToken;
157157
}
@@ -240,8 +240,8 @@ private IEnumerable<OrderDefinition> ParseOrderDefinitions(IEnumerator<SqlToken>
240240
case SqlTokenType.Text:
241241
if (blockLevel != 0) break;
242242

243-
if (token.Equals("asc", StringComparison.InvariantCultureIgnoreCase)
244-
|| token.Equals("desc", StringComparison.InvariantCultureIgnoreCase))
243+
if (token.Equals("asc", StringComparison.OrdinalIgnoreCase)
244+
|| token.Equals("desc", StringComparison.OrdinalIgnoreCase))
245245
{
246246
orderEndToken = prevToken;
247247
directionToken = token;
@@ -272,7 +272,7 @@ private IEnumerable<OrderDefinition> ParseOrderDefinitions(IEnumerator<SqlToken>
272272
private OrderDefinition ParseOrderDefinition(SqlToken beginToken, SqlToken endToken, SqlToken directionToken)
273273
{
274274
var isDescending = directionToken != null &&
275-
directionToken.Equals("desc", StringComparison.InvariantCultureIgnoreCase);
275+
directionToken.Equals("desc", StringComparison.OrdinalIgnoreCase);
276276

277277
var columnNameOrIndex = beginToken == endToken
278278
? beginToken.Value
@@ -301,8 +301,8 @@ private bool TryGetColumnDefinition(string columnNameOrIndex, out ColumnDefiniti
301301

302302
foreach (var column in _columns)
303303
{
304-
if (columnNameOrIndex.Equals(column.Name, StringComparison.InvariantCultureIgnoreCase)
305-
|| columnNameOrIndex.Equals(column.Alias, StringComparison.InvariantCultureIgnoreCase))
304+
if (columnNameOrIndex.Equals(column.Name, StringComparison.OrdinalIgnoreCase)
305+
|| columnNameOrIndex.Equals(column.Alias, StringComparison.OrdinalIgnoreCase))
306306
{
307307
result = column;
308308
return true;
@@ -316,9 +316,9 @@ private bool TryGetColumnDefinition(string columnNameOrIndex, out ColumnDefiniti
316316
var text = Sql.ToString(sqlIndex, sqlLength);
317317
foreach (var column in _columns)
318318
{
319-
if (text.Equals(column.Name, StringComparison.InvariantCultureIgnoreCase) ||
320-
text.Equals(column.Alias, StringComparison.InvariantCultureIgnoreCase) ||
321-
text.Equals(Sql.ToString(column.SqlIndex, column.SqlLength), StringComparison.InvariantCultureIgnoreCase))
319+
if (text.Equals(column.Name, StringComparison.OrdinalIgnoreCase) ||
320+
text.Equals(column.Alias, StringComparison.OrdinalIgnoreCase) ||
321+
text.Equals(Sql.ToString(column.SqlIndex, column.SqlLength), StringComparison.OrdinalIgnoreCase))
322322
{
323323
result = column;
324324
return true;

0 commit comments

Comments
 (0)