mirror of
https://github.com/Alfresco/alfresco-community-repo.git
synced 2025-10-29 15:21:53 +00:00
6afb44e712
125606 rmunteanu: Merged 5.1.1 (5.1.1) to 5.1.N (5.1.2)
125515 slanglois: MNT-16155 Update source headers - add new Copyrights for Java and JSP source files + automatic check in the build
git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/BRANCHES/DEV/5.2.N/root@125788 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
978 lines
42 KiB
Java
978 lines
42 KiB
Java
/*
|
|
* #%L
|
|
* Alfresco Repository
|
|
* %%
|
|
* Copyright (C) 2005 - 2016 Alfresco Software Limited
|
|
* %%
|
|
* This file is part of the Alfresco software.
|
|
* If the software was purchased under a paid Alfresco license, the terms of
|
|
* the paid license agreement will prevail. Otherwise, the software is
|
|
* provided under the following open source license terms:
|
|
*
|
|
* Alfresco is free software: you can redistribute it and/or modify
|
|
* it under the terms of the GNU Lesser General Public License as published by
|
|
* the Free Software Foundation, either version 3 of the License, or
|
|
* (at your option) any later version.
|
|
*
|
|
* Alfresco is distributed in the hope that it will be useful,
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
* GNU Lesser General Public License for more details.
|
|
*
|
|
* You should have received a copy of the GNU Lesser General Public License
|
|
* along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
|
|
* #L%
|
|
*/
|
|
package org.alfresco.repo.links;
|
|
|
|
import static org.junit.Assert.assertEquals;
|
|
import static org.junit.Assert.assertFalse;
|
|
import static org.junit.Assert.assertNotNull;
|
|
import static org.junit.Assert.assertTrue;
|
|
|
|
import java.util.ArrayList;
|
|
import java.util.Date;
|
|
import java.util.List;
|
|
|
|
import org.alfresco.model.ContentModel;
|
|
import org.alfresco.query.PagingRequest;
|
|
import org.alfresco.query.PagingResults;
|
|
import org.alfresco.repo.policy.BehaviourFilter;
|
|
import org.alfresco.repo.security.authentication.AuthenticationUtil;
|
|
import org.alfresco.repo.site.SiteModel;
|
|
import org.alfresco.repo.transaction.RetryingTransactionHelper;
|
|
import org.alfresco.repo.transaction.RetryingTransactionHelper.RetryingTransactionCallback;
|
|
import org.alfresco.service.cmr.links.LinkInfo;
|
|
import org.alfresco.service.cmr.links.LinksService;
|
|
import org.alfresco.service.cmr.repository.ContentService;
|
|
import org.alfresco.service.cmr.repository.NodeRef;
|
|
import org.alfresco.service.cmr.repository.NodeService;
|
|
import org.alfresco.service.cmr.security.MutableAuthenticationService;
|
|
import org.alfresco.service.cmr.security.PermissionService;
|
|
import org.alfresco.service.cmr.security.PersonService;
|
|
import org.alfresco.service.cmr.site.SiteInfo;
|
|
import org.alfresco.service.cmr.site.SiteService;
|
|
import org.alfresco.service.cmr.site.SiteVisibility;
|
|
import org.alfresco.service.cmr.tagging.TaggingService;
|
|
import org.alfresco.util.ApplicationContextHelper;
|
|
import org.alfresco.util.PropertyMap;
|
|
import org.junit.After;
|
|
import org.junit.AfterClass;
|
|
import org.junit.Before;
|
|
import org.junit.BeforeClass;
|
|
import org.junit.Test;
|
|
import org.springframework.context.ApplicationContext;
|
|
|
|
/**
|
|
* Test cases for {@link LinksServiceImpl}.
|
|
*
|
|
* @author Nick Burch
|
|
* @since 4.0
|
|
*/
|
|
public class LinksServiceImplTest
|
|
{
|
|
private static final String TEST_SITE_PREFIX = "LinksSiteTest";
|
|
private static final long ONE_DAY_MS = 24*60*60*1000;
|
|
|
|
private static final ApplicationContext testContext = ApplicationContextHelper.getApplicationContext();
|
|
|
|
// injected services
|
|
private static MutableAuthenticationService AUTHENTICATION_SERVICE;
|
|
private static BehaviourFilter BEHAVIOUR_FILTER;
|
|
private static LinksService LINKS_SERVICE;
|
|
private static NodeService NODE_SERVICE;
|
|
private static NodeService PUBLIC_NODE_SERVICE;
|
|
private static PersonService PERSON_SERVICE;
|
|
private static RetryingTransactionHelper TRANSACTION_HELPER;
|
|
private static PermissionService PERMISSION_SERVICE;
|
|
private static SiteService SITE_SERVICE;
|
|
private static ContentService CONTENT_SERVICE;
|
|
private static TaggingService TAGGING_SERVICE;
|
|
|
|
private static final String TEST_USER = LinksServiceImplTest.class.getSimpleName() + "_testuser";
|
|
private static final String ADMIN_USER = AuthenticationUtil.getAdminUserName();
|
|
|
|
private static SiteInfo LINKS_SITE;
|
|
private static SiteInfo ALTERNATE_LINKS_SITE;
|
|
|
|
/**
|
|
* Temporary test nodes (created during a test method) that need deletion after the test method.
|
|
*/
|
|
private List<NodeRef> testNodesToTidy = new ArrayList<NodeRef>();
|
|
/**
|
|
* Temporary test nodes (created BeforeClass) that need deletion after this test class.
|
|
*/
|
|
private static List<NodeRef> CLASS_TEST_NODES_TO_TIDY = new ArrayList<NodeRef>();
|
|
|
|
@BeforeClass public static void initTestsContext() throws Exception
|
|
{
|
|
AUTHENTICATION_SERVICE = (MutableAuthenticationService)testContext.getBean("authenticationService");
|
|
BEHAVIOUR_FILTER = (BehaviourFilter)testContext.getBean("policyBehaviourFilter");
|
|
LINKS_SERVICE = (LinksService)testContext.getBean("LinksService");
|
|
NODE_SERVICE = (NodeService)testContext.getBean("nodeService");
|
|
PUBLIC_NODE_SERVICE = (NodeService)testContext.getBean("NodeService");
|
|
PERSON_SERVICE = (PersonService)testContext.getBean("personService");
|
|
TRANSACTION_HELPER = (RetryingTransactionHelper)testContext.getBean("retryingTransactionHelper");
|
|
PERMISSION_SERVICE = (PermissionService)testContext.getBean("permissionService");
|
|
SITE_SERVICE = (SiteService)testContext.getBean("siteService");
|
|
CONTENT_SERVICE = (ContentService)testContext.getBean("ContentService");
|
|
TAGGING_SERVICE = (TaggingService)testContext.getBean("TaggingService");
|
|
|
|
// Do the setup as admin
|
|
AuthenticationUtil.setFullyAuthenticatedUser(ADMIN_USER);
|
|
createUser(TEST_USER);
|
|
|
|
// We need to create the test site as the test user so that they can contribute content to it in tests below.
|
|
AuthenticationUtil.setFullyAuthenticatedUser(TEST_USER);
|
|
createTestSites();
|
|
}
|
|
|
|
@Test public void createNewEntry() throws Exception
|
|
{
|
|
LinkInfo link;
|
|
|
|
// Nothing to start with
|
|
PagingResults<LinkInfo> results =
|
|
LINKS_SERVICE.listLinks(LINKS_SITE.getShortName(), new PagingRequest(10));
|
|
assertEquals(0, results.getPage().size());
|
|
|
|
|
|
// Get with an arbitrary name gives nothing
|
|
link = LINKS_SERVICE.getLink(LINKS_SITE.getShortName(), "madeUp");
|
|
assertEquals(null, link);
|
|
|
|
link = LINKS_SERVICE.getLink(LINKS_SITE.getShortName(), "madeUp2");
|
|
assertEquals(null, link);
|
|
|
|
// Create one
|
|
link = LINKS_SERVICE.createLink(
|
|
LINKS_SITE.getShortName(), "Title", "Description",
|
|
"http://www.alfresco.com/", false);
|
|
|
|
|
|
// Ensure it got a noderef, and the correct site
|
|
assertNotNull(link.getNodeRef());
|
|
assertNotNull(link.getSystemName());
|
|
|
|
NodeRef container = PUBLIC_NODE_SERVICE.getPrimaryParent(link.getNodeRef()).getParentRef();
|
|
NodeRef site = PUBLIC_NODE_SERVICE.getPrimaryParent(container).getParentRef();
|
|
assertEquals(LINKS_SITE.getNodeRef(), site);
|
|
|
|
|
|
// Check the details on the object
|
|
assertEquals("Title", link.getTitle());
|
|
assertEquals("Description", link.getDescription());
|
|
assertEquals("http://www.alfresco.com/", link.getURL());
|
|
assertEquals(false, link.isInternal());
|
|
assertEquals(ADMIN_USER, link.getCreator());
|
|
assertEquals(0, link.getTags().size());
|
|
|
|
|
|
// Fetch it, and check the details
|
|
link = LINKS_SERVICE.getLink(LINKS_SITE.getShortName(), link.getSystemName());
|
|
assertEquals("Title", link.getTitle());
|
|
assertEquals("Description", link.getDescription());
|
|
assertEquals("http://www.alfresco.com/", link.getURL());
|
|
assertEquals(false, link.isInternal());
|
|
assertEquals(ADMIN_USER, link.getCreator());
|
|
assertEquals(0, link.getTags().size());
|
|
|
|
|
|
// Mark it as done with
|
|
testNodesToTidy.add(link.getNodeRef());
|
|
}
|
|
|
|
@Test public void createUpdateDeleteEntry() throws Exception
|
|
{
|
|
LinkInfo link;
|
|
|
|
// Run as the test user instead
|
|
AuthenticationUtil.setFullyAuthenticatedUser(TEST_USER);
|
|
|
|
|
|
// Create a link
|
|
link = LINKS_SERVICE.createLink(
|
|
LINKS_SITE.getShortName(), "Title", "Description",
|
|
"http://www.alfresco.com/", false);
|
|
|
|
|
|
// Check it
|
|
assertEquals("Title", link.getTitle());
|
|
assertEquals("Description", link.getDescription());
|
|
assertEquals("http://www.alfresco.com/", link.getURL());
|
|
assertEquals(false, link.isInternal());
|
|
assertEquals(TEST_USER, link.getCreator());
|
|
assertEquals(0, link.getTags().size());
|
|
|
|
|
|
// Check the underlying node
|
|
assertEquals("Title", PUBLIC_NODE_SERVICE.getProperty(link.getNodeRef(), LinksModel.PROP_TITLE));
|
|
assertEquals("Description", PUBLIC_NODE_SERVICE.getProperty(link.getNodeRef(), LinksModel.PROP_DESCRIPTION));
|
|
assertEquals("http://www.alfresco.com/", PUBLIC_NODE_SERVICE.getProperty(link.getNodeRef(), LinksModel.PROP_URL));
|
|
assertEquals("http://www.alfresco.com/", CONTENT_SERVICE.getReader(link.getNodeRef(), ContentModel.PROP_CONTENT).getContentString());
|
|
assertEquals(false, PUBLIC_NODE_SERVICE.hasAspect(link.getNodeRef(), LinksModel.ASPECT_INTERNAL_LINK));
|
|
|
|
|
|
// Change it
|
|
link.setTitle("New Title");
|
|
link.setURL("http://share.alfresco.com/");
|
|
link.setInternal(true);
|
|
|
|
LINKS_SERVICE.updateLink(link);
|
|
|
|
|
|
// Fetch, and check
|
|
link = LINKS_SERVICE.getLink(LINKS_SITE.getShortName(), link.getSystemName());
|
|
assertEquals("New Title", link.getTitle());
|
|
assertEquals("Description", link.getDescription());
|
|
assertEquals("http://share.alfresco.com/", link.getURL());
|
|
assertEquals(true, link.isInternal());
|
|
assertEquals(TEST_USER, link.getCreator());
|
|
assertEquals(0, link.getTags().size());
|
|
|
|
|
|
// Check the underlying node now
|
|
assertEquals("New Title", PUBLIC_NODE_SERVICE.getProperty(link.getNodeRef(), LinksModel.PROP_TITLE));
|
|
assertEquals("Description", PUBLIC_NODE_SERVICE.getProperty(link.getNodeRef(), LinksModel.PROP_DESCRIPTION));
|
|
assertEquals("http://share.alfresco.com/", PUBLIC_NODE_SERVICE.getProperty(link.getNodeRef(), LinksModel.PROP_URL));
|
|
assertEquals("http://share.alfresco.com/", CONTENT_SERVICE.getReader(link.getNodeRef(), ContentModel.PROP_CONTENT).getContentString());
|
|
assertEquals(true, PUBLIC_NODE_SERVICE.hasAspect(link.getNodeRef(), LinksModel.ASPECT_INTERNAL_LINK));
|
|
|
|
|
|
// Delete it
|
|
LINKS_SERVICE.deleteLink(link);
|
|
|
|
// Check it went
|
|
assertEquals(null, LINKS_SERVICE.getLink(LINKS_SITE.getShortName(), link.getSystemName()));
|
|
}
|
|
|
|
/**
|
|
* Ensures that when we try to write an entry to the
|
|
* container of a new site, it is correctly setup for us.
|
|
* This test does it's own transactions
|
|
*/
|
|
@Test public void newContainerSetup() throws Exception
|
|
{
|
|
final String TEST_SITE_NAME = "LinksTestNewTestSite";
|
|
|
|
TRANSACTION_HELPER.doInTransaction(new RetryingTransactionHelper.RetryingTransactionCallback<Void>()
|
|
{
|
|
@Override
|
|
public Void execute() throws Throwable
|
|
{
|
|
if (SITE_SERVICE.getSite(TEST_SITE_NAME) != null)
|
|
{
|
|
SITE_SERVICE.deleteSite(TEST_SITE_NAME);
|
|
}
|
|
SITE_SERVICE.createSite(
|
|
TEST_SITE_PREFIX, TEST_SITE_NAME, "Test", "Test", SiteVisibility.PUBLIC);
|
|
|
|
// Won't have the container to start with
|
|
assertFalse(SITE_SERVICE.hasContainer(TEST_SITE_NAME, LinksServiceImpl.LINKS_COMPONENT));
|
|
|
|
// Create a link
|
|
LINKS_SERVICE.createLink(
|
|
TEST_SITE_NAME, "Title", "Description",
|
|
"http://www.alfresco.com/", false);
|
|
|
|
// It will now exist
|
|
assertTrue(SITE_SERVICE.hasContainer(TEST_SITE_NAME, LinksServiceImpl.LINKS_COMPONENT));
|
|
|
|
// It'll be a tag scope too
|
|
NodeRef container = SITE_SERVICE.getContainer(TEST_SITE_NAME, LinksServiceImpl.LINKS_COMPONENT);
|
|
assertTrue(TAGGING_SERVICE.isTagScope(container));
|
|
|
|
// Tidy up
|
|
SITE_SERVICE.deleteSite(TEST_SITE_NAME);
|
|
return null;
|
|
}
|
|
});
|
|
}
|
|
|
|
@Test public void tagging() throws Exception
|
|
{
|
|
LinkInfo link;
|
|
final String TAG_1 = "link_tag_1";
|
|
final String TAG_2 = "link_tag_2";
|
|
final String TAG_3 = "link_tag_3";
|
|
|
|
// Create one without tagging
|
|
link = LINKS_SERVICE.createLink(
|
|
LINKS_SITE.getShortName(), "Title", "Description",
|
|
"http://www.alfresco.com/", false);
|
|
testNodesToTidy.add(link.getNodeRef());
|
|
|
|
// Check
|
|
assertEquals(0, link.getTags().size());
|
|
|
|
link = LINKS_SERVICE.getLink(LINKS_SITE.getShortName(), link.getSystemName());
|
|
assertEquals(0, link.getTags().size());
|
|
|
|
|
|
// Update it to have tags
|
|
link.getTags().add(TAG_1);
|
|
link.getTags().add(TAG_2);
|
|
link.getTags().add(TAG_1);
|
|
assertEquals(3, link.getTags().size());
|
|
LINKS_SERVICE.updateLink(link);
|
|
|
|
// Check
|
|
link = LINKS_SERVICE.getLink(LINKS_SITE.getShortName(), link.getSystemName());
|
|
assertEquals(2, link.getTags().size());
|
|
assertEquals(true, link.getTags().contains(TAG_1));
|
|
assertEquals(true, link.getTags().contains(TAG_2));
|
|
assertEquals(false, link.getTags().contains(TAG_3));
|
|
|
|
|
|
// Update it to have different tags
|
|
link.getTags().remove(TAG_2);
|
|
link.getTags().add(TAG_3);
|
|
link.getTags().add(TAG_1);
|
|
LINKS_SERVICE.updateLink(link);
|
|
|
|
// Check it as-is
|
|
assertEquals(3, link.getTags().size()); // Includes duplicate tag until re-loaded
|
|
assertEquals(true, link.getTags().contains(TAG_1));
|
|
assertEquals(false, link.getTags().contains(TAG_2));
|
|
assertEquals(true, link.getTags().contains(TAG_3));
|
|
|
|
// Now load and re-check
|
|
link = LINKS_SERVICE.getLink(LINKS_SITE.getShortName(), link.getSystemName());
|
|
assertEquals(2, link.getTags().size()); // Duplicate now gone
|
|
assertEquals(true, link.getTags().contains(TAG_1));
|
|
assertEquals(false, link.getTags().contains(TAG_2));
|
|
assertEquals(true, link.getTags().contains(TAG_3));
|
|
|
|
|
|
// Update it to have no tags
|
|
link.getTags().clear();
|
|
LINKS_SERVICE.updateLink(link);
|
|
|
|
// Check
|
|
link = LINKS_SERVICE.getLink(LINKS_SITE.getShortName(), link.getSystemName());
|
|
assertEquals(0, link.getTags().size());
|
|
|
|
|
|
// Update it to have tags again
|
|
link.getTags().add(TAG_1);
|
|
link.getTags().add(TAG_2);
|
|
link.getTags().add(TAG_3);
|
|
LINKS_SERVICE.updateLink(link);
|
|
|
|
// Check
|
|
link = LINKS_SERVICE.getLink(LINKS_SITE.getShortName(), link.getSystemName());
|
|
assertEquals(3, link.getTags().size());
|
|
assertEquals(true, link.getTags().contains(TAG_1));
|
|
assertEquals(true, link.getTags().contains(TAG_2));
|
|
assertEquals(true, link.getTags().contains(TAG_3));
|
|
|
|
// Tidy
|
|
LINKS_SERVICE.deleteLink(link);
|
|
}
|
|
|
|
/**
|
|
* Tests for listing the links of a site, possibly by user or date range
|
|
*/
|
|
@Test public void linksListing() throws Exception
|
|
{
|
|
PagingRequest paging = new PagingRequest(10);
|
|
AuthenticationUtil.setFullyAuthenticatedUser(TEST_USER);
|
|
|
|
// Nothing to start with
|
|
PagingResults<LinkInfo> results =
|
|
LINKS_SERVICE.listLinks(LINKS_SITE.getShortName(), paging);
|
|
assertEquals(0, results.getPage().size());
|
|
|
|
// Add a few
|
|
LinkInfo linkA = LINKS_SERVICE.createLink(
|
|
LINKS_SITE.getShortName(), "TitleA", "Description",
|
|
"http://www.alfresco.com/", false);
|
|
LinkInfo linkB = LINKS_SERVICE.createLink(
|
|
LINKS_SITE.getShortName(), "TitleB", "Description",
|
|
"http://www.alfresco.com/", false);
|
|
LinkInfo linkC = LINKS_SERVICE.createLink(
|
|
LINKS_SITE.getShortName(), "TitleC", "Description",
|
|
"http://www.alfresco.com/", false);
|
|
testNodesToTidy.add(linkA.getNodeRef());
|
|
testNodesToTidy.add(linkB.getNodeRef());
|
|
testNodesToTidy.add(linkC.getNodeRef());
|
|
|
|
// Check now, should be newest first
|
|
results = LINKS_SERVICE.listLinks(LINKS_SITE.getShortName(), paging);
|
|
assertEquals(3, results.getPage().size());
|
|
assertEquals("TitleC", results.getPage().get(0).getTitle());
|
|
assertEquals("TitleB", results.getPage().get(1).getTitle());
|
|
assertEquals("TitleA", results.getPage().get(2).getTitle());
|
|
|
|
// Add one more, as a different user, and drop the page size
|
|
AuthenticationUtil.setFullyAuthenticatedUser(ADMIN_USER);
|
|
LinkInfo linkD = LINKS_SERVICE.createLink(
|
|
LINKS_SITE.getShortName(), "TitleD", "Description",
|
|
"http://www.alfresco.com/", false);
|
|
testNodesToTidy.add(linkD.getNodeRef());
|
|
AuthenticationUtil.setFullyAuthenticatedUser(TEST_USER);
|
|
|
|
paging = new PagingRequest(3);
|
|
results = LINKS_SERVICE.listLinks(LINKS_SITE.getShortName(), paging);
|
|
assertEquals(3, results.getPage().size());
|
|
assertEquals("TitleD", results.getPage().get(0).getTitle());
|
|
assertEquals("TitleC", results.getPage().get(1).getTitle());
|
|
assertEquals("TitleB", results.getPage().get(2).getTitle());
|
|
|
|
paging = new PagingRequest(3, 3);
|
|
results = LINKS_SERVICE.listLinks(LINKS_SITE.getShortName(), paging);
|
|
assertEquals(1, results.getPage().size());
|
|
assertEquals("TitleA", results.getPage().get(0).getTitle());
|
|
|
|
|
|
// Now check filtering by user
|
|
paging = new PagingRequest(10);
|
|
|
|
results = LINKS_SERVICE.listLinks(LINKS_SITE.getShortName(), TEST_USER, paging);
|
|
assertEquals(3, results.getPage().size());
|
|
assertEquals("TitleC", results.getPage().get(0).getTitle());
|
|
assertEquals("TitleB", results.getPage().get(1).getTitle());
|
|
assertEquals("TitleA", results.getPage().get(2).getTitle());
|
|
|
|
results = LINKS_SERVICE.listLinks(LINKS_SITE.getShortName(), ADMIN_USER, paging);
|
|
assertEquals(1, results.getPage().size());
|
|
assertEquals("TitleD", results.getPage().get(0).getTitle());
|
|
|
|
|
|
// Now check filtering by date range
|
|
pushCreatedDateBack(linkB, 10);
|
|
pushCreatedDateBack(linkC, 100);
|
|
|
|
Date today = new Date();
|
|
Date tomorrow = new Date(today.getTime()+ONE_DAY_MS);
|
|
Date yesterday = new Date(today.getTime()-ONE_DAY_MS);
|
|
Date twoWeeksAgo = new Date(today.getTime()-14*ONE_DAY_MS);
|
|
|
|
// Very recent ones
|
|
results = LINKS_SERVICE.listLinks(LINKS_SITE.getShortName(), yesterday, tomorrow, paging);
|
|
assertEquals(2, results.getPage().size());
|
|
assertEquals("TitleD", results.getPage().get(0).getTitle());
|
|
assertEquals("TitleA", results.getPage().get(1).getTitle());
|
|
|
|
// Fairly old ones
|
|
results = LINKS_SERVICE.listLinks(LINKS_SITE.getShortName(), twoWeeksAgo, yesterday, paging);
|
|
assertEquals(1, results.getPage().size());
|
|
assertEquals("TitleB", results.getPage().get(0).getTitle());
|
|
|
|
// Fairly old to current
|
|
results = LINKS_SERVICE.listLinks(LINKS_SITE.getShortName(), twoWeeksAgo, tomorrow, paging);
|
|
assertEquals(3, results.getPage().size());
|
|
assertEquals("TitleD", results.getPage().get(0).getTitle());
|
|
assertEquals("TitleA", results.getPage().get(1).getTitle());
|
|
assertEquals("TitleB", results.getPage().get(2).getTitle());
|
|
|
|
|
|
// Tidy
|
|
paging = new PagingRequest(10);
|
|
AuthenticationUtil.setFullyAuthenticatedUser(ADMIN_USER);
|
|
results = LINKS_SERVICE.listLinks(LINKS_SITE.getShortName(), paging);
|
|
for (LinkInfo link : results.getPage())
|
|
{
|
|
PUBLIC_NODE_SERVICE.deleteNode(link.getNodeRef());
|
|
}
|
|
results = LINKS_SERVICE.listLinks(ALTERNATE_LINKS_SITE.getShortName(), paging);
|
|
for (LinkInfo link : results.getPage())
|
|
{
|
|
PUBLIC_NODE_SERVICE.deleteNode(link.getNodeRef());
|
|
}
|
|
}
|
|
|
|
/**
|
|
* Checks that the correct permission checking occurs on fetching
|
|
* links listings (which go through canned queries)
|
|
*/
|
|
@Test public void linksListingPermissionsChecking() throws Exception
|
|
{
|
|
PagingRequest paging = new PagingRequest(10);
|
|
PagingResults<LinkInfo> results;
|
|
|
|
// Nothing to start with in either site
|
|
results = LINKS_SERVICE.listLinks(LINKS_SITE.getShortName(), paging);
|
|
assertEquals(0, results.getPage().size());
|
|
results = LINKS_SERVICE.listLinks(ALTERNATE_LINKS_SITE.getShortName(), paging);
|
|
assertEquals(0, results.getPage().size());
|
|
|
|
// Double check that we're only allowed to see the 1st site
|
|
TRANSACTION_HELPER.doInTransaction(new RetryingTransactionHelper.RetryingTransactionCallback<Void>()
|
|
{
|
|
@Override public Void execute() throws Throwable
|
|
{
|
|
assertEquals(true, SITE_SERVICE.isMember(LINKS_SITE.getShortName(), TEST_USER));
|
|
assertEquals(false, SITE_SERVICE.isMember(ALTERNATE_LINKS_SITE.getShortName(), TEST_USER));
|
|
return null;
|
|
}
|
|
});
|
|
|
|
|
|
// Now become the test user
|
|
AuthenticationUtil.setFullyAuthenticatedUser(TEST_USER);
|
|
|
|
|
|
// Add two events to one site and three to the other
|
|
// Note - add the events as a different user for the site that the
|
|
// test user isn't a member of!
|
|
LinkInfo linkA = LINKS_SERVICE.createLink(
|
|
LINKS_SITE.getShortName(), "TitleA", "Description",
|
|
"http://www.alfresco.com/", false);
|
|
LinkInfo linkB = LINKS_SERVICE.createLink(
|
|
LINKS_SITE.getShortName(), "TitleB", "Description",
|
|
"http://www.alfresco.com/", false);
|
|
testNodesToTidy.add(linkA.getNodeRef());
|
|
testNodesToTidy.add(linkB.getNodeRef());
|
|
|
|
AuthenticationUtil.setFullyAuthenticatedUser(ADMIN_USER);
|
|
LinkInfo linkPrivA = LINKS_SERVICE.createLink(
|
|
ALTERNATE_LINKS_SITE.getShortName(), "PrivTitleA", "Description",
|
|
"http://team.alfresco.com/", false);
|
|
LinkInfo linkPrivB = LINKS_SERVICE.createLink(
|
|
ALTERNATE_LINKS_SITE.getShortName(), "PrivTitleB", "Description",
|
|
"http://team.alfresco.com/", false);
|
|
LinkInfo linkPrivC = LINKS_SERVICE.createLink(
|
|
ALTERNATE_LINKS_SITE.getShortName(), "PrivTitleC", "Description",
|
|
"http://team.alfresco.com/", false);
|
|
testNodesToTidy.add(linkPrivA.getNodeRef());
|
|
testNodesToTidy.add(linkPrivB.getNodeRef());
|
|
testNodesToTidy.add(linkPrivC.getNodeRef());
|
|
AuthenticationUtil.setFullyAuthenticatedUser(TEST_USER);
|
|
|
|
|
|
// Check again, as we're not in the 2nd site won't see any there
|
|
results = LINKS_SERVICE.listLinks(LINKS_SITE.getShortName(), paging);
|
|
assertEquals(2, results.getPage().size());
|
|
results = LINKS_SERVICE.listLinks(ALTERNATE_LINKS_SITE.getShortName(), paging);
|
|
assertEquals(0, results.getPage().size());
|
|
|
|
|
|
// Join the site, now we can see both
|
|
TRANSACTION_HELPER.doInTransaction(new RetryingTransactionHelper.RetryingTransactionCallback<Void>()
|
|
{
|
|
@Override
|
|
public Void execute() throws Throwable
|
|
{
|
|
AuthenticationUtil.setFullyAuthenticatedUser(ADMIN_USER);
|
|
SITE_SERVICE.setMembership(ALTERNATE_LINKS_SITE.getShortName(), TEST_USER, SiteModel.SITE_COLLABORATOR);
|
|
AuthenticationUtil.setFullyAuthenticatedUser(TEST_USER);
|
|
return null;
|
|
}
|
|
});
|
|
|
|
results = LINKS_SERVICE.listLinks(LINKS_SITE.getShortName(), paging);
|
|
assertEquals(2, results.getPage().size());
|
|
results = LINKS_SERVICE.listLinks(ALTERNATE_LINKS_SITE.getShortName(), paging);
|
|
assertEquals(3, results.getPage().size());
|
|
|
|
|
|
// Explicitly remove their permissions from one node, check it vanishes from the list
|
|
PERMISSION_SERVICE.setInheritParentPermissions(linkPrivC.getNodeRef(), false);
|
|
PERMISSION_SERVICE.clearPermission(linkPrivC.getNodeRef(), TEST_USER);
|
|
|
|
results = LINKS_SERVICE.listLinks(LINKS_SITE.getShortName(), paging);
|
|
assertEquals(2, results.getPage().size());
|
|
results = LINKS_SERVICE.listLinks(ALTERNATE_LINKS_SITE.getShortName(), paging);
|
|
assertEquals(2, results.getPage().size());
|
|
|
|
|
|
// Leave, they go away again
|
|
TRANSACTION_HELPER.doInTransaction(new RetryingTransactionHelper.RetryingTransactionCallback<Void>()
|
|
{
|
|
@Override
|
|
public Void execute() throws Throwable
|
|
{
|
|
AuthenticationUtil.setFullyAuthenticatedUser(ADMIN_USER);
|
|
SITE_SERVICE.removeMembership(ALTERNATE_LINKS_SITE.getShortName(), TEST_USER);
|
|
AuthenticationUtil.setFullyAuthenticatedUser(TEST_USER);
|
|
return null;
|
|
}
|
|
});
|
|
|
|
results = LINKS_SERVICE.listLinks(LINKS_SITE.getShortName(), paging);
|
|
assertEquals(2, results.getPage().size());
|
|
results = LINKS_SERVICE.listLinks(ALTERNATE_LINKS_SITE.getShortName(), paging);
|
|
assertEquals(0, results.getPage().size());
|
|
|
|
|
|
// Tidy
|
|
paging = new PagingRequest(10);
|
|
AuthenticationUtil.setFullyAuthenticatedUser(ADMIN_USER);
|
|
results = LINKS_SERVICE.listLinks(LINKS_SITE.getShortName(), paging);
|
|
for (LinkInfo link : results.getPage())
|
|
{
|
|
PUBLIC_NODE_SERVICE.deleteNode(link.getNodeRef());
|
|
}
|
|
results = LINKS_SERVICE.listLinks(ALTERNATE_LINKS_SITE.getShortName(), paging);
|
|
for (LinkInfo link : results.getPage())
|
|
{
|
|
PUBLIC_NODE_SERVICE.deleteNode(link.getNodeRef());
|
|
}
|
|
}
|
|
|
|
/**
|
|
* The lucene based searching
|
|
*/
|
|
@Test public void linksSearching() throws Exception
|
|
{
|
|
AuthenticationUtil.setFullyAuthenticatedUser(TEST_USER);
|
|
PagingRequest paging = new PagingRequest(10);
|
|
PagingResults<LinkInfo> results;
|
|
|
|
final String TAG_1 = "link_tag_1";
|
|
final String TAG_2 = "link_tag_2";
|
|
final String TAG_3 = "link_tag_3";
|
|
final String TAG_4 = "link_tag_4";
|
|
|
|
// Nothing to start with in either site
|
|
results = LINKS_SERVICE.findLinks(LINKS_SITE.getShortName(), null, null, null, null, paging);
|
|
assertEquals(0, results.getPage().size());
|
|
results = LINKS_SERVICE.findLinks(ALTERNATE_LINKS_SITE.getShortName(), null, null, null, null, paging);
|
|
assertEquals(0, results.getPage().size());
|
|
|
|
|
|
// Add some entries to the two sites
|
|
LinkInfo linkA = LINKS_SERVICE.createLink(
|
|
LINKS_SITE.getShortName(), "TitleA", "Description",
|
|
"http://www.alfresco.com/", false);
|
|
LinkInfo linkB = LINKS_SERVICE.createLink(
|
|
LINKS_SITE.getShortName(), "TitleB", "Description",
|
|
"http://www.alfresco.com/", false);
|
|
testNodesToTidy.add(linkA.getNodeRef());
|
|
testNodesToTidy.add(linkB.getNodeRef());
|
|
|
|
AuthenticationUtil.setFullyAuthenticatedUser(ADMIN_USER);
|
|
LinkInfo linkPrivA = LINKS_SERVICE.createLink(
|
|
ALTERNATE_LINKS_SITE.getShortName(), "PrivTitleA", "Description",
|
|
"http://team.alfresco.com/", false);
|
|
LinkInfo linkPrivB = LINKS_SERVICE.createLink(
|
|
ALTERNATE_LINKS_SITE.getShortName(), "PrivTitleB", "Description",
|
|
"http://team.alfresco.com/", false);
|
|
LinkInfo linkPrivC = LINKS_SERVICE.createLink(
|
|
ALTERNATE_LINKS_SITE.getShortName(), "PrivTitleC", "Description",
|
|
"http://team.alfresco.com/", false);
|
|
testNodesToTidy.add(linkPrivA.getNodeRef());
|
|
testNodesToTidy.add(linkPrivB.getNodeRef());
|
|
testNodesToTidy.add(linkPrivC.getNodeRef());
|
|
AuthenticationUtil.setFullyAuthenticatedUser(TEST_USER);
|
|
|
|
|
|
// Check again, won't be able to see the alternate site ones yet
|
|
results = LINKS_SERVICE.findLinks(LINKS_SITE.getShortName(), null, null, null, null, paging);
|
|
assertEquals(2, results.getPage().size());
|
|
results = LINKS_SERVICE.findLinks(ALTERNATE_LINKS_SITE.getShortName(), null, null, null, null, paging);
|
|
assertEquals(0, results.getPage().size());
|
|
|
|
|
|
// Now join the site, will be allowed to see things
|
|
TRANSACTION_HELPER.doInTransaction(new RetryingTransactionHelper.RetryingTransactionCallback<Void>()
|
|
{
|
|
@Override
|
|
public Void execute() throws Throwable
|
|
{
|
|
AuthenticationUtil.setFullyAuthenticatedUser(ADMIN_USER);
|
|
SITE_SERVICE.setMembership(ALTERNATE_LINKS_SITE.getShortName(), TEST_USER, SiteModel.SITE_COLLABORATOR);
|
|
AuthenticationUtil.setFullyAuthenticatedUser(TEST_USER);
|
|
return null;
|
|
}
|
|
});
|
|
|
|
results = LINKS_SERVICE.findLinks(LINKS_SITE.getShortName(), null, null, null, null, paging);
|
|
assertEquals(2, results.getPage().size());
|
|
results = LINKS_SERVICE.findLinks(ALTERNATE_LINKS_SITE.getShortName(), null, null, null, null, paging);
|
|
assertEquals(3, results.getPage().size());
|
|
|
|
|
|
// Alter some of the links' created dates
|
|
pushCreatedDateBack(linkB, 10);
|
|
pushCreatedDateBack(linkPrivB, 10);
|
|
pushCreatedDateBack(linkPrivC, 100);
|
|
|
|
// Wait for everything to catch up (lucene indexing etc)
|
|
Thread.sleep(100);
|
|
|
|
|
|
// Add tags and another entry
|
|
linkA.getTags().add(TAG_1);
|
|
linkA.getTags().add(TAG_2);
|
|
LINKS_SERVICE.updateLink(linkA);
|
|
linkPrivA.getTags().add(TAG_1);
|
|
linkPrivA.getTags().add(TAG_3);
|
|
LINKS_SERVICE.updateLink(linkPrivA);
|
|
linkPrivB.getTags().add(TAG_1);
|
|
LINKS_SERVICE.updateLink(linkPrivB);
|
|
|
|
LinkInfo linkPrivD = LINKS_SERVICE.createLink(
|
|
ALTERNATE_LINKS_SITE.getShortName(), "PrivTitleD", "Description",
|
|
"http://team.alfresco.com/", false);
|
|
testNodesToTidy.add(linkPrivD.getNodeRef());
|
|
|
|
|
|
// Filter by username
|
|
results = LINKS_SERVICE.findLinks(LINKS_SITE.getShortName(), TEST_USER, null, null, null, paging);
|
|
assertEquals(2, results.getPage().size());
|
|
results = LINKS_SERVICE.findLinks(ALTERNATE_LINKS_SITE.getShortName(), TEST_USER, null, null, null, paging);
|
|
assertEquals(1, results.getPage().size());
|
|
|
|
results = LINKS_SERVICE.findLinks(LINKS_SITE.getShortName(), ADMIN_USER, null, null, null, paging);
|
|
assertEquals(0, results.getPage().size());
|
|
results = LINKS_SERVICE.findLinks(ALTERNATE_LINKS_SITE.getShortName(), ADMIN_USER, null, null, null, paging);
|
|
assertEquals(3, results.getPage().size());
|
|
|
|
|
|
// Filter by date
|
|
Date today = new Date();
|
|
Date tomorrow = new Date(today.getTime()+ONE_DAY_MS);
|
|
Date yesterday = new Date(today.getTime()-ONE_DAY_MS);
|
|
Date twoWeeksAgo = new Date(today.getTime()-14*ONE_DAY_MS);
|
|
|
|
// The A's are on today, as is PrivD
|
|
results = LINKS_SERVICE.findLinks(LINKS_SITE.getShortName(), null, yesterday, tomorrow, null, paging);
|
|
assertEquals(1, results.getPage().size());
|
|
results = LINKS_SERVICE.findLinks(ALTERNATE_LINKS_SITE.getShortName(), null, yesterday, tomorrow, null, paging);
|
|
assertEquals(2, results.getPage().size());
|
|
|
|
// The B's are 10 days ago
|
|
results = LINKS_SERVICE.findLinks(LINKS_SITE.getShortName(), null, twoWeeksAgo, yesterday, null, paging);
|
|
assertEquals(1, results.getPage().size());
|
|
results = LINKS_SERVICE.findLinks(ALTERNATE_LINKS_SITE.getShortName(), null, twoWeeksAgo, yesterday, null, paging);
|
|
assertEquals(1, results.getPage().size());
|
|
|
|
// Get both A's, B's and D, but not C as it's 100 days ago
|
|
results = LINKS_SERVICE.findLinks(LINKS_SITE.getShortName(), null, twoWeeksAgo, tomorrow, null, paging);
|
|
assertEquals(2, results.getPage().size());
|
|
results = LINKS_SERVICE.findLinks(ALTERNATE_LINKS_SITE.getShortName(), null, twoWeeksAgo, tomorrow, null, paging);
|
|
assertEquals(3, results.getPage().size());
|
|
|
|
|
|
// Filter by tag
|
|
// Tag1 is on A, PrivA and PrivB
|
|
results = LINKS_SERVICE.findLinks(LINKS_SITE.getShortName(), null, null, null, TAG_1, paging);
|
|
assertEquals(1, results.getPage().size());
|
|
results = LINKS_SERVICE.findLinks(ALTERNATE_LINKS_SITE.getShortName(), null, null, null, TAG_1, paging);
|
|
assertEquals(2, results.getPage().size());
|
|
|
|
// Tag2 is on A
|
|
results = LINKS_SERVICE.findLinks(LINKS_SITE.getShortName(), null, null, null, TAG_2, paging);
|
|
assertEquals(1, results.getPage().size());
|
|
results = LINKS_SERVICE.findLinks(ALTERNATE_LINKS_SITE.getShortName(), null, null, null, TAG_2, paging);
|
|
assertEquals(0, results.getPage().size());
|
|
|
|
// Tag3 is on PrivA
|
|
results = LINKS_SERVICE.findLinks(LINKS_SITE.getShortName(), null, null, null, TAG_3, paging);
|
|
assertEquals(0, results.getPage().size());
|
|
results = LINKS_SERVICE.findLinks(ALTERNATE_LINKS_SITE.getShortName(), null, null, null, TAG_3, paging);
|
|
assertEquals(1, results.getPage().size());
|
|
|
|
// Tag4 isn't on anything
|
|
results = LINKS_SERVICE.findLinks(LINKS_SITE.getShortName(), null, null, null, TAG_4, paging);
|
|
assertEquals(0, results.getPage().size());
|
|
results = LINKS_SERVICE.findLinks(ALTERNATE_LINKS_SITE.getShortName(), null, null, null, TAG_4, paging);
|
|
assertEquals(0, results.getPage().size());
|
|
|
|
|
|
// Filter by username and date and tag
|
|
results = LINKS_SERVICE.findLinks(LINKS_SITE.getShortName(), ADMIN_USER, yesterday, tomorrow, TAG_1, paging);
|
|
assertEquals(0, results.getPage().size()); // Wrong user
|
|
results = LINKS_SERVICE.findLinks(ALTERNATE_LINKS_SITE.getShortName(), ADMIN_USER, yesterday, tomorrow, TAG_1, paging);
|
|
assertEquals(1, results.getPage().size()); // Only PrivA
|
|
}
|
|
|
|
|
|
// --------------------------------------------------------------------------------
|
|
|
|
|
|
/**
|
|
* Alters the created date on a link entry for testing
|
|
*/
|
|
private void pushCreatedDateBack(LinkInfo link, int daysAgo) throws Exception
|
|
{
|
|
final NodeRef node = link.getNodeRef();
|
|
|
|
final Date created = link.getCreatedAt();
|
|
final Date newCreated = new Date(created.getTime() - daysAgo*ONE_DAY_MS);
|
|
|
|
// Update the created date
|
|
TRANSACTION_HELPER.doInTransaction(new RetryingTransactionCallback<Void>()
|
|
{
|
|
@Override
|
|
public Void execute() throws Throwable
|
|
{
|
|
BEHAVIOUR_FILTER.disableBehaviour(ContentModel.ASPECT_AUDITABLE);
|
|
NODE_SERVICE.setProperty(node, ContentModel.PROP_CREATED, newCreated);
|
|
return null;
|
|
}
|
|
}, false, true);
|
|
|
|
// Change something else too in the public nodeservice, to force a re-index
|
|
TRANSACTION_HELPER.doInTransaction(new RetryingTransactionCallback<Void>()
|
|
{
|
|
@Override
|
|
public Void execute() throws Throwable
|
|
{
|
|
BEHAVIOUR_FILTER.disableBehaviour(ContentModel.ASPECT_AUDITABLE);
|
|
PUBLIC_NODE_SERVICE.setProperty(node, ContentModel.PROP_CREATED, newCreated);
|
|
PUBLIC_NODE_SERVICE.setProperty(node, ContentModel.PROP_DESCRIPTION, "Forced Change");
|
|
return null;
|
|
}
|
|
}, false, true);
|
|
|
|
// Check it was taken
|
|
TRANSACTION_HELPER.doInTransaction(new RetryingTransactionCallback<Void>()
|
|
{
|
|
@Override
|
|
public Void execute() throws Throwable
|
|
{
|
|
assertEquals(newCreated, NODE_SERVICE.getProperty(node, ContentModel.PROP_CREATED));
|
|
assertEquals(newCreated, PUBLIC_NODE_SERVICE.getProperty(node, ContentModel.PROP_CREATED));
|
|
return null;
|
|
}
|
|
}, false, true);
|
|
|
|
// Update the object itself
|
|
((LinkInfoImpl)link).setCreatedAt(newCreated);
|
|
}
|
|
|
|
private static void createTestSites() throws Exception
|
|
{
|
|
final LinksServiceImpl privateCalendarService = (LinksServiceImpl)testContext.getBean("linksService");
|
|
|
|
LINKS_SITE = TRANSACTION_HELPER.doInTransaction(new RetryingTransactionHelper.RetryingTransactionCallback<SiteInfo>()
|
|
{
|
|
@Override
|
|
public SiteInfo execute() throws Throwable
|
|
{
|
|
SiteInfo site = SITE_SERVICE.createSite(
|
|
TEST_SITE_PREFIX,
|
|
LinksServiceImplTest.class.getSimpleName() + "_testSite" + System.currentTimeMillis(),
|
|
"test site title", "test site description",
|
|
SiteVisibility.PUBLIC);
|
|
privateCalendarService.getSiteLinksContainer(site.getShortName(), true);
|
|
CLASS_TEST_NODES_TO_TIDY.add(site.getNodeRef());
|
|
return site;
|
|
}
|
|
});
|
|
|
|
// Create the alternate site as admin
|
|
AuthenticationUtil.setFullyAuthenticatedUser(ADMIN_USER);
|
|
ALTERNATE_LINKS_SITE = TRANSACTION_HELPER.doInTransaction(new RetryingTransactionHelper.RetryingTransactionCallback<SiteInfo>()
|
|
{
|
|
@Override
|
|
public SiteInfo execute() throws Throwable
|
|
{
|
|
SiteInfo site = SITE_SERVICE.createSite(
|
|
TEST_SITE_PREFIX,
|
|
LinksServiceImplTest.class.getSimpleName() + "_testAltSite" + System.currentTimeMillis(),
|
|
"alternate site title", "alternate site description",
|
|
SiteVisibility.PRIVATE);
|
|
privateCalendarService.getSiteLinksContainer(site.getShortName(), true);
|
|
CLASS_TEST_NODES_TO_TIDY.add(site.getNodeRef());
|
|
return site;
|
|
}
|
|
});
|
|
AuthenticationUtil.setFullyAuthenticatedUser(TEST_USER);
|
|
}
|
|
|
|
/**
|
|
* By default, all tests are run as the admin user.
|
|
*/
|
|
@Before public void setAdminUser()
|
|
{
|
|
AuthenticationUtil.setFullyAuthenticatedUser(ADMIN_USER);
|
|
}
|
|
|
|
@After public void deleteTestNodes() throws Exception
|
|
{
|
|
performDeletionOfNodes(testNodesToTidy);
|
|
}
|
|
|
|
@AfterClass public static void deleteClassTestNodesAndUsers() throws Exception
|
|
{
|
|
performDeletionOfNodes(CLASS_TEST_NODES_TO_TIDY);
|
|
deleteUser(TEST_USER);
|
|
}
|
|
|
|
/**
|
|
* Deletes the specified NodeRefs, if they exist.
|
|
* @param nodesToDelete
|
|
*/
|
|
private static void performDeletionOfNodes(final List<NodeRef> nodesToDelete)
|
|
{
|
|
TRANSACTION_HELPER.doInTransaction(new RetryingTransactionHelper.RetryingTransactionCallback<Void>()
|
|
{
|
|
@Override
|
|
public Void execute() throws Throwable
|
|
{
|
|
AuthenticationUtil.setFullyAuthenticatedUser(ADMIN_USER);
|
|
|
|
for (NodeRef node : nodesToDelete)
|
|
{
|
|
if (NODE_SERVICE.exists(node))
|
|
{
|
|
// st:site nodes can only be deleted via the SiteService
|
|
if (NODE_SERVICE.getType(node).equals(SiteModel.TYPE_SITE))
|
|
{
|
|
|
|
SiteInfo siteInfo = SITE_SERVICE.getSite(node);
|
|
SITE_SERVICE.deleteSite(siteInfo.getShortName());
|
|
}
|
|
else
|
|
{
|
|
NODE_SERVICE.deleteNode(node);
|
|
}
|
|
}
|
|
}
|
|
|
|
return null;
|
|
}
|
|
});
|
|
}
|
|
|
|
private static void createUser(final String userName)
|
|
{
|
|
TRANSACTION_HELPER.doInTransaction(new RetryingTransactionHelper.RetryingTransactionCallback<Void>()
|
|
{
|
|
@Override
|
|
public Void execute() throws Throwable
|
|
{
|
|
if (!AUTHENTICATION_SERVICE.authenticationExists(userName))
|
|
{
|
|
AUTHENTICATION_SERVICE.createAuthentication(userName, "PWD".toCharArray());
|
|
}
|
|
|
|
if (!PERSON_SERVICE.personExists(userName))
|
|
{
|
|
PropertyMap ppOne = new PropertyMap();
|
|
ppOne.put(ContentModel.PROP_USERNAME, userName);
|
|
ppOne.put(ContentModel.PROP_FIRSTNAME, "firstName");
|
|
ppOne.put(ContentModel.PROP_LASTNAME, "lastName");
|
|
ppOne.put(ContentModel.PROP_EMAIL, "email@email.com");
|
|
ppOne.put(ContentModel.PROP_JOBTITLE, "jobTitle");
|
|
|
|
PERSON_SERVICE.createPerson(ppOne);
|
|
}
|
|
|
|
return null;
|
|
}
|
|
});
|
|
}
|
|
|
|
private static void deleteUser(final String userName)
|
|
{
|
|
TRANSACTION_HELPER.doInTransaction(new RetryingTransactionHelper.RetryingTransactionCallback<Void>()
|
|
{
|
|
@Override
|
|
public Void execute() throws Throwable
|
|
{
|
|
if (PERSON_SERVICE.personExists(userName))
|
|
{
|
|
PERSON_SERVICE.deletePerson(userName);
|
|
}
|
|
|
|
return null;
|
|
}
|
|
});
|
|
}
|
|
}
|