Fix issue in DSS error message.

When current params is disabled, its getting removed from WSDL also. So, no need to add null element when its disabled.
wso2 · Aug 29, 2023 · 46f1eda · 46f1eda
1 parent fc668f4
commit 46f1eda
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/...ices.core/src/main/java/org/wso2/micro/integrator/dataservices/core/DataServiceFault.java b/...ices.core/src/main/java/org/wso2/micro/integrator/dataservices/core/DataServiceFault.java
@@ -181,11 +181,15 @@ public String getFullMessage() {
         if (this.getDsFaultMessage() != null) {
             buff.append("DS Fault Message: " + this.getDsFaultMessage() + "\n");
         }
-        if (this.getCurrentParams() != null && !("true".equalsIgnoreCase(DBUtils.getCurrentParamsDisabledProperty()))) {
-            buff.append("Current Params: " + this.getCurrentParams() + "\n");
-            getPropertyMap().put(DBConstants.FaultParams.CURRENT_PARAMS, this.getCurrentParams().toString());
-        } else {
-            getPropertyMap().put(DBConstants.FaultParams.CURRENT_PARAMS, null);
+        // if skipCurrentParams is not set to true, we don't have current params in WSDL
+        // so no need to add a null value
+        if (!("true".equalsIgnoreCase(DBUtils.getCurrentParamsDisabledProperty()))) {
+            if (this.getCurrentParams() != null) {
+                buff.append("Current Params: " + this.getCurrentParams() + "\n");
+                getPropertyMap().put(DBConstants.FaultParams.CURRENT_PARAMS, this.getCurrentParams().toString());
+            } else {
+                getPropertyMap().put(DBConstants.FaultParams.CURRENT_PARAMS, null);
+            }
         }
         if (this.getCurrentRequestName() != null) {
             buff.append("Current Request Name: " + this.getCurrentRequestName() + "\n");