Merge pull request #350 from mapzen/chuck/remove-external-storage-per…

…missions Remove scary external storage permissions on all platform versions
mapzen · Nov 7, 2014 · bc636a8 · bc636a8
2 parents 86df5a5 + 9ea92ce
commit bc636a8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/AndroidManifest.xml b/AndroidManifest.xml
@@ -14,10 +14,6 @@
     <uses-permission android:name="android.permission.ACCESS_WIFI_STATE" />
     <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />
     <uses-permission android:name="android.permission.INTERNET" />
-    <uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE"
-        android:maxSdkVersion="18" />
-    <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE"
-        android:maxSdkVersion="18" />
 
     <application
         android:name=".MapzenApplication"

diff --git a/src/main/java/com/mapzen/open/core/StyleDownLoader.java b/src/main/java/com/mapzen/open/core/StyleDownLoader.java
@@ -47,6 +47,10 @@ public void download() {
         (new AsyncTask<Void, Void, Void>() {
             @Override
             protected Void doInBackground(Void... params) {
+                if (client == null) {
+                    return null;
+                }
+
                 URL url = null;
                 try {
                     url = new URL(host + "manifest");