Fixes 2001

Author: bnasslahsen

pom.xml

@@ -11,7 +11,7 @@
 	<parent>
 		<groupId>org.springframework.boot</groupId>
 		<artifactId>spring-boot-starter-parent</artifactId>
-		<version>2.7.5</version>
+		<version>2.7.6</version>
 	</parent>
 
 	<licenses>

springdoc-openapi-common/src/main/java/org/springdoc/api/AbstractOpenApiResource.java

@@ -66,7 +66,6 @@
 import io.swagger.v3.oas.models.Paths;
 import io.swagger.v3.oas.models.media.StringSchema;
 import io.swagger.v3.oas.models.parameters.Parameter;
-import io.swagger.v3.oas.models.parameters.PathParameter;
 import io.swagger.v3.oas.models.responses.ApiResponses;
 import io.swagger.v3.oas.models.servers.Server;
 import org.apache.commons.lang3.ArrayUtils;

springdoc-openapi-common/src/main/java/org/springdoc/core/GenericResponseService.java

@@ -55,7 +55,6 @@
 import org.apache.commons.lang3.StringUtils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
-import org.springdoc.core.converters.AdditionalModelsConverter;
 import org.springdoc.core.providers.JavadocProvider;
 import org.springdoc.core.providers.ObjectMapperProvider;
 

springdoc-openapi-common/src/main/java/org/springdoc/core/MultipleOpenApiGroupsCondition.java

@@ -22,8 +22,6 @@
 package org.springdoc.core;
 
 
-import java.util.Collection;
-
 import org.springframework.boot.autoconfigure.condition.AnyNestedCondition;
 import org.springframework.boot.autoconfigure.condition.ConditionalOnBean;
 import org.springframework.boot.autoconfigure.condition.ConditionalOnProperty;

springdoc-openapi-common/src/main/java/org/springdoc/core/SpringDocUtils.java

@@ -23,7 +23,6 @@
 package org.springdoc.core;
 
 import java.lang.annotation.Annotation;
-import java.math.BigDecimal;
 import java.util.function.Predicate;
 
 import io.swagger.v3.oas.models.media.Schema;

springdoc-openapi-data-rest/src/main/java/org/springdoc/data/rest/core/DataRestOperationService.java

@@ -24,6 +24,7 @@
 
 import java.lang.reflect.Field;
 import java.lang.reflect.Method;
+import java.lang.reflect.Type;
 import java.util.Arrays;
 
 import io.swagger.v3.oas.annotations.enums.ParameterIn;
@@ -39,6 +40,7 @@
 import org.springdoc.core.SpringDocAnnotationsUtils;
 
 import org.springframework.core.MethodParameter;
+import org.springframework.core.ResolvableType;
 import org.springframework.core.annotation.AnnotatedElementUtils;
 import org.springframework.data.repository.query.Param;
 import org.springframework.data.rest.core.mapping.MethodResourceMapping;
@@ -54,6 +56,7 @@
 
 /**
  * The type Data rest operation builder.
+ *
  * @author bnasslahsen
  */
 public class DataRestOperationService {
@@ -202,17 +205,8 @@ private Operation buildSearchOperation(HandlerMethod handlerMethod, DataRestRepo
 			String pName = parameterMetadatum.getName();
 			ResourceDescription description = parameterMetadatum.getDescription();
 			if (description instanceof TypedResourceDescription) {
-				TypedResourceDescription typedResourceDescription = (TypedResourceDescription) description;
-				Field fieldType = FieldUtils.getField(TypedResourceDescription.class, "type", true);
-				Class<?> type;
-				try {
-					type = (Class<?>) fieldType.get(typedResourceDescription);
-				}
-				catch (IllegalAccessException e) {
-					LOGGER.warn(e.getMessage());
-					type = String.class;
-				}
-				Schema<?> schema = SpringDocAnnotationsUtils.resolveSchemaFromType(type, openAPI.getComponents(), null, null);
+				Type type = getParameterType(pName,method,description);
+				Schema<?> schema = SpringDocAnnotationsUtils.extractSchema(openAPI.getComponents(), type, null, null);
 				Parameter parameter = getParameterFromAnnotations(openAPI, methodAttributes, method, pName);
 				if (parameter == null)
 					parameter = new Parameter().name(pName).in(ParameterIn.QUERY.toString()).schema(schema);
@@ -231,6 +225,39 @@ private Operation buildSearchOperation(HandlerMethod handlerMethod, DataRestRepo
 		return operation;
 	}
 
+	/**
+	 * Gets parameter type.
+	 *
+	 * @param pName the p name
+	 * @param method the method
+	 * @param description the description
+	 * @return the parameter type
+	 */
+	private Type getParameterType(String pName, Method method, ResourceDescription description) {
+		Type type = null;
+		java.lang.reflect.Parameter[] parameters = method.getParameters();
+		for (int i = 0; i < parameters.length; i++) {
+			java.lang.reflect.Parameter parameter = parameters[i];
+			if (pName.equals(parameter.getName()) || pName.equals(parameter.getAnnotation(Param.class).value())) {
+				ResolvableType resolvableType = ResolvableType.forMethodParameter(method, i);
+				type = resolvableType.getType();
+				break;
+			}
+		}
+		if (type == null) {
+			TypedResourceDescription typedResourceDescription = (TypedResourceDescription) description;
+			Field fieldType = FieldUtils.getField(TypedResourceDescription.class, "type", true);
+			try {
+				type = (Type) fieldType.get(typedResourceDescription);
+			}
+			catch (IllegalAccessException e) {
+				LOGGER.warn(e.getMessage());
+				type = String.class;
+			}
+		}
+		return type;
+	}
+
 	/**
 	 * Update parameter from annotations parameter.
 	 *
@@ -279,6 +306,7 @@ private Operation initOperation(HandlerMethod handlerMethod, Class<?> domainType
 
 	/**
 	 * Add operation description.
+	 *
 	 * @param operation the operation
 	 * @param requestMethod the request method
 	 * @param entity the entity

springdoc-openapi-webmvc-core/src/test/java/test/org/springdoc/api/v30/app114/SpringDocApp114Test.java

@@ -27,7 +27,6 @@
 import javax.money.MonetaryAmount;
 
 import org.springdoc.core.SpringDocUtils;
-import org.springdoc.core.converters.AdditionalModelsConverter;
 import test.org.springdoc.api.v30.AbstractSpringDocV30Test;
 
 import org.springframework.boot.autoconfigure.SpringBootApplication;

springdoc-openapi-webmvc-core/src/test/java/test/org/springdoc/api/v30/app199/HelloController.java

@@ -23,19 +23,20 @@
 package test.org.springdoc.api.v30.app199;
 
 
+import java.util.LinkedHashMap;
+import java.util.Map;
+
 import com.fasterxml.jackson.databind.ObjectMapper;
 import io.swagger.v3.oas.models.Operation;
 import io.swagger.v3.oas.models.examples.Example;
 import org.springdoc.core.customizers.OperationCustomizer;
+
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.context.annotation.Bean;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.RestController;
 import org.springframework.web.method.HandlerMethod;
 
-import java.util.LinkedHashMap;
-import java.util.Map;
-
 import static org.springframework.http.MediaType.ALL_VALUE;
 import static org.springframework.http.MediaType.APPLICATION_JSON_VALUE;
 

springdoc-openapi-webmvc-core/src/test/java/test/org/springdoc/api/v30/app199/SpringDocApp199Test.java

@@ -22,9 +22,10 @@
 
 package test.org.springdoc.api.v30.app199;
 
-import org.springframework.boot.autoconfigure.SpringBootApplication;
 import test.org.springdoc.api.v30.AbstractSpringDocV30Test;
 
+import org.springframework.boot.autoconfigure.SpringBootApplication;
+
 public class SpringDocApp199Test extends AbstractSpringDocV30Test {
 
 	@SpringBootApplication