From: Ram Koya <rk541m@att.com>
Date: Thu, 13 Sep 2018 15:04:03 +0000 (+0000)
Subject: Merge "DcaeLocation : Fixed sonar issue"
X-Git-Url: https://gerrit.onap.org/r/gitweb?a=commitdiff_plain;h=83e39720bc969dd0f708d81213bde40d186b79bd;hp=81bbd0918799a33426737073dc956a5ba669582d;p=dmaap%2Fdbcapi.git

Merge "DcaeLocation : Fixed sonar issue"
---

diff --git a/src/main/java/org/onap/dmaap/dbcapi/resources/TopicResource.java b/src/main/java/org/onap/dmaap/dbcapi/resources/TopicResource.java
index 95f9d33..cab48ca 100644
--- a/src/main/java/org/onap/dmaap/dbcapi/resources/TopicResource.java
+++ b/src/main/java/org/onap/dmaap/dbcapi/resources/TopicResource.java
@@ -96,7 +96,7 @@ public class TopicResource extends BaseLoggingClass {
 	public Response  addTopic( 
 			Topic topic
 			) {
-		logger.info( "addTopic request: " + String.valueOf(topic) );
+		logger.info( "addTopic request: " + topic );
 		ApiService check = new ApiService();
 
 		try {
@@ -104,10 +104,10 @@ public class TopicResource extends BaseLoggingClass {
 			check.required( "topicDescription", topic.getTopicDescription(), "" );
 			check.required( "owner", topic.getOwner(), "" );
 		} catch( RequiredFieldException rfe ) {
+			logger.error("Error", rfe);
 			return check.error();
 		}
 		
-		//String repReq = topic.getReplicationRequest();
 		ReplicationType t = topic.getReplicationCase();
 		if ( t == null || t == ReplicationType.REPLICATION_NOT_SPECIFIED ) {
 			topic.setReplicationCase( mr_topicService.reviewTopic(topic));
@@ -163,6 +163,7 @@ public class TopicResource extends BaseLoggingClass {
 		try {
 			check.required( "fqtn", id, "" );
 		} catch( RequiredFieldException rfe ) {
+			logger.error("Error", rfe);
 			return check.error();
 		}
 		
@@ -192,6 +193,7 @@ public class TopicResource extends BaseLoggingClass {
 		try {
 			check.required( "topicName", id, "^\\S+$" );  //no white space allowed in topicName
 		} catch( RequiredFieldException rfe ) {
+			logger.error("Error", rfe);
 			return check.error();
 		}
 		Topic mrc =  mr_topicService.getTopic( id, check.getErr() );