Skip to content

Commit

Permalink
Fixing checkstyle errors
Browse files Browse the repository at this point in the history
  • Loading branch information
Fabricio Leotti committed Feb 27, 2014
1 parent 6b09436 commit ea98a78
Show file tree
Hide file tree
Showing 5 changed files with 20 additions and 17 deletions.
2 changes: 2 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@
target
work

.checkstyle

# IntelliJ
.idea/
*.iml
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -67,14 +67,14 @@ public BuildForm(final PipelineBuild pipelineBuild) {
dependencies.add(new BuildForm(downstream));
}
id = hashCode();
AbstractProject<?,?> project = pipelineBuild.getProject();
final AbstractProject<?, ?> project = pipelineBuild.getProject();
projectId = project.getFullName().hashCode();
ParametersDefinitionProperty params = project.getProperty(ParametersDefinitionProperty.class);
ArrayList<String> paramList = new ArrayList<String>();
if(params != null) {
for(String p : params.getParameterDefinitionNames()) {
paramList.add(p);
}
final ParametersDefinitionProperty params = project.getProperty(ParametersDefinitionProperty.class);
final ArrayList<String> paramList = new ArrayList<String>();
if (params != null) {
for (String p : params.getParameterDefinitionNames()) {
paramList.add(p);
}
}
parameters = paramList;
}
Expand Down Expand Up @@ -145,7 +145,7 @@ public Map<String, String> getParameters() {
}

public ArrayList<String> getParameterList() {
return parameters;
return parameters;
}

public Integer getProjectId() {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,8 +40,6 @@
import hudson.model.Descriptor.FormException;
import hudson.model.Hudson;
import hudson.model.ParametersAction;
import hudson.model.ParametersDefinitionProperty;
import hudson.model.Project;
import hudson.model.Run;
import hudson.model.User;
import hudson.model.View;
Expand Down Expand Up @@ -74,8 +72,9 @@
import au.com.centrumsystems.hudson.plugin.util.ProjectUtil;

/**
* This view displays the set of jobs that are related based on their upstream\downstream relationships as a pipeline. Each build pipeline
* becomes a row on the view.
* This view displays the set of jobs that are related
* based on their upstream\downstream relationships as a pipeline. Each
* build pipeline becomes a row on the view.
*
* @author Centrum Systems
*
Expand Down Expand Up @@ -222,9 +221,9 @@ public void print(final TaskListener listener) {
* Indicates whether only the latest job will be triggered.
* @param cssUrl
* URL for the custom CSS file.
*
* @param startsWithParameters
* Indicates whether the first job of the pipeline takes parameters
* Indicates whether the first job of the pipeline takes
* parameters
*/
public BuildPipelineView(final String name, final String buildViewTitle,
final ProjectGridBuilder gridBuilder, final String noOfDisplayedBuilds,
Expand Down Expand Up @@ -266,7 +265,7 @@ public BuildPipelineView(final String name, final String buildViewTitle,
* @param selectedJob
* the first job name in the pipeline. it can be set to null when gridBuilder is passed.
* @param startsWithParameters
* Indicates whether the first job of the pipeline takes parameters
* indicates whether the first job of the pipeline takes parameters
*/
@DataBoundConstructor
public BuildPipelineView(final String name, final String buildViewTitle, final ProjectGridBuilder gridBuilder,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -163,8 +163,8 @@ public boolean hasBuildPermission(BuildPipelineView owner) {

@Override
public boolean startsWithParameters(BuildPipelineView owner) {
AbstractProject<?,?> firstJob = this.getFirstJob(owner);
ParametersDefinitionProperty pdp = firstJob.getProperty(ParametersDefinitionProperty.class);
final AbstractProject<?, ?> firstJob = this.getFirstJob(owner);
final ParametersDefinitionProperty pdp = firstJob.getProperty(ParametersDefinitionProperty.class);
return pdp != null;
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,8 @@ public class ReadOnlyBuildPipelineView extends BuildPipelineView {
* is trigger only latest job?
* @param cssUrl
* URL for the custom CSS file.
* @param startsWithParameters
* indicates if the first job in the view takes parameters
*/
public ReadOnlyBuildPipelineView(final String displayName, final String description, final ProjectGridBuilder gridBuilder,
final String noOfDisplayedBuilds, final boolean triggerOnlyLatestJob, final String cssUrl, final boolean startsWithParameters) {
Expand Down

0 comments on commit ea98a78

Please sign in to comment.