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)