Skip to content

Commit ff04fef

Browse files
committed
Fix CaseInsensitiveStringComparer
1 parent b6f5147 commit ff04fef

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

src/MigrationTools.Clients.TfsObjectModel/Tools/TfsUserMappingTool.cs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,8 @@ public TfsUserMappingTool(IOptions<TfsUserMappingToolOptions> options, IServiceP
2222
{
2323
}
2424

25+
private readonly CaseInsensitiveStringComparer _workItemNameComparer = new();
26+
2527
private HashSet<string> GetUsersFromWorkItems(List<WorkItemData> workitems, List<string> identityFieldsToCheck)
2628
{
2729
HashSet<string> foundUsers = new(StringComparer.CurrentCultureIgnoreCase);
@@ -31,7 +33,7 @@ private HashSet<string> GetUsersFromWorkItems(List<WorkItemData> workitems, List
3133
{
3234
foreach (var fItem in rItem.Fields.Values)
3335
{
34-
if (identityFieldsToCheck.Contains(fItem.ReferenceName, new CaseInsensativeStringComparer()))
36+
if (identityFieldsToCheck.Contains(fItem.ReferenceName, _workItemNameComparer))
3537
{
3638
if (!foundUsers.Contains(fItem.Value) && !string.IsNullOrEmpty((string)fItem.Value))
3739
{
@@ -162,7 +164,7 @@ public List<IdentityMapData> GetUsersInSourceMappedToTargetForWorkItems(TfsProce
162164
}
163165
}
164166

165-
public class CaseInsensativeStringComparer : IEqualityComparer<string>
167+
internal class CaseInsensitiveStringComparer : IEqualityComparer<string>
166168
{
167169
public bool Equals(string x, string y)
168170
{

0 commit comments

Comments
 (0)