Analyze.java-Merge this if statement with the enclosing one 65/96565/2
authorThugutla sailakshmi <tsaila10@in.ibm.com>
Thu, 3 Oct 2019 13:22:02 +0000 (18:52 +0530)
committerThugutla Sai Lakshmi <tsaila10@in.ibm.com>
Thu, 3 Oct 2019 13:27:21 +0000 (13:27 +0000)
Merge this if statement with the enclosing one

Issue-ID: AAF-876
Change-Id: Ibd707a4bd159861dec750610711840d35d9da22c
Signed-off-by: Thugutla sailakshmi <tsaila10@in.ibm.com>
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);
                         }