[GitHub] cordova-plugin-camera pull request: CB-10270 android: Added back s...

2016-03-11 Thread jcesarmobile
Github user jcesarmobile commented on the pull request: https://github.com/apache/cordova-plugin-camera/pull/160#issuecomment-195337221 I'm going to send a PR that will use the API 19 method on android 11 and greater and will handle file urls --- If your project is set up for it, yo

[GitHub] cordova-plugin-camera pull request: CB-10270 android: Added back s...

2016-03-10 Thread riknoll
Github user riknoll commented on the pull request: https://github.com/apache/cordova-plugin-camera/pull/160#issuecomment-195084460 @jcesarmobile I don't even know if this is an issue in Jelly Bean. Do you have a device to test with? --- If your project is set up for it, you can repl

[GitHub] cordova-plugin-camera pull request: CB-10270 android: Added back s...

2016-03-10 Thread jcesarmobile
Github user jcesarmobile commented on the pull request: https://github.com/apache/cordova-plugin-camera/pull/160#issuecomment-195082019 #175 only fixes it on API 19 and greater, maybe it should be moved outside. --- If your project is set up for it, you can reply to this email and h

[GitHub] cordova-plugin-camera pull request: CB-10270 android: Added back s...

2016-03-10 Thread riknoll
Github user riknoll commented on the pull request: https://github.com/apache/cordova-plugin-camera/pull/160#issuecomment-195077141 Seems to have been fixed by #175, so I'm closing this --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[GitHub] cordova-plugin-camera pull request: CB-10270 android: Added back s...

2016-03-10 Thread riknoll
Github user riknoll closed the pull request at: https://github.com/apache/cordova-plugin-camera/pull/160 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] cordova-plugin-camera pull request: CB-10270 android: Added back s...

2016-02-12 Thread TanaseButcaru
Github user TanaseButcaru commented on the pull request: https://github.com/apache/cordova-plugin-camera/pull/160#issuecomment-183518294 I can confirm this is working for picture selection. @shahin8r you could use this commit to [prevent empty video content uri](https://github.com

[GitHub] cordova-plugin-camera pull request: CB-10270 android: Added back s...

2016-02-10 Thread shahin8r
Github user shahin8r commented on the pull request: https://github.com/apache/cordova-plugin-camera/pull/160#issuecomment-182563046 This doesn't seem to fix empty string for videos (mediaType: 1) but is working fine on photos. --- If your project is set up for it, you can reply to t

[GitHub] cordova-plugin-camera pull request: CB-10270 android: Added back s...

2016-02-09 Thread riknoll
Github user riknoll commented on the pull request: https://github.com/apache/cordova-plugin-camera/pull/160#issuecomment-181996971 Alright, the real solution to this problem in my opinion is to get rid of the `getRealPath` function altogether. Not all URIs have real paths and there a

[GitHub] cordova-plugin-camera pull request: CB-10270 android: Added back s...

2016-02-09 Thread olexme
Github user olexme commented on the pull request: https://github.com/apache/cordova-plugin-camera/pull/160#issuecomment-181818806 I had the same issue (empty string from getPicture() when trying to get the gallery picture) and I've just tried to use this PR by cordova plugin add

[GitHub] cordova-plugin-camera pull request: CB-10270 android: Added back s...

2016-02-09 Thread shaoner
Github user shaoner commented on the pull request: https://github.com/apache/cordova-plugin-camera/pull/160#issuecomment-181769966 @riknoll yes, actually if you don't set correctOrientation to true, getRealPath is never called, it returns the uri instead (I don't see why) so this is

[GitHub] cordova-plugin-camera pull request: CB-10270 android: Added back s...

2016-02-08 Thread anuj6117
Github user anuj6117 commented on the pull request: https://github.com/apache/cordova-plugin-camera/pull/160#issuecomment-181738797 Yes, I am getting an empty string back from the getPicture() call. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cordova-plugin-camera pull request: CB-10270 android: Added back s...

2016-02-08 Thread riknoll
Github user riknoll commented on the pull request: https://github.com/apache/cordova-plugin-camera/pull/160#issuecomment-181510405 @shaoner @anuj6117 Is that exception related to [the JIRA](https://issues.apache.org/jira/browse/CB-10270) this PR is addressing? That is, does it result

[GitHub] cordova-plugin-camera pull request: CB-10270 android: Added back s...

2016-02-07 Thread anuj6117
Github user anuj6117 commented on the pull request: https://github.com/apache/cordova-plugin-camera/pull/160#issuecomment-181195873 I am also facing the same exception if I use ALLMEDIA. --- If your project is set up for it, you can reply to this email and have your reply appear on G

[GitHub] cordova-plugin-camera pull request: CB-10270 android: Added back s...

2016-02-06 Thread shaoner
Github user shaoner commented on the pull request: https://github.com/apache/cordova-plugin-camera/pull/160#issuecomment-180798266 @riknoll, somehow an exception is triggered by this line for some uri: ```java try { String wholeID = DocumentsContract.getDocumentId(

[GitHub] cordova-plugin-camera pull request: CB-10270 android: Added back s...

2016-02-01 Thread riknoll
Github user riknoll commented on the pull request: https://github.com/apache/cordova-plugin-camera/pull/160#issuecomment-178228435 For reference, [this](https://github.com/apache/cordova-plugin-camera/commit/929733b8913311f2cfd504937268a8a1d22f4f9a) is the commit I mentioned. --- I

[GitHub] cordova-plugin-camera pull request: CB-10270 android: Added back s...

2016-02-01 Thread riknoll
GitHub user riknoll opened a pull request: https://github.com/apache/cordova-plugin-camera/pull/160 CB-10270 android: Added back support for file:// URIs to getRealPath Our `getRealPath()` function is unable to handle `file: //` URIs, which can occasionally lead it to return an empt