Add multi report

Author: totocaca

src/main/java/io/jenkins/plugins/reporter/steps/ReportConfiguration.java

@@ -0,0 +1,90 @@
+package io.jenkins.plugins.reporter.steps;
+
+import hudson.Extension;
+import hudson.model.AbstractDescribableImpl;
+import hudson.model.AbstractProject;
+import hudson.model.Descriptor;
+import hudson.model.Item;
+import hudson.util.FormValidation;
+import hudson.util.ListBoxModel;
+import io.jenkins.plugins.reporter.model.DisplayType;
+import io.jenkins.plugins.reporter.model.Provider;
+import io.jenkins.plugins.util.JenkinsFacade;
+import org.apache.commons.lang3.StringUtils;
+import org.kohsuke.stapler.AncestorInPath;
+import org.kohsuke.stapler.DataBoundConstructor;
+import org.kohsuke.stapler.DataBoundSetter;
+import org.kohsuke.stapler.QueryParameter;
+import org.kohsuke.stapler.verb.POST;
+
+/**
+ * Configuration for a single report in a FreeStyle project.
+ */
+public class ReportConfiguration extends AbstractDescribableImpl<ReportConfiguration> {
+    
+    private String name;
+    private Provider provider;
+    private String displayType = "dual";
+    
+    @DataBoundConstructor
+    public ReportConfiguration() {
+        // empty constructor required for Stapler
+    }
+    
+    public String getName() {
+        return name;
+    }
+    
+    @DataBoundSetter
+    public void setName(String name) {
+        this.name = name;
+    }
+    
+    public Provider getProvider() {
+        return provider;
+    }
+    
+    @DataBoundSetter
+    public void setProvider(Provider provider) {
+        this.provider = provider;
+    }
+    
+    public String getDisplayType() {
+        return displayType;
+    }
+    
+    @DataBoundSetter
+    public void setDisplayType(String displayType) {
+        this.displayType = displayType;
+    }
+    
+    @Extension
+    public static class DescriptorImpl extends Descriptor<ReportConfiguration> {
+        private static final JenkinsFacade JENKINS = new JenkinsFacade();
+        
+        @Override
+        public String getDisplayName() {
+            return "Report Configuration";
+        }
+        
+        @POST
+        public FormValidation doCheckName(@QueryParameter("name") String name) {
+            if (StringUtils.isEmpty(name)) {
+                return FormValidation.error("Field 'name' is required.");
+            }
+            return FormValidation.ok();
+        }
+        
+        @POST
+        public ListBoxModel doFillDisplayTypeItems(@AncestorInPath final AbstractProject<?, ?> project) {
+            if (JENKINS.hasPermission(Item.CONFIGURE, project)) {
+                ListBoxModel r = new ListBoxModel();
+                for (DisplayType dt : DisplayType.values()) {
+                    r.add(dt.name().toLowerCase());
+                }
+                return r;
+            }
+            return new ListBoxModel();
+        }
+    }
+}

src/main/java/io/jenkins/plugins/reporter/steps/ReportRecorder.java

@@ -29,16 +29,14 @@
 import org.kohsuke.stapler.verb.POST;
 
 import java.io.IOException;
+import java.util.ArrayList;
 import java.util.Arrays;
+import java.util.List;
 import java.util.Locale;
 
 public class ReportRecorder extends Recorder {
 
-    private String name;
-    
-    private Provider provider;
-    
-    private String displayType;
+    private List<ReportConfiguration> reports = new ArrayList<>();
 
     /**
      * Creates a new instance of {@link ReportRecorder}.
@@ -49,6 +47,15 @@ public ReportRecorder() {
 
         // empty constructor required for Stapler
     }
+    
+    @DataBoundSetter
+    public void setReports(List<ReportConfiguration> reports) {
+        this.reports = reports;
+    }
+    
+    public List<ReportConfiguration> getReports() {
+        return reports;
+    }
 
     /**
      * Called after de-serialization to retain backward compatibility or to populate new elements (that would be
@@ -57,9 +64,25 @@ public ReportRecorder() {
      * @return this
      */
     protected Object readResolve() {
+        // Backward compatibility: if old single report fields are set, convert to list
+        if (reports == null) {
+            reports = new ArrayList<>();
+        }
+        if (name != null && provider != null && reports.isEmpty()) {
+            ReportConfiguration config = new ReportConfiguration();
+            config.setName(name);
+            config.setProvider(provider);
+            config.setDisplayType(displayType != null ? displayType : "dual");
+            reports.add(config);
+        }
         return this;
     }
 
+    // Keep these for backward compatibility
+    private String name;
+    private Provider provider;
+    private String displayType;
+    
     @DataBoundSetter
     public void setName(String name) {
         this.name = name;
@@ -124,17 +147,31 @@ ReportResult perform(final Run<?, ?> run, final FilePath workspace, final TaskLi
 
     private ReportResult record(final Run<?, ?> run, final FilePath workspace, final TaskListener listener) 
             throws IOException, InterruptedException {
-    
-        Report report = scan(run, workspace, listener, provider);
-        report.setName(getName());
-
-        DisplayType dt = Arrays.stream(DisplayType.values())
-                .filter(e -> e.name().toLowerCase(Locale.ROOT).equals(getDisplayType()))
-                .findFirst().orElse(DisplayType.ABSOLUTE);
 
-        report.setDisplayType(dt);
+        // Ensure backward compatibility is handled
+        readResolve();
+        
+        ReportResult lastResult = null;
+        
+        // Process all configured reports
+        for (ReportConfiguration config : reports) {
+            if (config.getProvider() == null) {
+                continue;
+            }
+            
+            Report report = scan(run, workspace, listener, config.getProvider());
+            report.setName(config.getName());
+
+            DisplayType dt = Arrays.stream(DisplayType.values())
+                    .filter(e -> e.name().toLowerCase(Locale.ROOT).equals(config.getDisplayType()))
+                    .findFirst().orElse(DisplayType.ABSOLUTE);
+            
+            report.setDisplayType(dt);
+            
+            lastResult = publishReport(run, listener, config.getProvider().getSymbolName(), report);
+        }
 
-    return publishReport(run, listener, provider.getSymbolName(), report);
+        return lastResult;
     }
 
     ReportResult publishReport(final Run<?, ?> run, final TaskListener listener,

src/main/resources/io/jenkins/plugins/reporter/steps/ReportConfiguration/config.jelly

@@ -0,0 +1,6 @@
+<?jelly escape-by-default='true'?>
+<j:jelly xmlns:j="jelly:core" xmlns:report="/report">
+ 
+        <report:step/>
+            
+</j:jelly>

src/main/resources/io/jenkins/plugins/reporter/steps/ReportRecorder/config.jelly

@@ -1,6 +1,8 @@
 <?jelly escape-by-default='true'?>
-<j:jelly xmlns:j="jelly:core" xmlns:report="/report">
+<j:jelly xmlns:j="jelly:core" xmlns:f="/lib/form">
 
-        <report:step/>
+    <f:entry>
+        <f:repeatableProperty field="reports" add="Add Report" header="Report Configuration"/>
+    </f:entry>
 
-</j:jelly>
+</j:jelly>