Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[FIXED JENKINS-13112] Adding any post-build step as a build step caus…
…es exception
  • Loading branch information
imod committed Mar 31, 2012
1 parent e51b3ad commit a13c4d9
Showing 1 changed file with 7 additions and 1 deletion.
Expand Up @@ -34,6 +34,7 @@
import hudson.tasks.BuildStep;
import hudson.tasks.BuildStepDescriptor;
import hudson.tasks.Builder;
import hudson.tasks.Publisher;

import java.io.IOException;
import java.util.ArrayList;
Expand Down Expand Up @@ -135,7 +136,12 @@ public boolean configure(StaplerRequest req, JSONObject formData) throws FormExc
@Override
public Builder newInstance(StaplerRequest req, JSONObject formData) throws hudson.model.Descriptor.FormException {
ConditionalBuilder instance = req.bindJSON(ConditionalBuilder.class, formData);
instance.conditionalbuilders = Descriptor.newInstancesFromHeteroList(req, formData, "conditionalbuilders", Builder.all());
if (formData.opt("conditionalbuilders") != null) {
final List all = new ArrayList(Builder.all());
// as Any Build step also allows publishers to be used, we have to pass the publisher descriptors too...
all.addAll(Publisher.all());
instance.conditionalbuilders = Descriptor.newInstancesFromHeteroList(req, formData, "conditionalbuilders", all);
}
return instance;
}

Expand Down

0 comments on commit a13c4d9

Please sign in to comment.