Split cache loading in offline + online modes

Also, add a new "loading elevation data" step in progress dialog.
This closes #701.

3 files changed, 24 insertions, 13 deletions

diff --git a/main/res/values-fr/strings.xml b/main/res/values-fr/strings.xml
index 8defca7..12ff333 100644
--- a/main/res/values-fr/strings.xml
+++ b/main/res/values-fr/strings.xml
@@ -480,6 +480,7 @@
   <string name="cache_dialog_loading_details_status_logs">Chargement des entrées de carnets</string>
   <string name="cache_dialog_loading_details_status_waypoints">Traitement des points intermédiaires</string>
   <string name="cache_dialog_loading_details_status_gcvote">Chargement des votes</string>
+  <string name="cache_dialog_loading_details_status_elevation">Chargement de l\'altitude</string>
   <string name="cache_dialog_loading_details_status_render">Préparation de la page</string>
   <string name="cache_dialog_loading_description">Chargement de la description…</string>
   <string name="cache_dialog_offline_save_title">Hors ligne</string>
diff --git a/main/res/values/strings.xml b/main/res/values/strings.xml
index d5ba832..8bb16c2 100644
--- a/main/res/values/strings.xml
+++ b/main/res/values/strings.xml
@@ -490,6 +490,7 @@
   <string name="cache_dialog_loading_details_status_logs">Loading logs</string>
   <string name="cache_dialog_loading_details_status_waypoints">Processing waypoints</string>
   <string name="cache_dialog_loading_details_status_gcvote">Loading GCVote</string>
+  <string name="cache_dialog_loading_details_status_elevation">Loading elevation data</string>
   <string name="cache_dialog_loading_details_status_render">Render view</string>
   <string name="cache_dialog_loading_description">Loading cache description…</string>
   <string name="cache_dialog_offline_save_title">Offline</string>
diff --git a/main/src/cgeo/geocaching/cgBase.java b/main/src/cgeo/geocaching/cgBase.java
index 5775e7b..61eaaba 100644
--- a/main/src/cgeo/geocaching/cgBase.java
+++ b/main/src/cgeo/geocaching/cgBase.java
@@ -997,6 +997,18 @@ public class cgBase {
     }
 
     public static cgCacheWrap parseCache(final String page, final int reason, final Handler handler) {
+        final cgCacheWrap caches = parseCacheFromText(page, reason, handler);
+        if (!caches.cacheList.isEmpty()) {
+            final cgCache cache = caches.cacheList.get(0);
+            getExtraOnlineInfo(cache, page, handler);
+            cache.updated = System.currentTimeMillis();
+            cache.detailedUpdate = cache.updated;
+            cache.detailed = true;
+        }
+        return caches;
+    }
+
+    static cgCacheWrap parseCacheFromText(final String page, final int reason, final Handler handler) {
         sendLoadProgressDetail(handler, R.string.cache_dialog_loading_details_status_details);
 
         if (StringUtils.isBlank(page)) {
@@ -1278,11 +1290,6 @@ public class cgBase {
             Log.w(Settings.tag, "cgeoBase.parseCache: Failed to parse cache log count");
         }
 
-        // cache logs
-        sendLoadProgressDetail(handler, R.string.cache_dialog_loading_details_status_logs);
-
-        loadLogsFromDetails(page, cache);
-
         int wpBegin = 0;
         int wpEnd = 0;
 
@@ -1411,25 +1418,27 @@ public class cgBase {
             }
         }
 
+        caches.cacheList.add(cache);
+
+        return caches;
+    }
+
+    private static void getExtraOnlineInfo(final cgCache cache, final String page, final Handler handler) {
+        sendLoadProgressDetail(handler, R.string.cache_dialog_loading_details_status_logs);
+        loadLogsFromDetails(page, cache);
+
+        sendLoadProgressDetail(handler, R.string.cache_dialog_loading_details_status_elevation);
         if (cache.coords != null) {
             cache.elevation = getElevation(cache.coords);
         }
 
         sendLoadProgressDetail(handler, R.string.cache_dialog_loading_details_status_gcvote);
-
         final cgRating rating = GCVote.getRating(cache.guid, cache.geocode);
         if (rating != null) {
             cache.rating = rating.rating;
             cache.votes = rating.votes;
             cache.myVote = rating.myVote;
         }
-
-        cache.updated = System.currentTimeMillis();
-        cache.detailedUpdate = System.currentTimeMillis();
-        cache.detailed = true;
-        caches.cacheList.add(cache);
-
-        return caches;
     }
 
     /**