specified roles override controller level

avaje · Dec 8, 2024 · 082f3bd · 082f3bd
1 parent 4b06090
commit 082f3bd
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/http-generator-core/src/main/java/io/avaje/http/generator/core/MethodReader.java b/http-generator-core/src/main/java/io/avaje/http/generator/core/MethodReader.java
@@ -338,7 +338,9 @@ public void buildApiDocumentation() {
 
   public List<String> roles() {
     final var roles = new ArrayList<>(methodRoles);
-    roles.addAll(bean.roles());
+    if (roles.isEmpty()) {
+      roles.addAll(bean.roles());
+    }
     return roles;
   }
 
@@ -521,7 +523,7 @@ private List<PathSegments.Segment> namedSegments() {
 
   private boolean allArgParamNames() {
     for (int i = 0; i < params.size(); i++) {
-      if (!params.get(i).name().equals("arg" + i)) {
+      if (!("arg" + i).equals(params.get(i).name())) {
         return false;
       }
     }