Fixing my merge error for PR #1378 Optional event emit instead of fun…

…ction call for action buttons
phonegap · Apr 29, 2017 · 057cbd1 · 057cbd1
1 parent 83238fc
commit 057cbd1
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/src/js/push.js b/src/js/push.js
@@ -215,9 +215,10 @@ class PushNotification {
    */
 
   on(eventName, callback) {
-    if (this.handlers.hasOwnProperty(eventName)) {
-      this.handlers[eventName].push(callback);
+    if (!this.handlers.hasOwnProperty(eventName)) {
+      this.handlers[eventName] = [];
     }
+    this.handlers[eventName].push(callback);
   }
 
   /**

diff --git a/www/push.js b/www/push.js
@@ -258,9 +258,10 @@ var PushNotification = function () {
   }, {
     key: 'on',
     value: function on(eventName, callback) {
-      if (this.handlers.hasOwnProperty(eventName)) {
-        this.handlers[eventName].push(callback);
+      if (!this.handlers.hasOwnProperty(eventName)) {
+        this.handlers[eventName] = [];
       }
+      this.handlers[eventName].push(callback);
     }
 
     /**