Add comments

Author: misaochan

app/src/main/java/fr/free/nrw/commons/category/CategorizationFragment.java

@@ -121,8 +121,10 @@ public void writeToParcel(Parcel parcel, int flags) {
         }
     }
 
-    //TODO: Probably add 'suggest from filename' cats here. We want it to be displayed at start, not upon typing
-    //TODO: Search using MethodA query, but can't use MethodAUpdater because we don't want it updating when user types
+    /**
+     * Retrieves category suggestions from title input
+     * @return a list containing title-related categories
+     */
     protected ArrayList<String> titleCatQuery() {
 
         TitleCategories titleCategoriesSub;
@@ -132,6 +134,7 @@ protected ArrayList<String> titleCatQuery() {
         String title = titleDesc.getString("Title", "");
         Log.d(TAG, "Title: " + title);
 
+        //Override onPostExecute to access the results of async API call
         titleCategoriesSub = new TitleCategories(title) {
             @Override
             protected void onPostExecute(ArrayList<String> result) {
@@ -142,8 +145,11 @@ protected void onPostExecute(ArrayList<String> result) {
                 mergeLatch.countDown();
             }
         };
+
         titleCategoriesSub.execute();
         Log.d(TAG, "TitleCatItems in titleCatQuery: " + titleCatItems);
+
+        //Only return titleCatItems after API call has finished
         try {
             mergeLatch.await();
         } catch (InterruptedException e) {
@@ -152,15 +158,13 @@ protected void onPostExecute(ArrayList<String> result) {
         return titleCatItems;
     }
 
-
-
     /**
      * Retrieves recently-used categories
      * @return a list containing recent categories
      */
     protected ArrayList<String> recentCatQuery() {
         ArrayList<String> items = new ArrayList<String>();
-        
+
         try {
             Cursor cursor = client.query(
                     CategoryContentProvider.BASE_URI,

app/src/main/java/fr/free/nrw/commons/category/TitleCategories.java

@@ -13,7 +13,11 @@
 
 import fr.free.nrw.commons.CommonsApplication;
 
-
+/**
+ * Sends asynchronous queries to the Commons MediaWiki API to retrieve categories that are related to
+ * the title entered in previous screen. The 'srsearch' action-specific parameter is used for this
+ * purpose. This class should be subclassed in CategorizationFragment.java to add the results to recent and GPS cats.
+ */
 public class TitleCategories extends AsyncTask<Void, Void, ArrayList<String>> {
 
     private final static int SEARCH_CATS_LIMIT = 25;