Merge pull request #121 from mapbox/103-out-of-bounds-exception

#103 - only remove an offline region if there is are any
mapbox · Aug 9, 2016 · bbb93b8 · bbb93b8
2 parents 58d18b3 + 78bef60
commit bbb93b8
Showing 1 changed file with 14 additions and 13 deletions.
diff --git a/...src/main/java/com/mapbox/mapboxandroiddemo/examples/offline/SimpleOfflineMapActivity.java b/...src/main/java/com/mapbox/mapboxandroiddemo/examples/offline/SimpleOfflineMapActivity.java
@@ -140,19 +140,20 @@ public void onPause() {
         offlineManager.listOfflineRegions(new OfflineManager.ListOfflineRegionsCallback() {
             @Override
             public void onList(OfflineRegion[] offlineRegions) {
-
-                // delete the last item in the offlineRegions list which will be yosemite offline map
-                offlineRegions[(offlineRegions.length - 1)].delete(new OfflineRegion.OfflineRegionDeleteCallback() {
-                    @Override
-                    public void onDelete() {
-                        Toast.makeText(SimpleOfflineMapActivity.this, "Yosemite offline map deleted", Toast.LENGTH_LONG).show();
-                    }
-
-                    @Override
-                    public void onError(String error) {
-                        Log.e(TAG, "On Delete error: " + error);
-                    }
-                });
+                if (offlineRegions.length > 0) {
+                    // delete the last item in the offlineRegions list which will be yosemite offline map
+                    offlineRegions[(offlineRegions.length - 1)].delete(new OfflineRegion.OfflineRegionDeleteCallback() {
+                        @Override
+                        public void onDelete() {
+                            Toast.makeText(SimpleOfflineMapActivity.this, "Yosemite offline map deleted", Toast.LENGTH_LONG).show();
+                        }
+
+                        @Override
+                        public void onError(String error) {
+                            Log.e(TAG, "On Delete error: " + error);
+                        }
+                    });
+                }
             }
 
             @Override