From: Andrzej Popowski <a.popowski@samsung.com>
Date: Wed, 28 Dec 2016 10:12:47 +0000 (+0100)
Subject: [HAM] - fixing pedometer listeners
X-Git-Tag: submit/tizen_3.0/20161229.023338~4
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=dbe8157f28394a8bba37a3ecab194c67e360209a;p=platform%2Fcore%2Fapi%2Fwebapi-plugins.git

[HAM] - fixing pedometer listeners

Change-Id: Ia9acbdb41e269ded7e246d08948c9556c30f6735
Signed-off-by: Andrzej Popowski <a.popowski@samsung.com>
---

diff --git a/src/humanactivitymonitor/humanactivitymonitor_api.js b/src/humanactivitymonitor/humanactivitymonitor_api.js
index 2d2d65e1..5c36b72b 100755
--- a/src/humanactivitymonitor/humanactivitymonitor_api.js
+++ b/src/humanactivitymonitor/humanactivitymonitor_api.js
@@ -354,12 +354,17 @@ HumanActivityMonitorManager.prototype.stop = function(type) {
     {name: 'type', type: types_.ENUM, values: Object.keys(HumanActivityType)}
   ]);
 
-  stopListener('HumanActivityMonitor_'  + args.type,
-               'HumanActivityMonitorManager_stop',
-               { type: args.type });
-
   if (HumanActivityType.PEDOMETER === args.type) {
+    if (pedometerListener && !accumulativePedometerListener) {
+      stopListener('HumanActivityMonitor_PEDOMETER',
+                   'HumanActivityMonitorManager_stop',
+                   { type: HumanActivityType.PEDOMETER });
+    }
     pedometerListener = null;
+  } else {
+    stopListener('HumanActivityMonitor_'  + args.type,
+                 'HumanActivityMonitorManager_stop',
+                 { type: args.type });
   }
 
   if (HumanActivityType.GPS === args.type) {
@@ -382,13 +387,12 @@ HumanActivityMonitorManager.prototype.setAccumulativePedometerListener = functio
 };
 
 HumanActivityMonitorManager.prototype.unsetAccumulativePedometerListener = function() {
-  var oldPedometerListener = pedometerListener;
-
-  // calling stop() will overwrite pedometerListener, needs to be restored afterwards
-  this.stop(HumanActivityType.PEDOMETER);
-
+  if (accumulativePedometerListener && !pedometerListener) {
+    stopListener('HumanActivityMonitor_PEDOMETER',
+                 'HumanActivityMonitorManager_stop',
+                 { type: HumanActivityType.PEDOMETER });
+  }
   accumulativePedometerListener = null;
-  pedometerListener = oldPedometerListener;
 };