Fixed ignore case search option by respecting current culture

MBulli · Oct 22, 2017 · fa820ca · fa820ca
1 parent e83c1d0
commit fa820ca
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/SmartCmdArgs/SmartCmdArgs/Helper/Extensions.cs b/SmartCmdArgs/SmartCmdArgs/Helper/Extensions.cs
@@ -18,5 +18,10 @@ public static FileSystemWatcherDisabledContext TemporarilyDisable(this System.IO
         {
             return new FileSystemWatcherDisabledContext(watcher);
         }
+
+        public static bool Contains(this string str, string value, StringComparison comparisonType)
+        {
+            return str.IndexOf(value, comparisonType) >= 0;
+        }
     }
 }
diff --git a/SmartCmdArgs/SmartCmdArgs/ViewModel/ListViewModel.cs b/SmartCmdArgs/SmartCmdArgs/ViewModel/ListViewModel.cs
@@ -149,8 +149,7 @@ public void SetStringFilter(string filter, bool matchCase = false)
                     }
                     else
                     {
-                        filter = filter.ToLower();
-                        DataCollectionView.Filter = item => ((CmdArgItem) item).Command.ToLower().Contains(filter);
+                        DataCollectionView.Filter = item => ((CmdArgItem) item).Command.Contains(filter, StringComparison.CurrentCultureIgnoreCase);
                     }
                 }
             });
-Original file line number
+Diff line change
@@ Expand Up @@
                         }
                         else
                         {
-                            filter = filter.ToLower();
-                            DataCollectionView.Filter = item => ((CmdArgItem) item).Command.ToLower().Contains(filter);
+                            DataCollectionView.Filter = item => ((CmdArgItem) item).Command.Contains(filter, StringComparison.CurrentCultureIgnoreCase);
                         }
                     }
                 });
@@ Expand Down @@