code.onehippo.org is currently readonly. We are migrating to code.bloomreach.com, please continue working there on Monday 14/12. See: https://docs.bloomreach.com/display/engineering/GitLab

Commit 7509c078 authored by Ate Douma's avatar Ate Douma

CMS7-7024: better let service clients deal with non-existing or non-accessible Node exceptions

parent 20d764b3
......@@ -24,10 +24,8 @@ import java.util.Map;
import java.util.Set;
import java.util.TreeSet;
import javax.jcr.ItemNotFoundException;
import javax.jcr.Node;
import javax.jcr.NodeIterator;
import javax.jcr.PathNotFoundException;
import javax.jcr.Property;
import javax.jcr.PropertyIterator;
import javax.jcr.PropertyType;
......@@ -139,22 +137,12 @@ public class HippoContentTypeService implements ContentTypeService {
@Override
public DocumentType getDocumentType(String uuid, Session session) throws RepositoryException {
try {
return getDocumentType(session.getNodeByIdentifier(uuid));
}
catch (ItemNotFoundException inf) {
return null;
}
return getDocumentType(session.getNodeByIdentifier(uuid));
}
@Override
public DocumentType getDocumentType(Session session, String path) throws RepositoryException {
try {
return getDocumentType(session.getNode(path));
}
catch (PathNotFoundException inf) {
return null;
}
return getDocumentType(session.getNode(path));
}
private EffectiveNodeTypesCache loadEffectiveNodeTypes(Session session, boolean allowRetry) throws RepositoryException {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment