Fixed JENKINS-10584

jenkinsci · Aug 13, 2011 · 71f94b1 · 71f94b1
1 parent 1072d3f
commit 71f94b1
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 6 deletions.
diff --git a/src/main/java/com/thalesgroup/hudson/plugins/cccc/CcccPublisher.java b/src/main/java/com/thalesgroup/hudson/plugins/cccc/CcccPublisher.java
@@ -41,10 +41,12 @@ public class CcccPublisher extends Recorder implements Serializable {
 
     private final String metricFilePath;
 
+    private final boolean runOnFailedBuild;
 
     @DataBoundConstructor
-    public CcccPublisher(String metricFilePath) {
+    public CcccPublisher(String metricFilePath, boolean runOnFailedBuild) {
         this.metricFilePath = metricFilePath;
+        this.runOnFailedBuild = runOnFailedBuild;
     }
 
     @Override
@@ -59,7 +61,7 @@ protected boolean canContinue(final Result result) {
     @Override
     public boolean perform(AbstractBuild<?, ?> build, Launcher launcher, BuildListener listener) {
 
-        if (this.canContinue(build.getResult())) {
+        if (this.canContinue(build.getResult()) || runOnFailedBuild) {
 
             listener.getLogger().println("Parsing cccc results");
 

diff --git a/src/main/resources/com/thalesgroup/hudson/plugins/cccc/CcccPublisher/config.jelly b/src/main/resources/com/thalesgroup/hudson/plugins/cccc/CcccPublisher/config.jelly
@@ -23,10 +23,16 @@
 *******************************************************************************/
 -->
 <j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:d="jelly:define"
-	xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form" xmlns:u="/util">
+         xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form" xmlns:u="/util">
 
-	<f:entry title="Main XML report file path">
-		<f:textbox name="metricFilePath" value="${instance.metricFilePath}"/>
-	</f:entry>
+    <f:entry title="${%Main XML CCCC report file path}">
+        <f:textbox name="metricFilePath" value="${instance.metricFilePath}"/>
+    </f:entry>
+
+    <f:advanced>
+        <f:entry title="${%Run even if the build fails}">
+            <f:checkbox name="runOnFailedBuild" value="${instance.runOnFailedBuild}"/>
+        </f:entry>
+    </f:advanced>
 
 </j:jelly>