[CB-7639] Notification.confirm() fails on windows due to multiple issues

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Component/s: cordova-plugin-dialogs
• Assignee: Unassigned
• Reporter: Vladimir Kotikov
• Created: 25/Sep/14 13:34
• Updated: 23/Aug/18 07:17
• Resolved: 13/Nov/14 06:53

Description

There are bunch of issues related to this bug:

1. On WP8.1 it's not possible to show dialog with more than two buttons. Such attempt will cause WinRTError: The parameter is incorrect exception
2. On Windows8/8.1 it is not possible to add more than three buttons to MessageDialog instance. It will throw JavaScript runtime error: The operation attempted to access data outside the valid range error.

These issues should be noted in plugin's docs.

1. After dialog call, that causes any exception such as listed above, following attempts to show any dialog will have no effect because of windows's implementation logic. We need to handle exceptions inside of plugin and always set isAlertShowing flag to false to prevent queuing of future dialogs calls.
2. When confirm dialog is called with buttons array, if one of button's labels contains a comma, it will be splitted to two separate labels. For example ["yes","no","maybe,possible"] will be converted into ["yes","no","maybe", "possible"]

Also there is issue in manual tests: windows Javascript apps doesn't have built-in notification finctions, such as alert() or confirm(), so manual tests that calls this methods will fail. We need to add check for these methods exists to manual tests.

Issue Links

• relates to: CB-7237 Execution failure on Windows 8.0