[Mediacontroller] Setter fixes for shuffleMode, repeatMode end position in MediaContr... 16/208716/1
authorArkadiusz Pietraszek <a.pietraszek@partner.samsung.com>
Thu, 27 Jun 2019 09:14:58 +0000 (11:14 +0200)
committerPiotr Kosko/Native/Web API (PLT) /SRPOL/Professional/삼성전자 <p.kosko@samsung.com>
Thu, 27 Jun 2019 09:47:12 +0000 (11:47 +0200)
Previous implementation intended to check for null and undefined and in reality didn't allow for setting 'false' or '0' values.

Change-Id: I03194e81d5d0bd5ef97a88d01a014f758a6ca49d

src/mediacontroller/mediacontroller_api.js

index 46b1034..2dd6d3d 100755 (executable)
@@ -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
     },