apache / cordova-plugin-file-transfer

Apache Cordova File Transfer Plugin
https://cordova.apache.org/
Apache License 2.0
596 stars 887 forks source link

CB-9022 Resolve source URI on background thread #154

Closed vladimir-kotikov closed 8 years ago

vladimir-kotikov commented 8 years ago

Platforms affected

Android

What does this PR do?

This PR fixes application crash when trying to upload file from 'Content' file system, specified by cdvfile://localhost/content/... URI.

What testing has been done on this change?

Manual and automated testing

Checklist

We should call remapUri on background thread otherwise it throws IllegalStateException when trying to remap 'cdvfile://localhost/content/...' URIs via ContentFilesystem

cordova-qa commented 8 years ago

Cordova CI Build has completed successfully.

Commit - Link Dashboard - Link

Builder Name Console Output Test Report Device Logs
Windows 8.1 Store Link Link Link
Windows 10 Store Link Link Link
Windows 8.1 Phone Link Link Link
iOS Link Link Link
Android Link Link Link