You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cordova.apache.org by de...@apache.org on 2012/03/30 20:44:04 UTC

git commit: Add Gord's fix for PlayBook network.

Updated Branches:
  refs/heads/master 36caca0d2 -> 33f1fa37b


Add Gord's fix for PlayBook network.


Project: http://git-wip-us.apache.org/repos/asf/incubator-cordova-js/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-cordova-js/commit/33f1fa37
Tree: http://git-wip-us.apache.org/repos/asf/incubator-cordova-js/tree/33f1fa37
Diff: http://git-wip-us.apache.org/repos/asf/incubator-cordova-js/diff/33f1fa37

Branch: refs/heads/master
Commit: 33f1fa37b077c5a3c39a43d07e131e6b75171e64
Parents: 36caca0
Author: Drew Walters <de...@gmail.com>
Authored: Fri Mar 30 13:43:38 2012 -0500
Committer: Drew Walters <de...@gmail.com>
Committed: Fri Mar 30 13:43:38 2012 -0500

----------------------------------------------------------------------
 lib/playbook/plugin/playbook/manager.js |   10 ++++++----
 1 files changed, 6 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-cordova-js/blob/33f1fa37/lib/playbook/plugin/playbook/manager.js
----------------------------------------------------------------------
diff --git a/lib/playbook/plugin/playbook/manager.js b/lib/playbook/plugin/playbook/manager.js
index f83f47d..4a4b5b3 100644
--- a/lib/playbook/plugin/playbook/manager.js
+++ b/lib/playbook/plugin/playbook/manager.js
@@ -244,8 +244,8 @@ var webworks = require('cordova/plugin/webworks/manager'),
                 request;
 
             /**
-             * For PlayBooks, we currently only have WiFi connections, so return WiFi if there is
-             * any access at all.
+             * For PlayBooks, we currently only have WiFi connections, so
+             * return WiFi if there is any access at all.
              * TODO: update if/when PlayBook gets other connection types...
              */
             if (blackberry.system.hasDataCoverage()) {
@@ -254,14 +254,16 @@ var webworks = require('cordova/plugin/webworks/manager'),
             }
 
             //Register an event handler for the networkChange event
-            callbackID = blackberry.events.registerEventHandler("networkChange", win);
+            callbackID = blackberry.events.registerEventHandler("networkChange", function (status) {
+                win(status.type);
+            });
 
             //pass our callback id down to our network extension
             request = new blackberry.transport.RemoteFunctionCall("org/apache/cordova/getConnectionInfo");
             request.addParam("networkStatusChangedID", callbackID);
             request.makeSyncCall();
 
-            return { "status": cordova.callbackStatus.OK, "message": {"type": connectionType, "event": eventType } };
+            return { "status": cordova.callbackStatus.OK, "message": connectionType};
         }
     },
     notificationAPI = {