Skip to content

Commit

Permalink
Merge pull request #14356 from geoand/#14348
Browse files Browse the repository at this point in the history
Use Dotname instead of Type to determine @BeanParam class target
  • Loading branch information
geoand authored Jan 18, 2021
2 parents 5018a9f + 3fe3c8e commit f60af75
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
package io.quarkus.resteasy.reactive.server.test.simple;

import javax.validation.constraints.NotNull;
import javax.ws.rs.BeanParam;
import javax.ws.rs.GET;
import javax.ws.rs.HeaderParam;
Expand Down Expand Up @@ -45,7 +46,7 @@ private void checkInjections(String path, String query, String header, UriInfo u
public String param(@QueryParam("query") String query,
@HeaderParam("header") String header,
@Context UriInfo uriInfo,
@BeanParam BeanParamSubClass beanParam) {
@BeanParam @NotNull BeanParamSubClass beanParam) {
checkInjections("/injection-subclass/param2", query, header, uriInfo, this.beanParam, query, uriInfo, beanParam);
return "OK";
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -121,17 +121,17 @@ public static ResourceScanningResult scanResources(
// FIXME: this isn't right wrt generics
switch (target.kind()) {
case FIELD:
beanParams.add(target.asField().type().toString());
beanParams.add(target.asField().type().name().toString());
break;
case METHOD:
Type setterParamType = target.asMethod().parameters().get(0);
beanParams.add(setterParamType.toString());
beanParams.add(setterParamType.name().toString());
break;
case METHOD_PARAMETER:
MethodInfo method = target.asMethodParameter().method();
int paramIndex = target.asMethodParameter().position();
Type paramType = method.parameters().get(paramIndex);
beanParams.add(paramType.toString());
beanParams.add(paramType.name().toString());
break;
default:
break;
Expand Down

0 comments on commit f60af75

Please sign in to comment.