[GitHub] bvmensvoort commented on a change in pull request #322: CB-14097: (android) Fix crash when selecting some files with getPicture

[GitBox <gi...@apache.org>]

bvmensvoort commented on a change in pull request #322: CB-14097: (android) Fix crash when selecting some files with getPicture
URL: https://github.com/apache/cordova-plugin-camera/pull/322#discussion_r200380526
 
 

 ##########
 File path: src/android/FileHelper.java
 ##########
 @@ -97,10 +98,19 @@ public static String getRealPathFromURI_API11_And_Above(final Context context, f
             else if (isDownloadsDocument(uri)) {
 
                 final String id = DocumentsContract.getDocumentId(uri);
-                final Uri contentUri = ContentUris.withAppendedId(
-                        Uri.parse("content://downloads/public_downloads"), Long.valueOf(id));
-
-                return getDataColumn(context, contentUri, null, null);
+                if (!TextUtils.isEmpty(id)) {
 
 Review comment:
   Good idea to remove the dependency

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org