Skip to content

Commit

Permalink
JENKINS-40448: remove method added to S3ArtifactsAction
Browse files Browse the repository at this point in the history
As per comment on #97
  • Loading branch information
juliantcook committed Jan 2, 2017
1 parent b585d0d commit 3e43e4f
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 5 deletions.
4 changes: 0 additions & 4 deletions src/main/java/hudson/plugins/s3/S3ArtifactsAction.java
Expand Up @@ -66,10 +66,6 @@ public List<FingerprintRecord> getArtifacts() {
return artifacts;
}

public void addArtifacts(List<FingerprintRecord> artifacts) {
this.artifacts.addAll(artifacts);
}

public void doDownload(final StaplerRequest request, final StaplerResponse response) throws IOException, ServletException {
final String restOfPath = request.getRestOfPath();
if (restOfPath == null) {
Expand Down
2 changes: 1 addition & 1 deletion src/main/java/hudson/plugins/s3/S3BucketPublisher.java
Expand Up @@ -273,7 +273,7 @@ public void perform(@Nonnull Run<?, ?> run, @Nonnull FilePath ws, @Nonnull Launc
private void addS3ArtifactsAction(Run<?, ?> run, S3Profile profile, List<FingerprintRecord> artifacts) {
S3ArtifactsAction existingAction = run.getAction(S3ArtifactsAction.class);
if (existingAction != null) {
existingAction.addArtifacts(artifacts);
existingAction.getArtifacts().addAll(artifacts);
} else {
run.addAction(new S3ArtifactsAction(run, profile, artifacts));
}
Expand Down

0 comments on commit 3e43e4f

Please sign in to comment.