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

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

Github user Bubkastor commented on a diff in the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/64#discussion_r25779060
  
    --- Diff: src/windows/CameraProxy.js ---
    @@ -165,7 +165,16 @@ module.exports = {
     
                                 var storageFolder = 
Windows.Storage.ApplicationData.current.localFolder;
                                 file.copyAsync(storageFolder, file.name, 
Windows.Storage.NameCollisionOption.replaceExisting).then(function 
(storageFile) {
    -                                
successCallback(URL.createObjectURL(storageFile));
    +                                var res;
    +                                if (destinationType == 
Camera.DestinationType.NATIVE_URI) {
    +                                    res = storageFile.path
    +                                        .replace(/\\/g, '/')
    +                                        .replace(':','%3A');
    --- End diff --
    
    there is no need to replace


> Camera plugin on windows does not return native URL for photolibrary source
> ---------------------------------------------------------------------------
>
>                 Key: CB-8498
>                 URL: https://issues.apache.org/jira/browse/CB-8498
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: Plugin Camera
>    Affects Versions: 3.5.0
>         Environment: Windows 8.1, platform windows
>            Reporter: Vladimir Avdonin
>
> getPicture called with options 
> {
> sourceType: Camera.PictureSourceType.PHOTOLIBRARY,
> destinationType: Camera.DestinationType.NATIVE_URI
> }
> returns blob: URI. Expected to return native file URI.



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

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

Reply via email to