From: Pawel Andruszkiewicz Date: Wed, 3 Jun 2015 07:24:33 +0000 (+0200) Subject: [Badge] Fixed validation of arguments. X-Git-Tag: submit/tizen_mobile/20150612.133019^2~2^2~59^2 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=3b73a3baf3fc3e894b34c59507278169304ddcfd;p=platform%2Fcore%2Fapi%2Fwebapi-plugins.git [Badge] Fixed validation of arguments. [Verification] TCT pass rate: 26/26. Change-Id: Ie4c3d25607533fad864e1b6e129afff53c45cfa4 Signed-off-by: Pawel Andruszkiewicz --- diff --git a/src/badge/badge_api.js b/src/badge/badge_api.js index c380453b..4a2ab7c1 100755 --- a/src/badge/badge_api.js +++ b/src/badge/badge_api.js @@ -58,18 +58,15 @@ function BadgeManager() { BadgeManager.prototype.setBadgeCount = function() { xwalk.utils.checkPrivilegeAccess(xwalk.utils.privilege.NOTIFICATION); - if (arguments.length < 2) - throw new WebAPIException(WebAPIException.INVALID_VALUES_ERR, - 'incorrect number of arguments'); - var args = validator_.validateArgs(arguments, [ {name: 'appId', type: types_.STRING}, {name: 'count', type: types_.LONG} ]); - if (args.count < 0) + if (args.count < 0) { throw new WebAPIException(WebAPIException.INVALID_VALUES_ERR, - 'Count parameter is negative!'); + 'Count parameter is negative!'); + } var ret = native_.callSync('BadgeManager_setBadgeCount', { appId: args.appId, @@ -89,9 +86,6 @@ BadgeManager.prototype.setBadgeCount = function() { BadgeManager.prototype.getBadgeCount = function() { xwalk.utils.checkPrivilegeAccess(xwalk.utils.privilege.NOTIFICATION); - if (arguments.length < 1) - throw new WebAPIException(WebAPIException.INVALID_VALUES_ERR, - 'incorrect number of arguments'); var args = validator_.validateArgs(arguments, [ {name: 'appId', type: types_.STRING} ]); @@ -114,9 +108,6 @@ BadgeManager.prototype.getBadgeCount = function() { BadgeManager.prototype.addChangeListener = function() { xwalk.utils.checkPrivilegeAccess(xwalk.utils.privilege.NOTIFICATION); - if (arguments.length < 2) - throw new WebAPIException(WebAPIException.TYPE_MISMATCH_ERR, - 'incorrect number of arguments'); var args = validator_.validateArgs(arguments, [ { name: 'appIdList', @@ -158,10 +149,6 @@ BadgeManager.prototype.addChangeListener = function() { BadgeManager.prototype.removeChangeListener = function() { xwalk.utils.checkPrivilegeAccess(xwalk.utils.privilege.NOTIFICATION); - if (arguments.length < 1) - throw new WebAPIException(WebAPIException.TYPE_MISMATCH_ERR, - 'incorrect number of arguments'); - var args = validator_.validateArgs(arguments, [ { name: 'appIdList',