From 9f695c33a7b0fb16f3dfa74355144e69fe4513f5 Mon Sep 17 00:00:00 2001
From: ramunteanu <raluca.andreea.munteanu@gmail.com>
Date: Tue, 6 Mar 2018 10:06:46 +0200
Subject: [PATCH] REPO-3344: Web Api Parameter `required` is not being used
 (#49)

* REPO-3344: Web Api Parameter `required` is not being used
   - Added condition that checks if a body is optional and not provided
---
 .../framework/webscripts/ResourceWebScriptPost.java   | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/main/java/org/alfresco/rest/framework/webscripts/ResourceWebScriptPost.java b/src/main/java/org/alfresco/rest/framework/webscripts/ResourceWebScriptPost.java
index 9671af65eb..98b5f09175 100644
--- a/src/main/java/org/alfresco/rest/framework/webscripts/ResourceWebScriptPost.java
+++ b/src/main/java/org/alfresco/rest/framework/webscripts/ResourceWebScriptPost.java
@@ -171,7 +171,16 @@ public class ResourceWebScriptPost extends AbstractResourceWebScript implements
                         Object jsonContent = null;
                         if (objType != null)
                         {
-                            jsonContent = extractJsonContent(req, assistant.getJsonHelper(), objType);
+                            // check if the body is optional and is not provided
+                            if (!resourceParameter.isRequired() && Integer.valueOf(req.getHeader("content-length")) <= 0)
+                            {
+                                // in some cases the body is optional and the json doesn't need to be extracted
+                                return null;
+                            }
+                            else
+                            {
+                                jsonContent = extractJsonContent(req, assistant.getJsonHelper(), objType);
+                            }
                         }
                         
                         if (isTypeOperation)