[jira] [Commented] (CB-9548) camera.getPicture() - invalid FILE_URI when returning original image from device

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

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

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

GitHub user TanaseButcaru opened a pull request:

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

    separate FILE_URI and NATIVE_URI / take in account user defined destType

    Right now, on Android, ``FILE_URI`` and ``NATIVE_URI`` returns the path with ``content://`` scheme, excepting the case when the image has been modified.
    
    These changes take in account what is the user defined ``destType`` and returns the path as requested.
    
    I know there were discussions on JIRA ([CB-9548](https://issues.apache.org/jira/browse/CB-9548)) about how ``FILE_URI`` and ``NATIVE_URI`` should look, but I came to think that it should be like this:
    * FILE_URI should return the ``file://`` scheme;
    * NATIVE_URI should return the ``content://`` scheme;

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

    $ git pull https://github.com/TanaseButcaru/cordova-plugin-camera-unofficial fileuri-nativeuri

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

    https://github.com/apache/cordova-plugin-camera/pull/169.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 #169
    
----
commit 5e496f143c9b980ece203141204452bbf9178702
Author: TanaseButcaru <co...@butcaru.com>
Date:   2016-02-15T02:04:08Z

    separate FILE_URI and NATIVE_URI

----


> camera.getPicture() - invalid FILE_URI when returning original image from device
> --------------------------------------------------------------------------------
>
>                 Key: CB-9548
>                 URL: https://issues.apache.org/jira/browse/CB-9548
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: Plugin Camera
>         Environment: Archlinux OS
> Cordova CLI (5.2.0)
> Android platform (4.1.1)
> cordova-plugin-camera (1.2.0)
> ---------------------
> Android Simulator (4.4.4 & 5.0.1)
> Xperia Z3Compact (5.1.1)
> Xperia Sola (4.4.4)
>            Reporter: Tanase Butcaru
>            Assignee: Richard B Knoll
>              Labels: Android, reproduced, triaged
>
> Camera plugin returns a NATIVE_URI path when trying to get the original image with +getPicture()+ method instead of returning the FILE_URI path (even though the _destinationType_ is set to _navigator.camera.DestinationType.FILE_URI_).
> Until now I was using the camera plugin to get modified images only (by passing the _targetHeight_ & _targetWidth_ and/or _allowEdit_ arguments to +getPicture()+ method), but when I tried to get the original image I found this bug.
> Tested with devices mentioned in the Environment section for both Android & iOS platforms, but only Android manifests this behaviour..
> _I've created a github repo where I intend to reproduce all cordova core plugin bugs that I discover. I think this is the best way to test/reproduce any issue._
>  *How to reproduce the bug*:
> {code}
> git clone https://github.com/TanaseButcaru/cdv.bugs.git
> cd cdv.bugs/bugs/getPictureFileURI
> cordova run/emulate android
> {code}
> When the app starts you have two options for the _getPictureFileURI_ bug: 
> * getPicture with edits (targetHeight/Width) - OK
> * getPicture without edits (original image) - *wrong FILE_URI*
> For the second option, where the bug appears, I have provided a external (temporary) solution: I'm using the [cordova-plugin-filepath|https://github.com/hiddentao/cordova-plugin-filepath] plugin to get the FILE_URI from the NATIVE_URI and as you can see in the sample app, it works well, but it would be better if the camera plugin would do this by default when returning an unmodified image.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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