mirror of
https://github.com/Alfresco/alfresco-community-repo.git
synced 2025-08-14 17:58:59 +00:00
Merged 5.2.N (5.2.1) to HEAD (5.2)
129169 mmuller: Merged RETURN-OF-THE-API (5.2.0) to 5.2.N (5.2.1) 128525 jvonka: REPO-874: Improve REST fwk: improve error message if a POST operation is attempted with multiple items - follow-on to fix fallout git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@129343 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
This commit is contained in:
@@ -148,8 +148,9 @@ public class ResourceWebScriptPost extends AbstractResourceWebScript implements
|
||||
{
|
||||
return null;
|
||||
}
|
||||
|
||||
|
||||
Class<?> objType = resourceMeta.getObjectType(operation);
|
||||
boolean isTypeOperation = resourceMeta.getType().equals(ResourceMetadata.RESOURCE_TYPE.OPERATION);
|
||||
List<ResourceParameter> params = operation.getParameters();
|
||||
|
||||
if (!params.isEmpty())
|
||||
@@ -157,7 +158,6 @@ public class ResourceWebScriptPost extends AbstractResourceWebScript implements
|
||||
for (ResourceParameter resourceParameter : params)
|
||||
{
|
||||
// POST to collection may or may not support List as json body, Operations don't support a List as json body
|
||||
boolean isTypeOperation = resourceMeta.getType().equals(ResourceMetadata.RESOURCE_TYPE.OPERATION);
|
||||
boolean notMultiple = ((! resourceParameter.isAllowMultiple()) || isTypeOperation);
|
||||
|
||||
if (ResourceParameter.KIND.HTTP_BODY_OBJECT.equals(resourceParameter.getParamType()) && notMultiple)
|
||||
@@ -168,7 +168,7 @@ public class ResourceWebScriptPost extends AbstractResourceWebScript implements
|
||||
Object jsonContent = null;
|
||||
if (objType != null)
|
||||
{
|
||||
jsonContent = ResourceWebScriptHelper.extractJsonContent(req,assistant.getJsonHelper(), objType);
|
||||
jsonContent = ResourceWebScriptHelper.extractJsonContent(req, assistant.getJsonHelper(), objType);
|
||||
}
|
||||
|
||||
if (isTypeOperation)
|
||||
@@ -194,7 +194,21 @@ public class ResourceWebScriptPost extends AbstractResourceWebScript implements
|
||||
}
|
||||
}
|
||||
}
|
||||
return ResourceWebScriptHelper.extractJsonContentAsList(req, assistant.getJsonHelper(), objType);
|
||||
|
||||
if (objType == null)
|
||||
{
|
||||
return null;
|
||||
}
|
||||
|
||||
if (isTypeOperation)
|
||||
{
|
||||
// Operations don't support a List as json body
|
||||
return ResourceWebScriptHelper.extractJsonContent(req, assistant.getJsonHelper(), objType);
|
||||
}
|
||||
else
|
||||
{
|
||||
return ResourceWebScriptHelper.extractJsonContentAsList(req, assistant.getJsonHelper(), objType);
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
|
Reference in New Issue
Block a user