[
https://issues.apache.org/jira/browse/SLING-7985?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
David Bosschaert resolved SLING-7985.
-------------------------------------
Resolution: Fixed
Thanks for your pull request [~simone.tripodi], it's merged!
> 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)
