From 819ec795bdc9891c48402ddc8fff834d63d425b3 Mon Sep 17 00:00:00 2001
From: misaochan <josephinelim86@gmail.com>
Date: Sun, 18 Mar 2018 18:51:17 +1000
Subject: [PATCH] Add check for empty direct categories

---
 .../nrw/commons/category/CategorizationFragment.java     | 9 ++++++---
 1 file changed, 6 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 a62452e37..bdf000288 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
@@ -89,6 +89,7 @@ public class CategorizationFragment extends CommonsDaggerSupportFragment {
     private HashMap<String, ArrayList<String>> categoriesCache;
     private List<CategoryItem> selectedCategories = new ArrayList<>();
     private TitleTextWatcher textWatcher = new TitleTextWatcher();
+    private boolean hasDirectCategories = false;
 
     private final CategoriesAdapterFactory adapterFactory = new CategoriesAdapterFactory(item -> {
         if (item.isSelected()) {
@@ -275,11 +276,13 @@ public class CategorizationFragment extends CommonsDaggerSupportFragment {
 
     private Observable<CategoryItem> directCategories() {
         String directCategory = directPrefs.getString("Category", "");
+        List<String> categoryList = new ArrayList<>();
         Timber.d("Direct category found: " + directCategory);
 
-        List<String> categoryList = new ArrayList<>();
-        categoryList.add(directCategory);
-
+        if (!directCategory.equals("")) {
+            hasDirectCategories = true;
+            categoryList.add(directCategory);
+        }
         return Observable.fromIterable(categoryList).map(name -> new CategoryItem(name, false));
     }