[Mediacontroller] Setter fixes for shuffleMode, repeatMode end position in MediaContr... 12/208712/3
authorArkadiusz Pietraszek <a.pietraszek@partner.samsung.com>
Thu, 27 Jun 2019 09:14:58 +0000 (11:14 +0200)
committerArkadiusz Pietraszek <a.pietraszek@partner.samsung.com>
Thu, 27 Jun 2019 09:27:37 +0000 (11:27 +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 720c84923cd1a70ac6e26daf61736ae5a852d5ef..da9422e951496e850cdd2e812cdd2f18f75cb749 100755 (executable)
@@ -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
         },