inteligr8/alfresco-community-repo
1
0
Fork 0
mirror of https://github.com/Alfresco/alfresco-community-repo.git synced 2025-07-31 17:39:05 +00:00
Code Issues Packages Projects Releases Wiki Activity

RM-5611 Code review fixes.

Browse Source 
Remove nested try catch. Add additional check that we don't have duplicate
topics selected.
This commit is contained in:
Tom Page
2017-09-08 14:10:47 +01:00
parent b2ac4f89ee
commit d8f16a990c
1 changed files with 5 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/core/v0/BaseAPI.java
View File

@@ -404,16 +404,12 @@ public abstract class BaseAPI
try try
{ {
String bodyString = EntityUtils.toString(response.getEntity()); responseBody = new JSONObject(EntityUtils.toString(response.getEntity()));
try
{
responseBody = new JSONObject(bodyString);
} }
catch (JSONException error) catch (JSONException error)
{ {
LOGGER.error("Converting message body to JSON failed. Body: {}", responseBody, error); LOGGER.error("Converting message body to JSON failed. Body: {}", responseBody, error);
} }
}
catch (ParseException | IOException error) catch (ParseException | IOException error)
{ {
LOGGER.error("Parsing message body failed.", error); LOGGER.error("Parsing message body failed.", error);