Merge "Analyze.java-Merge this if statement with the enclosing one"
authorJonathan Gathman <jonathan.gathman@att.com>
Tue, 8 Oct 2019 12:27:38 +0000 (12:27 +0000)
committerGerrit Code Review <gerrit@onap.org>
Tue, 8 Oct 2019 12:27:38 +0000 (12:27 +0000)
auth/auth-batch/src/main/java/org/onap/aaf/auth/batch/reports/Analyze.java

index 5780c60..d7d97ad 100644 (file)
@@ -391,8 +391,7 @@ public class Analyze extends Batch {
                                 return;
                             }
                             // Just let expired UserRoles sit until deleted
-                            if(futureRange.inRange(ur.expires())) {
-                                if(!mur.containsKey(ur.user() + '|' + ur.role())) {
+                            if(futureRange.inRange(ur.expires())&&(!mur.containsKey(ur.user() + '|' + ur.role()))) {    
                                     // Cannot just delete owners, unless there is at least one left. Process later
                                     if ("owner".equals(ur.rname())) {
                                         Set<UserRole> urs = owners.get(ur.role());
@@ -410,8 +409,7 @@ public class Analyze extends Batch {
                                             }
                                         }
                                     }
-                                }
-                            }
+                             }
                         } catch (OrganizationException e) {
                             noAvg.error().log(e);
                         }