From 3eb196f15815b3d8d4725a1a7738b08f71b4a346 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tomasz=20P=C3=B3=C5=82grabia?= <tpsa.s.a@gmail.com>
Date: Sun, 11 Sep 2016 18:48:31 +0200
Subject: [PATCH] Seems working on start and on scroll to the bottom
 (autoloading). Now - remove remaing buttons to load (and paginate).

---
 .../urbanexplorer/fragments/HomeFragment.java | 87 ++++++++++++-------
 1 file changed, 58 insertions(+), 29 deletions(-)

diff --git a/app/src/main/java/pl/tpolgrabia/urbanexplorer/fragments/HomeFragment.java b/app/src/main/java/pl/tpolgrabia/urbanexplorer/fragments/HomeFragment.java
index 61c87b0..8ddc435 100644
--- a/app/src/main/java/pl/tpolgrabia/urbanexplorer/fragments/HomeFragment.java
+++ b/app/src/main/java/pl/tpolgrabia/urbanexplorer/fragments/HomeFragment.java
@@ -46,6 +46,7 @@ public class HomeFragment extends Fragment implements LocationListener {
     private String locationProvider;
     private boolean locationServicesActivated = false;
     private AQuery aq;
+    private boolean initialized = false;
 
     private View inflatedView;
     private TextView pageSizeWidget;
@@ -123,38 +124,13 @@ public class HomeFragment extends Fragment implements LocationListener {
 
                 if (firstVisibleItem <= 0) {
                     // scrolled to the top
+                    Log.v(CLASS_TAG, "Scrolled to the top");
                 }
 
                 if (firstVisibleItem + visibleItemCount >= totalItemCount) {
+                    Log.v(CLASS_TAG, "Scrolled to the bottom");
                     // scrolled to the bottom
-                    Location location = locationService.getLastKnownLocation(locationProvider);
-                    PanoramioUtils.fetchPanoramioImages(
-                        getActivity(),
-                        location.getLatitude(),
-                        location.getLongitude(),
-                        fetchRadiusX(),
-                        fetchRadiusY(),
-                        (long)(firstVisibleItem + visibleItemCount),
-                        fetchLocationPageSize(),
-                        new PanoramioResponseCallback() {
-                            @Override
-                            public void callback(PanoramioResponseStatus status, List<PanoramioImageInfo> images, Long imagesCount) {
-                                if (status != PanoramioResponseStatus.SUCCESS) {
-                                    return;
-                                }
-
-                                PanoramioAdapter adapter = (PanoramioAdapter) locations.getAdapter();
-                                adapter.addAll(images);
-
-                                // TODO loading on end scroll should now working
-                                // TODO we can remove pagination
-                                // TODO we can think about removing first items also and last if the number
-                                // TODO of items exceeds the limit (to save the memory)
-
-                            }
-                        }
-
-                    );
+                    fetchAdditionalPhotos(firstVisibleItem, visibleItemCount);
 
                 }
 
@@ -254,9 +230,61 @@ public class HomeFragment extends Fragment implements LocationListener {
             }
         });
 
+        initialized = true;
+
+        fetchAdditionalPhotos(0, 10);
+        // FIXME hardcoded values
+
         return inflatedView;
     }
 
+    private void fetchAdditionalPhotos(int firstVisibleItem, int visibleItemCount) {
+        if (!initialized) {
+            Log.v(CLASS_TAG, "Fetching additional photos blocked till system is initialized");
+            return;
+        }
+
+        Log.v(CLASS_TAG, "Fetching additional photos");
+        Location location = locationService.getLastKnownLocation(locationProvider);
+        PanoramioUtils.fetchPanoramioImages(
+            getActivity(),
+            location.getLatitude(),
+            location.getLongitude(),
+            fetchRadiusX(),
+            fetchRadiusY(),
+            (long)(firstVisibleItem + visibleItemCount),
+            fetchLocationPageSize(),
+            new PanoramioResponseCallback() {
+                @Override
+                public void callback(PanoramioResponseStatus status, List<PanoramioImageInfo> images, Long imagesCount) {
+                    Log.v(CLASS_TAG, "Fetched with status: " + status + ", images: " + images + ", count: " +
+                        imagesCount);
+                    if (status != PanoramioResponseStatus.SUCCESS) {
+                        return;
+                    }
+
+                    PanoramioAdapter adapter = (PanoramioAdapter) locations.getAdapter();
+                    if (adapter != null) {
+                        adapter.addAll(images);
+                    } else {
+                        locations.setAdapter(new PanoramioAdapter(getActivity(),
+                            R.layout.location_item,
+                            images));
+                    }
+
+                    // TODO loading on end scroll should now working
+                    // TODO we can remove pagination
+                    // TODO we can think about removing first items also and last if the number
+                    // TODO of items exceeds the limit (to save the memory)
+
+                    Log.v(CLASS_TAG, "Finished loading additional photos");
+
+                }
+            }
+
+        );
+    }
+
     private void fetchPanoramioLocations() {
 
         final Location location = locationService.getLastKnownLocation(locationProvider);
@@ -288,7 +316,8 @@ public class HomeFragment extends Fragment implements LocationListener {
     }
 
     private Long fetchLocationPageSize() {
-        return NumberUtils.safeParseLong(pageSizeWidget.getText());
+        final CharSequence sPageSize = pageSizeWidget != null ? pageSizeWidget.getText() : null;
+        return NumberUtils.safeParseLong(sPageSize);
     }
 
     private Long fetchLocationPageId() {