From a6fb2f1cbca4ccc7db32352ae385add8ddc38cf6 Mon Sep 17 00:00:00 2001
From: cagache <claudia.agache@ness.com>
Date: Mon, 20 May 2019 16:58:35 +0300
Subject: [PATCH] addressed code review comments

---
 .../main/java/org/alfresco/rest/v0/RMRolesAndActionsAPI.java   | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/v0/RMRolesAndActionsAPI.java b/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/v0/RMRolesAndActionsAPI.java
index 94e0b25269..2a98261c7e 100644
--- a/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/v0/RMRolesAndActionsAPI.java
+++ b/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/v0/RMRolesAndActionsAPI.java
@@ -465,8 +465,7 @@ public class RMRolesAndActionsAPI extends BaseAPI
     public HttpResponse createHold(String user, String password, String holdName, String reason, String description)
     {
         // if the hold already exists don't try to create it again
-        final String holdsContainerPath = Utility.buildPath(getFilePlanPath(), HOLDS_CONTAINER);
-        final String fullHoldPath = holdsContainerPath + holdName;
+        final String fullHoldPath = Utility.buildPath(getFilePlanPath(), HOLDS_CONTAINER) + holdName;
         final CmisObject hold = getObjectByPath(user, password, fullHoldPath);
         if (hold != null)
         {