diff --git a/source/java/org/alfresco/rest/api/impl/NodesImpl.java b/source/java/org/alfresco/rest/api/impl/NodesImpl.java index 8492aaf24d..97ebb6bd13 100644 --- a/source/java/org/alfresco/rest/api/impl/NodesImpl.java +++ b/source/java/org/alfresco/rest/api/impl/NodesImpl.java @@ -131,6 +131,9 @@ public class NodesImpl implements Nodes private final static String PARAM_RELATIVE_PATH = "relativePath"; // TODO wip private final static String PARAM_SELECT_PROPERTIES = "properties"; + private final static String PARAM_SELECT_PATH = "path"; + private final static String PARAM_SELECT_ASPECTNAMES = "aspectNames"; + private final static String PARAM_SELECT_ISLINK = "isLink"; // TODO ... private NodeService nodeService; private DictionaryService dictionaryService; @@ -512,18 +515,24 @@ public class NodesImpl implements Nodes QName typeQName = nodeService.getType(nodeRef); - List selectParam = parameters.getSelectedProperties(); - return getFolderOrDocument(nodeRef, getParentNodeRef(nodeRef), typeQName, selectParam, false, null); + List selectParam = new ArrayList<>(); + selectParam.addAll(parameters.getSelectedProperties()); + + // Add basic info for single get (above & beyond minimal that is used for listing collections) + selectParam.add(PARAM_SELECT_ASPECTNAMES); + selectParam.add(PARAM_SELECT_PROPERTIES); + + return getFolderOrDocument(nodeRef, getParentNodeRef(nodeRef), typeQName, selectParam, null); } - private Node getFolderOrDocument(final NodeRef nodeRef, NodeRef parentNodeRef, QName typeQName, List selectParam, boolean minimalInfo, Map mapUserInfo) + private Node getFolderOrDocument(final NodeRef nodeRef, NodeRef parentNodeRef, QName typeQName, List selectParam, Map mapUserInfo) { if (mapUserInfo == null) { mapUserInfo = new HashMap<>(2); } PathInfo pathInfo = null; - if (!minimalInfo) + if (selectParam.contains(PARAM_SELECT_PATH)) { pathInfo = lookupPathInfo(nodeRef); } @@ -547,9 +556,13 @@ public class NodesImpl implements Nodes throw new InvalidArgumentException("Node is not a folder or file"); } - if (! minimalInfo) + if (selectParam.size() > 0) { node.setProperties(mapFromNodeProperties(properties, selectParam, mapUserInfo)); + } + + if (selectParam.contains(PARAM_SELECT_ASPECTNAMES)) + { node.setAspectNames(mapFromNodeAspects(nodeService.getAspects(nodeRef))); } @@ -746,7 +759,6 @@ public class NodesImpl implements Nodes final NodeRef parentNodeRef = validateOrLookupNode(parentFolderNodeId, null); final List selectParam = parameters.getSelectedProperties(); - final boolean minimalInfo = (selectParam.size() == 0); boolean includeFolders = true; boolean includeFiles = true; @@ -816,7 +828,7 @@ public class NodesImpl implements Nodes FileInfo fInfo = page.get(index); // minimal info by default (unless "select"ed otherwise) - return getFolderOrDocument(fInfo.getNodeRef(), parentNodeRef, fInfo.getType(), selectParam, minimalInfo, mapUserInfo); + return getFolderOrDocument(fInfo.getNodeRef(), parentNodeRef, fInfo.getType(), selectParam, mapUserInfo); } @Override @@ -1200,7 +1212,7 @@ public class NodesImpl implements Nodes private Node createUploadResponse(NodeRef parentNodeRef, NodeRef newFileNodeRef) { - return getFolderOrDocument(newFileNodeRef, parentNodeRef, ContentModel.TYPE_CONTENT, Collections.emptyList(), true, null); + return getFolderOrDocument(newFileNodeRef, parentNodeRef, ContentModel.TYPE_CONTENT, Collections.emptyList(), null); } private String getStringOrNull(String value)