[
https://issues.apache.org/jira/browse/CB-8038?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14216692#comment-14216692
]
ASF GitHub Bot commented on CB-8038:
------------------------------------
GitHub user stacic opened a pull request:
https://github.com/apache/cordova-plugin-dialogs/pull/42
CB-8038 backslash getting escaped twice in bb10
Removed unnecessary escaping of backslash char in notifications for bb10,
as they are already escaped in the js.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/stacic/cordova-plugin-dialogs CB-8038
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cordova-plugin-dialogs/pull/42.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 #42
----
commit 33da80badd0b6cda8cc0c19feb977922f718906b
Author: Staci Cooper <smcoo...@us.ibm.com>
Date: 2014-11-18T19:55:23Z
CB-8038 backslash getting escaped twice in bb10
Removed unnecessary escaping of backslash char in notifications for bb10,
as they are already escaped in the js.
----
> BB10 dialogs escapes backslash twice
> ------------------------------------
>
> Key: CB-8038
> URL: https://issues.apache.org/jira/browse/CB-8038
> Project: Apache Cordova
> Issue Type: Bug
> Components: BlackBerry, Plugin Dialogs
> Reporter: Staci Cooper
> Assignee: Staci Cooper
> Priority: Minor
>
> navigator.notification.confirm("\\\\", function(){}, "hello");
> On android/ios the above will be printed as '\\'. On BlackBerry it prints '\'.
> Looks like the js escapes the backslashes as expected, but they are then
> unnecessarily escaped again in the plugin code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org
