X-Git-Url: https://gerrit.onap.org/r/gitweb?p=dmaap%2Fdatarouter.git;a=blobdiff_plain;f=datarouter-prov%2Fsrc%2Fmain%2Fjava%2Forg%2Fonap%2Fdmaap%2Fdatarouter%2Fprovisioning%2Fbeans%2FFeed.java;h=3462e2347bbe57bd11c6292afcfecf2ec38bc488;hp=1cfd0f6bc892341008f9d3a4a0010d037c4d4c8e;hb=8ebd052e0b1066c432f5dba6ef69599eaf5ec27e;hpb=b10c179229cea56e36a2fb9b6f2c17baf0d83f13 diff --git a/datarouter-prov/src/main/java/org/onap/dmaap/datarouter/provisioning/beans/Feed.java b/datarouter-prov/src/main/java/org/onap/dmaap/datarouter/provisioning/beans/Feed.java index 1cfd0f6b..3462e234 100644 --- a/datarouter-prov/src/main/java/org/onap/dmaap/datarouter/provisioning/beans/Feed.java +++ b/datarouter-prov/src/main/java/org/onap/dmaap/datarouter/provisioning/beans/Feed.java @@ -162,7 +162,7 @@ public class Feed extends Syncable { } public static Collection getAllFeeds() { - Map map = new HashMap(); + Map map = new HashMap<>(); try { DB db = new DB(); @SuppressWarnings("resource") @@ -208,7 +208,7 @@ public class Feed extends Syncable { } public static List getFilteredFeedUrlList(final String name, final String val) { - List list = new ArrayList(); + List list = new ArrayList<>(); String sql = "select SELF_LINK from FEEDS where DELETED = 0"; if (name.equals("name")) { sql += " and NAME = ?"; @@ -329,10 +329,8 @@ public class Feed extends Syncable { this.groupid = jo.optInt("groupid"); this.name = jo.getString("name"); this.aaf_instance = jo.optString("aaf_instance", "legacy"); - if(!(aaf_instance.equalsIgnoreCase("legacy"))){ - if (aaf_instance.length() > 255){ + if(!(aaf_instance.equalsIgnoreCase("legacy")) && aaf_instance.length() > 255){ throw new InvalidObjectException("aaf_instance field is too long"); - } } if (name.length() > 255) throw new InvalidObjectException("name field is too long"); @@ -365,7 +363,7 @@ public class Feed extends Syncable { throw new InvalidObjectException("password field is too long ("+ fid.getPassword()+")"); //Fortify scan fixes - Privacy Violation this.authorization.getEndpoint_ids().add(fid); } - if (this.authorization.getEndpoint_ids().size() < 1) + if (this.authorization.getEndpoint_ids().isEmpty()) throw new InvalidObjectException("need to specify at least one endpoint_id"); endPointIds = jauth.getJSONArray("endpoint_addrs"); for (int i = 0; i < endPointIds.length(); i++) {