Fix #837 search text is always lowercase

mezz · Apr 30, 2017 · 13964bf · 13964bf
1 parent 4cfa615
commit 13964bf
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 10 deletions.
diff --git a/src/main/java/mezz/jei/config/Config.java b/src/main/java/mezz/jei/config/Config.java
@@ -152,13 +152,12 @@ public enum SearchMode {
 	}
 
 	public static boolean setFilterText(String filterText) {
-		String lowercaseFilterText = filterText.toLowerCase();
-		if (values.filterText.equals(lowercaseFilterText)) {
+		if (values.filterText.equals(filterText)) {
 			return false;
+		} else {
+			values.filterText = filterText;
+			return true;
 		}
-
-		values.filterText = lowercaseFilterText;
-		return true;
 	}
 
 	public static String getFilterText() {

diff --git a/src/main/java/mezz/jei/ingredients/IngredientFilterInternals.java b/src/main/java/mezz/jei/ingredients/IngredientFilterInternals.java
@@ -123,15 +123,16 @@ private void buildSuffixTrees(List<IIngredientListElement> ingredientList) {
 	}
 
 	public ImmutableList<Object> getIngredientList() {
-		if (!Config.getFilterText().equals(filterCached)) {
-			ingredientListCached = getIngredientListUncached();
-			filterCached = Config.getFilterText();
+		String filterText = Config.getFilterText().toLowerCase();
+		if (!filterText.equals(filterCached)) {
+			ingredientListCached = getIngredientListUncached(filterText);
+			filterCached = filterText;
 		}
 		return ingredientListCached;
 	}
 
-	private ImmutableList<Object> getIngredientListUncached() {
-		String[] filters = Config.getFilterText().split("\\|");
+	private ImmutableList<Object> getIngredientListUncached(String filterText) {
+		String[] filters = filterText.split("\\|");
 
 		if (filters.length == 1) {
 			String filter = filters[0];