inteligr8/alfresco-community-repo
1
0
Fork 0
mirror of https://github.com/Alfresco/alfresco-community-repo.git synced 2025-07-31 17:39:05 +00:00
Code Issues Packages Projects Releases Wiki Activity

RM-7068 updates from review

Browse Source
This commit is contained in:
Sara Aspery
2019-11-28 17:07:20 +00:00
parent fe502aadec
commit 976748ee0f
2 changed files with 2 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/hold/HoldServiceImpl.java
View File

@@ -552,14 +552,7 @@ public class HoldServiceImpl extends ServiceBaseImpl
*/ */
private void checkPermissionsForDeleteHold(NodeRef hold) private void checkPermissionsForDeleteHold(NodeRef hold)
{ {
List<NodeRef> held = AuthenticationUtil.runAsSystem(new RunAsWork<List<NodeRef>>() List<NodeRef> held = AuthenticationUtil.runAsSystem(() -> getHeld(hold));
{
@Override
public List<NodeRef> doWork()
{
return getHeld(hold);
}
});
List<String> heldNames = new ArrayList<>(); List<String> heldNames = new ArrayList<>();
for (NodeRef nodeRef : held) for (NodeRef nodeRef : held)

2
rm-community/rm-community-repo/unit-test/java/org/alfresco/module/org_alfresco_module_rm/hold/HoldServiceImplUnitTest.java
View File

@@ -320,7 +320,7 @@ public class HoldServiceImplUnitTest extends BaseUnitTest
// TODO check interactions with policy component!!! // TODO check interactions with policy component!!!
} }
@Test (expected = AlfrescoRuntimeException.class) @Test (expected = AccessDeniedException.class)
public void deleteHoldNoPermissionsOnContent() public void deleteHoldNoPermissionsOnContent()
{ {
mockPoliciesForDeleteHold(); mockPoliciesForDeleteHold();