diff options
| author | Bananeweizen <bananeweizen@gmx.de> | 2011-11-01 15:12:48 +0100 |
|---|---|---|
| committer | Bananeweizen <bananeweizen@gmx.de> | 2011-11-01 15:12:48 +0100 |
| commit | 04c853e79301398c2920983b4c3086f5b3ff8b69 (patch) | |
| tree | 66ce80943b21d755ddb7cefb0aa58d75d375c77c /main/src/cgeo | |
| parent | 3c82f15f68a60cd47c122c660c58219b265f7437 (diff) | |
| download | cgeo-04c853e79301398c2920983b4c3086f5b3ff8b69.zip cgeo-04c853e79301398c2920983b4c3086f5b3ff8b69.tar.gz cgeo-04c853e79301398c2920983b4c3086f5b3ff8b69.tar.bz2 | |
fix #741: no more caches button still loads next page
Diffstat (limited to 'main/src/cgeo')
| -rw-r--r-- | main/src/cgeo/geocaching/cgeocaches.java | 42 |
1 files changed, 4 insertions, 38 deletions
diff --git a/main/src/cgeo/geocaching/cgeocaches.java b/main/src/cgeo/geocaching/cgeocaches.java index ed9f583..a5a39ee 100644 --- a/main/src/cgeo/geocaching/cgeocaches.java +++ b/main/src/cgeo/geocaching/cgeocaches.java @@ -204,16 +204,7 @@ public class cgeocaches extends AbstractListActivity { setMoreCaches(false); } else { final Integer count = cgeoapplication.getTotal(search); - - if (count != null && count > 0) { - if (cacheList.size() < cgeoapplication.getTotal(search) && cacheList.size() < MAX_LIST_ITEMS) { - setMoreCaches(true); - } else { - setMoreCaches(false); - } - } else { - setMoreCaches(false); - } + setMoreCaches(count != null && count > 0 && cacheList != null && cacheList.size() < count && cacheList.size() < MAX_LIST_ITEMS); } if (cacheList != null && cgeoapplication.getError(search) == StatusCode.UNAPPROVED_LICENSE) { @@ -304,15 +295,7 @@ public class cgeocaches extends AbstractListActivity { setMoreCaches(false); } else { final Integer count = cgeoapplication.getTotal(search); - if (count != null && count > 0) { - if (cacheList.size() < cgeoapplication.getTotal(search) && cacheList.size() < MAX_LIST_ITEMS) { - setMoreCaches(true); - } else { - setMoreCaches(false); - } - } else { - setMoreCaches(false); - } + setMoreCaches(count != null && count > 0 && cacheList != null && cacheList.size() < count && cacheList.size() < MAX_LIST_ITEMS); } if (cgeoapplication.getError(search) != null) { @@ -335,8 +318,6 @@ public class cgeocaches extends AbstractListActivity { Log.e(Settings.tag, "cgeocaches.loadNextPageHandler: " + e.toString()); } - listFooter.setOnClickListener(new moreCachesListener()); - hideLoading(); showProgress(false); @@ -1445,27 +1426,12 @@ public class cgeocaches extends AbstractListActivity { dir = app.startDir(this, dirUpdate); } - if (cacheList != null) { - setTitle(title); - } - if (CollectionUtils.isNotEmpty(cacheList)) { final Integer count = cgeoapplication.getTotal(search); - if (count != null && count > 0) { - setTitle(title); - if (cacheList.size() < cgeoapplication.getTotal(search) && cacheList.size() < MAX_LIST_ITEMS) { - setMoreCaches(true); - } else { - setMoreCaches(false); - } - } else { - setTitle(title); - setMoreCaches(false); - } - } else { - setTitle(title); + setMoreCaches(count != null && count > 0 && cacheList.size() < count && cacheList.size() < MAX_LIST_ITEMS); } + setTitle(title); setAdapter(); if (geo != null) { |