Changes Listed below:

[music.git] / src / main / java / org / onap / music / response / jsonobjects / JsonResponse.java

diff --git a/src/main/java/org/onap/music/response/jsonobjects/JsonResponse.java b/src/main/java/org/onap/music/response/jsonobjects/JsonResponse.java
index 207911d..66fec9c 100644 (file)
--- a/src/main/java/org/onap/music/response/jsonobjects/JsonResponse.java
+++ b/src/main/java/org/onap/music/response/jsonobjects/JsonResponse.java
@@ -19,12 +19,13 @@
  * ============LICENSE_END=============================================
  * ====================================================================
  */
+
 package org.onap.music.response.jsonobjects;
 
 import java.util.HashMap;
 import java.util.Map;
 
-import org.onap.music.lockingservice.MusicLockState.LockStatus;
+import org.onap.music.lockingservice.cassandra.MusicLockState.LockStatus;
 import org.onap.music.main.ResultType;
 
 
@@ -34,7 +35,7 @@ import io.swagger.annotations.ApiModelProperty;
 @ApiModel(value = "JsonResponse", description = "General Response JSON")
 public class JsonResponse {
 
-       /* Status is required */
+    /* Status is required */
     private ResultType status;
     
     /* Standard informational fields */
@@ -43,6 +44,7 @@ public class JsonResponse {
     
     /* versioning */
     private String musicVersion;
+    private String musicBuild;
     
     /* Data Fields */
     private Map<String, HashMap<String, Object>> dataResult;
@@ -64,7 +66,7 @@ public class JsonResponse {
         this.status = status;
     }
 
-       /**
+    /**
      * 
      * @return
      */
@@ -121,33 +123,52 @@ public class JsonResponse {
     
     
     /**
-     * 
+     * .
      * @return the music version
      */
     public String getMusicVersion() {
-       return this.musicVersion;
+        return this.musicVersion;
     }
     
     /**
-     * 
+     * .
      * @param version of music
      * @return
      */
     public JsonResponse setMusicVersion(String version) {
-       this.musicVersion = version;
-       return this;
+        this.musicVersion = version;
+        return this;
     }
 
+    /**
+     * .
+     * @return the music version
+     */
+    public String getMusicBuild() {
+        return this.musicBuild;
+    }
+    
+    /**
+     * .
+     * @param build of music
+     * @return
+     */
+    public JsonResponse setMusicBuild(String build) {
+        this.musicBuild = build;
+        return this;
+    }
+
+
     public Map<String, HashMap<String, Object>> getDataResult() {
-       return this.dataResult;
+        return this.dataResult;
     }
     
     public JsonResponse setDataResult(Map<String, HashMap<String, Object>> map) {
-       this.dataResult = map;
-       return this;
+        this.dataResult = map;
+        return this;
     }
 
-       /**
+    /**
      * 
      * @return
      */
@@ -226,36 +247,40 @@ public class JsonResponse {
     public Map<String, Object> toMap() {
         Map<String, Object> fullMap = new HashMap<>();
         fullMap.put("status", status);
-        if (error!=null) {
-            fullMap.put("error", error);
+        if (error != null && !"".equals(error)) { 
+            fullMap.put("error", error); 
         }
-        if (message!=null) {
-            fullMap.put("message", message);
+        if (message != null) { 
+            fullMap.put("message", message); 
         }
         
-        if (musicVersion!=null) {
+        if (musicVersion != null) {
             fullMap.put("version", musicVersion);
         }
         
-        if (dataResult!=null) {
-               fullMap.put("result", dataResult);
+        if (musicBuild != null) {
+            fullMap.put("build", musicBuild);
+        }
+    
+        if (dataResult != null) {
+            fullMap.put("result", dataResult);
         }
         
-        if (lock!=null) {
-               Map<String, Object> lockMap = new HashMap<>();
-               if (lock!=null) {
-                   lockMap.put("lock", lock);
-               }
-               if (lockStatus!=null) {
-                   lockMap.put("lock-status", lockStatus);
-               }
-               if (lockHolder!=null) {
-                   lockMap.put("lock-holder", lockHolder);
-               }
-               if (lockLease!=null) {
-                   lockMap.put("lock-lease", lockLease);
-               }
-               fullMap.put("lock", lockMap);
+        if (lock != null) {
+            Map<String, Object> lockMap = new HashMap<>();
+            if (lock != null) {
+                lockMap.put("lock", lock);
+            }
+            if (lockStatus != null) {
+                lockMap.put("lock-status", lockStatus);
+            }
+            if (lockHolder != null) {
+                lockMap.put("lock-holder", lockHolder);
+            }
+            if (lockLease != null) {
+                lockMap.put("lock-lease", lockLease);
+            }
+            fullMap.put("lock", lockMap);
         }
 
         return fullMap;