You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by da...@apache.org on 2019/06/19 04:01:11 UTC

[camel] 04/06: CAMEL-13658: Using OutputNode instead of isOutputSupported method to mark if a node/definition supports outputs or not.

This is an automated email from the ASF dual-hosted git repository.

davsclaus pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel.git

commit e682604b9ad076048595eedbe9e2f5d2b503c7e3
Author: Claus Ibsen <cl...@gmail.com>
AuthorDate: Tue Jun 18 21:27:50 2019 +0200

    CAMEL-13658: Using OutputNode instead of isOutputSupported method to mark if a node/definition supports outputs or not.
---
 .../src/main/java/org/apache/camel/model/ExpressionNode.java  | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/core/camel-core/src/main/java/org/apache/camel/model/ExpressionNode.java b/core/camel-core/src/main/java/org/apache/camel/model/ExpressionNode.java
index e5d47f1..8de221c 100644
--- a/core/camel-core/src/main/java/org/apache/camel/model/ExpressionNode.java
+++ b/core/camel-core/src/main/java/org/apache/camel/model/ExpressionNode.java
@@ -127,9 +127,14 @@ public abstract class ExpressionNode extends ProcessorDefinition<ExpressionNode>
 
     @Override
     public ExpressionNode id(String id) {
-        // let parent handle assigning the id, as we do not support outputs
-        getParent().id(id);
-        return this;
+        if (!(this instanceof OutputNode)) {
+            // let parent handle assigning the id, as we do not support outputs
+            getParent().id(id);
+            return this;
+        } else {
+            return super.id(id);
+        }
     }
+
 }