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: accepted/tizen/unified/20190704.115630~3
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=3e8f64043ab57095b309f6c9aac4231904485031;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 46b1034..2dd6d3d 100755
--- a/src/mediacontroller/mediacontroller_api.js
+++ b/src/mediacontroller/mediacontroller_api.js
@@ -385,7 +385,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
     },
@@ -403,7 +403,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
     },
@@ -413,7 +413,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
     },