Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[JENKINS-40612] More efficient idiom for hasValidDestination.
  • Loading branch information
jglick committed Oct 9, 2017
1 parent ef2735e commit 075a19d
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 18 deletions.
10 changes: 5 additions & 5 deletions pom.xml
Expand Up @@ -5,7 +5,7 @@
<parent>
<groupId>org.jenkins-ci.plugins</groupId>
<artifactId>plugin</artifactId>
<version>2.33</version>
<version>2.36</version>
<relativePath />
</parent>

Expand All @@ -19,13 +19,13 @@
</description>
<url>https://wiki.jenkins-ci.org/display/JENKINS/CloudBees+Folders+Plugin</url>
<properties>
<jenkins.version>1.642.3</jenkins.version>
<jenkins.version>2.46.3</jenkins.version>
<no-test-jar>false</no-test-jar>
</properties>
<licenses>
<license>
<name>MIT</name>
<url>http://opensource.org/licenses/MIT</url>
<url>https://opensource.org/licenses/MIT</url>
</license>
</licenses>

Expand All @@ -39,13 +39,13 @@
<repositories>
<repository>
<id>repo.jenkins-ci.org</id>
<url>http://repo.jenkins-ci.org/public/</url>
<url>https://repo.jenkins-ci.org/public/</url>
</repository>
</repositories>
<pluginRepositories>
<pluginRepository>
<id>repo.jenkins-ci.org</id>
<url>http://repo.jenkins-ci.org/public/</url>
<url>https://repo.jenkins-ci.org/public/</url>
</pluginRepository>
</pluginRepositories>

Expand Down
Expand Up @@ -24,7 +24,6 @@

package com.cloudbees.hudson.plugins.folder.relocate;

import com.cloudbees.hudson.plugins.folder.AbstractFolder;
import com.cloudbees.hudson.plugins.folder.Folder;
import hudson.Extension;
import hudson.model.AbstractItem;
Expand All @@ -33,6 +32,7 @@
import hudson.model.Items;
import hudson.model.Job;
import hudson.model.TopLevelItem;
import hudson.security.ACL;
import hudson.security.AccessControlled;
import java.io.IOException;
import java.util.ArrayList;
Expand All @@ -48,9 +48,11 @@
/**
* Handler which can move items which are both {@link AbstractItem} and {@link TopLevelItem} into a {@link DirectlyModifiableTopLevelItemGroup}.
*/
@SuppressWarnings("rawtypes")
@Restricted(NoExternalUse.class)
@Extension(ordinal=-1000) public final class StandardHandler extends RelocationHandler {

@Override
public HandlingMode applicability(Item item) {
if (item instanceof TopLevelItem
&& item instanceof AbstractItem
Expand Down Expand Up @@ -78,16 +80,15 @@ private static <I extends AbstractItem & TopLevelItem> I doMove(Item item, Direc
}

public boolean hasValidDestination(Item item) {
Jenkins instance = Jenkins.getActiveInstance();
Jenkins instance = Jenkins.getInstance();
if (permitted(item, instance) && instance.getItem(item.getName()) == null) {
// we can move to the root if there is none with the same name.
return true;
}
// TODO use Items.allItems(instance, Item.class) once baseline Jenkins 2.37+
ITEM: for (Item g : instance.getAllItems()) {
ITEM: for (Item g : Items.allItems(ACL.SYSTEM, instance, Item.class)) {
if (g instanceof DirectlyModifiableTopLevelItemGroup) {
DirectlyModifiableTopLevelItemGroup itemGroup = (DirectlyModifiableTopLevelItemGroup) g;
if (!permitted(item, itemGroup)) {
if (!permitted(item, itemGroup) || /* unlikely since we just checked CREATE, but just in case: */ !g.hasPermission(Item.READ)) {
continue;
}
// Cannot move a folder into itself or a descendant
Expand Down Expand Up @@ -126,8 +127,8 @@ public boolean hasValidDestination(Item item) {

@Override
public List<? extends ItemGroup<?>> validDestinations(Item item) {
List<DirectlyModifiableTopLevelItemGroup> result = new ArrayList<DirectlyModifiableTopLevelItemGroup>();
Jenkins instance = Jenkins.getActiveInstance();
List<DirectlyModifiableTopLevelItemGroup> result = new ArrayList<>();
Jenkins instance = Jenkins.getInstance();
// ROOT context is only added in case there is not any item with the same name
// But we add it in case the one is there is the item itself and not a different job with the same name
// No-op by default
Expand Down
Expand Up @@ -29,10 +29,9 @@
import hudson.model.Item;
import hudson.model.User;
import hudson.security.ACL;
import hudson.security.ACLContext;
import java.util.Arrays;
import jenkins.model.Jenkins;
import org.acegisecurity.context.SecurityContext;
import org.acegisecurity.context.SecurityContextHolder;
import static org.junit.Assert.*;
import org.junit.Rule;
import org.junit.Test;
Expand All @@ -52,15 +51,12 @@ public class StandardHandlerTest {
r.jenkins.setAuthorizationStrategy(new MockAuthorizationStrategy().
grant(Jenkins.READ, Item.READ).everywhere().to("joe").
grant(Item.CREATE).onItems(d2).to("joe"));
SecurityContext sc = ACL.impersonate(User.get("joe").impersonate());
try {
try (ACLContext ctx = ACL.as(User.get("joe"))) {
assertEquals(Arrays.asList(d1, d2), new StandardHandler().validDestinations(j));
assertEquals(Arrays.asList(r.jenkins, d2), new StandardHandler().validDestinations(d1));

assertNotEquals(Arrays.asList(r.jenkins, d3), new StandardHandler().validDestinations(j));
assertNotEquals(Arrays.asList(d1, d3), new StandardHandler().validDestinations(d1));
} finally {
SecurityContextHolder.setContext(sc);
}
}

Expand Down

0 comments on commit 075a19d

Please sign in to comment.