From 2719399e85920c3fb616fc5b0de23d33f509a529 Mon Sep 17 00:00:00 2001
From: M66B <M66B@users.noreply.github.com>
Date: Fri, 30 Sep 2022 15:10:14 +0200
Subject: [PATCH] Refactoring

---
 .../faircode/email/BoundaryCallbackMessages.java   | 13 ++-----------
 .../main/java/eu/faircode/email/Fts4DbHelper.java  | 14 +++++++++-----
 2 files changed, 11 insertions(+), 16 deletions(-)

diff --git a/app/src/main/java/eu/faircode/email/BoundaryCallbackMessages.java b/app/src/main/java/eu/faircode/email/BoundaryCallbackMessages.java
index e0eb00a930..69a8c18281 100644
--- a/app/src/main/java/eu/faircode/email/BoundaryCallbackMessages.java
+++ b/app/src/main/java/eu/faircode/email/BoundaryCallbackMessages.java
@@ -788,17 +788,8 @@ public class BoundaryCallbackMessages extends PagedList.BoundaryCallback<TupleMe
     private static boolean contains(String text, String query) {
         if (TextUtils.isEmpty(text))
             return false;
-
-        text = text.toLowerCase();
-        query = query.toLowerCase();
-
-        query = Normalizer.normalize(query.toLowerCase(), Normalizer.Form.NFKD)
-                .replaceAll("[\\p{InCombiningDiacriticalMarks}]", "");
-
-        text = Normalizer.normalize(text, Normalizer.Form.NFKD)
-                .replaceAll("[\\p{InCombiningDiacriticalMarks}]", "");
-
-        return text.contains(query);
+        return Fts4DbHelper.preprocessText(text)
+                .contains(Fts4DbHelper.preprocessText(query));
     }
 
     State getState() {
diff --git a/app/src/main/java/eu/faircode/email/Fts4DbHelper.java b/app/src/main/java/eu/faircode/email/Fts4DbHelper.java
index d04f6f2544..643ee399b4 100644
--- a/app/src/main/java/eu/faircode/email/Fts4DbHelper.java
+++ b/app/src/main/java/eu/faircode/email/Fts4DbHelper.java
@@ -134,18 +134,22 @@ public class Fts4DbHelper extends SQLiteOpenHelper {
         db.delete("message", "rowid = ?", new String[]{Long.toString(id)});
     }
 
+    static String preprocessText(String text) {
+        return Normalizer.normalize(text.trim().toLowerCase(), Normalizer.Form.NFKD)
+                .replaceAll("[\\p{InCombiningDiacriticalMarks}]", "");
+    }
+
     private static String breakText(String text) {
         if (TextUtils.isEmpty(text))
             return "";
 
+        text = preprocessText(text);
+
         if (Build.VERSION.SDK_INT < Build.VERSION_CODES.N)
             return text;
 
         // https://www.sqlite.org/fts3.html#tokenizer
 
-        text = Normalizer.normalize(text.toLowerCase(), Normalizer.Form.NFKD)
-                .replaceAll("[\\p{InCombiningDiacriticalMarks}]", "");
-
         StringBuilder sb = new StringBuilder();
         android.icu.text.BreakIterator boundary = android.icu.text.BreakIterator.getWordInstance();
         boundary.setText(text);
@@ -172,7 +176,7 @@ public class Fts4DbHelper extends SQLiteOpenHelper {
                         " GROUP BY term" +
                         " ORDER BY SUM(occurrences) DESC" +
                         " LIMIT " + max,
-                new String[]{query})) {
+                new String[]{preprocessText(query)})) {
             while (cursor != null && cursor.moveToNext())
                 result.add(cursor.getString(0));
         }
@@ -185,7 +189,7 @@ public class Fts4DbHelper extends SQLiteOpenHelper {
             Long account, Long folder, long[] exclude,
             BoundaryCallbackMessages.SearchCriteria criteria) {
 
-        String query = breakText(criteria.query.trim());
+        String query = breakText(criteria.query);
 
         List<String> word = new ArrayList<>();
         List<String> plus = new ArrayList<>();