Skip to content

Commit 960dbf2

Browse files
committed
Revert "2703 remove null value parameters from workflow definition"
This reverts commit 415f13b.
1 parent e684bc0 commit 960dbf2

File tree

1 file changed

+5
-7
lines changed

1 file changed

+5
-7
lines changed

server/libs/platform/platform-configuration/platform-configuration-service/src/main/java/com/bytechef/platform/configuration/facade/WorkflowNodeParameterFacadeImpl.java

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ public ParameterResultDTO deleteClusterElementParameter(
123123

124124
String[] parameterPathParts = parameterPath.split("\\.");
125125

126-
setParameter(parameterPathParts, null, workflowNodeStructure.parameterMap);
126+
setParameter(parameterPathParts, null, true, workflowNodeStructure.parameterMap);
127127

128128
// For now only check the first, root level of properties on which other properties could depend on
129129

@@ -175,7 +175,7 @@ public ParameterResultDTO deleteWorkflowNodeParameter(
175175

176176
String[] parameterPathParts = parameterPath.split("\\.");
177177

178-
setParameter(parameterPathParts, null, workflowNodeStructure.parameterMap);
178+
setParameter(parameterPathParts, null, true, workflowNodeStructure.parameterMap);
179179

180180
// For now only check the first, root level of properties on which other properties could depend on
181181

@@ -283,7 +283,7 @@ public ParameterResultDTO updateClusterElementParameter(
283283

284284
String[] parameterPathParts = parameterPath.split("\\.");
285285

286-
setParameter(parameterPathParts, value, workflowNodeStructure.parameterMap);
286+
setParameter(parameterPathParts, value, false, workflowNodeStructure.parameterMap);
287287

288288
// For now only check the first, root level of properties on which other properties could depend on
289289

@@ -335,7 +335,7 @@ public ParameterResultDTO updateWorkflowNodeParameter(
335335

336336
String[] parameterPathParts = parameterPath.split("\\.");
337337

338-
setParameter(parameterPathParts, value, workflowNodeStructure.parameterMap);
338+
setParameter(parameterPathParts, value, false, workflowNodeStructure.parameterMap);
339339

340340
// For now only check the first, root level of properties on which other properties could depend on
341341

@@ -1219,9 +1219,7 @@ private static String replaceIndexes(String expression, List<Integer> indexes) {
12191219

12201220
@SuppressWarnings("unchecked")
12211221
private void setParameter(
1222-
String[] parameterPathParts, Object value, Map<String, ?> parameterMap) {
1223-
1224-
boolean removeValue = value == null;
1222+
String[] parameterPathParts, Object value, boolean removeValue, Map<String, ?> parameterMap) {
12251223

12261224
Map<String, Object> map = (Map<String, Object>) parameterMap;
12271225

0 commit comments

Comments
 (0)