You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Simone Tripodi (JIRA)" <ji...@apache.org> on 2018/10/05 10:01:00 UTC
[jira] [Created] (SLING-7985) ProjectHelper#getConfigValue() puts
null values to the resulting Set values
Simone Tripodi created SLING-7985:
-------------------------------------
Summary: 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
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)