From 464d6e0bac390937b7fa7a523fd839efb3b51175 Mon Sep 17 00:00:00 2001
From: misaochan <josephinelim86@gmail.com>
Date: Thu, 27 Oct 2016 16:21:21 +1300
Subject: [PATCH] Add logs and additional var filteredItems to store list

---
 .../fr/free/nrw/commons/category/CategorizationFragment.java | 2 --
 .../java/fr/free/nrw/commons/category/PrefixUpdater.java     | 5 ++++-
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/app/src/main/java/fr/free/nrw/commons/category/CategorizationFragment.java b/app/src/main/java/fr/free/nrw/commons/category/CategorizationFragment.java
index aad549bef..ca8300172 100644
--- a/app/src/main/java/fr/free/nrw/commons/category/CategorizationFragment.java
+++ b/app/src/main/java/fr/free/nrw/commons/category/CategorizationFragment.java
@@ -222,8 +222,6 @@ public class CategorizationFragment extends Fragment {
         //Needs to be an ArrayList and not a List unless we want to modify a big portion of preexisting code
         ArrayList<String> mergedItemsList = new ArrayList<String>(mergedItems);
 
-
-
         Log.d(TAG, "Merged item list: " + mergedItemsList);
         return mergedItemsList;
     }
diff --git a/app/src/main/java/fr/free/nrw/commons/category/PrefixUpdater.java b/app/src/main/java/fr/free/nrw/commons/category/PrefixUpdater.java
index fb50137d9..015b397a7 100644
--- a/app/src/main/java/fr/free/nrw/commons/category/PrefixUpdater.java
+++ b/app/src/main/java/fr/free/nrw/commons/category/PrefixUpdater.java
@@ -43,6 +43,7 @@ public class PrefixUpdater extends AsyncTask<Void, Void, ArrayList<String>> {
     }
 
     protected ArrayList<String> filterYears(ArrayList<String> items) {
+
         //Copy to Iterator to prevent ConcurrentModificationException when removing item
         for(Iterator<String> item = items.iterator(); item.hasNext();) {
             String s = item.next();
@@ -54,7 +55,8 @@ public class PrefixUpdater extends AsyncTask<Void, Void, ArrayList<String>> {
             }
         }
 
-        return items;
+        ArrayList<String> filteredItems = new ArrayList<String>(items);
+        return filteredItems;
     }
 
     @Override
@@ -62,6 +64,7 @@ public class PrefixUpdater extends AsyncTask<Void, Void, ArrayList<String>> {
         //If user hasn't typed anything in yet, get GPS and recent items
         if(TextUtils.isEmpty(filter)) {
             ArrayList<String> mergedItems = new ArrayList<String>(catFragment.mergeItems());
+            Log.d(TAG, "Merged items, waiting for filter");
             ArrayList<String> filteredItems = new ArrayList<String>(filterYears(mergedItems));
             return filteredItems;
         }