diff options
4 files changed, 22 insertions, 16 deletions
diff --git a/POLICY-SDK-APP/src/main/java/org/onap/policy/admin/PolicyManagerServlet.java b/POLICY-SDK-APP/src/main/java/org/onap/policy/admin/PolicyManagerServlet.java index 93fe82c34..16a8ff7a9 100644 --- a/POLICY-SDK-APP/src/main/java/org/onap/policy/admin/PolicyManagerServlet.java +++ b/POLICY-SDK-APP/src/main/java/org/onap/policy/admin/PolicyManagerServlet.java @@ -1276,7 +1276,7 @@ public class PolicyManagerServlet extends HttpServlet { } } }else{ - List<String> activePoliciesInPDP = new ArrayList<String>(); + List<String> activePoliciesInPDP = new ArrayList<>(); if(!policyEntityobjects.isEmpty()){ for(Object object : policyEntityobjects){ policyEntity = (PolicyEntity) object; diff --git a/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/AutoPushController.java b/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/AutoPushController.java index e0388e5d6..96e104656 100644 --- a/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/AutoPushController.java +++ b/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/AutoPushController.java @@ -119,8 +119,8 @@ public class AutoPushController extends RestrictedBaseController{ @RequestMapping(value={"/get_AutoPushPoliciesContainerData"}, method={org.springframework.web.bind.annotation.RequestMethod.GET} , produces=MediaType.APPLICATION_JSON_VALUE) public void getPolicyGroupContainerData(HttpServletRequest request, HttpServletResponse response){ try{ - Set<String> scopes = null; - List<String> roles = null; + Set<String> scopes; + List<String> roles; data = new ArrayList<>(); String userId = UserUtils.getUserSession(request).getOrgUserId(); Map<String, Object> model = new HashMap<>(); @@ -139,7 +139,7 @@ public class AutoPushController extends RestrictedBaseController{ scopes.add(multipleScopes[i]); } }else{ - if(!userRole.getScope().equals("")){ + if(!"".equals(userRole.getScope())){ scopes.add(userRole.getScope()); } } @@ -220,7 +220,7 @@ public class AutoPushController extends RestrictedBaseController{ // Get the current selection String selectedItem = policyId; // - assert (selectedItem != null); + assert selectedItem != null; // create the id of the target file // Our standard for file naming is: // <domain>.<filename>.<version>.xml @@ -328,6 +328,10 @@ public class AutoPushController extends RestrictedBaseController{ JsonMessage msg = new JsonMessage(mapper.writeValueAsString(groups)); JSONObject j = new JSONObject(msg); out.write(j.toString()); + // + // Why is this here? This defeats the purpose of the loop?? + // Sonar says to remove it or make it conditional + // return null; } } @@ -336,7 +340,7 @@ public class AutoPushController extends RestrictedBaseController{ request.setCharacterEncoding("UTF-8"); PrintWriter out = response.getWriter(); logger.error(e); - out.write(PolicyUtils.CATCH_EXCEPTION);; + out.write(PolicyUtils.CATCH_EXCEPTION); } return null; } @@ -393,7 +397,7 @@ public class AutoPushController extends RestrictedBaseController{ request.setCharacterEncoding("UTF-8"); PrintWriter out = response.getWriter(); logger.error(e); - out.write(PolicyUtils.CATCH_EXCEPTION);; + out.write(PolicyUtils.CATCH_EXCEPTION); } return null; } diff --git a/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/CreateClosedLoopFaultController.java b/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/CreateClosedLoopFaultController.java index 7cfc4fed4..d942939d1 100644 --- a/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/CreateClosedLoopFaultController.java +++ b/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/CreateClosedLoopFaultController.java @@ -168,7 +168,7 @@ public class CreateClosedLoopFaultController extends RestrictedBaseController{ @SuppressWarnings("unchecked") - private String connectTriggerSignature(int index, ArrayList<Object> triggerSignatures, Object object) { + private String connectTriggerSignature(int index, List<Object> triggerSignatures, Object object) { StringBuilder resultBody = new StringBuilder(); Map<String, String> connectTraps = (Map<String, String>) triggerSignatures.get(index); try{ @@ -381,7 +381,7 @@ public class CreateClosedLoopFaultController extends RestrictedBaseController{ //connect traps data set to JSON Body as String @SuppressWarnings({ "unchecked", "rawtypes" }) - private String getUIConnectTraps(ArrayList<Object> connectTrapSignatures) { + private String getUIConnectTraps(List<Object> connectTrapSignatures) { StringBuilder resultBody = new StringBuilder(); String connectMainBody = ""; for(int j = 0; j < connectTrapSignatures.size(); j++){ @@ -617,8 +617,8 @@ class ClosedLoopGridJSONData{ private String clearTimeOut; private String trapMaxAge; private String verificationclearTimeOut; - private ArrayList<Object> connecttriggerSignatures; - private ArrayList<Object> connectVerificationSignatures; + private List<Object> connecttriggerSignatures; + private List<Object> connectVerificationSignatures; public String getClearTimeOut() { return clearTimeOut; @@ -640,16 +640,16 @@ class ClosedLoopGridJSONData{ } - public ArrayList<Object> getConnecttriggerSignatures() { + public List<Object> getConnecttriggerSignatures() { return connecttriggerSignatures; } - public void setConnecttriggerSignatures(ArrayList<Object> connecttriggerSignatures) { + public void setConnecttriggerSignatures(List<Object> connecttriggerSignatures) { this.connecttriggerSignatures = connecttriggerSignatures; } - public ArrayList<Object> getConnectVerificationSignatures() { + public List<Object> getConnectVerificationSignatures() { return connectVerificationSignatures; } - public void setConnectVerificationSignatures(ArrayList<Object> connectVerificationSignatures) { + public void setConnectVerificationSignatures(List<Object> connectVerificationSignatures) { this.connectVerificationSignatures = connectVerificationSignatures; } }
\ No newline at end of file diff --git a/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/CreateFirewallController.java b/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/CreateFirewallController.java index 2750ff505..7001aa9a8 100644 --- a/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/CreateFirewallController.java +++ b/POLICY-SDK-APP/src/main/java/org/onap/policy/controller/CreateFirewallController.java @@ -113,7 +113,9 @@ public class CreateFirewallController extends RestrictedBaseController { CreateFirewallController.commonClassDao = commonClassDao; } - public CreateFirewallController(){} + public CreateFirewallController(){ + // Empty constructor + } private List<String> termCollectorList; |