commit 53bacf736379f91295957360a8ad3a127f38b5fb
author alsorokin <alexander.sorokin@akvelon.com> Tue May 12 16:32:02 2015 +0300
committer Dmitry Blotsky <dmitry.blotsky@gmail.com> Tue May 12 14:15:49 2015 -0700
tree 462c9ff0db18e6b9a82c43ff082f180790365079
parent b0463746dd842818c1f08560e998ec847460596c

CB-8996 (Windows) Fixed invalid null comparison. This closes #110.

src/windows/exec.js

diff --git a/src/windows/exec.js b/src/windows/exec.js
index d7f0c0f..18f559a 100644
--- a/src/windows/exec.js
+++ b/src/windows/exec.js
@@ -61,7 +61,11 @@
             onSuccess = function (result, callbackOptions) {
                 callbackOptions = callbackOptions || {};
                 var callbackStatus;
-                if (callbackOptions.status !== null) {
+                // covering both undefined and null.
+                // strict null comparison was causing callbackStatus to be undefined
+                // and then no callback was called because of the check in cordova.callbackFromNative
+                // see CB-8996 Mobilespec app hang on windows
+                if (callbackOptions.status !== undefined && callbackOptions.status !== null) {
                     callbackStatus = callbackOptions.status;
                 }
                 else {
@@ -77,7 +81,11 @@
             onError = function (err, callbackOptions) {
                 callbackOptions = callbackOptions || {};
                 var callbackStatus;
-                if (callbackOptions.status !== null) {
+                // covering both undefined and null.
+                // strict null comparison was causing callbackStatus to be undefined
+                // and then no callback was called because of the check in cordova.callbackFromNative
+                // see CB-8996 Mobilespec app hang on windows
+                if (callbackOptions.status !== undefined && callbackOptions.status !== null) {
                     callbackStatus = callbackOptions.status;
                 }
                 else {