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

ASF GitHub Bot commented on CB-13307:
-------------------------------------

GitHub user DavidBriglio opened a pull request:

    https://github.com/apache/cordova-plugin-camera/pull/286

    CB-13307: (Android) Fixed issue when unable to load image with mime t…

    …ype "image/*", and failing to recognize new contentprovider uri authority.
    
    ### Platforms affected
    Android
    
    ### What does this PR do?
    Allows for image mimetype "image/*" to be accepted as a valid image.
    Allows for google photos uri authority ".contentprovider" to be recognized.
    
    ### What testing has been done on this change?
    When running "npm test" results in "invalid config loglevel='notice'".
    When running "npm run test-build" results in "missing script: test-build".
    Manual testing has been done on Android 8.0 Google Pixel with cordova cli 
6.5.0.


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/DavidBriglio/cordova-plugin-camera 
CB-13307-cordova-plugin-camera-MimeType-ContentProvider

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cordova-plugin-camera/pull/286.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #286
    
----
commit 0d0116a3ed0b593253d96cfdd705e849b4ebca61
Author: David Briglio <[email protected]>
Date:   2017-09-21T13:49:47Z

    CB-13307: (Android) Fixed issue when unable to load image with mime type 
"image/*", and failing to recognize new contentprovider uri authority.

----


> Accessing Google Photos on Pixel from cordova-plugin-camera
> -----------------------------------------------------------
>
>                 Key: CB-13307
>                 URL: https://issues.apache.org/jira/browse/CB-13307
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: cordova-plugin-camera
>            Reporter: David Briglio
>
> Currently when trying to access some photos from Google Photos, it fails to 
> load. Android 8.0 on Google Pixel, cordova version 6.5.0, 
> cordova-plugin-camera version 2.4.1.
> Causes:
> 1) Only mimetypes "image/jpeg" and "image/png" are accepted, all other types 
> are considered an invalid image. The mimetype of the image I am loading gets 
> set to "image/*".
> 2) In FileHelper.java: when hitting the isGooglePhotosUri method, the uri 
> authority is compared against "com.google.android.apps.photos.content", where 
> the actual is "com.google.android.apps.photos.contentprovider". This causes 
> the logic to dismiss the image as not from Google Photos.
> After applying fixes for these two areas, I am able to successfully load all 
> of my images.
> Merge request [here|https://github.com/apache/cordova-plugin-camera/pull/285].



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to