From 8c52ab41665a0fa0f3a3e5875c6a9d2ce3ed3c10 Mon Sep 17 00:00:00 2001
From: burdziak <olaf.burdziakowski@nokia.com>
Date: Tue, 22 May 2018 13:54:27 +0200
Subject: [PATCH] Fix sonar issues in Mapper1_0

Change-Id: I15746954d91610cbcd517d56fa25b98535a76e0e
Issue-ID: AAF-313
Signed-off-by: burdziak <olaf.burdziakowski@nokia.com>
---
 .../org/onap/aaf/auth/cm/mapper/Mapper1_0.java     | 88 +++++++++++-----------
 1 file changed, 44 insertions(+), 44 deletions(-)

diff --git a/auth/auth-certman/src/main/java/org/onap/aaf/auth/cm/mapper/Mapper1_0.java b/auth/auth-certman/src/main/java/org/onap/aaf/auth/cm/mapper/Mapper1_0.java
index 3d865d30..16517c95 100644
--- a/auth/auth-certman/src/main/java/org/onap/aaf/auth/cm/mapper/Mapper1_0.java
+++ b/auth/auth-certman/src/main/java/org/onap/aaf/auth/cm/mapper/Mapper1_0.java
@@ -97,50 +97,51 @@ public class Mapper1_0 implements Mapper<BaseRequest,CertInfo,Artifacts,Error> {
 	 */
 	@Override
 	public Result<CertInfo> toCert(AuthzTrans trans, Result<CertResp> in, boolean withTrustChain) throws IOException {
-		if(in.isOK()) {
-			CertResp cin = in.value;
-			CertInfo cout = newInstance(API.CERT);
-			cout.setPrivatekey(cin.privateString());
-			String value;
-			if((value=cin.challenge())!=null) {
-				cout.setChallenge(value);
-			}
-			cout.getCerts().add(cin.asCertString());
-			if(cin.trustChain()!=null) {
-				for(String c : cin.trustChain()) {
-					if(c!=null) {
-						cout.getCerts().add(c);
-					}
+		if(!in.isOK()) {
+			return Result.err(in);
+		}
+
+		CertResp cin = in.value;
+		CertInfo cout = newInstance(API.CERT);
+		cout.setPrivatekey(cin.privateString());
+		String value;
+		if((value=cin.challenge())!=null) {
+			cout.setChallenge(value);
+		}
+		cout.getCerts().add(cin.asCertString());
+		if(cin.trustChain()!=null) {
+			for(String c : cin.trustChain()) {
+				if(c!=null) {
+					cout.getCerts().add(c);
 				}
 			}
-			// Adding all the Certs in one response is a mistake.  Makes it very hard for Agent to setup 
-			// Certs in keystore versus Truststore.  Separate in Version 2_0
-			if(cin.trustCAs()!=null) {
-				for(String c : cin.trustCAs()) {
-					if(c!=null) {
-						cout.getCerts().add(c);
-					} 
+		}
+		// Adding all the Certs in one response is a mistake.  Makes it very hard for Agent to setup
+		// Certs in keystore versus Truststore.  Separate in Version 2_0
+		if(cin.trustCAs()!=null) {
+			for(String c : cin.trustCAs()) {
+				if(c!=null) {
+					cout.getCerts().add(c);
 				}
 			}
-			if(cin.notes()!=null) {
-				boolean first = true;
-				StringBuilder sb = new StringBuilder();
-				for(String n : cin.notes()) {
-					if(first) {
-						first = false;
-					} else {
-						sb.append('\n');
-					}
-					sb.append(n);
+		}
+		if(cin.notes()!=null) {
+			boolean first = true;
+			StringBuilder sb = new StringBuilder();
+			for(String n : cin.notes()) {
+				if(first) {
+					first = false;
+				} else {
+					sb.append('\n');
 				}
-				cout.setNotes(sb.toString());
+				sb.append(n);
 			}
-			cout.getCaIssuerDNs().addAll(cin.caIssuerDNs());
-			cout.setEnv(cin.env());
-			return Result.ok(cout);
-		} else {
-			return Result.err(in);
+			cout.setNotes(sb.toString());
 		}
+		cout.getCaIssuerDNs().addAll(cin.caIssuerDNs());
+		cout.setEnv(cin.env());
+		return Result.ok(cout);
+
 	}
 
 	@Override
@@ -171,9 +172,10 @@ public class Mapper1_0 implements Mapper<BaseRequest,CertInfo,Artifacts,Error> {
 
 		CertReq out = new CertReq();
 		CertmanValidator v = new CertmanValidator();
-		v.isNull("CertRequest", req)
-			.nullOrBlank("MechID", out.mechid=in.getMechid());
-		v.nullBlankMin("FQDNs", out.fqdns=in.getFqdns(),1);
+		out.mechid=in.getMechid();
+		out.fqdns=in.getFqdns();
+		v.isNull("CertRequest", req).nullOrBlank("MechID", out.mechid);
+		v.nullBlankMin("FQDNs", out.fqdns,1);
 		if(v.err()) {
 			return Result.err(Result.ERR_BadData, v.errs());
 		}
@@ -206,7 +208,7 @@ public class Mapper1_0 implements Mapper<BaseRequest,CertInfo,Artifacts,Error> {
 	 */
 	@Override
 	public List<ArtiDAO.Data> toArtifact(AuthzTrans trans, Artifacts artifacts) {
-		List<ArtiDAO.Data> ladd = new ArrayList<ArtiDAO.Data>();
+		List<ArtiDAO.Data> ladd = new ArrayList<>();
 		for(Artifact arti : artifacts.getArtifact()) {
 			ArtiDAO.Data data = new ArtiDAO.Data();
 			data.mechid = arti.getMechid();
@@ -226,10 +228,8 @@ public class Mapper1_0 implements Mapper<BaseRequest,CertInfo,Artifacts,Error> {
 			
 			// Derive Optional Data from Machine (Domain) if exists
 			if(data.machine!=null) {
-				if(data.ca==null) {
-					if(data.machine.endsWith(".att.com")) {
+				if(data.ca==null && data.machine.endsWith(".att.com")) {
 						data.ca = "aaf"; // default
-					}
 				}
 				if(data.ns==null ) {
 					data.ns=FQI.reverseDomain(data.machine);
-- 
2.16.6