From 6e7ad0f27dd6dcc3ad17f045ae7c51f3b13a6f34 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Dominik=20Sch=C3=BCrmann?= <dominik@dominikschuermann.de>
Date: Thu, 26 Jan 2017 23:42:37 +0100
Subject: [PATCH] Disable search view from SelectPublicKeyFragment.

Needs to be fixed. When searching, previously
selected keys are no longer selected.
---
 .../remote/ui/SelectPublicKeyFragment.java    | 27 ++++++++++---------
 1 file changed, 14 insertions(+), 13 deletions(-)

diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/remote/ui/SelectPublicKeyFragment.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/remote/ui/SelectPublicKeyFragment.java
index 920c36c4a..eac5189ea 100644
--- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/remote/ui/SelectPublicKeyFragment.java
+++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/remote/ui/SelectPublicKeyFragment.java
@@ -54,7 +54,7 @@ public class SelectPublicKeyFragment extends RecyclerFragment<SelectEncryptKeyAd
         implements TextWatcher, LoaderManager.LoaderCallbacks<Cursor> {
     public static final String ARG_PRESELECTED_KEY_IDS = "preselected_key_ids";
 
-    private EditText mSearchView;
+//    private EditText mSearchView;
     private long mSelectedMasterKeyIds[];
     private String mQuery;
 
@@ -110,17 +110,18 @@ public class SelectPublicKeyFragment extends RecyclerFragment<SelectEncryptKeyAd
         LinearLayout innerListContainer = new LinearLayout(context);
         innerListContainer.setOrientation(LinearLayout.VERTICAL);
 
-        mSearchView = new EditText(context);
-        mSearchView.setId(android.R.id.input);
-        mSearchView.setHint(R.string.menu_search);
-        mSearchView.setCompoundDrawablesWithIntrinsicBounds(
-                ContextCompat.getDrawable(
-                        context,
-                        R.drawable.ic_search_grey_24dp
-                ), null, null, null);
-
-        innerListContainer.addView(mSearchView, new LinearLayout.LayoutParams(
-                ViewGroup.LayoutParams.MATCH_PARENT, ViewGroup.LayoutParams.WRAP_CONTENT));
+        // TODO: search is broken: When searching, previously selected keys are no longer selected
+//        mSearchView = new EditText(context);
+//        mSearchView.setId(android.R.id.input);
+//        mSearchView.setHint(R.string.menu_search);
+//        mSearchView.setCompoundDrawablesWithIntrinsicBounds(
+//                ContextCompat.getDrawable(
+//                        context,
+//                        R.drawable.ic_search_grey_24dp
+//                ), null, null, null);
+//
+//        innerListContainer.addView(mSearchView, new LinearLayout.LayoutParams(
+//                ViewGroup.LayoutParams.MATCH_PARENT, ViewGroup.LayoutParams.WRAP_CONTENT));
 
         RecyclerView listView = new RecyclerView(context);
         listView.setId(INTERNAL_LIST_VIEW_ID);
@@ -155,7 +156,7 @@ public class SelectPublicKeyFragment extends RecyclerFragment<SelectEncryptKeyAd
         // Give some text to display if there is no data. In a real
         // application this would come from a resource.
         setEmptyText(getString(R.string.list_empty));
-        mSearchView.addTextChangedListener(this);
+//        mSearchView.addTextChangedListener(this);
 
         setAdapter(new SelectEncryptKeyAdapter(getContext(), null));
         setLayoutManager(new LinearLayoutManager(getContext()));