update anonymous inner classes with lambdas 79/76279/1
authorDriptaroop Das <driptaroop.das@in.ibm.com>
Thu, 24 Jan 2019 11:15:41 +0000 (16:45 +0530)
committerDriptaroop Das <driptaroop.das@in.ibm.com>
Thu, 24 Jan 2019 11:15:53 +0000 (16:45 +0530)
update anonymous inner classes of some files with lambdas

Issue-ID: AAF-731
Change-Id: Ifc37d1aef4017943a5d33ee7c89ca919df474436
Signed-off-by: Driptaroop Das <driptaroop.das@in.ibm.com>
auth/auth-batch/src/main/java/org/onap/aaf/auth/batch/actions/URFutureApproveExec.java
auth/auth-batch/src/main/java/org/onap/aaf/auth/batch/reports/Expiring.java
auth/auth-batch/src/main/java/org/onap/aaf/auth/batch/reports/NotInOrg.java
auth/auth-batch/src/main/java/org/onap/aaf/auth/batch/reports/NsRoleUserReport.java
auth/auth-certman/src/main/java/org/onap/aaf/auth/cm/ca/JscepCA.java
auth/auth-cmd/src/main/java/org/onap/aaf/auth/cmd/Cmd.java

index 631a811..6636f45 100644 (file)
@@ -66,15 +66,8 @@ public class URFutureApproveExec extends ActionDAO<List<Approval>, OP_STATUS, Fu
                 }
             }
             Result<OP_STATUS> rv = f.performFutureOp(trans, FUTURE_OP.A, future.fdd,
-                new Lookup<List<ApprovalDAO.Data>>() {
-                    @Override
-                    public List<Data> get(AuthzTrans trans, Object ... noop) {
-                        return apprs;
-                    }
-                },
-                new Lookup<UserRoleDAO.Data>() {
-                    @Override
-                    public UserRoleDAO.Data get(AuthzTrans trans, Object ... keys) {
+                    (trans1, noop) -> apprs,
+                    (trans12, keys) -> {
                         List<UserRole> lur = UserRole.getByUser().get(keys[0]);
                         if (lur!=null) {
                             for (UserRole ur : lur) {
@@ -84,8 +77,7 @@ public class URFutureApproveExec extends ActionDAO<List<Approval>, OP_STATUS, Fu
                             }
                         }
                         return null;
-                    }
-                });
+                    });
             if (rv.isOK()) {
                 switch(rv.value) {
                     case D:
index e171d17..11eacd4 100644 (file)
@@ -193,18 +193,15 @@ public class Expiring extends Batch {
                        }
                        
                        trans.info().log("Checking for Expired X509s");
-                       X509.load(trans, session, new Visitor<X509>() {
-                               @Override
-                               public void visit(X509 x509) {
-                                       try {
-                                               for(Certificate cert : Factory.toX509Certificate(x509.x509)) {
-                                                       writeAnalysis(trans, x509, (X509Certificate)cert);
-                                               }
-                                       } catch (CertificateException | IOException e) {
-                                               trans.error().log(e, "Error Decrypting X509");
+                       X509.load(trans, session, x509 -> {
+                               try {
+                                       for(Certificate cert : Factory.toX509Certificate(x509.x509)) {
+                                               writeAnalysis(trans, x509, (X509Certificate)cert);
                                        }
-                                       
+                               } catch (CertificateException | IOException e) {
+                                       trans.error().log(e, "Error Decrypting X509");
                                }
+
                        });
                } catch (FileNotFoundException e) {
                        trans.info().log(e);
index fe754c1..ab8b6e8 100644 (file)
@@ -105,16 +105,13 @@ public class NotInOrg extends Batch {
                        trans.info().log("User Roles");
                        
                        final AuthzTrans transNoAvg = trans.env().newTransNoAvg();
-                       UserRole.load(trans, session, UserRole.v2_0_11, new Visitor<UserRole>() {
-                               @Override
-                               public void visit(UserRole ur) {
-                                       try {
-                                               if(!check(transNoAvg, checked, ur.user())) {
-                                                       ur.row(whichWriter(transNoAvg,ur.user()));
-                                               }
-                                       } catch (OrganizationException e) {
-                                               trans.error().log(e, "Error Decrypting X509");
+                       UserRole.load(trans, session, UserRole.v2_0_11, ur -> {
+                               try {
+                                       if(!check(transNoAvg, checked, ur.user())) {
+                                               ur.row(whichWriter(transNoAvg,ur.user()));
                                        }
+                               } catch (OrganizationException e) {
+                                       trans.error().log(e, "Error Decrypting X509");
                                }
                        });
                        
index 8281e74..1a7d774 100644 (file)
@@ -86,24 +86,21 @@ public class NsRoleUserReport extends Batch {
                        trans.info().log("Create Report on Roles by NS");
                        
                        final AuthzTrans transNoAvg = trans.env().newTransNoAvg();
-                       UserRole.load(transNoAvg, session, UserRole.v2_0_11, new Visitor<UserRole>() {
-                               @Override
-                               public void visit(UserRole ur) {
-                                       if(ur.expires().after(now)) {
-                                               Map<String, Integer> roleCount = theMap.get(ur.ns());
-                                               Integer count;
-                                               if(roleCount==null) {
-                                                       roleCount = new TreeMap<>();
-                                                       theMap.put(ur.ns(),roleCount);
+                       UserRole.load(transNoAvg, session, UserRole.v2_0_11, ur -> {
+                               if(ur.expires().after(now)) {
+                                       Map<String, Integer> roleCount = theMap.get(ur.ns());
+                                       Integer count;
+                                       if(roleCount==null) {
+                                               roleCount = new TreeMap<>();
+                                               theMap.put(ur.ns(),roleCount);
+                                               count = 0;
+                                       } else {
+                                               count = roleCount.get(ur.rname());
+                                               if(count == null) {
                                                        count = 0;
-                                               } else {
-                                                       count = roleCount.get(ur.rname());
-                                                       if(count == null) {
-                                                               count = 0;
-                                                       }
                                                }
-                                               roleCount.put(ur.rname(), count+1);
                                        }
+                                       roleCount.put(ur.rname(), count+1);
                                }
                        });
                        
index 4caa339..b7dd069 100644 (file)
@@ -225,14 +225,11 @@ public class JscepCA extends CA {
         protected Client _newClient(String urlinfo) throws LocatorException {
             try {
                 String[] info = Split.split('/', urlinfo);
-                Client c = new Client(new URL(JscepCA.CA_PREFIX + info[0] + JscepCA.CA_POSTFIX), 
-                        new CertificateVerifier() {
-                        @Override
-                        public boolean verify(X509Certificate cert) {
+                Client c = new Client(new URL(JscepCA.CA_PREFIX + info[0] + JscepCA.CA_POSTFIX),
+                        cert -> {
                             //TODO checkIssuer
                             return true;
                         }
-                    }
                 );
                 // Map URL to Client, because Client doesn't expose Connection
                 mxcwiC.put(c, mxcwiS.get(urlinfo));
index 0597640..49fd486 100644 (file)
@@ -358,12 +358,7 @@ public abstract class Cmd {
     
             // Save Server time by Sorting locally
             List<Item> items = history.getItem();
-            java.util.Collections.sort(items, new Comparator<Item>() {
-                @Override
-                public int compare(Item o1, Item o2) {
-                    return o2.getTimestamp().compare(o1.getTimestamp());
-                }
-            });
+            java.util.Collections.sort(items, (Comparator<Item>) (o1, o2) -> o2.getTimestamp().compare(o1.getTimestamp()));
             
             for (History.Item item : items) {
                 GregorianCalendar gc = item.getTimestamp().toGregorianCalendar();