[
https://issues.apache.org/jira/browse/SLING-7985?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16640677#comment-16640677
]
Simone Tripodi commented on SLING-7985:
---------------------------------------
Awersome [~bosschaert], thanks! :)
> ProjectHelper#getConfigValue() puts null values to the resulting Set<String>
> values
> -----------------------------------------------------------------------------------
>
> Key: SLING-7985
> URL: https://issues.apache.org/jira/browse/SLING-7985
> Project: Sling
> Issue Type: Bug
> Components: Feature Model, Maven Plugins and Archetypes
> Reporter: Simone Tripodi
> Assignee: David Bosschaert
> Priority: Major
>
> Looks like {{null}} values can be erroneously added in the {{Set<String>}}
> values and that are preferred to the default value.
> A safer way is adding more safe guards, i.e.
> {noformat}
> if ( pluginNode != null
> && pluginNode.getValue() != null
> && !pluginNode.getValue().isEmpty() ) {
> values.add(pluginNode.getValue());
> }
> {noformat}
> PR is coming
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
