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

Merge pull request #2446 from Alfresco/feature/mkinas-ACS-5506-exception-fix

Browse Source 
[ACS-5506] Fix string to node ref cast exception
This commit is contained in:
MichalKinas
2024-02-13 19:29:26 +01:00
committed by GitHub
parent af69c12be3 e3407e5a53
commit e6e05c8890
2 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
remote-api/src/main/java/org/alfresco/rest/api/impl/GroupsImpl.java
View File

@@ -147,7 +147,7 @@ public class GroupsImpl implements Groups
}
Map<QName, Serializable> props = new HashMap<>();
if (StringUtils.isNotEmpty(group.getDescription()))
if (group.getDescription() != null)
{
props.put(ContentModel.PROP_DESCRIPTION, group.getDescription());
}
@@ -170,7 +170,7 @@ public class GroupsImpl implements Groups
try
{
if (StringUtils.isNotEmpty(group.getDescription()))
if (group.getDescription() != null)
{
authorityService.setAuthorityDisplayNameAndDescription(groupId, group.getDisplayName(), group.getDescription());
}

5
repository/src/main/java/org/alfresco/repo/security/permissions/impl/acegi/ACLEntryAfterInvocationProvider.java
View File

@@ -424,6 +424,11 @@ public class ACLEntryAfterInvocationProvider implements AfterInvocationProvider,
@SuppressWarnings("rawtypes")
private Pair decide(Authentication authentication, Object object, ConfigAttributeDefinition config, Pair returnedObject) throws AccessDeniedException
{
if (returnedObject.getSecond() != null && !NodeRef.class.isAssignableFrom(returnedObject.getSecond().getClass()))
{
return returnedObject;
}
NodeRef nodeRef = (NodeRef) returnedObject.getSecond();
decide(authentication, object, config, nodeRef);
// the noderef was allowed