From d909c69245f6d64a241e645a3632aa10caf9ebf1 Mon Sep 17 00:00:00 2001
From: M66B <M66B@users.noreply.github.com>
Date: Wed, 24 Feb 2021 19:18:50 +0100
Subject: [PATCH] Fixed searching w/o index

---
 .../email/BoundaryCallbackMessages.java       | 24 +++----------------
 .../faircode/email/FragmentDialogSearch.java  |  3 ++-
 2 files changed, 5 insertions(+), 22 deletions(-)

diff --git a/app/src/main/java/eu/faircode/email/BoundaryCallbackMessages.java b/app/src/main/java/eu/faircode/email/BoundaryCallbackMessages.java
index b1cf90aba7..2c575f731c 100644
--- a/app/src/main/java/eu/faircode/email/BoundaryCallbackMessages.java
+++ b/app/src/main/java/eu/faircode/email/BoundaryCallbackMessages.java
@@ -201,7 +201,7 @@ public class BoundaryCallbackMessages extends PagedList.BoundaryCallback<TupleMe
         SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(context);
         boolean fts = prefs.getBoolean("fts", false);
         boolean pro = ActivityBilling.isPro(context);
-        if (fts && pro && criteria.isQueryOnly()) {
+        if (fts && pro && criteria.fts && criteria.query != null) {
             if (state.ids == null) {
                 SQLiteDatabase sdb = FtsDbHelper.getInstance(context);
                 state.ids = FtsDbHelper.match(sdb, account, folder, criteria);
@@ -641,6 +641,7 @@ public class BoundaryCallbackMessages extends PagedList.BoundaryCallback<TupleMe
 
     static class SearchCriteria implements Serializable {
         String query;
+        boolean fts = false;
         boolean in_senders = true;
         boolean in_recipients = true;
         boolean in_subject = true;
@@ -663,26 +664,6 @@ public class BoundaryCallbackMessages extends PagedList.BoundaryCallback<TupleMe
             this.query = query;
         }
 
-        boolean isQueryOnly() {
-            return (!TextUtils.isEmpty(query) &&
-                    in_senders &&
-                    in_recipients &&
-                    in_subject &&
-                    in_keywords &&
-                    in_message &&
-                    isWithout());
-        }
-
-        boolean isWithout() {
-            return !(with_unseen ||
-                    with_flagged ||
-                    with_hidden ||
-                    with_encrypted ||
-                    with_attachments ||
-                    with_types != null ||
-                    with_size != null);
-        }
-
         boolean isExpression() {
             if (this.query == null)
                 return false;
@@ -890,6 +871,7 @@ public class BoundaryCallbackMessages extends PagedList.BoundaryCallback<TupleMe
         @Override
         public String toString() {
             return query +
+                    " fts=" + fts +
                     " senders=" + in_senders +
                     " recipients=" + in_recipients +
                     " subject=" + in_subject +
diff --git a/app/src/main/java/eu/faircode/email/FragmentDialogSearch.java b/app/src/main/java/eu/faircode/email/FragmentDialogSearch.java
index 6364b7c98d..b80645d756 100644
--- a/app/src/main/java/eu/faircode/email/FragmentDialogSearch.java
+++ b/app/src/main/java/eu/faircode/email/FragmentDialogSearch.java
@@ -282,7 +282,8 @@ public class FragmentDialogSearch extends FragmentDialogBase {
                         else
                             prefs.edit().putString("last_search", criteria.query).apply();
 
-                        if (!cbSearchIndex.isChecked()) {
+                        criteria.fts = cbSearchIndex.isChecked();
+                        if (!fts) {
                             criteria.in_senders = cbSenders.isChecked();
                             criteria.in_recipients = cbRecipients.isChecked();
                             criteria.in_subject = cbSubject.isChecked();