diff --git a/config/alfresco/public-rest-context.xml b/config/alfresco/public-rest-context.xml index 087344e713..c4ccbdc933 100644 --- a/config/alfresco/public-rest-context.xml +++ b/config/alfresco/public-rest-context.xml @@ -146,6 +146,7 @@ + diff --git a/source/java/org/alfresco/rest/api/impl/NodesImpl.java b/source/java/org/alfresco/rest/api/impl/NodesImpl.java index 0d089d8365..29e77875b8 100644 --- a/source/java/org/alfresco/rest/api/impl/NodesImpl.java +++ b/source/java/org/alfresco/rest/api/impl/NodesImpl.java @@ -1273,8 +1273,6 @@ public class NodesImpl implements Nodes } props.put(ContentModel.PROP_NAME, nodeName); - validatePropValues(props); - QName assocQName = QName.createQName(NamespaceService.CONTENT_MODEL_1_0_URI, QName.createValidLocalName(nodeName)); try { @@ -1301,20 +1299,6 @@ public class NodesImpl implements Nodes } } - // special cases: additional validation of property values (if not done by underlying foundation services) - private void validatePropValues(Map props) - { - String newOwner = (String)props.get(ContentModel.PROP_OWNER); - if (newOwner != null) - { - // validate that user exists - if (! personService.personExists(newOwner)) - { - throw new InvalidArgumentException("Unknown owner: "+newOwner); - } - } - } - @Override public Node updateNode(String nodeId, Node nodeInfo, Parameters parameters) { @@ -1446,8 +1430,6 @@ public class NodesImpl implements Nodes if (props.size() > 0) { - validatePropValues(props); - try { // update node properties - note: null will unset the specified property diff --git a/source/test-java/org/alfresco/rest/api/tests/NodeApiTest.java b/source/test-java/org/alfresco/rest/api/tests/NodeApiTest.java index bd711fce63..517656dd27 100644 --- a/source/test-java/org/alfresco/rest/api/tests/NodeApiTest.java +++ b/source/test-java/org/alfresco/rest/api/tests/NodeApiTest.java @@ -2010,7 +2010,7 @@ public class NodeApiTest extends AbstractBaseApiTest dUpdate = new Document(); dUpdate.setProperties(props); - put(URL_NODES, user1, d1Id, toJsonAsStringNonNull(dUpdate), null, 400); + put(URL_NODES, user1, d1Id, toJsonAsStringNonNull(dUpdate), null, 422); AuthenticationUtil.setFullyAuthenticatedUser(user2);