Don't include "nullVersion" in swagger API 15/85515/2
authorJim Hahn <jrh3@att.com>
Tue, 16 Apr 2019 19:32:12 +0000 (15:32 -0400)
committerJim Hahn <jrh3@att.com>
Tue, 16 Apr 2019 19:42:47 +0000 (15:42 -0400)
Swagger sitll uses jackson, thus it is including
ToscaPolicyIdentifierOptVersion.isNullVersion() in the API that it
generates.  Added @JsonIgnore annotation to prevent it from being
included.  Similar issue with get/setVersion() in PdpGroup.
Created POLICY-1653 to remove the isNullVersion() method altogether in
El Alto.

Change-Id: I6cb7b7d1d75258eeb9d6f9b6c7a48eeb32174307
Issue-ID: POLICY-1542
Signed-off-by: Jim Hahn <jrh3@att.com>
models-pdp/src/main/java/org/onap/policy/models/pdp/concepts/PdpGroup.java
models-tosca/src/main/java/org/onap/policy/models/tosca/authorative/concepts/ToscaPolicyIdentifierOptVersion.java

index 4cb0ac5..8cad58c 100644 (file)
@@ -21,6 +21,7 @@
 
 package org.onap.policy.models.pdp.concepts;
 
+import com.fasterxml.jackson.annotation.JsonIgnore;
 import java.util.HashSet;
 import java.util.LinkedHashMap;
 import java.util.List;
@@ -121,12 +122,14 @@ public class PdpGroup implements PfNameVersion, Comparable<PdpGroup> {
     }
 
     @Override
+    @JsonIgnore
     public String getVersion() {
         // We need to pass a version for keying in the database
         return PfKey.NULL_KEY_VERSION;
     }
 
     @Override
+    @JsonIgnore
     public void setVersion(String version) {
         // Just ignore any version that is set
     }
index adfd616..a6aec08 100644 (file)
@@ -20,6 +20,7 @@
 
 package org.onap.policy.models.tosca.authorative.concepts;
 
+import com.fasterxml.jackson.annotation.JsonIgnore;
 import com.google.gson.annotations.SerializedName;
 import lombok.Data;
 import lombok.NoArgsConstructor;
@@ -61,6 +62,7 @@ public class ToscaPolicyIdentifierOptVersion implements Comparable<ToscaPolicyId
      *
      * @return {@code true} if the version is null/missing, {@code false}
      */
+    @JsonIgnore
     public boolean isNullVersion() {
         return (version == null);
     }