[ 
https://issues.apache.org/jira/browse/CB-5398?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13977552#comment-13977552
 ] 

Randy Lau edited comment on CB-5398 at 4/22/14 10:22 PM:
---------------------------------------------------------

I managed to get a temporary fix working, in line with what people mentioned 
above in previous months.  I based this from the stackoverflow solution Mike 
Billau suggested here: http://stackoverflow.com/a/20559175/368762

I haven't tested this extensively on any phone other than HTC One on Android 
4.4, so let me know if you see any potential gotchas or run into any issues.

In /platforms/android/src/org/apache/cordova/camera/CameraLauncher.java
{code}
private void processResultFromGallery(int destType, Intent intent) {
        Uri uri = intent.getData();

        // process the uri with string detection
        Context currentContext = 
this.cordova.getActivity().getApplicationContext();
        String processedUriString = getPath(currentContext, uri);
        Uri newUri = Uri.parse(processedUriString);
        uri = newUri;

        ....
{code}

And I added these file utility functions in CameraLauncher.java:
{code}
//Additions for stackoverflow answer on handling different media providers
import android.content.ContentResolver;
import android.content.ContentUris;
import android.content.Context;
import android.content.Intent;
import android.database.Cursor;
import android.database.DatabaseUtils;
import android.graphics.Bitmap;
import android.net.Uri;
import android.os.Build;
import android.os.Environment;
import android.provider.DocumentsContract;
import android.provider.MediaStore;
import android.util.Log;
import android.webkit.MimeTypeMap;

import java.io.File;
import java.io.FileFilter;
import java.text.DecimalFormat;
import java.util.Comparator;
{code}

{code}
/**
     * Get a file path from a Uri. This will get the the path for Storage Access
     * Framework Documents, as well as the _data field for the MediaStore and
     * other file-based ContentProviders.
     *
     * //@param context The context.
     * //@param uri The Uri to query.
     * //@author paulburke
     */
    public static String getPath(final Context context, final Uri uri) {

        final boolean isKitKat = Build.VERSION.SDK_INT >= 
Build.VERSION_CODES.KITKAT;

        // DocumentProvider
        if (isKitKat && DocumentsContract.isDocumentUri(context, uri)) {
            LOG.e("IceCreamCordovaWebViewClient", "isKitKat + 
DocumentsContract");
            // ExternalStorageProvider
            if (isExternalStorageDocument(uri)) {
                LOG.e("IceCreamCordovaWebViewClient", "isKitKat + external 
storage doc");
                final String docId = DocumentsContract.getDocumentId(uri);
                final String[] split = docId.split(":");
                final String type = split[0];

                if ("primary".equalsIgnoreCase(type)) {
                    return Environment.getExternalStorageDirectory() + "/" + 
split[1];
                }

                // TODO handle non-primary volumes
            }
            // DownloadsProvider
            else if (isDownloadsDocument(uri)) {
                LOG.e("IceCreamCordovaWebViewClient", "isKitKat + 
isDownloadsDocument storage doc");
                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);
            }
            // MediaProvider
            else if (isMediaDocument(uri)) {
                LOG.e("IceCreamCordovaWebViewClient", "isKitKat + 
isMediaDocument storage doc");
                final String docId = DocumentsContract.getDocumentId(uri);
                final String[] split = docId.split(":");
                final String type = split[0];

                Uri contentUri = null;
                if ("image".equals(type)) {
                    contentUri = MediaStore.Images.Media.EXTERNAL_CONTENT_URI;
                } else if ("video".equals(type)) {
                    contentUri = MediaStore.Video.Media.EXTERNAL_CONTENT_URI;
                } else if ("audio".equals(type)) {
                    contentUri = MediaStore.Audio.Media.EXTERNAL_CONTENT_URI;
                }

                final String selection = "_id=?";
                final String[] selectionArgs = new String[] {
                        split[1]
                };

                return getDataColumn(context, contentUri, selection, 
selectionArgs);
            }
        }
        // MediaStore (and general)
        else if ("content".equalsIgnoreCase(uri.getScheme())) {
            LOG.e("IceCreamCordovaWebViewClient", "isKitKat + MediaStore 
storage doc");
            return getDataColumn(context, uri, null, null);
        }
        // File
        else if ("file".equalsIgnoreCase(uri.getScheme())) {
            LOG.e("IceCreamCordovaWebViewClient", "isKitKat + file storage 
doc");
            return uri.getPath();
        }

        return null;
    }

    /**
     * Get the value of the data column for this Uri. This is useful for
     * MediaStore Uris, and other file-based ContentProviders.
     *
     * //@param context The context.
     * //@param uri The Uri to query.
     * //@param selection (Optional) Filter used in the query.
     * //@param selectionArgs (Optional) Selection arguments used in the query.
     * //@return The value of the _data column, which is typically a file path.
     */
    public static String getDataColumn(Context context, Uri uri, String 
selection,
            String[] selectionArgs) {
        LOG.e("IceCreamCordovaWebViewClient", "getDataColumn called");

        Cursor cursor = null;
        final String column = "_data";
        final String[] projection = {
                column
        };

        try {
            cursor = context.getContentResolver().query(uri, projection, 
selection, selectionArgs,
                    null);
            if (cursor != null && cursor.moveToFirst()) {
                final int column_index = cursor.getColumnIndexOrThrow(column);
                return cursor.getString(column_index);
            }
        } finally {
            if (cursor != null)
                cursor.close();
        }
        return null;
    }


    /**
     * //@param uri The Uri to check.
     * //@return Whether the Uri authority is ExternalStorageProvider.
     */
    public static boolean isExternalStorageDocument(Uri uri) {
        return 
"com.android.externalstorage.documents".equals(uri.getAuthority());
    }

    /**
     * //@param uri The Uri to check.
     * //@return Whether the Uri authority is DownloadsProvider.
     */
    public static boolean isDownloadsDocument(Uri uri) {
        return 
"com.android.providers.downloads.documents".equals(uri.getAuthority());
    }

    /**
     * //@param uri The Uri to check.
     * //@return Whether the Uri authority is MediaProvider.
     */
    public static boolean isMediaDocument(Uri uri) {
        return 
"com.android.providers.media.documents".equals(uri.getAuthority());
    }
{code}

These file utility functions return a relative path, so I added a "file://" 
before passing the string to window.resolveLocalFileSystemURL on the javascript 
side.

{code}
if (imageUrlString.indexOf("file://") == -1) {
        imageUrlString = "file://" + imageUrlString;
      }

window.resolveLocalFileSystemURL(imageUrlString, function(fileEntry) {
      var resolvedFileURL = fileEntry.toURL();
      console.log('resolvedFileURL', resolvedFileURL);
      $scope.emitFileAvailableEvent(resolvedFileURL, imageUrlString);
    });
{code}


was (Author: rlau1115):
I managed to get a temporary fix working, in line with what people mentioned 
above in previous months.  I based this from the stackoverflow solution Mike 
Billau suggested here: http://stackoverflow.com/a/20559175/368762

I haven't tested this extensively on any phone other than HTC One on Android 
4.4, so let me know if you see any potential gotchas or run into any issues.

In /platforms/android/src/org/apache/cordova/camera/CameraLauncher.java
{code}
private void processResultFromGallery(int destType, Intent intent) {
        Uri uri = intent.getData();

        // process the uri with string detection
        Context currentContext = 
this.cordova.getActivity().getApplicationContext();
        String processedUriString = getPath(currentContext, uri);
        Uri newUri = Uri.parse(processedUriString);
        uri = newUri;

        ....
{code}

And I added these file utility functions in CameraLauncher.java:
{code}
//Additions for stackoverflow answer on handling different media providers
import android.content.ContentResolver;
import android.content.ContentUris;
import android.content.Context;
import android.content.Intent;
import android.database.Cursor;
import android.database.DatabaseUtils;
import android.graphics.Bitmap;
import android.net.Uri;
import android.os.Build;
import android.os.Environment;
import android.provider.DocumentsContract;
import android.provider.MediaStore;
import android.util.Log;
import android.webkit.MimeTypeMap;

import java.io.File;
import java.io.FileFilter;
import java.text.DecimalFormat;
import java.util.Comparator;
{code}

{code}
/**
     * Get a file path from a Uri. This will get the the path for Storage Access
     * Framework Documents, as well as the _data field for the MediaStore and
     * other file-based ContentProviders.
     *
     * //@param context The context.
     * //@param uri The Uri to query.
     * //@author paulburke
     */
    public static String getPath(final Context context, final Uri uri) {

        final boolean isKitKat = Build.VERSION.SDK_INT >= 
Build.VERSION_CODES.KITKAT;

        // DocumentProvider
        if (isKitKat && DocumentsContract.isDocumentUri(context, uri)) {
            LOG.e("IceCreamCordovaWebViewClient", "isKitKat + 
DocumentsContract");
            // ExternalStorageProvider
            if (isExternalStorageDocument(uri)) {
                LOG.e("IceCreamCordovaWebViewClient", "isKitKat + external 
storage doc");
                final String docId = DocumentsContract.getDocumentId(uri);
                final String[] split = docId.split(":");
                final String type = split[0];

                if ("primary".equalsIgnoreCase(type)) {
                    return Environment.getExternalStorageDirectory() + "/" + 
split[1];
                }

                // TODO handle non-primary volumes
            }
            // DownloadsProvider
            else if (isDownloadsDocument(uri)) {
                LOG.e("IceCreamCordovaWebViewClient", "isKitKat + 
isDownloadsDocument storage doc");
                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);
            }
            // MediaProvider
            else if (isMediaDocument(uri)) {
                LOG.e("IceCreamCordovaWebViewClient", "isKitKat + 
isMediaDocument storage doc");
                final String docId = DocumentsContract.getDocumentId(uri);
                final String[] split = docId.split(":");
                final String type = split[0];

                Uri contentUri = null;
                if ("image".equals(type)) {
                    contentUri = MediaStore.Images.Media.EXTERNAL_CONTENT_URI;
                } else if ("video".equals(type)) {
                    contentUri = MediaStore.Video.Media.EXTERNAL_CONTENT_URI;
                } else if ("audio".equals(type)) {
                    contentUri = MediaStore.Audio.Media.EXTERNAL_CONTENT_URI;
                }

                final String selection = "_id=?";
                final String[] selectionArgs = new String[] {
                        split[1]
                };

                return getDataColumn(context, contentUri, selection, 
selectionArgs);
            }
        }
        // MediaStore (and general)
        else if ("content".equalsIgnoreCase(uri.getScheme())) {
            LOG.e("IceCreamCordovaWebViewClient", "isKitKat + MediaStore 
storage doc");
            return getDataColumn(context, uri, null, null);
        }
        // File
        else if ("file".equalsIgnoreCase(uri.getScheme())) {
            LOG.e("IceCreamCordovaWebViewClient", "isKitKat + file storage 
doc");
            return uri.getPath();
        }

        return null;
    }

    /**
     * Get the value of the data column for this Uri. This is useful for
     * MediaStore Uris, and other file-based ContentProviders.
     *
     * //@param context The context.
     * //@param uri The Uri to query.
     * //@param selection (Optional) Filter used in the query.
     * //@param selectionArgs (Optional) Selection arguments used in the query.
     * //@return The value of the _data column, which is typically a file path.
     */
    public static String getDataColumn(Context context, Uri uri, String 
selection,
            String[] selectionArgs) {
        LOG.e("IceCreamCordovaWebViewClient", "getDataColumn called");

        Cursor cursor = null;
        final String column = "_data";
        final String[] projection = {
                column
        };

        try {
            cursor = context.getContentResolver().query(uri, projection, 
selection, selectionArgs,
                    null);
            if (cursor != null && cursor.moveToFirst()) {
                final int column_index = cursor.getColumnIndexOrThrow(column);
                return cursor.getString(column_index);
            }
        } finally {
            if (cursor != null)
                cursor.close();
        }
        return null;
    }


    /**
     * //@param uri The Uri to check.
     * //@return Whether the Uri authority is ExternalStorageProvider.
     */
    public static boolean isExternalStorageDocument(Uri uri) {
        return 
"com.android.externalstorage.documents".equals(uri.getAuthority());
    }

    /**
     * //@param uri The Uri to check.
     * //@return Whether the Uri authority is DownloadsProvider.
     */
    public static boolean isDownloadsDocument(Uri uri) {
        return 
"com.android.providers.downloads.documents".equals(uri.getAuthority());
    }

    /**
     * //@param uri The Uri to check.
     * //@return Whether the Uri authority is MediaProvider.
     */
    public static boolean isMediaDocument(Uri uri) {
        return 
"com.android.providers.media.documents".equals(uri.getAuthority());
    }
{code}

These file utility functions return a relative path, so I added a "file://" 
before passing the string to window.resolveLocalFileSystemURL on the javascript 
side.

{code}
if (imageUrlString.indexOf("file://") == -1) {
        imageUrlString = "file://" + imageUrlString;
      }

window.resolveLocalFileSystemURL(imageUrlStringQuotesRemoved, 
function(fileEntry) {
      var resolvedFileURL = fileEntry.toURL();
      console.log('resolvedFileURL', resolvedFileURL);
      $scope.emitFileAvailableEvent(resolvedFileURL, imageUrlString);
    });
{code}

> Pick image from Library or Photo album on android 4.4
> -----------------------------------------------------
>
>                 Key: CB-5398
>                 URL: https://issues.apache.org/jira/browse/CB-5398
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: Android, Plugin Camera
>    Affects Versions: 2.9.0, 3.2.0
>         Environment: android 4.4
>            Reporter: julio cesar
>            Assignee: Mike Billau
>             Fix For: 3.5.0
>
>
> An android 4.4 try to pick a photo using pictureSource.PHOTOLIBRARY or 
> pictureSource.SAVEDPHOTOALBUM and return type destinationType.FILE_URI.
> Now android 4.4, when you select the above options, it opens an "open from" 
> dialog that  let you choose from new places as "Recent", "Drive", "Images" 
> and "Downloads" (the names might not be the same as I use the device in 
> spanish and translated it).
> If you choose any of them, you get an error, AndroidProtocolHandler, unable 
> to open content URL: the url here with a content://com.android.providers 
> format.
> I've tested on phonegap 2.9 because this is the version I use, but I suppose 
> it affects all of them. (in fact I use 2.9.1)



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to