[ranger] branch master updated: RANGER-3312 : Role is getting removed from policy when user present in that policy is deleted

[pradeep]

This is an automated email from the ASF dual-hosted git repository.

pradeep pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ranger.git


The following commit(s) were added to refs/heads/master by this push:
     new 91ef9b5  RANGER-3312 : Role is getting removed from policy when user 
present in that policy is deleted
91ef9b5 is described below

commit 91ef9b5cee87998c2e6a1a215525476094148f58
Author: Dineshkumar Yadav <dineshkumar.ya...@outlook.com>
AuthorDate: Thu Jun 10 12:17:42 2021 +0530

    RANGER-3312 : Role is getting removed from policy when user present in that 
policy is deleted
    
    Signed-off-by: pradeep <prad...@apache.org>
---
 security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java 
b/security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java
index fad332a..6242326 100755
--- a/security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java
+++ b/security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java
@@ -2429,7 +2429,7 @@ public class XUserMgr extends XUserMgrBase {
                        if(StringUtils.isNotEmpty(group)) {
                                policyItem.getGroups().remove(group);
                        }
-                       if(policyItem.getUsers().isEmpty() && 
policyItem.getGroups().isEmpty()) {
+                       if(policyItem.getUsers().isEmpty() && 
policyItem.getGroups().isEmpty() && policyItem.getRoles().isEmpty()) {
                                if(itemsToRemove == null) {
                                        itemsToRemove = new ArrayList<T>();
                                }