Only allow item selection if orgs are non-empty

kevinsawicki · kevinsawicki · commit bea5c05a98e5 · 2012-09-17T10:11:28.000-07:00
The org count being empty means they did not successfully
load so don't allow further navigation in this case.
diff --git a/app/src/main/java/com/github/mobile/ui/user/HomeActivity.java b/app/src/main/java/com/github/mobile/ui/user/HomeActivity.java
@@ -216,7 +216,7 @@ public boolean onNavigationItemSelected(int itemPosition, long itemId) {
         if (homeAdapter.isOrgPosition(itemPosition)) {
             homeAdapter.setSelected(itemPosition);
             setOrg(orgs.get(itemPosition));
-        } else {
+        } else if (homeAdapter.getOrgCount() > 0) {
             switch (homeAdapter.getAction(itemPosition)) {
             case ACTION_GISTS:
                 startActivity(new Intent(this, GistsActivity.class));
diff --git a/app/src/main/java/com/github/mobile/ui/user/HomeDropdownListAdapter.java b/app/src/main/java/com/github/mobile/ui/user/HomeDropdownListAdapter.java
@@ -83,7 +83,7 @@ public HomeDropdownListAdapter(final Context context,
      *
      * @return org count
      */
-    private int getOrgCount() {
+    public int getOrgCount() {
         return getCount() - NON_ORG_ITEMS;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,7 @@ public HomeDropdownListAdapter(final Context context,`
`83`	`83`	`*`
`84`	`84`	`* @return org count`
`85`	`85`	`*/`
`86`		`- private int getOrgCount() {`
	`86`	`+ public int getOrgCount() {`
`87`	`87`	`return getCount() - NON_ORG_ITEMS;`
`88`	`88`	`}`
`89`	`89`