diff --git a/commons/src/main/java/fr/free/nrw/commons/category/CategorizationFragment.java b/commons/src/main/java/fr/free/nrw/commons/category/CategorizationFragment.java index 2b786ecc0..5bfd20b4f 100644 --- a/commons/src/main/java/fr/free/nrw/commons/category/CategorizationFragment.java +++ b/commons/src/main/java/fr/free/nrw/commons/category/CategorizationFragment.java @@ -335,60 +335,59 @@ public class CategorizationFragment extends SherlockFragment{ return rootView; } - final CountDownLatch latch = new CountDownLatch(1); - - private class PrefixUpdaterSub extends PrefixUpdater { - - public PrefixUpdaterSub() { - super(CategorizationFragment.this); - } - - @Override - protected ArrayList doInBackground(Void... voids) { - ArrayList result = new ArrayList(); - try { - result = super.doInBackground(); - latch.await(); - } - catch (InterruptedException e) { - Log.w(TAG, e); - } - return result; - } - - @Override - protected void onPostExecute(ArrayList result) { - super.onPostExecute(result); - - results.addAll(result); - categoriesAdapter.notifyDataSetChanged(); - } - } - - - private class MethodAUpdaterSub extends MethodAUpdater { - - public MethodAUpdaterSub() { - super(CategorizationFragment.this); - } - - @Override - protected void onPostExecute(ArrayList result) { - super.onPostExecute(result); - - results.clear(); - results.addAll(result); - categoriesAdapter.notifyDataSetChanged(); - - latch.countDown(); - } - } private void startUpdatingCategoryList() { + final CountDownLatch latch = new CountDownLatch(1); + class PrefixUpdaterSub extends PrefixUpdater { + + public PrefixUpdaterSub() { + super(CategorizationFragment.this); + } + + @Override + protected ArrayList doInBackground(Void... voids) { + ArrayList result = new ArrayList(); + try { + result = super.doInBackground(); + latch.await(); + } + catch (InterruptedException e) { + Log.w(TAG, e); + } + return result; + } + + @Override + protected void onPostExecute(ArrayList result) { + super.onPostExecute(result); + + results.addAll(result); + categoriesAdapter.notifyDataSetChanged(); + } + } + + + class MethodAUpdaterSub extends MethodAUpdater { + + public MethodAUpdaterSub() { + super(CategorizationFragment.this); + } + + @Override + protected void onPostExecute(ArrayList result) { + super.onPostExecute(result); + + results.clear(); + results.addAll(result); + categoriesAdapter.notifyDataSetChanged(); + + latch.countDown(); + } + } if (lastUpdater != null) { lastUpdater.cancel(true); @@ -404,15 +403,6 @@ public class CategorizationFragment extends SherlockFragment{ Utils.executeAsyncTask(prefixUpdaterSub); Utils.executeAsyncTask(methodAUpdaterSub); -/* - ArrayList itemList = new ArrayList(itemSet); - - methodAUpdater = new MethodAUpdater(); - lastUpdater = new PrefixUpdater(); - - Utils.executeAsyncTask(lastUpdater, executor); - Utils.executeAsyncTask(methodAUpdater, executor); - */ }