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/approvalsets/Pending.java
auth/auth-batch/src/main/java/org/onap/aaf/auth/batch/helpers/LastNotified.java
auth/auth-core/src/main/java/org/onap/aaf/auth/org/OrganizationFactory.java
auth/auth-core/src/main/java/org/onap/aaf/auth/rserv/TypedCode.java
auth/auth-fs/src/main/java/org/onap/aaf/auth/fs/AAF_FS.java
auth/auth-hello/src/main/java/org/onap/aaf/auth/hello/API_Hello.java
auth/auth-locate/src/main/java/org/onap/aaf/auth/locate/AAF_Locate.java
auth/auth-locate/src/main/java/org/onap/aaf/auth/locate/service/LocateServiceImpl.java

index eed6733..9a66698 100644 (file)
@@ -3,6 +3,8 @@
  * org.onap.aaf
  * ===========================================================================
  * Copyright (c) 2018 AT&T Intellectual Property. All rights reserved.
+ *
+ * Modifications Copyright (C) 2019 IBM.
  * ===========================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -36,12 +38,12 @@ public class Pending {
     
     /**
      * Use this constructor to indicate when last Notified
-     * @param last_notified
+     * @param lastNotified
      */
-    public Pending(Date last_notified) {
+    public Pending(Date lastNotified) {
         qty = 1;
-        hasNew = last_notified==null;
-        earliest = last_notified;
+        hasNew = lastNotified==null;
+        earliest = lastNotified;
     }
 
     /**
index 6f228e0..dfacd81 100644 (file)
@@ -45,7 +45,7 @@ import com.datastax.driver.core.SimpleStatement;
 import com.datastax.driver.core.Statement;
 
 public class LastNotified {
-    private Map<String,Date> lastNotified = new TreeMap<>();
+    private Map<String,Date> lastNotifiedVar = new TreeMap<>();
     private Session session;
     public static final Date NEVER = new Date(0);
     private static final String SELECT = "SELECT user,target,key,last FROM authz.notified";
@@ -67,7 +67,7 @@ public class LastNotified {
             query.append('\'');
             if(cnt>=30) {
                 endQuery(query);
-                add(session.execute(query.toString()),lastNotified, (x,y) -> false);
+                add(session.execute(query.toString()),lastNotifiedVar, (x,y) -> false);
                 query.setLength(0);
                 startQuery(query);
                 cnt=0;
@@ -75,7 +75,7 @@ public class LastNotified {
         }
         if(cnt>0) {
             endQuery(query);
-            add(session.execute(query.toString()),lastNotified, (x,y) -> false);
+            add(session.execute(query.toString()),lastNotifiedVar, (x,y) -> false);
         }
     }
 
@@ -93,7 +93,7 @@ public class LastNotified {
     }
     
     public Date lastNotified(String key) {
-        Date d = lastNotified.get(key);
+        Date d = lastNotifiedVar.get(key);
         return d==null?NEVER:d;
     }
     
@@ -150,7 +150,7 @@ public class LastNotified {
         try {
             Statement stmt = new SimpleStatement( SELECT );
             results = session.execute(stmt);
-            add(results,lastNotified, (fullKey, last) ->  {
+            add(results,lastNotifiedVar, (fullKey, last) ->  {
                 if(delRange.inRange(last)) {
                     String[] params = Split.splitTrim('|', fullKey,3);
                     if(params.length==3) {
index 4f2ba4f..2ada75d 100644 (file)
@@ -142,7 +142,7 @@ public class OrganizationFactory {
         return org;
     }
 
-    public static Organization get(AuthzTrans trans) throws OrganizationException {
+    public static Organization get(AuthzTrans trans){
         String domain = FQI.reverseDomain(trans.user());
         Organization org = orgs.get(domain);
         if (org==null) {
index 1a66cf7..6f1372f 100644 (file)
@@ -128,7 +128,7 @@ public class TypedCode<TRANS extends Trans> extends Content<TRANS> {
             return type.y.y.add(new Pair<String,Object>(tag,"version".equals(tag)?new Version(value):value));
         }
         
-        public Pair<String, Pair<HttpCode<TRANS, ?>, List<Pair<String, Object>>>> prep(TRANS trans, String compare) throws IOException, ServletException {
+        public Pair<String, Pair<HttpCode<TRANS, ?>, List<Pair<String, Object>>>> prep(TRANS trans, String compare){
             Pair<String, Pair<HttpCode<TRANS,?>, List<Pair<String, Object>>>> c,rv=null;
             if (types.size()==1 && "".equals((c=types.get(0)).x)) { // if there are no checks for type, skip
                 rv = c;
index df05da9..cee06dd 100644 (file)
@@ -49,7 +49,7 @@ import org.onap.aaf.misc.env.APIException;
 
 public class AAF_FS extends AbsService<AuthzEnv, AuthzTrans>  {
 
-    public AAF_FS(final AuthzEnv env) throws APIException, IOException, CadiException {
+    public AAF_FS(final AuthzEnv env) throws IOException, CadiException {
         super(env.access(),env);
         try {
             ///////////////////////  
index 7fef0ba..814dff2 100644 (file)
@@ -56,7 +56,7 @@ public class API_Hello {
      * @param facade
      * @throws Exception
      */
-    public static void init(final AAF_Hello oauthHello) throws Exception {
+    public static void init(final AAF_Hello oauthHello){
         ////////
         // Simple "GET" API
         ///////
index b481ff9..0ad14c8 100644 (file)
@@ -170,7 +170,7 @@ public class AAF_Locate extends AbsService<AuthzEnv, AuthzTrans> {
         // Add other Supported APIs here as created
     }
     
-    public void routeAll(HttpMethods meth, String path, API api, LocateCode code) throws Exception {
+    public void routeAll(HttpMethods meth, String path, API api, LocateCode code){
         route(env,meth,path,code,""); // this will always match
     }
 
index e080e7e..1ace384 100644 (file)
@@ -47,7 +47,7 @@ public class LocateServiceImpl<IN,OUT,ERROR>
         private ConfigDAO configDAO;
         private boolean permToRegister;
     
-        public LocateServiceImpl(AuthzTrans trans, AAF_Locate locate, Mapper<IN,OUT,Endpoints,MgmtEndpoints,Configuration,ERROR> mapper) throws APIException {
+        public LocateServiceImpl(AuthzTrans trans, AAF_Locate locate, Mapper<IN,OUT,Endpoints,MgmtEndpoints,Configuration,ERROR> mapper){
             this.mapper = mapper;
             this.locateDAO = locate.locateDAO;
             this.configDAO = locate.configDAO;