We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
With #17088 we introduced lists.
Before this was not supported by the ConfigSource still it was possible by convention to get comma separated values as lists.
ConfigSource
Now we have some "hack" in the ConfigurationImpl.java to support both methods:
ConfigurationImpl.java
if (!propertiesService.isListProperty(propertyName)) { final String value = getValue(propertyName); values = ConfigListUtils.createList(value); } else { values = getValues(propertyName); }
This should be refactored and in the same way the AbstractConfigSource can be refactored to support list properties in a generic way
AbstractConfigSource
The text was updated successfully, but these errors were encountered:
No branches or pull requests
With #17088 we introduced lists.
Before this was not supported by the
ConfigSource
still it was possible by convention to get comma separated values as lists.Now we have some "hack" in the
ConfigurationImpl.java
to support both methods:This should be refactored and in the same way the
AbstractConfigSource
can be refactored to support list properties in a generic wayThe text was updated successfully, but these errors were encountered: