Skip to content

Commit

Permalink
[JENKINS-16917] hudson.plugins.downstream_ext.DownstreamTrigger.Descr…
Browse files Browse the repository at this point in the history
…iptorImpl should not be extending BuildTrigger.DescriptorImpl since it does not produce a BuildTrigger from newInstance.

(Generally there is no reason to subclass a Descriptor type which is already an @extension rather than an explicit abstract supertype, but this is a bit more conservative change than just making it final.)
  • Loading branch information
jglick committed Feb 21, 2013
1 parent 940d743 commit fd0657e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion core/src/main/java/hudson/tasks/BuildTrigger.java
Expand Up @@ -296,7 +296,7 @@ public String getHelpFile() {
}

@Override
public Publisher newInstance(StaplerRequest req, JSONObject formData) throws FormException {
public BuildTrigger newInstance(StaplerRequest req, JSONObject formData) throws FormException {
String childProjectsString = formData.getString("childProjects").trim();
if (childProjectsString.endsWith(",")) {
childProjectsString = childProjectsString.substring(0, childProjectsString.length() - 1).trim();
Expand Down

0 comments on commit fd0657e

Please sign in to comment.