Skip to content
Permalink
Browse files

[JENKINS-17680] Changed a confusiong method name.

  • Loading branch information...
ikedam committed Aug 8, 2013
1 parent 3c6babe commit 355aedb310d61d2f26e235630b4fc99c7c886e73
Showing with 3 additions and 3 deletions.
  1. +3 −3 src/main/java/hudson/plugins/copyartifact/CopyArtifact.java
@@ -146,7 +146,7 @@ private static synchronized void setUpgradeNeeded() {
}

// get all CopyArtifacts configured to AbstractProject. This works both for Project and MatrixProject.
private static List<CopyArtifact> getCopiers(AbstractProject<?,?> project) throws IOException {
private static List<CopyArtifact> getCopyArtifactsInProject(AbstractProject<?,?> project) throws IOException {
DescribableList<Builder,Descriptor<Builder>> list =
project instanceof Project ? ((Project<?,?>)project).getBuildersList()
: (project instanceof MatrixProject ?
@@ -165,7 +165,7 @@ public static void upgradeCopyArtifact() {
boolean isUpgraded = false;
for (AbstractProject<?,?> project: Jenkins.getInstance().getAllItems(AbstractProject.class)) {
try {
for (CopyArtifact target: getCopiers(project)) {
for (CopyArtifact target: getCopyArtifactsInProject(project)) {
try {
if (target.upgradeIfNecessary(project)) {
isUpgraded = true;
@@ -405,7 +405,7 @@ else if (ca.getProjectName().startsWith(oldName + '/'))
}

private static List<CopyArtifact> getCopiers(AbstractProject<?,?> project) throws IOException {
List<CopyArtifact> copiers = CopyArtifact.getCopiers(project);
List<CopyArtifact> copiers = CopyArtifact.getCopyArtifactsInProject(project);
for (CopyArtifact copier : copiers) {
copier.upgradeIfNecessary(project);
}

0 comments on commit 355aedb

Please sign in to comment.
You can’t perform that action at this time.