Merge branch 'master' of https://github.com/chrisupb/springdoc-openapi

Author: bnasslahsen

springdoc-openapi-data-rest/src/main/java/org/springdoc/data/rest/customisers/QuerydslPredicateOperationCustomizer.java

@@ -116,10 +116,13 @@ public Operation customize(Operation operation, HandlerMethod handlerMethod) {
 			fieldsToAdd.addAll(aliases);
 			fieldsToAdd.addAll(whiteList);
 
-			boolean excludeUnlistedProperties = getFieldValueOfBoolean(bindings, "excludeUnlistedProperties");
+			// if only listed properties should be included, remove all other fields from fieldsToAdd
+			if (getFieldValueOfBoolean(bindings, "excludeUnlistedProperties")) {
+				fieldsToAdd.removeIf(s -> !whiteList.contains(s));
+			}
 
 			for (String fieldName : fieldsToAdd) {
-				Type type = getFieldType(fieldName, pathSpecMap, predicate.root(), excludeUnlistedProperties);
+				Type type = getFieldType(fieldName, pathSpecMap, predicate.root());
 				if (type != null) {
 					Parameter newParameter = buildParam(type, fieldName);
 					parametersToAddToOperation.add(newParameter);
@@ -232,19 +235,18 @@ private Optional<Path<?>> getPathFromPathSpec(Object instance) {
 	 * Tries to figure out the Type of the field. It first checks the Qdsl pathSpecMap before checking the root class. Defaults to String.class
 	 * @param fieldName The name of the field used as reference to get the type  
 	 * @param pathSpecMap The Qdsl path specifications as defined in the resolved bindings  
-	 * @param root The root type where the paths are gotten  
-	 * @param excludeUnlistedProperties the exclude unlisted properties  
+	 * @param root The root type where the paths are gotten
 	 * @return The type of the field. Returns
 	 */
-	private Type getFieldType(String fieldName, Map<String, Object> pathSpecMap, Class<?> root, boolean excludeUnlistedProperties) {
+	private Type getFieldType(String fieldName, Map<String, Object> pathSpecMap, Class<?> root) {
 		Type genericType = null;
 		try {
 			Object pathAndBinding = pathSpecMap.get(fieldName);
 			Optional<Path<?>> path = getPathFromPathSpec(pathAndBinding);
 			Field declaredField;
 			if (path.isPresent()) {
 				genericType = path.get().getType();
-			} else if (!excludeUnlistedProperties) {
+			} else {
 				declaredField = root.getDeclaredField(fieldName);
 				genericType = declaredField.getGenericType();
 			}

springdoc-openapi-data-rest/src/test/java/test/org/springdoc/api/app19/ApplicationPredicate.java

@@ -11,6 +11,8 @@ public class ApplicationPredicate implements QuerydslBinderCustomizer<QApplicati
 	public void customize(QuerydslBindings bindings, QApplication root) {
 		bindings.excludeUnlistedProperties(true);
 		bindings.bind(root.name).first(StringExpression::containsIgnoreCase);
+		bindings.including(root.icon);
+		bindings.including(root.name);
 	}
 
-}
+}

springdoc-openapi-data-rest/src/test/resources/results/app19.json

@@ -32,6 +32,13 @@
             "schema": {
               "type": "string"
             }
+          },
+          {
+            "name": "icon",
+            "in": "query",
+            "schema": {
+              "type": "string"
+            }
           }
         ],
         "responses": {