Repository: ambari Updated Branches: refs/heads/branch-2.5 a31a011ce -> 5e3d7b47c
AMBARI-19341. Not able to update the role for a given user (onechiporenko) Project: http://git-wip-us.apache.org/repos/asf/ambari/repo Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/5e3d7b47 Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/5e3d7b47 Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/5e3d7b47 Branch: refs/heads/branch-2.5 Commit: 5e3d7b47c60a61db3abedc2c3c9f783bd2827819 Parents: a31a011 Author: Oleg Nechiporenko <[email protected]> Authored: Wed Jan 4 18:39:05 2017 +0200 Committer: Oleg Nechiporenko <[email protected]> Committed: Thu Jan 5 10:30:20 2017 +0200 ---------------------------------------------------------------------- .../app/scripts/controllers/clusters/UserAccessListCtrl.js | 7 ++++--- .../ui/admin-web/app/views/clusters/userAccessList.html | 2 +- .../ui/admin-web/test/unit/services/PermissionSaver_test.js | 5 +++++ 3 files changed, 10 insertions(+), 4 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/ambari/blob/5e3d7b47/ambari-admin/src/main/resources/ui/admin-web/app/scripts/controllers/clusters/UserAccessListCtrl.js ---------------------------------------------------------------------- diff --git a/ambari-admin/src/main/resources/ui/admin-web/app/scripts/controllers/clusters/UserAccessListCtrl.js b/ambari-admin/src/main/resources/ui/admin-web/app/scripts/controllers/clusters/UserAccessListCtrl.js index 4f00aeb..9869d0b 100644 --- a/ambari-admin/src/main/resources/ui/admin-web/app/scripts/controllers/clusters/UserAccessListCtrl.js +++ b/ambari-admin/src/main/resources/ui/admin-web/app/scripts/controllers/clusters/UserAccessListCtrl.js @@ -166,9 +166,10 @@ function($scope, $location, Cluster, $modal, $rootScope, $routeParams, Permissio // Process when it's NONE privilege or higher than current effective group privilege if (userIndex <= groupIndex || user.permission_name == $scope.NONE_ROLE.permission_name) { - var privilege_ids = []; - privilegesOfTypeUser.forEach(function(privilegeOfTypeUser) { - privilege_ids.push(privilegeOfTypeUser.privilege_id); + var privilege_ids = privilegesOfTypeUser.filter(function(privilegeOfTypeUser) { + return privilegeOfTypeUser.principal_type !== 'ROLE'; + }).map(function (privilegeOfTypeUser) { + return privilegeOfTypeUser.privilege_id; }); // Purge existing user level privileges if there is any http://git-wip-us.apache.org/repos/asf/ambari/blob/5e3d7b47/ambari-admin/src/main/resources/ui/admin-web/app/views/clusters/userAccessList.html ---------------------------------------------------------------------- diff --git a/ambari-admin/src/main/resources/ui/admin-web/app/views/clusters/userAccessList.html b/ambari-admin/src/main/resources/ui/admin-web/app/views/clusters/userAccessList.html index 7b54f9f..8b29157 100644 --- a/ambari-admin/src/main/resources/ui/admin-web/app/views/clusters/userAccessList.html +++ b/ambari-admin/src/main/resources/ui/admin-web/app/views/clusters/userAccessList.html @@ -68,7 +68,7 @@ </td> <td> <div ng-show="!user.editable">{{user.permission_label}}</div> - <select class="role-select" ng-show="user.editable" ng-model="user.permission_name" + <select class="form-control role-select" ng-show="user.editable" ng-model="user.permission_name" ng-options="role.permission_name as role.permission_label for role in roleValueOptions"> </select> <span ng-show="user.principal_type != $parent.currentTypeFilter.value"> {{'common.fromGroupMark' | translate}}</span> http://git-wip-us.apache.org/repos/asf/ambari/blob/5e3d7b47/ambari-admin/src/main/resources/ui/admin-web/test/unit/services/PermissionSaver_test.js ---------------------------------------------------------------------- diff --git a/ambari-admin/src/main/resources/ui/admin-web/test/unit/services/PermissionSaver_test.js b/ambari-admin/src/main/resources/ui/admin-web/test/unit/services/PermissionSaver_test.js index 6c662f2..6a409e4 100644 --- a/ambari-admin/src/main/resources/ui/admin-web/test/unit/services/PermissionSaver_test.js +++ b/ambari-admin/src/main/resources/ui/admin-web/test/unit/services/PermissionSaver_test.js @@ -40,6 +40,7 @@ describe('PermissionSaver Service', function () { PermissionInfo: { permission_name: 'CLUSTER.ADMINISTRATOR' }, + ROLE: {}, USER: ['u0', 'u1', 'g0'], GROUP: ['g0', 'g1', 'u0'] }, @@ -47,6 +48,7 @@ describe('PermissionSaver Service', function () { PermissionInfo: { permission_name: 'CLUSTER.OPERATOR' }, + ROLE: {}, USER: ['g1'], GROUP: ['u1'] } @@ -118,6 +120,7 @@ describe('PermissionSaver Service', function () { PermissionInfo: { permission_name: 'CLUSTER.ADMINISTRATOR' }, + ROLE: {}, USER: ['u0', 'u1'], GROUP: ['g0', 'g1'] }, @@ -125,6 +128,7 @@ describe('PermissionSaver Service', function () { PermissionInfo: { permission_name: 'CLUSTER.OPERATOR' }, + ROLE: {}, USER: ['u0'], GROUP: ['g2'] }, @@ -132,6 +136,7 @@ describe('PermissionSaver Service', function () { PermissionInfo: { permission_name: 'CLUSTER.USER' }, + ROLE: {}, USER: ['u2'], GROUP: ['g0'] }
