ppalaga commented on a change in pull request #1462:
URL: https://github.com/apache/camel-quarkus/pull/1462#discussion_r463453838
##########
File path:
extensions-core/core/runtime/src/main/java/org/apache/camel/quarkus/core/BaseModel.java
##########
@@ -110,6 +112,91 @@ public synchronized RouteDefinition
getRouteDefinition(String id) {
return null;
}
+ @Override
+ public List<RouteTemplateDefinition> getRouteTemplateDefinitions() {
+ return routeTemplateDefinitions;
+ }
+
+ @Override
+ public RouteTemplateDefinition getRouteTemplateDefinition(String id) {
+ for (RouteTemplateDefinition route : routeTemplateDefinitions) {
+ if
(route.idOrCreate(camelContext.adapt(ExtendedCamelContext.class).getNodeIdFactory()).equals(id))
{
+ return route;
+ }
+ }
+ return null;
+ }
+
+ @Override
+ public void
addRouteTemplateDefinitions(Collection<RouteTemplateDefinition>
routeTemplateDefinitions) throws Exception {
+ if (routeTemplateDefinitions == null ||
routeTemplateDefinitions.isEmpty()) {
+ return;
+ }
+ this.routeTemplateDefinitions.addAll(routeTemplateDefinitions);
+ }
+
+ @Override
+ public void addRouteTemplateDefinition(RouteTemplateDefinition
routeTemplateDefinition) throws Exception {
+ this.routeTemplateDefinitions.addAll(routeTemplateDefinitions);
+ }
+
+ @Override
+ public void
removeRouteTemplateDefinitions(Collection<RouteTemplateDefinition>
routeTemplateDefinitions) throws Exception {
+ routeTemplateDefinitions.removeAll(routeTemplateDefinitions);
Review comment:
Does not look right either. Maybe this?
```suggestion
this.routeTemplateDefinitions.removeAll(routeTemplateDefinitions);
```
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
