fix: application.properties: Unrecognized property 'quarkus.log.level',

it is not referenced in any Java files microprofile (unknown)

See redhat-developer/intellij-quarkus#1480

Signed-off-by: azerr <[email protected]>

Author: angelozerr

quarkus.jdt.ext/com.redhat.microprofile.jdt.quarkus.test/projects/maven/config-mapping/src/main/java/org/acme/classes/ServerClass.java

@@ -0,0 +1,12 @@
+package org.acme.classes;
+
+import java.util.Map;
+
+import io.smallrye.config.ConfigMapping;
+import java.util.logging.Level;
+
+@ConfigMapping(prefix = "server.classes")
+public interface ServerClass {
+
+	Level level();
+}

quarkus.jdt.ext/com.redhat.microprofile.jdt.quarkus.test/projects/maven/config-mapping/src/main/java/org/acme/map/Server.java

@@ -3,13 +3,22 @@
 import java.util.Map;
 
 import io.smallrye.config.ConfigMapping;
+import io.smallrye.config.WithName;
+import java.util.logging.Level;
 
 @ConfigMapping(prefix = "server.map")
 public interface Server {
 
-	String host();
+    String host();
 
-	int port();
+    int port();
 
-	Map<String, String> form();
+    Map<String, String> form();
+
+    @WithName("category")
+    Map<String, CategoryConfig> categories();
+
+    interface CategoryConfig {
+        Level level();
+    }
 }

quarkus.jdt.ext/com.redhat.microprofile.jdt.quarkus.test/src/main/java/com/redhat/microprofile/jdt/quarkus/configmapping/QuarkusConfigMappingTest.java

@@ -66,6 +66,8 @@ public void configMapping() throws Exception {
 				p(null, "server.map.port", "int", null, false, "org.acme.map.Server", null, "port()I", 0, null),
 				p(null, "server.map.form.{*}", "java.util.Map", null, false, "org.acme.map.Server", null,
 						"form()QMap<QString;QString;>;", 0, null),
+				 p(null, "server.map.category.{*}.level", "java.util.logging.Level", null, false, "org.acme.map.Server$CategoryConfig", null,
+	                        "level()QLevel;", 0, null),
 
 				// 3) Naming strategy
 
@@ -150,7 +152,21 @@ public void configMapping() throws Exception {
 				p(null, "server.withparentname.port", "int", null, false, "org.acme.withparentname.ServerHostAndPort",
 						null, "port()I", 0, null),
 				p(null, "server.withparentname.name", "java.lang.String", null, false,
-						"org.acme.withparentname.ServerInfo", null, "name()QString;", 0, null)
+						"org.acme.withparentname.ServerInfo", null, "name()QString;", 0, null),
+				
+                // 9) Class
+                // import java.util.logging.Level;
+                //
+                // @ConfigMapping(prefix = "server.classes")
+                // public interface Server {
+                //
+                //	Level level;
+                //}
+                //}
+
+                p(null, "server.classes.level", "java.util.logging.Level", null, false, "org.acme.classes.ServerClass", null,
+                        "level()QLevel;", 0, null)
+
 
 		);
 

quarkus.jdt.ext/com.redhat.microprofile.jdt.quarkus/src/main/java/com/redhat/microprofile/jdt/internal/quarkus/core/properties/QuarkusConfigMappingProvider.java

@@ -31,10 +31,10 @@
 import static org.eclipse.lsp4mp.jdt.core.utils.JDTTypeUtils.getSourceMethod;
 import static org.eclipse.lsp4mp.jdt.core.utils.JDTTypeUtils.getSourceType;
 import static org.eclipse.lsp4mp.jdt.core.utils.JDTTypeUtils.isOptional;
-import static org.eclipse.lsp4mp.jdt.core.utils.JDTTypeUtils.isPrimitiveType;
 
 import java.util.ArrayList;
 import java.util.Arrays;
+import java.util.Collections;
 import java.util.HashSet;
 import java.util.List;
 import java.util.Set;
@@ -111,17 +111,27 @@ private void processConfigMapping(IJavaElement javaElement, IAnnotation configMa
 			return;
 		}
 		// @ConfigMapping(prefix="server") case
-		List<IType> allInterfaces = new ArrayList<>(Arrays.asList(findInterfaces(configMappingType, monitor)));
-		allInterfaces.add(0, configMappingType);
+		// @ConfigMapping(prefix="server") case
+        Set<IType> allInterfaces = findInterfaces(configMappingType, monitor);
 		for (IType configMappingInterface : allInterfaces) {
 			populateConfigObject(configMappingInterface, prefix, extensionName, new HashSet<>(),
 					configMappingAnnotation, collector, monitor);
 		}
 	}
 
-	private static IType[] findInterfaces(IType type, IProgressMonitor progressMonitor) throws JavaModelException {
+	private static Set<IType> findInterfaces(IType type, IProgressMonitor progressMonitor) throws JavaModelException {
+		// No reason to use a JDK interface to generate a config class? Primarily to fix the java.nio.file.Path case.
+        // see https://github.com/smallrye/smallrye-config/blob/22635f24dc7634706867cc52e28d5bd82d15f54e/implementation/src/main/java/io/smallrye/config/ConfigMappingInterface.java#L782C9-L783C58
+        if (type.getFullyQualifiedName() == null || type.getFullyQualifiedName().startsWith("java")) {
+            return Collections.emptySet();
+        }
+        Set<IType> result = new HashSet<>();
+        result.add(type);
+        
 		ITypeHierarchy typeHierarchy = type.newSupertypeHierarchy(progressMonitor);
-		return typeHierarchy.getAllSuperInterfaces(type);
+		IType[] allSuperInterfaces = typeHierarchy.getAllSuperInterfaces(type);
+		result.addAll(Arrays.asList(allSuperInterfaces));
+		return result;
 	}
 
 	private void populateConfigObject(IType configMappingType, String prefixStr, String extensionName,
@@ -156,22 +166,7 @@ private void populateConfigObject(IType configMappingType, String prefixStr, Str
 				}
 
 				IType returnType = findType(method.getJavaProject(), resolvedTypeSignature);
-				boolean simpleType = isSimpleType(resolvedTypeSignature, returnType);
-				if (!simpleType) {
-					if (returnType != null && !returnType.isInterface()) {
-						// When type is not an interface, it requires Converters
-						// ex :
-						// interface Server {Log log; class Log {}}
-						// throw the error;
-						// java.lang.IllegalArgumentException: SRCFG00013: No Converter registered for
-						// class org.acme.Server2$Log
-						// at
-						// io.smallrye.config.SmallRyeConfig.requireConverter(SmallRyeConfig.java:466)
-						// at
-						// io.smallrye.config.ConfigMappingContext.getConverter(ConfigMappingContext.java:113)
-						continue;
-					}
-				}
+				boolean leafType = isLeafType(returnType);
 
 				String defaultValue = getWithDefault(method);
 				String propertyName = getPropertyName(method, prefixStr, configMappingAnnotation);
@@ -190,27 +185,26 @@ private void populateConfigObject(IType configMappingType, String prefixStr, Str
 				IType enclosedType = getEnclosedType(returnType, resolvedTypeSignature, method.getJavaProject());
 				super.updateHint(collector, enclosedType);
 
-				if (!simpleType) {
+				if (!leafType) {
 					if (isMap(returnType, resolvedTypeSignature)) {
 						// Map<String, String>
 						propertyName += ".{*}";
-						simpleType = true;
+						leafType = true;
 					} else if (isCollection(returnType, resolvedTypeSignature)) {
 						// List<String>, List<App>
 						propertyName += "[*]"; // Generate indexed property.
-						String[] typeArguments = org.eclipse.jdt.core.Signature
-								.getTypeArguments(returnTypeSignature);
+						String[] typeArguments = org.eclipse.jdt.core.Signature.getTypeArguments(returnTypeSignature);
 						if (typeArguments.length < 1) {
 							continue;
 						}
 						String genericTypeName = typeArguments[0];
 						resolvedTypeSignature = JavaModelUtil.getResolvedTypeName(genericTypeName, configMappingType);
 						returnType = findType(method.getJavaProject(), resolvedTypeSignature);
-						simpleType = isSimpleType(resolvedTypeSignature, returnType);
+						leafType = isLeafType(returnType);
 					}
 				}
 
-				if (simpleType) {
+				if (leafType) {
 					// String, int, etc
 					ItemMetadata metadata = super.addItemMetadata(collector, propertyName, type, description,
 							sourceType, null, sourceMethod, defaultValue, extensionName, method.isBinary());
@@ -224,8 +218,15 @@ private void populateConfigObject(IType configMappingType, String prefixStr, Str
 		}
 	}
 
-	private boolean isSimpleType(String resolvedTypeSignature, IType returnType) {
-		return returnType == null || isPrimitiveType(resolvedTypeSignature);
+	/**
+	 * Returns true if the given return type should be treated as a leaf in the
+	 * configuration tree, i.e. it is null or not an interface, and therefore not
+	 * recursively visited.
+	 * 
+	 * @throws JavaModelException
+	 */
+	private static boolean isLeafType(IType returnType) throws JavaModelException {
+		return returnType == null || !returnType.isInterface();
 	}
 
 	private static boolean isMap(IType type, String typeName) {
@@ -276,7 +277,7 @@ private static String convertName(IMember member, IAnnotation configMappingAnnot
 		// public interface ServerVerbatimNamingStrategy
 		// --> See https://quarkus.io/guides/config-mappings#namingstrategy
 		NamingStrategy namingStrategy = getNamingStrategy(configMappingAnnotation);
-		if (namingStrategy != null) {			
+		if (namingStrategy != null) {
 			switch (namingStrategy) {
 			case VERBATIM:
 				// The method name is used as is to map the configuration property.
@@ -321,7 +322,7 @@ private static NamingStrategy getNamingStrategy(IAnnotation configMappingAnnotat
 		}
 		return null;
 	}
-    
+
 	/**
 	 * Returns the value of @WithDefault("a value") and null otherwise.
 	 *