From: Arkadiusz Pietraszek <a.pietraszek@partner.samsung.com>
Date: Thu, 27 Jun 2019 09:14:58 +0000 (+0200)
Subject: [Mediacontroller] Setter fixes for shuffleMode, repeatMode end position in MediaContr... 
X-Git-Tag: submit/tizen/20190805.070446~4^2^2^2~1^2~3
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=2faf59bd06f4b290552947f76d734efa4595c5ee;p=platform%2Fcore%2Fapi%2Fwebapi-plugins.git

[Mediacontroller] Setter fixes for shuffleMode, repeatMode end position in MediaControllerPlaybackInfo

Previous implementation intended to check for null and undefined and in reality didn't allow for setting 'false' or '0' values.

Change-Id: I03194e81d5d0bd5ef97a88d01a014f758a6ca49d
---

diff --git a/src/mediacontroller/mediacontroller_api.js b/src/mediacontroller/mediacontroller_api.js
index 720c8492..da9422e9 100755
--- a/src/mediacontroller/mediacontroller_api.js
+++ b/src/mediacontroller/mediacontroller_api.js
@@ -398,7 +398,7 @@ var MediaControllerPlaybackInfo = function(data) {
                 return _position;
             },
             set: function(v) {
-                _position = edit_.isAllowed && v ? v : _position;
+                _position = edit_.isAllowed ? converter_.toLong(v) : _position;
             },
             enumerable: true
         },
@@ -407,7 +407,7 @@ var MediaControllerPlaybackInfo = function(data) {
                 return _shuffleMode;
             },
             set: function(v) {
-                _shuffleMode = edit_.isAllowed && v ? v : _shuffleMode;
+                _shuffleMode = edit_.isAllowed ? converter_.toBoolean(v) : _shuffleMode;
             },
             enumerable: true
         },
@@ -416,7 +416,7 @@ var MediaControllerPlaybackInfo = function(data) {
                 return _repeatMode;
             },
             set: function(v) {
-                _repeatMode = edit_.isAllowed && v ? v : _repeatMode;
+                _repeatMode = edit_.isAllowed ? converter_.toBoolean(v) : _repeatMode;
             },
             enumerable: true
         },