[JENKINS-43094] fix default values on the web configuration page of t…

…he Dependencies Fingerprint Publisher
jenkinsci · Jun 14, 2017 · 88ca34c · 88ca34c
1 parent 74ccdec
commit 88ca34c
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 5 deletions.
diff --git a/...ava/org/jenkinsci/plugins/pipeline/maven/publishers/DependenciesFingerprintPublisher.java b/...ava/org/jenkinsci/plugins/pipeline/maven/publishers/DependenciesFingerprintPublisher.java
@@ -45,7 +45,7 @@ public class DependenciesFingerprintPublisher extends MavenPublisher {
 
     private boolean includeScopeCompile = true;
 
-    private boolean includeScopeRuntime;
+    private boolean includeScopeRuntime = true;
 
     private boolean includeScopeTest;
 
@@ -170,6 +170,14 @@ public void process(@Nonnull StepContext context, @Nonnull Element mavenSpyLogsE
         }
     }
 
+    @Override
+    public String toString() {
+        return getClass().getName() + "[" +
+                "disabled=" + isDisabled() + ", " +
+                "scopes=" + getIncludedScopes() + ", " +
+                "versions={snapshot:" + isIncludeSnapshotVersions() + ", release:" + isIncludeReleaseVersions() + "}" +
+                ']';
+    }
     /**
      * @param mavenSpyLogs Root XML element
      * @return list of {@link MavenSpyLogProcessor.MavenArtifact}

diff --git a/...jenkinsci/plugins/pipeline/maven/publishers/DependenciesFingerprintPublisher/config.jelly b/...jenkinsci/plugins/pipeline/maven/publishers/DependenciesFingerprintPublisher/config.jelly
@@ -29,14 +29,14 @@ THE SOFTWARE.
     <st:include page="maven-publisher" class="${descriptor.clazz}"/>
     <f:section title="Dependencies filtering">
         <f:entry title="${%Include versions}">
-            <f:checkbox title="${%Snapshots}" field="includeSnapshotVersions"/>
+            <f:checkbox title="${%Snapshots}" field="includeSnapshotVersions" default="true"/>
             <f:checkbox title="${%Releases}" field="includeReleaseVersions"/>
         </f:entry>
 
         <f:entry title="${%Include Scopes}">
-            <f:checkbox title="${%Compile}" field="includeScopeCompile"/>
-            <f:checkbox title="${%Runtime}" field="includeScopeRuntime"/>
-            <f:checkbox title="${%Provided}" field="includeScopeProvided"/>
+            <f:checkbox title="${%Compile}" field="includeScopeCompile" default="true"/>
+            <f:checkbox title="${%Runtime}" field="includeScopeRuntime" default="true"/>
+            <f:checkbox title="${%Provided}" field="includeScopeProvided" default="true"/>
             <f:checkbox title="${%Test}" field="includeScopeTest"/>
         </f:entry>
     </f:section>