Merge pull request #48567 from geoand/#48215

Properly set `typeOfHandler` field of `RouteBuildItem`

Author: geoand

extensions/vertx-http/deployment-spi/src/main/java/io/quarkus/vertx/http/deployment/spi/RouteBuildItem.java

@@ -81,14 +81,15 @@ public enum RouteType {
 
     private final Handler<RoutingContext> handler;
 
-    private final HandlerType typeOfHandler = HandlerType.NORMAL;
+    private final HandlerType typeOfHandler;
 
     private final boolean displayOnNotFoundPage;
     private final String notFoundPageTitle;
 
     private final String routeConfigKey;
 
     @SuppressWarnings("OptionalUsedAsFieldOrParameterType")
+    @Deprecated(since = "3.25", forRemoval = true)
     public RouteBuildItem(RouteType typeOfRoute, String path, Consumer<Route> customizer,
             boolean isManagement,
             Handler<RoutingContext> handler,
@@ -97,6 +98,7 @@ public RouteBuildItem(RouteType typeOfRoute, String path, Consumer<Route> custom
             String routeConfigKey, OptionalInt order) {
         this.order = order;
         this.typeOfRoute = typeOfRoute;
+        this.typeOfHandler = HandlerType.NORMAL;
         this.path = path;
         this.handler = handler;
         this.displayOnNotFoundPage = displayOnNotFoundPage;
@@ -106,6 +108,20 @@ public RouteBuildItem(RouteType typeOfRoute, String path, Consumer<Route> custom
         this.isManagement = isManagement;
     }
 
+    @SuppressWarnings("OptionalUsedAsFieldOrParameterType")
+    private RouteBuildItem(Builder builder) {
+        this.order = builder.order;
+        this.typeOfRoute = builder.typeOfRoute;
+        this.typeOfHandler = builder.typeOfHandler;
+        this.path = builder.path;
+        this.handler = builder.handler;
+        this.displayOnNotFoundPage = builder.displayOnNotFoundPage;
+        this.notFoundPageTitle = builder.notFoundPageTitle;
+        this.routeConfigKey = builder.routeConfigKey;
+        this.customizer = builder.customizer;
+        this.isManagement = builder.isManagement;
+    }
+
     public RouteType getTypeOfRoute() {
         return typeOfRoute;
     }
@@ -391,9 +407,7 @@ public RouteBuildItem build() {
                 throw new IllegalArgumentException("The route handler must be set");
             }
 
-            return new RouteBuildItem(typeOfRoute, path, customizer, isManagement, handler, displayOnNotFoundPage,
-                    notFoundPageTitle,
-                    routeConfigKey, order);
+            return new RouteBuildItem(this);
         }
     }