[sling-slingfeature-maven-plugin] 01/01: SLING-7985 - ProjectHelper#getConfigValue() puts null values to the resulting Set values

[simonetripodi]

This is an automated email from the ASF dual-hosted git repository.

simonetripodi pushed a commit to branch SLING-7985
in repository 
https://gitbox.apache.org/repos/asf/sling-slingfeature-maven-plugin.git

commit 419321093bd4565c754bb260feae95e97a401af9
Author: Simo Tripodi <strip...@adobe.com>
AuthorDate: Fri Oct 5 12:01:27 2018 +0200

    SLING-7985 - ProjectHelper#getConfigValue() puts null values to the
    resulting Set<String> values
    
    proposed fix
---
 src/main/java/org/apache/sling/feature/maven/ProjectHelper.java | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/main/java/org/apache/sling/feature/maven/ProjectHelper.java 
b/src/main/java/org/apache/sling/feature/maven/ProjectHelper.java
index dc76e78..1b8dc73 100644
--- a/src/main/java/org/apache/sling/feature/maven/ProjectHelper.java
+++ b/src/main/java/org/apache/sling/feature/maven/ProjectHelper.java
@@ -180,7 +180,9 @@ public abstract class ProjectHelper {
         for(final PluginExecution exec : plugin.getExecutions()) {
             final Xpp3Dom cfg = (Xpp3Dom)exec.getConfiguration();
             final Xpp3Dom pluginNode = (cfg == null ? null : 
cfg.getChild(name));
-            if ( pluginNode != null ) {
+            if ( pluginNode != null
+                    && pluginNode.getValue() != null
+                    && !pluginNode.getValue().isEmpty() ) {
                 values.add(pluginNode.getValue());
             }
         }