faster description pop-up

[mobile-ledger-staging.git] / app / src / main / java / net / ktnx / mobileledger / utils / MLDB.java

diff --git a/app/src/main/java/net/ktnx/mobileledger/utils/MLDB.java b/app/src/main/java/net/ktnx/mobileledger/utils/MLDB.java
index 5741a2e6fb45bf8357b7a8b07d66df3685ba1b62..07aea52b34b91a2e425ffda643c77d2a94f57e5f 100644 (file)
--- a/app/src/main/java/net/ktnx/mobileledger/utils/MLDB.java
+++ b/app/src/main/java/net/ktnx/mobileledger/utils/MLDB.java
@@ -20,16 +20,15 @@ package net.ktnx.mobileledger.utils;
 import android.annotation.TargetApi;
 import android.content.Context;
 import android.database.Cursor;
-import android.database.MatrixCursor;
 import android.database.sqlite.SQLiteDatabase;
 import android.os.AsyncTask;
 import android.os.Build;
-import android.provider.FontsContract;
-import android.view.View;
 import android.widget.AutoCompleteTextView;
 import android.widget.FilterQueryProvider;
 import android.widget.SimpleCursorAdapter;
 
+import androidx.annotation.NonNull;
+
 import net.ktnx.mobileledger.App;
 import net.ktnx.mobileledger.async.DbOpQueue;
 import net.ktnx.mobileledger.async.DescriptionSelectedCallback;
@@ -38,8 +37,6 @@ import net.ktnx.mobileledger.model.MobileLedgerProfile;
 
 import org.jetbrains.annotations.NonNls;
 
-import java.util.Locale;
-
 import static net.ktnx.mobileledger.utils.Logger.debug;
 
 public final class MLDB {
@@ -137,13 +134,13 @@ public final class MLDB {
     public static void hookAutocompletionAdapter(final Context context,
                                                  final AutoCompleteTextView view,
                                                  final String table, final String field) {
-        hookAutocompletionAdapter(context, view, table, field, true, null, null, null);
+        hookAutocompletionAdapter(context, view, table, field, true, null, null);
     }
     @TargetApi(Build.VERSION_CODES.N)
     public static void hookAutocompletionAdapter(final Context context,
                                                  final AutoCompleteTextView view,
                                                  final String table, final String field,
-                                                 final boolean profileSpecific, final View nextView,
+                                                 final boolean profileSpecific,
                                                  final DescriptionSelectedCallback callback,
                                                  final MobileLedgerProfile profile) {
         String[] from = {field};
@@ -158,60 +155,43 @@ public final class MLDB {
 
             String str = constraint.toString().toUpperCase();
             debug("autocompletion", "Looking for " + str);
-            String[] col_names = {FontsContract.Columns._ID, field};
-            MatrixCursor c = new MatrixCursor(col_names);
 
             String sql;
             String[] params;
             if (profileSpecific) {
                 MobileLedgerProfile p = (profile == null) ? Data.profile.getValue() : profile;
                 if (p == null) throw new AssertionError();
-                sql = String.format("SELECT %s as a, case when %s_upper LIKE ?||'%%' then 1 " +
-                                    "WHEN %s_upper LIKE '%%:'||?||'%%' then 2 " +
-                                    "WHEN %s_upper LIKE '%% '||?||'%%' then 3 else 9 end " +
-                                    "FROM %s " +
-                                    "WHERE profile=? AND %s_upper LIKE '%%'||?||'%%' " +
-                                    "ORDER BY 2, 1;", field, field, field, field, table, field);
+                sql = String.format(
+                        "SELECT rowid as _id, %s, CASE WHEN %s_upper LIKE ?||'%%' THEN 1 " +
+                        "WHEN %s_upper LIKE '%%:'||?||'%%' then 2 " +
+                        "WHEN %s_upper LIKE '%% '||?||'%%' THEN 3 " + "ELSE 9 END " + "FROM %s " +
+                        "WHERE profile=? AND %s_upper LIKE '%%'||?||'%%' " +
+                        "ORDER BY 3, %s_upper, 1;", field, field, field, field, table, field,
+                        field);
                 params = new String[]{str, str, str, p.getUuid(), str};
             }
             else {
-                sql = String.format("SELECT %s as a, case when %s_upper LIKE ?||'%%' then 1 " +
-                                    "WHEN %s_upper LIKE '%%:'||?||'%%' then 2 " +
-                                    "WHEN %s_upper LIKE '%% '||?||'%%' then 3 " + "else 9 end " +
-                                    "FROM %s " + "WHERE %s_upper LIKE '%%'||?||'%%' " +
-                                    "ORDER BY 2, 1;", field, field, field, field, table, field);
+                sql = String.format(
+                        "SELECT rowid as _id, %s, CASE WHEN %s_upper LIKE ?||'%%' THEN 1 " +
+                        "WHEN %s_upper LIKE '%%:'||?||'%%' THEN 2 " +
+                        "WHEN %s_upper LIKE '%% '||?||'%%' THEN 3 " + "ELSE 9 END " + "FROM %s " +
+                        "WHERE %s_upper LIKE '%%'||?||'%%' " + "ORDER BY 3, %s_upper, 1;", field,
+                        field, field, field, table, field, field);
                 params = new String[]{str, str, str, str};
             }
             debug("autocompletion", sql);
             SQLiteDatabase db = App.getDatabase();
 
-            try (Cursor matches = db.rawQuery(sql, params)) {
-                int i = 0;
-                while (matches.moveToNext()) {
-                    String match = matches.getString(0);
-                    int order = matches.getInt(1);
-                    debug("autocompletion",
-                            String.format(Locale.ENGLISH, "match: %s |%d", match, order));
-                    c.newRow().add(i++).add(match);
-                }
-            }
-
-            return c;
-
+            return db.rawQuery(sql, params);
         };
 
         adapter.setFilterQueryProvider(provider);
 
         view.setAdapter(adapter);
 
-        if (nextView != null) {
-            view.setOnItemClickListener((parent, itemView, position, id) -> {
-                nextView.requestFocus(View.FOCUS_FORWARD);
-                if (callback != null) {
-                    callback.descriptionSelected(String.valueOf(view.getText()));
-                }
-            });
-        }
+        if (callback != null) view.setOnItemClickListener((parent, itemView, position, id) -> {
+            callback.descriptionSelected(String.valueOf(view.getText()));
+        });
     }
 }