JiriOndrusek commented on a change in pull request #1462:
URL: https://github.com/apache/camel-quarkus/pull/1462#discussion_r463454909



##########
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);

Review comment:
       I agree that current state doesn't make sense. I'll fix it.




----------------------------------------------------------------
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:
[email protected]


Reply via email to