rmannibucau commented on code in PR #109:
URL: https://github.com/apache/deltaspike/pull/109#discussion_r1474915026


##########
deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/config/TypedResolverImpl.java:
##########
@@ -504,6 +530,66 @@ else if (Double.class.equals(configEntryType))
         {
             result = Double.parseDouble(value);
         }
+        else if (Character.class.equals(configEntryType))

Review Comment:
   I guess it is cause you didn't set a converter on the injection and there is 
no default built-in producer of such a type (this is the challenge with this 
PR, to modify 
org.apache.deltaspike.core.impl.config.ConfigurationExtension#collectDynamicTypes
 for default case when there is a known converter in the converter "registry" 
but avoid to load and define everything upfront which would make the whole 
container slower for a very poor gain for most apps)



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@deltaspike.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to