From fe6bb7c6a878490ef0795a5cc8a80fb57087dc73 Mon Sep 17 00:00:00 2001
From: cip <cip@gmx.at>
Date: Fri, 5 Apr 2013 10:03:54 +0200
Subject: [PATCH] article suggestion: Changed implementation to use filter

Cleaner, and does not block ui thread while retrieving article titles
from zim file
---
 .../kiwixmobile/KiwixMobileActivity.java      | 91 +++++++++++++------
 1 file changed, 64 insertions(+), 27 deletions(-)

diff --git a/src/org/kiwix/kiwixmobile/KiwixMobileActivity.java b/src/org/kiwix/kiwixmobile/KiwixMobileActivity.java
index 50f46ca7a..05c2764f0 100644
--- a/src/org/kiwix/kiwixmobile/KiwixMobileActivity.java
+++ b/src/org/kiwix/kiwixmobile/KiwixMobileActivity.java
@@ -11,6 +11,7 @@ import java.util.Arrays;
 import android.app.Activity;
 import android.content.ActivityNotFoundException;
 import android.content.ComponentName;
+import android.content.Context;
 import android.content.Intent;
 import android.content.SharedPreferences;
 import android.net.Uri;
@@ -30,6 +31,8 @@ import android.webkit.WebView;
 import android.webkit.WebViewClient;
 import android.widget.ArrayAdapter;
 import android.widget.AutoCompleteTextView;
+import android.widget.Filter;
+import android.widget.Filterable;
 import android.widget.TextView;
 import android.widget.TextView.OnEditorActionListener;
 import android.widget.Toast;
@@ -45,6 +48,65 @@ public class KiwixMobileActivity extends Activity {
 	private static final String PREFS_KIWIX_MOBILE = "kiwix-mobile";
 	
 	
+	public class AutoCompleteAdapter extends ArrayAdapter<String> implements Filterable {
+		private ArrayList<String> mData;
+
+		public AutoCompleteAdapter(Context context, int textViewResourceId) {
+			super(context, textViewResourceId);
+			mData = new ArrayList<String>();
+		}
+
+		@Override
+		public int getCount() {
+			return mData.size();
+		}
+
+		@Override
+		public String getItem(int index) {
+			return mData.get(index);
+		}
+
+		@Override
+		public Filter getFilter() {
+			Filter myFilter = new Filter() {
+				@Override
+				protected FilterResults performFiltering(CharSequence constraint) {
+					FilterResults filterResults = new FilterResults();
+					ArrayList<String> data = new ArrayList<String>();
+					if(constraint != null) {
+						// A class that queries a web API, parses the data and returns an ArrayList<Style>
+						try {
+							ZimContentProvider.searchSuggestions(constraint.toString(), 20);
+							String suggestion;
+							
+							data.clear();
+							while ((suggestion = ZimContentProvider.getNextSuggestion())!=null) {
+								data.add(suggestion);        
+							}    
+						}
+						catch(Exception e) {}
+						// Now assign the values and count to the FilterResults object
+						filterResults.values = data;
+						filterResults.count = data.size();
+					}
+					return filterResults;
+				}
+
+				@SuppressWarnings("unchecked")
+				@Override
+				protected void publishResults(CharSequence contraint, FilterResults results) {
+					if(results != null && results.count > 0) {
+						notifyDataSetChanged();
+						mData = (ArrayList<String>)results.values;
+					}
+					else {
+						notifyDataSetInvalidated();
+					}
+				}
+			};
+			return myFilter;
+		}
+	} 
     @Override
     public void onCreate(Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);
@@ -59,10 +121,9 @@ public class KiwixMobileActivity extends Activity {
         
      // Get a reference to the AutoCompleteTextView in the layout
         AutoCompleteTextView articleSearchtextView = (AutoCompleteTextView) findViewById(R.id.articleSearchTextView);
-        ArrayList<String> emptyList = new ArrayList<String>();
         // Create the adapter and set it to the AutoCompleteTextView 
-        adapter = 
-                new ArrayAdapter<String>(this, android.R.layout.simple_list_item_1, emptyList);
+        adapter = new AutoCompleteAdapter(this, android.R.layout.simple_list_item_1); 
+               
         articleSearchtextView.setAdapter(adapter);
         articleSearchtextView.setOnEditorActionListener(new OnEditorActionListener() {
 
@@ -87,30 +148,6 @@ public class KiwixMobileActivity extends Activity {
             			return true;
             		}
          }});
-        articleSearchtextView.addTextChangedListener(new TextWatcher()
-        {
-        public void afterTextChanged(Editable s)
-        {
-                                                                        // Abstract Method of TextWatcher Interface.
-        }
-        public void beforeTextChanged(CharSequence s,
-        int start, int count, int after)
-        {
-        // Abstract Method of TextWatcher Interface.
-        }
-        public void onTextChanged(CharSequence s,
-        		int start, int before, int count)
-        {        
-        	AutoCompleteTextView articleSearchtextView = (AutoCompleteTextView) findViewById(R.id.articleSearchTextView);
-        	Log.d("zimgap", "Adapter:"+adapter.getCount());
-        	adapter.clear();
-        	ZimContentProvider.searchSuggestions(s.toString(), 20);
-        	String suggestion;
-        	while ((suggestion = ZimContentProvider.getNextSuggestion())!=null) {
-        		adapter.add(suggestion);        
-        	}
-        }
-        });
         
     	 
         // js includes will not happen unless we enable JS