From af2e069b2eabcd5433cee39d83ec06bad6fc69a0 Mon Sep 17 00:00:00 2001 From: Alex Mukha Date: Wed, 23 Aug 2017 12:27:15 +0100 Subject: [PATCH] Merge branch 'fix/REPO-2365-GET-/shared-links-returns-403' into 'develop' REPO-2365 : See merge request !19 --- .../repo/quickshare/QuickShareServiceImpl.java | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/src/main/java/org/alfresco/repo/quickshare/QuickShareServiceImpl.java b/src/main/java/org/alfresco/repo/quickshare/QuickShareServiceImpl.java index c8ba3c3b17..9f98f24e59 100644 --- a/src/main/java/org/alfresco/repo/quickshare/QuickShareServiceImpl.java +++ b/src/main/java/org/alfresco/repo/quickshare/QuickShareServiceImpl.java @@ -1009,17 +1009,24 @@ public class QuickShareServiceImpl implements QuickShareService, private String getSiteName(NodeRef nodeRef) { NodeRef parent = nodeService.getPrimaryParent(nodeRef).getParentRef(); + while (parent != null && !nodeService.getType(parent).equals(SiteModel.TYPE_SITE)) { // check that we can read parent name - String parentName = (String) nodeService.getProperty(parent, ContentModel.PROP_NAME); + if (permissionService.hasReadPermission(parent) == AccessStatus.ALLOWED) + { + String parentName = (String) nodeService.getProperty(parent,ContentModel.PROP_NAME); + } + else + { + return null; + } if (nodeService.getPrimaryParent(nodeRef) != null) { parent = nodeService.getPrimaryParent(parent).getParentRef(); } } - if (parent == null) { return null;