Merge pull request #7 from getrostt/Bugfix/JENKINS-39344

Fix issue JENKINS-39344
jenkinsci · Jan 14, 2018 · fcc34c7 · fcc34c7
2 parents 546f020 + f7bf01c
commit fcc34c7
Show file tree

Hide file tree

Showing 3 changed files with 38 additions and 3 deletions.
diff --git a/pom.xml b/pom.xml
@@ -75,7 +75,7 @@
         <dependency>
             <groupId>org.jenkins-ci.plugins</groupId>
             <artifactId>publish-over</artifactId>
-            <version>0.20</version>
+            <version>0.21</version>
         </dependency>
         <dependency>
             <groupId>org.jenkins-ci.plugins</groupId>

diff --git a/src/main/java/jenkins/plugins/publish_over_cifs/CifsPublisher.java b/src/main/java/jenkins/plugins/publish_over_cifs/CifsPublisher.java
@@ -90,5 +90,4 @@ public int hashCode() {
     public String toString() {
         return addToToString(new ToStringBuilder(this, ToStringStyle.SHORT_PREFIX_STYLE)).toString();
     }
-
 }
diff --git a/src/main/java/jenkins/plugins/publish_over_cifs/CifsPublisherPlugin.java b/src/main/java/jenkins/plugins/publish_over_cifs/CifsPublisherPlugin.java
@@ -32,13 +32,15 @@
 import jenkins.plugins.publish_over.BPBuildInfo;
 import jenkins.plugins.publish_over.BPPlugin;
 import jenkins.plugins.publish_over.BPPluginDescriptor;
+import jenkins.plugins.publish_over.ParamPublish;
 import jenkins.plugins.publish_over_cifs.descriptor.CifsPublisherPluginDescriptor;
 import org.apache.commons.lang.builder.EqualsBuilder;
 import org.apache.commons.lang.builder.HashCodeBuilder;
 import org.apache.commons.lang.builder.ToStringBuilder;
 import org.apache.commons.lang.builder.ToStringStyle;
 import org.jenkinsci.Symbol;
 import org.kohsuke.stapler.DataBoundConstructor;
+import org.kohsuke.stapler.DataBoundSetter;
 
 import java.util.ArrayList;
 import java.util.List;
@@ -48,37 +50,71 @@ public class CifsPublisherPlugin extends BPPlugin<CifsPublisher, CifsClient, Obj
 
     private static final long serialVersionUID = 1L;
 
-    @DataBoundConstructor
     public CifsPublisherPlugin(final ArrayList<CifsPublisher> publishers, final boolean continueOnError, final boolean failOnError,
                                final boolean alwaysPublishFromMaster, final String masterNodeName, final CifsParamPublish paramPublish) {
         super(Messages.console_message_prefix(), publishers, continueOnError, failOnError, alwaysPublishFromMaster, masterNodeName,
                 paramPublish);
     }
 
+    @DataBoundConstructor
+    public CifsPublisherPlugin() {
+        super(Messages.console_message_prefix());
+    }
+
     public List<CifsPublisher> getPublishers() {
         return this.getDelegate().getPublishers();
     }
 
+    @DataBoundSetter
+    public void setPublishers(final ArrayList<CifsPublisher> publishers) {
+        this.getDelegate().setPublishers(publishers);
+    }
+
     public boolean isContinueOnError() {
         return this.getDelegate().isContinueOnError();
     }
 
+    @DataBoundSetter
+    public void setContinueOnError(final boolean continueOnError) {
+        this.getDelegate().setContinueOnError(continueOnError);
+    }
+
     public boolean isFailOnError() {
         return this.getDelegate().isFailOnError();
     }
 
+    @DataBoundSetter
+    public void setFailOnError(final boolean failOnError) {
+        this.getDelegate().setFailOnError(failOnError);
+    }
+
     public boolean isAlwaysPublishFromMaster() {
         return this.getDelegate().isAlwaysPublishFromMaster();
     }
 
+    @DataBoundSetter
+    public void setAlwaysPublishFromMaster(final boolean alwaysPublishFromMaster) {
+        this.getDelegate().setAlwaysPublishFromMaster(alwaysPublishFromMaster);
+    }
+
     public String getMasterNodeName() {
         return this.getDelegate().getMasterNodeName();
     }
 
+    @DataBoundSetter
+    public void setMasterNodeName(final String masterNodeName) {
+        this.getDelegate().setMasterNodeName(masterNodeName);
+    }
+
     public CifsParamPublish getParamPublish() {
         return (CifsParamPublish) this.getDelegate().getParamPublish();
     }
 
+    @DataBoundSetter
+    public void setParamPublish(final ParamPublish paramPublish) {
+        this.getDelegate().setParamPublish(paramPublish);
+    }
+
     @Override
     protected void fixup(final Run<?, ?> build, final BPBuildInfo buildInfo) {
         final Jenkins jenkins = Jenkins.getInstance();