Merge pull request nostra13#311 from sherifelkhatib/master

Allowing Apps to omit "android.permission.WRITE_EXTERNAL_STORAGE" even when sdcard is mounted

Author: nostra13

library/src/com/nostra13/universalimageloader/utils/StorageUtils.java

@@ -19,6 +19,7 @@
 import java.io.IOException;
 
 import android.content.Context;
+import android.content.pm.PackageManager;
 import android.os.Environment;
 
 /**
@@ -44,7 +45,7 @@ private StorageUtils() {
 	 */
 	public static File getCacheDirectory(Context context) {
 		File appCacheDir = null;
-		if (Environment.getExternalStorageState().equals(android.os.Environment.MEDIA_MOUNTED)) {
+		if (Environment.getExternalStorageState().equals(android.os.Environment.MEDIA_MOUNTED) && hasExternalPermission(context)) {
 			appCacheDir = getExternalCacheDir(context);
 		}
 		if (appCacheDir == null) {
@@ -86,7 +87,7 @@ public static File getIndividualCacheDirectory(Context context) {
 	 */
 	public static File getOwnCacheDirectory(Context context, String cacheDir) {
 		File appCacheDir = null;
-		if (Environment.getExternalStorageState().equals(android.os.Environment.MEDIA_MOUNTED)) {
+		if (Environment.getExternalStorageState().equals(android.os.Environment.MEDIA_MOUNTED) && hasExternalPermission(context)) {
 			appCacheDir = new File(Environment.getExternalStorageDirectory(), cacheDir);
 		}
 		if (appCacheDir == null || (!appCacheDir.exists() && !appCacheDir.mkdirs())) {
@@ -111,4 +112,11 @@ private static File getExternalCacheDir(Context context) {
 		}
 		return appCacheDir;
 	}
+	
+	private static boolean hasExternalPermission(Context cxt)
+	{
+	    String permission = "android.permission.WRITE_EXTERNAL_STORAGE";
+	    int res = cxt.checkCallingOrSelfPermission(permission);
+	    return (res == PackageManager.PERMISSION_GRANTED);            
+	}
 }