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

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


The following commit(s) were added to refs/heads/yaml-optional by this push:
     new c9b8a16  CAMEL-16604: route template parameter should be able to be 
marked explicit as required.
c9b8a16 is described below

commit c9b8a16289f20f4d5bac1b336e7c9f2da8af5553
Author: Claus Ibsen <claus.ib...@gmail.com>
AuthorDate: Wed May 12 07:56:24 2021 +0200

    CAMEL-16604: route template parameter should be able to be marked explicit 
as required.
---
 .../apache/camel/catalog/models/routeTemplate.json |  4 +--
 .../org/apache/camel/model/routeTemplate.json      |  4 +--
 .../camel/model/RouteTemplateDefinition.java       | 34 +++++++++-------------
 3 files changed, 17 insertions(+), 25 deletions(-)

diff --git 
a/catalog/camel-catalog/src/generated/resources/org/apache/camel/catalog/models/routeTemplate.json
 
b/catalog/camel-catalog/src/generated/resources/org/apache/camel/catalog/models/routeTemplate.json
index 25c4d73..9d681a0 100644
--- 
a/catalog/camel-catalog/src/generated/resources/org/apache/camel/catalog/models/routeTemplate.json
+++ 
b/catalog/camel-catalog/src/generated/resources/org/apache/camel/catalog/models/routeTemplate.json
@@ -11,8 +11,8 @@
     "output": true
   },
   "properties": {
-    "templateParameter": { "kind": "element", "displayName": "Template 
Parameter", "required": false, "type": "array", "javaType": 
"java.util.List<org.apache.camel.model.RouteTemplateParameterDefinition>", 
"deprecated": false, "autowired": false, "secret": false, "description": "Adds 
a required parameter the route template uses." },
-    "templateBean": { "kind": "element", "displayName": "Template Bean", 
"required": false, "type": "array", "javaType": 
"java.util.List<org.apache.camel.model.RouteTemplateBeanDefinition>", 
"deprecated": false, "autowired": false, "secret": false, "description": "Adds 
a local bean the route template uses." },
+    "templateParameter": { "kind": "element", "displayName": "Template 
Parameter", "required": false, "type": "array", "javaType": 
"java.util.List<org.apache.camel.model.RouteTemplateParameterDefinition>", 
"deprecated": false, "autowired": false, "secret": false, "description": "Adds 
a template parameter the route template uses" },
+    "templateBean": { "kind": "element", "displayName": "Template Bean", 
"required": false, "type": "array", "javaType": 
"java.util.List<org.apache.camel.model.RouteTemplateBeanDefinition>", 
"deprecated": false, "autowired": false, "secret": false, "description": "Adds 
a local bean the route template uses" },
     "route": { "kind": "element", "displayName": "Route", "required": true, 
"type": "object", "javaType": "org.apache.camel.model.RouteDefinition", 
"deprecated": false, "autowired": false, "secret": false, "description": "To 
define the route in the template" },
     "id": { "kind": "attribute", "displayName": "Id", "required": false, 
"type": "string", "javaType": "java.lang.String", "deprecated": false, 
"autowired": false, "secret": false, "description": "Sets the id of this node" 
},
     "description": { "kind": "element", "displayName": "Description", 
"required": false, "type": "object", "javaType": 
"org.apache.camel.model.DescriptionDefinition", "deprecated": false, 
"autowired": false, "secret": false, "description": "Sets the description of 
this node" }
diff --git 
a/core/camel-core-model/src/generated/resources/org/apache/camel/model/routeTemplate.json
 
b/core/camel-core-model/src/generated/resources/org/apache/camel/model/routeTemplate.json
index 25c4d73..9d681a0 100644
--- 
a/core/camel-core-model/src/generated/resources/org/apache/camel/model/routeTemplate.json
+++ 
b/core/camel-core-model/src/generated/resources/org/apache/camel/model/routeTemplate.json
@@ -11,8 +11,8 @@
     "output": true
   },
   "properties": {
-    "templateParameter": { "kind": "element", "displayName": "Template 
Parameter", "required": false, "type": "array", "javaType": 
"java.util.List<org.apache.camel.model.RouteTemplateParameterDefinition>", 
"deprecated": false, "autowired": false, "secret": false, "description": "Adds 
a required parameter the route template uses." },
-    "templateBean": { "kind": "element", "displayName": "Template Bean", 
"required": false, "type": "array", "javaType": 
"java.util.List<org.apache.camel.model.RouteTemplateBeanDefinition>", 
"deprecated": false, "autowired": false, "secret": false, "description": "Adds 
a local bean the route template uses." },
+    "templateParameter": { "kind": "element", "displayName": "Template 
Parameter", "required": false, "type": "array", "javaType": 
"java.util.List<org.apache.camel.model.RouteTemplateParameterDefinition>", 
"deprecated": false, "autowired": false, "secret": false, "description": "Adds 
a template parameter the route template uses" },
+    "templateBean": { "kind": "element", "displayName": "Template Bean", 
"required": false, "type": "array", "javaType": 
"java.util.List<org.apache.camel.model.RouteTemplateBeanDefinition>", 
"deprecated": false, "autowired": false, "secret": false, "description": "Adds 
a local bean the route template uses" },
     "route": { "kind": "element", "displayName": "Route", "required": true, 
"type": "object", "javaType": "org.apache.camel.model.RouteDefinition", 
"deprecated": false, "autowired": false, "secret": false, "description": "To 
define the route in the template" },
     "id": { "kind": "attribute", "displayName": "Id", "required": false, 
"type": "string", "javaType": "java.lang.String", "deprecated": false, 
"autowired": false, "secret": false, "description": "Sets the id of this node" 
},
     "description": { "kind": "element", "displayName": "Description", 
"required": false, "type": "object", "javaType": 
"org.apache.camel.model.DescriptionDefinition", "deprecated": false, 
"autowired": false, "secret": false, "description": "Sets the description of 
this node" }
diff --git 
a/core/camel-core-model/src/main/java/org/apache/camel/model/RouteTemplateDefinition.java
 
b/core/camel-core-model/src/main/java/org/apache/camel/model/RouteTemplateDefinition.java
index 81a5bc7..987b635 100644
--- 
a/core/camel-core-model/src/main/java/org/apache/camel/model/RouteTemplateDefinition.java
+++ 
b/core/camel-core-model/src/main/java/org/apache/camel/model/RouteTemplateDefinition.java
@@ -45,9 +45,10 @@ import org.apache.camel.spi.Metadata;
 public class RouteTemplateDefinition extends OptionalIdentifiedDefinition {
 
     @XmlElement(name = "templateParameter")
+    @Metadata(description = "Adds a template parameter the route template 
uses")
     private List<RouteTemplateParameterDefinition> templateParameters;
     @XmlElement(name = "templateBean")
-    @Metadata(description = "Adds a local bean the route template uses.")
+    @Metadata(description = "Adds a local bean the route template uses")
     private List<RouteTemplateBeanDefinition> templateBeans;
     @XmlElement(name = "route", required = true)
     private RouteDefinition route = new RouteDefinition();
@@ -135,7 +136,7 @@ public class RouteTemplateDefinition extends 
OptionalIdentifiedDefinition {
     }
 
     /**
-     * Adds a required parameter the route template uses.
+     * Adds a required parameter the route template uses
      *
      * @param name the name of the parameter
      */
@@ -145,7 +146,7 @@ public class RouteTemplateDefinition extends 
OptionalIdentifiedDefinition {
     }
 
     /**
-     * Adds an optional parameter the route template uses.
+     * Adds an optional parameter the route template uses
      *
      * @param name the name of the parameter
      */
@@ -155,7 +156,7 @@ public class RouteTemplateDefinition extends 
OptionalIdentifiedDefinition {
     }
 
     /**
-     * Adds an optional parameter the route template uses.
+     * Adds an optional parameter the route template uses
      *
      * @param name        the name of the parameter
      * @param description the description of the parameter
@@ -166,7 +167,7 @@ public class RouteTemplateDefinition extends 
OptionalIdentifiedDefinition {
     }
 
     /**
-     * Adds a parameter (will use default value if not provided) the route 
template uses.
+     * Adds a parameter (will use default value if not provided) the route 
template uses
      *
      * @param name         the name of the parameter
      * @param defaultValue default value of the parameter
@@ -177,7 +178,7 @@ public class RouteTemplateDefinition extends 
OptionalIdentifiedDefinition {
     }
 
     /**
-     * Adds a parameter (will use default value if not provided) the route 
template uses.
+     * Adds a parameter (will use default value if not provided) the route 
template uses
      *
      * @param name         the name of the parameter
      * @param defaultValue default value of the parameter
@@ -202,7 +203,7 @@ public class RouteTemplateDefinition extends 
OptionalIdentifiedDefinition {
     }
 
     /**
-     * Adds a local bean the route template uses.
+     * Adds a local bean the route template uses
      *
      * @param name the name of the bean
      * @param type the type of the bean to associate the binding
@@ -219,7 +220,7 @@ public class RouteTemplateDefinition extends 
OptionalIdentifiedDefinition {
     }
 
     /**
-     * Adds a local bean the route template uses.
+     * Adds a local bean the route template uses
      *
      * @param name the name of the bean
      * @param bean the bean or a supplier for the bean
@@ -243,7 +244,7 @@ public class RouteTemplateDefinition extends 
OptionalIdentifiedDefinition {
     }
 
     /**
-     * Adds a local bean the route template uses.
+     * Adds a local bean the route template uses
      *
      * @param name the name of the bean
      * @param bean the supplier for the bean
@@ -260,7 +261,7 @@ public class RouteTemplateDefinition extends 
OptionalIdentifiedDefinition {
     }
 
     /**
-     * Adds a local bean the route template uses.
+     * Adds a local bean the route template uses
      *
      * @param name the name of the bean
      * @param type the type of the bean to associate the binding
@@ -279,7 +280,7 @@ public class RouteTemplateDefinition extends 
OptionalIdentifiedDefinition {
     }
 
     /**
-     * Adds a local bean the route template uses.
+     * Adds a local bean the route template uses
      *
      * @param name     the name of the bean
      * @param language the language to use
@@ -298,7 +299,7 @@ public class RouteTemplateDefinition extends 
OptionalIdentifiedDefinition {
     }
 
     /**
-     * Adds a local bean the route template uses (via fluent builder).
+     * Adds a local bean the route template uses (via fluent builder)
      *
      * @param  name the name of the bean
      * @return      fluent builder to choose which language and script to use 
for creating the bean
@@ -334,16 +335,10 @@ public class RouteTemplateDefinition extends 
OptionalIdentifiedDefinition {
         return "RouteTemplate[" + route.getInput().getLabel() + "]";
     }
 
-    /**
-     * Adds a parameter the route template uses.
-     */
     private void addTemplateParameter(String name, String defaultValue) {
         addTemplateParameter(name, defaultValue, null);
     }
 
-    /**
-     * Adds a parameter the route template uses.
-     */
     private void addTemplateParameter(String name, String defaultValue, String 
description) {
         if (this.templateParameters == null) {
             this.templateParameters = new ArrayList<>();
@@ -351,9 +346,6 @@ public class RouteTemplateDefinition extends 
OptionalIdentifiedDefinition {
         this.templateParameters.add(new RouteTemplateParameterDefinition(name, 
defaultValue, description));
     }
 
-    /**
-     * Adds an optional parameter the route template uses.
-     */
     private void addTemplateOptionalParameter(String name, String description) 
{
         if (this.templateParameters == null) {
             this.templateParameters = new ArrayList<>();

Reply via email to