[
https://issues.apache.org/jira/browse/CB-9238?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14605842#comment-14605842
]
ASF GitHub Bot commented on CB-9238:
------------------------------------
Github user robpaveza commented on a diff in the pull request:
https://github.com/apache/cordova-plugin-media/pull/58#discussion_r33482012
--- Diff: src/windows/MediaProxy.js ---
@@ -203,20 +234,64 @@ module.exports = {
var id = args[0];
var thisM = Media.get(id);
- var normalizedSrc = thisM.src.replace(/\//g, '\\');
+ var success = function () {
+ Media.onStatus(id, Media.MEDIA_STATE, Media.MEDIA_STOPPED);
+ win();
+ };
+
+ var error = function (reason) {
+ Media.onStatus(id, Media.MEDIA_ERROR, reason);
+ lose(reason);
+ };
+
+ var fsTypes = {
+ PERSISTENT: 'PERSISTENT',
+ TEMPORARY: 'TEMPORARY'
+ };
+ var fsType;
+
+ var normalizedSrc = thisM.src;
+ normalizedSrc = setTemporaryFsByDefault(normalizedSrc);
+
+ if (normalizedSrc.indexOf('ms-appdata:///local/') === 0) {
--- End diff --
Is this component going to allow for interoperable code among platforms? I
just want to ensure that the developer isn't expected to utter
"ms-appdata:///local/" - instead they're getting that from some service (or
we're converting into that from some other argument) so that the developer can
just write "persistent" or "temporary" and it saves.
> Media plugin cannot record audio on windows
> -------------------------------------------
>
> Key: CB-9238
> URL: https://issues.apache.org/jira/browse/CB-9238
> Project: Apache Cordova
> Issue Type: Bug
> Components: Plugin Media, Windows
> Reporter: Alexander Sorokin
>
> Steps to reproduce:
> - create mobile spec app with media plugin
> - run mobile spec app on windows store or windows phone
> - go to media manual tests
> - click "Record Audio 10 sec"
> The result is: RecordAudio():Audio Error: 1
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
