[maven-release-plugin][skip ci] prepare release 23.6.0.18

backport MNT-24776 to 23.N

.github/workflows/ci.yml

@@ -42,17 +42,16 @@ jobs:
       !contains(github.event.head_commit.message, '[force')
     steps:
       - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
       - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.16.0
       - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.16.0
       - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.16.0
-      - uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
       - id: changed-files
-        uses: Alfresco/alfresco-build-tools/.github/actions/github-list-changes@v6.1.0
+        uses: Alfresco/alfresco-build-tools/.github/actions/github-list-changes@v8.16.0
         with:
           write-list-to-env: true
-      - uses: Alfresco/alfresco-build-tools/.github/actions/pre-commit@v6.1.0
+      - uses: Alfresco/alfresco-build-tools/.github/actions/pre-commit@v8.16.0
       - name: "Init"
         run: bash ./scripts/ci/init.sh
       - name: "Prepare maven cache and check compilation"
@@ -146,7 +145,7 @@ jobs:
       - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.16.0
       - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.16.0
       - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.16.0
-      - uses: Alfresco/ya-pmd-scan@v4.0.0
+      - uses: Alfresco/ya-pmd-scan@v4.3.0
         with:
           classpath-build-command: "mvn test-compile -ntp -Pags -pl \"-:alfresco-community-repo-docker\""
 
@@ -267,6 +266,7 @@ jobs:
       - name: "Set transformers tag"
         run: echo "TRANSFORMERS_TAG=$(mvn help:evaluate -Dexpression=dependency.alfresco-transform-core.version -q -DforceStdout)" >> $GITHUB_ENV
       - name: "Set up the environment"
+        timeout-minutes: ${{ fromJSON(env.GITHUB_ACTIONS_DEPLOY_TIMEOUT) }}
         run: docker compose -f ./scripts/ci/docker-compose/docker-compose.yaml --profile ${{ matrix.compose-profile }} up -d
       - name: "Prepare Report Portal"
         if: github.ref_name == 'master'
@@ -875,6 +875,7 @@ jobs:
             echo "HOSTNAME_VERIFICATION_DISABLED=false" >> "$GITHUB_ENV"
           fi
       - name: "Set up the environment"
+        timeout-minutes: ${{ fromJSON(env.GITHUB_ACTIONS_DEPLOY_TIMEOUT) }}
         run: docker compose -f ./scripts/ci/docker-compose/docker-compose.yaml --profile ${{ matrix.compose-profile }} up -d
       - name: "Prepare Report Portal"
         if: github.ref_name == 'master'
@@ -976,6 +977,7 @@ jobs:
           bash ./scripts/ci/init.sh
           bash ./scripts/ci/build.sh
       - name: "Set up the environment"
+        timeout-minutes: ${{ fromJSON(env.GITHUB_ACTIONS_DEPLOY_TIMEOUT) }}
         run: |
           ${{ env.TAS_SCRIPTS }}/start-compose.sh ${{ env.TAS_ENVIRONMENT }}/docker-compose-minimal+transforms.yml
           ${{ env.TAS_SCRIPTS }}/wait-for-alfresco-start.sh "http://localhost:8082/alfresco"

.github/workflows/master_release.yml

@@ -35,7 +35,7 @@ jobs:
         with:
           persist-credentials: false
       - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.16.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.16.0      
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.16.0
       - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.16.0
       - name: "Init"
         run: bash ./scripts/ci/init.sh
@@ -64,7 +64,7 @@ jobs:
         with:
           persist-credentials: false
       - uses: Alfresco/alfresco-build-tools/.github/actions/get-build-info@v8.16.0
-      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.16.0      
+      - uses: Alfresco/alfresco-build-tools/.github/actions/free-hosted-runner-disk-space@v8.16.0
       - uses: Alfresco/alfresco-build-tools/.github/actions/setup-java-build@v8.16.0
       - name: "Init"
         run: bash ./scripts/ci/init.sh

.secrets.baseline

@@ -147,7 +147,7 @@
         "filename": ".github/workflows/ci.yml",
         "hashed_secret": "128f14373ccfaff49e3664045d3a11b50cbb7b39",
         "is_verified": false,
-        "line_number": 901
+        "line_number": 902
       }
     ],
     ".github/workflows/master_release.yml": [
@@ -166,7 +166,7 @@
         "filename": "amps/ags/rm-automation/rm-automation-community-rest-api/src/test/java/org/alfresco/rest/rm/community/base/TestData.java",
         "hashed_secret": "5baa61e4c9b93f3f0682250b6cf8331b7ee68fd8",
         "is_verified": false,
-        "line_number": 58,
+        "line_number": 57,
         "is_secret": false
       }
     ],
@@ -176,7 +176,7 @@
         "filename": "amps/ags/rm-automation/rm-automation-community-rest-api/src/test/java/org/alfresco/rest/rm/community/retentionschedule/RetentionScheduleStepTests.java",
         "hashed_secret": "e3557f3b0f2189660980d213f75d1a7828d56af1",
         "is_verified": false,
-        "line_number": 85,
+        "line_number": 83,
         "is_secret": false
       }
     ],
@@ -731,7 +731,7 @@
         "filename": "remote-api/src/test/java/org/alfresco/rest/api/tests/AbstractBaseApiTest.java",
         "hashed_secret": "d033e22ae348aeb5660fc2140aec35850c4da997",
         "is_verified": false,
-        "line_number": 120,
+        "line_number": 111,
         "is_secret": false
       }
     ],
@@ -1273,7 +1273,7 @@
         "filename": "repository/src/main/resources/alfresco/repository.properties",
         "hashed_secret": "84551ae5442affc9f1a2d3b4c86ae8b24860149d",
         "is_verified": false,
-        "line_number": 770,
+        "line_number": 771,
         "is_secret": false
       }
     ],
@@ -1627,7 +1627,7 @@
         "filename": "repository/src/test/java/org/alfresco/repo/security/authentication/identityservice/SpringBasedIdentityServiceFacadeUnitTest.java",
         "hashed_secret": "5baa61e4c9b93f3f0682250b6cf8331b7ee68fd8",
         "is_verified": false,
-        "line_number": 46,
+        "line_number": 48,
         "is_secret": false
       }
     ],
@@ -1888,5 +1888,5 @@
       }
     ]
   },
-  "generated_at": "2025-03-31T12:42:09Z"
+  "generated_at": "2025-05-15T21:47:13Z"
 }

README.md

@@ -126,4 +126,3 @@ Please use [**this guide**](CONTRIBUTING.md) to make a contribution to the proje
 
 - [Alfresco Content Services Documentation](https://docs.alfresco.com/content-services/latest/)
 - [Alfresco Platform](https://www.hyland.com/en/products/alfresco-platform)
-

amps/ags/pom.xml

@@ -7,7 +7,7 @@
    <parent>
       <groupId>org.alfresco</groupId>
       <artifactId>alfresco-community-repo-amps</artifactId>
-      <version>23.3.7.2</version>
+      <version>23.6.0.18</version>
    </parent>
 
    <modules>

amps/ags/rm-automation/pom.xml

@@ -7,7 +7,7 @@
    <parent>
       <groupId>org.alfresco</groupId>
       <artifactId>alfresco-governance-services-community-parent</artifactId>
-      <version>23.3.7.2</version>
+      <version>23.6.0.18</version>
    </parent>
 
    <modules>

amps/ags/rm-automation/rm-automation-community-rest-api/pom.xml

@@ -7,7 +7,7 @@
    <parent>
       <groupId>org.alfresco</groupId>
       <artifactId>alfresco-governance-services-automation-community-repo</artifactId>
-      <version>23.3.7.2</version>
+      <version>23.6.0.18</version>
    </parent>
 
    <build>
@@ -98,7 +98,7 @@
       <dependency>
          <groupId>com.github.docker-java</groupId>
          <artifactId>docker-java</artifactId>
-         <version>3.3.6</version>
+         <version>3.4.0</version>
          <exclusions>
             <exclusion>
                <groupId>org.bouncycastle</groupId>

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/core/RMRestProperties.java

@@ -26,11 +26,12 @@
  */
 package org.alfresco.rest.core;
 
-import lombok.Getter;
 import org.springframework.beans.factory.annotation.Value;
 import org.springframework.context.annotation.Configuration;
 import org.springframework.context.annotation.PropertySource;
 
+import lombok.Getter;
+
 /**
  * Extends {@link RestProperties} to be able to change/add properties
  *
@@ -44,18 +45,18 @@ import org.springframework.context.annotation.PropertySource;
 @PropertySource(value = "classpath:local.properties", ignoreResourceNotFound = true)
 public class RMRestProperties extends RestProperties
 {
-    @Value("${alfresco.scheme}")
+    @Value ("${alfresco.scheme}")
     private String scheme;
 
-    @Value("${alfresco.server}")
+    @Value ("${alfresco.server}")
     private String server;
 
-    @Value("${alfresco.port}")
+    @Value ("${alfresco.port}")
     private String port;
 
-    @Value("${rest.rmPath}")
+    @Value ("${rest.rmPath}")
     private String restRmPath;
 
-    @Value("${docker.host}")
+    @Value ("${docker.host}")
     private String dockerHost;
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/core/RMRestWrapper.java

@@ -27,11 +27,6 @@
 package org.alfresco.rest.core;
 
 import io.restassured.builder.RequestSpecBuilder;
-import lombok.Getter;
-import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.context.annotation.Scope;
-import org.springframework.http.HttpStatus;
-import org.springframework.stereotype.Service;
 
 import org.alfresco.rest.exception.EmptyJsonResponseException;
 import org.alfresco.rest.model.RestErrorModel;
@@ -43,6 +38,12 @@ import org.alfresco.rest.requests.search.SearchAPI;
 import org.alfresco.rest.rm.community.requests.gscore.GSCoreAPI;
 import org.alfresco.utility.model.StatusModel;
 import org.alfresco.utility.model.UserModel;
+import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.context.annotation.Scope;
+import org.springframework.http.HttpStatus;
+import org.springframework.stereotype.Service;
+
+import lombok.Getter;
 
 /**
  * Extends {@link RestWrapper} in order to call GS APIs with our own properties
@@ -152,18 +153,15 @@ public class RMRestWrapper
     }
 
     /**
-     * You can handle the request sent to server by calling this method. If for example you want to sent multipart form data you can use:
+     * You can handle the request sent to server by calling this method.
-     * 
+     * If for example you want to sent multipart form data you can use: <pre>
-     * <pre>
      * restClient.configureRequestSpec()
-     *         .addMultiPart("filedata", Utility.getResourceTestDataFile("restapi-resource"))
+     *              .addMultiPart("filedata", Utility.getResourceTestDataFile("restapi-resource"))
-     *         .addFormParam("renditions", "doclib")
+     *              .addFormParam("renditions", "doclib")
-     *         .addFormParam("autoRename", true);
+     *              .addFormParam("autoRename", true);
      *
      * restClient.withCoreAPI().usingNode(ContentModel.my()).createNode();
-     * </pre>
+     * </pre> This will create the node using the multipart data defined.
-     * 
-     * This will create the node using the multipart data defined.
      */
     public RequestSpecBuilder configureRequestSpec()
     {
@@ -173,8 +171,7 @@ public class RMRestWrapper
     /**
      * Process a response that returns a html
      *
-     * @throws EmptyJsonResponseException
+     * @throws EmptyJsonResponseException If there is no response from the server.
-     *             If there is no response from the server.
      */
     public RestHtmlResponse processHtmlResponse(RestRequest simpleRequest)
     {

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/core/RestAPIFactory.java

@@ -32,10 +32,6 @@ import jakarta.annotation.Resource;
 
 import lombok.Getter;
 import lombok.Setter;
-import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.context.annotation.Scope;
-import org.springframework.stereotype.Service;
-
 import org.alfresco.rest.requests.Node;
 import org.alfresco.rest.requests.coreAPI.RestCoreAPI;
 import org.alfresco.rest.requests.search.SearchAPI;
@@ -49,14 +45,17 @@ import org.alfresco.rest.rm.community.requests.gscore.api.RMUserAPI;
 import org.alfresco.rest.rm.community.requests.gscore.api.RecordCategoryAPI;
 import org.alfresco.rest.rm.community.requests.gscore.api.RecordFolderAPI;
 import org.alfresco.rest.rm.community.requests.gscore.api.RecordsAPI;
-import org.alfresco.rest.rm.community.requests.gscore.api.RetentionScheduleAPI;
 import org.alfresco.rest.rm.community.requests.gscore.api.TransferAPI;
 import org.alfresco.rest.rm.community.requests.gscore.api.TransferContainerAPI;
 import org.alfresco.rest.rm.community.requests.gscore.api.UnfiledContainerAPI;
 import org.alfresco.rest.rm.community.requests.gscore.api.UnfiledRecordFolderAPI;
+import org.alfresco.rest.rm.community.requests.gscore.api.RetentionScheduleAPI;
 import org.alfresco.utility.data.DataUserAIS;
 import org.alfresco.utility.model.RepoTestModel;
 import org.alfresco.utility.model.UserModel;
+import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.context.annotation.Scope;
+import org.springframework.stereotype.Service;
 
 /**
  * REST API Factory which provides access to the APIs
@@ -69,7 +68,7 @@ import org.alfresco.utility.model.UserModel;
 public class RestAPIFactory
 {
     @Autowired
-    @Getter(value = PROTECTED)
+    @Getter (value = PROTECTED)
     private DataUserAIS dataUser;
 
     @Resource(name = "RMRestWrapper")

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/core/search/SearchRequestBuilder.java

@@ -44,13 +44,11 @@ public class SearchRequestBuilder extends SearchRequest
     {
         new SearchRequest();
     }
-
     /**
      * Set the sql statement for the SearchRequest
      *
-     * @param query
+     * @param query sql statement
-     *            sql statement
+     * @return search  request
-     * @return search request
      */
     public SearchRequestBuilder setQueryBuilder(RestRequestQueryModel query)
     {
@@ -61,8 +59,7 @@ public class SearchRequestBuilder extends SearchRequest
     /**
      * Set the paging statement for the SearchRequest
      *
-     * @param paging
+     * @param paging pagination requested
-     *            pagination requested
      * @return search request
      */
     public SearchRequestBuilder setPagingBuilder(Pagination paging)
@@ -85,8 +82,7 @@ public class SearchRequestBuilder extends SearchRequest
     /**
      * Set the requested fields for the SearchRequest
      *
-     * @param fields
+     * @param fields requested fields
-     *            requested fields
      * @return search request
      */
     public SearchRequestBuilder setFieldsBuilder(List<String> fields)

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/core/v0/APIUtils.java

@@ -26,12 +26,12 @@
  */
 package org.alfresco.rest.core.v0;
 
+import jakarta.json.Json;
+import jakarta.json.JsonReader;
 import java.io.IOException;
 import java.io.InputStream;
 import java.time.format.DateTimeFormatter;
 import java.time.format.DateTimeFormatterBuilder;
-import jakarta.json.Json;
-import jakarta.json.JsonReader;
 
 import org.apache.commons.io.IOUtils;
 import org.apache.http.HttpEntity;
@@ -52,20 +52,20 @@ public class APIUtils
 {
     /** Logger for this class. */
     private static final Logger LOGGER = LoggerFactory.getLogger(APIUtils.class);
-    /**
+    /** The ISO instant formatter that formats or parses an instant in UTC, such as '2011-12-03T10:15:305Z'
-     * The ISO instant formatter that formats or parses an instant in UTC, such as '2011-12-03T10:15:305Z' similar with {@link DateTimeFormatter#ISO_INSTANT}, but with only 3 nanoseconds
+     * similar with {@link DateTimeFormatter#ISO_INSTANT}, but with only 3 nanoseconds*/
-     */
+    public static final DateTimeFormatter ISO_INSTANT_FORMATTER =
-    public static final DateTimeFormatter ISO_INSTANT_FORMATTER = new DateTimeFormatterBuilder().appendInstant(3).toFormatter();
+            new DateTimeFormatterBuilder().appendInstant(3).toFormatter();
 
     /** Private constructor for helper class. */
     private APIUtils()
-    {}
+    {
+    }
 
     /**
      * Extract the body of a HTTP response as a JSON object.
      *
-     * @param httpResponse
+     * @param httpResponse The HTTP response.
-     *            The HTTP response.
      * @return A JSON representation of the object.
      */
     public static JSONObject convertHTTPResponseToJSON(HttpResponse httpResponse)
@@ -86,8 +86,7 @@ public class APIUtils
     /**
      * Util method to extract the message string from the HTTP response
      *
-     * @param httpResponse
+     * @param httpResponse http response
-     *            http response
      * @return error message from the http response
      */
     public static String extractErrorMessageFromHttpResponse(HttpResponse httpResponse)

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/core/v0/BaseAPI.java

@@ -39,6 +39,9 @@ import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 
+import org.alfresco.dataprep.AlfrescoHttpClient;
+import org.alfresco.dataprep.AlfrescoHttpClientFactory;
+import org.alfresco.dataprep.ContentService;
 import org.apache.chemistry.opencmis.client.api.CmisObject;
 import org.apache.chemistry.opencmis.commons.exceptions.CmisObjectNotFoundException;
 import org.apache.http.HttpResponse;
@@ -61,10 +64,6 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.beans.factory.annotation.Autowired;
 
-import org.alfresco.dataprep.AlfrescoHttpClient;
-import org.alfresco.dataprep.AlfrescoHttpClientFactory;
-import org.alfresco.dataprep.ContentService;
-
 /**
  * The base API class containing common methods for making v0 API requests
  *
@@ -99,11 +98,9 @@ public abstract class BaseAPI
     /**
      * Helper method to extract list of properties values from result.
      *
-     * @param result
+     * @param result the response
-     *            the response
      * @return list of specified property values in result
-     * @throws RuntimeException
+     * @throws RuntimeException for malformed response
-     *             for malformed response
      */
     protected List<String> getPropertyValues(JSONObject result, String propertyName)
     {
@@ -127,7 +124,7 @@ public abstract class BaseAPI
 
     /**
      * Helper method to extract the property value for the given nodeRef and property name
-     * 
+     *
      * @param result
      * @param nodeRef
      * @param propertyName
@@ -142,7 +139,7 @@ public abstract class BaseAPI
             for (int i = 0; i < items.length(); i++)
             {
                 JSONObject item = items.getJSONObject(i);
-                if (nodeRef.equals(item.getString("nodeRef")))
+                if(nodeRef.equals(item.getString("nodeRef")))
                 {
                     propertyValue = item.getJSONObject("properties").getString(propertyName);
                 }
@@ -159,11 +156,9 @@ public abstract class BaseAPI
     /**
      * Helper method to extract property values from request result and put them in map as a list that corresponds to a unique property value.
      *
-     * @param requestResult
+     * @param requestResult the request response
-     *            the request response
      * @return a map containing information about multiple properties values that correspond to a unique one
-     * @throws RuntimeException
+     * @throws RuntimeException for malformed response
-     *             for malformed response
      */
     protected Map<String, List<String>> getPropertyValuesByUniquePropertyValue(JSONObject requestResult, String uniqueProperty, List<String> otherProperties)
     {
@@ -194,12 +189,11 @@ public abstract class BaseAPI
     /**
      * Retrieves the nodeRef of an item (category, folder or record) with the given path
      *
-     * @param username
+     * @param username the username
-     *            the username
+     * @param password the password
-     * @param password
+     * @param path     the path to the container eg. in case of a category it would be the category name,
-     *            the password
+     *                 in case of a folder it would be /categoryName/folderName
-     * @param path
+     *                 when trying to get File Plan, the path would be ""
-     *            the path to the container eg. in case of a category it would be the category name, in case of a folder it would be /categoryName/folderName when trying to get File Plan, the path would be ""
      * @return the container nodeRef
      */
     public String getItemNodeRef(String username, String password, String path)
@@ -210,12 +204,9 @@ public abstract class BaseAPI
     /**
      * Retrieve a Cmis object by its path
      *
-     * @param username
+     * @param username the user's username
-     *            the user's username
+     * @param password its password
-     * @param password
+     * @param path     the object path
-     *            its password
-     * @param path
-     *            the object path
      * @return the object in case it exists, null if its does not exist
      */
     protected CmisObject getObjectByPath(String username, String password, String path)
@@ -224,8 +215,7 @@ public abstract class BaseAPI
         try
         {
             object = contentService.getCMISSession(username, password).getObjectByPath(path);
-        }
+        } catch (CmisObjectNotFoundException notFoundError)
-        catch (CmisObjectNotFoundException notFoundError)
         {
             return null;
         }
@@ -235,12 +225,9 @@ public abstract class BaseAPI
     /**
      * Generic faceted request.
      *
-     * @param username
+     * @param username the username
-     *            the username
+     * @param password the password
-     * @param password
+     * @param parameters if the request has parameters
-     *            the password
-     * @param parameters
-     *            if the request has parameters
      * @return result object (see API reference for more details), null for any errors
      */
     protected JSONObject facetedRequest(String username, String password, List<NameValuePair> parameters, String requestURI)
@@ -250,13 +237,13 @@ public abstract class BaseAPI
 
         if (parameters == null || parameters.isEmpty())
         {
-            requestURL = MessageFormat.format(
+             requestURL = MessageFormat.format(
                     requestURI,
                     client.getAlfrescoUrl());
         }
         else
         {
-            requestURL = MessageFormat.format(
+             requestURL = MessageFormat.format(
                     requestURI,
                     client.getAlfrescoUrl(),
                     URLEncodedUtils.format(parameters, "UTF-8"));
@@ -268,26 +255,21 @@ public abstract class BaseAPI
 
     /**
      * Helper method for GET requests
-     * 
+     * @param adminUser user with administrative privileges
-     * @param adminUser
+     * @param adminPassword password for adminUser
-     *            user with administrative privileges
+     * @param urlTemplate request URL template
-     * @param adminPassword
+     * @param urlTemplateParams zero or more parameters used with <i>urlTemplate</i>
-     *            password for adminUser
-     * @param urlTemplate
-     *            request URL template
-     * @param urlTemplateParams
-     *            zero or more parameters used with <i>urlTemplate</i>
      */
     protected JSONObject doGetRequest(String adminUser,
-            String adminPassword,
+        String adminPassword,
-            String urlTemplate,
+        String urlTemplate,
-            String... urlTemplateParams)
+        String ... urlTemplateParams)
     {
         AlfrescoHttpClient client = alfrescoHttpClientFactory.getObject();
         String requestUrl = MessageFormat.format(
-                urlTemplate,
+            urlTemplate,
-                client.getApiUrl(),
+            client.getApiUrl(),
-                urlTemplateParams);
+            urlTemplateParams);
         client.close();
 
         try
@@ -302,26 +284,21 @@ public abstract class BaseAPI
 
     /**
      * Helper method for Delete requests
-     * 
+     * @param adminUser user with administrative privileges
-     * @param adminUser
+     * @param adminPassword password for adminUser
-     *            user with administrative privileges
+     * @param urlTemplate request URL template
-     * @param adminPassword
+     * @param urlTemplateParams zero or more parameters used with <i>urlTemplate</i>
-     *            password for adminUser
-     * @param urlTemplate
-     *            request URL template
-     * @param urlTemplateParams
-     *            zero or more parameters used with <i>urlTemplate</i>
      */
     protected JSONObject doDeleteRequest(String adminUser,
-            String adminPassword,
+        String adminPassword,
-            String urlTemplate,
+        String urlTemplate,
-            String... urlTemplateParams)
+        String ... urlTemplateParams)
     {
         AlfrescoHttpClient client = alfrescoHttpClientFactory.getObject();
         String requestUrl = MessageFormat.format(
-                urlTemplate,
+            urlTemplate,
-                client.getApiUrl(),
+            client.getApiUrl(),
-                urlTemplateParams);
+            urlTemplateParams);
         client.close();
 
         try
@@ -336,29 +313,23 @@ public abstract class BaseAPI
 
     /**
      * Helper method for PUT requests
-     * 
+     * @param adminUser user with administrative privileges
-     * @param adminUser
+     * @param adminPassword password for adminUser
-     *            user with administrative privileges
+     * @param requestParams zero or more endpoint specific request parameters
-     * @param adminPassword
+     * @param urlTemplate request URL template
-     *            password for adminUser
+     * @param urlTemplateParams zero or more parameters used with <i>urlTemplate</i>
-     * @param requestParams
-     *            zero or more endpoint specific request parameters
-     * @param urlTemplate
-     *            request URL template
-     * @param urlTemplateParams
-     *            zero or more parameters used with <i>urlTemplate</i>
      */
     protected JSONObject doPutRequest(String adminUser,
-            String adminPassword,
+        String adminPassword,
-            JSONObject requestParams,
+        JSONObject requestParams,
-            String urlTemplate,
+        String urlTemplate,
-            String... urlTemplateParams)
+        String ... urlTemplateParams)
     {
         AlfrescoHttpClient client = alfrescoHttpClientFactory.getObject();
         String requestUrl = MessageFormat.format(
-                urlTemplate,
+            urlTemplate,
-                client.getApiUrl(),
+            client.getApiUrl(),
-                urlTemplateParams);
+            urlTemplateParams);
         client.close();
 
         try
@@ -374,25 +345,19 @@ public abstract class BaseAPI
     /**
      * Helper method for PUT requests
      *
-     * @param adminUser
+     * @param adminUser user with administrative privileges
-     *            user with administrative privileges
+     * @param adminPassword password for adminUser
-     * @param adminPassword
+     * @param expectedStatusCode The expected return status code.
-     *            password for adminUser
+     * @param requestParams zero or more endpoint specific request parameters
-     * @param expectedStatusCode
+     * @param urlTemplate request URL template
-     *            The expected return status code.
+     * @param urlTemplateParams zero or more parameters used with <i>urlTemplate</i>
-     * @param requestParams
-     *            zero or more endpoint specific request parameters
-     * @param urlTemplate
-     *            request URL template
-     * @param urlTemplateParams
-     *            zero or more parameters used with <i>urlTemplate</i>
      */
     protected HttpResponse doPutJsonRequest(String adminUser,
-            String adminPassword,
+                String adminPassword,
-            int expectedStatusCode,
+                int expectedStatusCode,
-            JSONObject requestParams,
+                JSONObject requestParams,
-            String urlTemplate,
+                String urlTemplate,
-            String... urlTemplateParams)
+                String... urlTemplateParams)
     {
         AlfrescoHttpClient client = alfrescoHttpClientFactory.getObject();
         return doPutJsonRequest(adminUser, adminPassword, expectedStatusCode, client.getApiUrl(), requestParams, urlTemplate, urlTemplateParams);
@@ -401,30 +366,22 @@ public abstract class BaseAPI
     /**
      * Helper method for PUT requests
      *
-     * @param adminUser
+     * @param adminUser user with administrative privileges
-     *            user with administrative privileges
+     * @param adminPassword password for adminUser
-     * @param adminPassword
+     * @param expectedStatusCode The expected return status code.
-     *            password for adminUser
+     * @param urlStart the start of the URL (for example "alfresco/s/slingshot").
-     * @param expectedStatusCode
+     * @param requestParams zero or more endpoint specific request parameters
-     *            The expected return status code.
+     * @param urlTemplate request URL template
-     * @param urlStart
+     * @param urlTemplateParams zero or more parameters used with <i>urlTemplate</i>
-     *            the start of the URL (for example "alfresco/s/slingshot").
+     * @throws AssertionError if the returned status code is not as expected.
-     * @param requestParams
-     *            zero or more endpoint specific request parameters
-     * @param urlTemplate
-     *            request URL template
-     * @param urlTemplateParams
-     *            zero or more parameters used with <i>urlTemplate</i>
-     * @throws AssertionError
-     *             if the returned status code is not as expected.
      */
     private HttpResponse doPutJsonRequest(String adminUser,
-            String adminPassword,
+                String adminPassword,
-            int expectedStatusCode,
+                int expectedStatusCode,
-            String urlStart,
+                String urlStart,
-            JSONObject requestParams,
+                JSONObject requestParams,
-            String urlTemplate,
+                String urlTemplate,
-            String... urlTemplateParams)
+                String... urlTemplateParams)
     {
         String requestUrl = formatRequestUrl(urlStart, urlTemplate, urlTemplateParams);
         try
@@ -442,12 +399,9 @@ public abstract class BaseAPI
     /**
      * Fill in the parameters for a URL template.
      *
-     * @param urlStart
+     * @param urlStart The start of the URL.
-     *            The start of the URL.
+     * @param urlTemplate The template.
-     * @param urlTemplate
+     * @param urlTemplateParams Any parameters that need to be filled into the URL template.
-     *            The template.
-     * @param urlTemplateParams
-     *            Any parameters that need to be filled into the URL template.
      * @return The resultant URL.
      */
     private String formatRequestUrl(String urlStart, String urlTemplate, String[] urlTemplateParams)
@@ -462,29 +416,23 @@ public abstract class BaseAPI
 
     /**
      * Helper method for POST requests
-     * 
+     * @param adminUser user with administrative privileges
-     * @param adminUser
+     * @param adminPassword password for adminUser
-     *            user with administrative privileges
+     * @param requestParams zero or more endpoint specific request parameters
-     * @param adminPassword
+     * @param urlTemplate request URL template
-     *            password for adminUser
+     * @param urlTemplateParams zero or more parameters used with <i>urlTemplate</i>
-     * @param requestParams
-     *            zero or more endpoint specific request parameters
-     * @param urlTemplate
-     *            request URL template
-     * @param urlTemplateParams
-     *            zero or more parameters used with <i>urlTemplate</i>
      */
     protected JSONObject doPostRequest(String adminUser,
-            String adminPassword,
+        String adminPassword,
-            JSONObject requestParams,
+        JSONObject requestParams,
-            String urlTemplate,
+        String urlTemplate,
-            String... urlTemplateParams)
+        String ... urlTemplateParams)
     {
         AlfrescoHttpClient client = alfrescoHttpClientFactory.getObject();
         String requestUrl = MessageFormat.format(
-                urlTemplate,
+            urlTemplate,
-                client.getApiUrl(),
+            client.getApiUrl(),
-                urlTemplateParams);
+            urlTemplateParams);
         client.close();
 
         try
@@ -500,25 +448,19 @@ public abstract class BaseAPI
     /**
      * Helper method for POST requests
      *
-     * @param adminUser
+     * @param adminUser user with administrative privileges
-     *            user with administrative privileges
+     * @param adminPassword password for adminUser
-     * @param adminPassword
+     * @param expectedStatusCode The expected return status code.
-     *            password for adminUser
+     * @param requestParams zero or more endpoint specific request parameters
-     * @param expectedStatusCode
+     * @param urlTemplate request URL template
-     *            The expected return status code.
+     * @param urlTemplateParams zero or more parameters used with <i>urlTemplate</i>
-     * @param requestParams
-     *            zero or more endpoint specific request parameters
-     * @param urlTemplate
-     *            request URL template
-     * @param urlTemplateParams
-     *            zero or more parameters used with <i>urlTemplate</i>
      */
     protected HttpResponse doPostJsonRequest(String adminUser,
-            String adminPassword,
+                                    String adminPassword,
-            int expectedStatusCode,
+                                    int expectedStatusCode,
-            JSONObject requestParams,
+                                    JSONObject requestParams,
-            String urlTemplate,
+                                    String urlTemplate,
-            String... urlTemplateParams)
+                                    String... urlTemplateParams)
     {
         AlfrescoHttpClient client = alfrescoHttpClientFactory.getObject();
         return doPostJsonRequest(adminUser, adminPassword, expectedStatusCode, client.getApiUrl(), requestParams, urlTemplate, urlTemplateParams);
@@ -527,25 +469,19 @@ public abstract class BaseAPI
     /**
      * Helper method for POST requests to slingshot.
      *
-     * @param adminUser
+     * @param adminUser user with administrative privileges
-     *            user with administrative privileges
+     * @param adminPassword password for adminUser
-     * @param adminPassword
+     * @param expectedStatusCode The expected return status code.
-     *            password for adminUser
+     * @param requestParams zero or more endpoint specific request parameters
-     * @param expectedStatusCode
+     * @param urlTemplate request URL template
-     *            The expected return status code.
+     * @param urlTemplateParams zero or more parameters used with <i>urlTemplate</i>
-     * @param requestParams
-     *            zero or more endpoint specific request parameters
-     * @param urlTemplate
-     *            request URL template
-     * @param urlTemplateParams
-     *            zero or more parameters used with <i>urlTemplate</i>
      */
     protected HttpResponse doSlingshotPostJsonRequest(String adminUser,
-            String adminPassword,
+                String adminPassword,
-            int expectedStatusCode,
+                int expectedStatusCode,
-            JSONObject requestParams,
+                JSONObject requestParams,
-            String urlTemplate,
+                String urlTemplate,
-            String... urlTemplateParams)
+                String... urlTemplateParams)
     {
         AlfrescoHttpClient client = alfrescoHttpClientFactory.getObject();
         return doPostJsonRequest(adminUser, adminPassword, expectedStatusCode, client.getAlfrescoUrl() + SLINGSHOT_PREFIX, requestParams, urlTemplate, urlTemplateParams);
@@ -554,30 +490,22 @@ public abstract class BaseAPI
     /**
      * Helper method for POST requests
      *
-     * @param adminUser
+     * @param adminUser user with administrative privileges
-     *            user with administrative privileges
+     * @param adminPassword password for adminUser
-     * @param adminPassword
+     * @param expectedStatusCode The expected return status code.
-     *            password for adminUser
+     * @param urlStart the start of the URL (for example "alfresco/s/slingshot").
-     * @param expectedStatusCode
+     * @param requestParams zero or more endpoint specific request parameters
-     *            The expected return status code.
+     * @param urlTemplate request URL template
-     * @param urlStart
+     * @param urlTemplateParams zero or more parameters used with <i>urlTemplate</i>
-     *            the start of the URL (for example "alfresco/s/slingshot").
+     * @throws AssertionError if the returned status code is not as expected.
-     * @param requestParams
-     *            zero or more endpoint specific request parameters
-     * @param urlTemplate
-     *            request URL template
-     * @param urlTemplateParams
-     *            zero or more parameters used with <i>urlTemplate</i>
-     * @throws AssertionError
-     *             if the returned status code is not as expected.
      */
     private HttpResponse doPostJsonRequest(String adminUser,
-            String adminPassword,
+                String adminPassword,
-            int expectedStatusCode,
+                int expectedStatusCode,
-            String urlStart,
+                String urlStart,
-            JSONObject requestParams,
+                JSONObject requestParams,
-            String urlTemplate,
+                String urlTemplate,
-            String... urlTemplateParams)
+                String... urlTemplateParams)
     {
         String requestUrl;
         requestUrl = formatRequestUrl(urlStart, urlTemplate, urlTemplateParams);
@@ -601,8 +529,7 @@ public abstract class BaseAPI
     /**
      * Try to convert the response body to a JSON object.
      *
-     * @param response
+     * @param response The response.
-     *            The response.
      * @return The JSON object or null if it was not possible to convert the response.
      */
     private JSONObject responseBodyToJson(HttpResponse response)
@@ -627,29 +554,21 @@ public abstract class BaseAPI
 
     /**
      * Helper method for handling generic HTTP requests
-     * 
+     * @param requestType request type (a subclass of {@link HttpRequestBase})
-     * @param requestType
+     * @param requestUrl URL the request is to be sent to
-     *            request type (a subclass of {@link HttpRequestBase})
+     * @param adminUser user with administrative privileges
-     * @param requestUrl
+     * @param adminPassword password for adminUser
-     *            URL the request is to be sent to
+     * @param requestParams endpoint specific request parameters
-     * @param adminUser
-     *            user with administrative privileges
-     * @param adminPassword
-     *            password for adminUser
-     * @param requestParams
-     *            endpoint specific request parameters
      * @return response body
-     * @throws IllegalAccessException
+     * @throws IllegalAccessException for invalid <i>requestType</i>
-     *             for invalid <i>requestType</i>
+     * @throws InstantiationException for invalid <i>requestType</i>
-     * @throws InstantiationException
-     *             for invalid <i>requestType</i>
      */
     private <T extends HttpRequestBase> JSONObject doRequest(
-            Class<T> requestType,
+        Class<T> requestType,
-            String requestUrl,
+        String requestUrl,
-            String adminUser,
+        String adminUser,
-            String adminPassword,
+        String adminPassword,
-            JSONObject requestParams) throws InstantiationException, IllegalAccessException
+        JSONObject requestParams) throws InstantiationException, IllegalAccessException
     {
         AlfrescoHttpClient client = alfrescoHttpClientFactory.getObject();
         T request = requestType.newInstance();
@@ -674,34 +593,34 @@ public abstract class BaseAPI
 
             switch (response.getStatusLine().getStatusCode())
             {
-            case HttpStatus.SC_OK:
+                case HttpStatus.SC_OK:
-            case HttpStatus.SC_CREATED:
+                case HttpStatus.SC_CREATED:
-                // request successful
+                    // request successful
-                if (responseBody != null)
+                    if (responseBody != null)
-                {
+                    {
-                    returnValues = responseBody;
+                        returnValues = responseBody;
-                }
+                    }
-                break;
+                    break;
 
-            case HttpStatus.SC_INTERNAL_SERVER_ERROR:
+                case HttpStatus.SC_INTERNAL_SERVER_ERROR:
-                if (responseBody != null && responseBody.has(EXCEPTION_KEY))
+					if (responseBody != null  && responseBody.has(EXCEPTION_KEY))
-                {
+                    {
-                    LOGGER.error("Request failed with error message: {}", responseBody.getString(MESSAGE_KEY));
+                        LOGGER.error("Request failed with error message: {}", responseBody.getString(MESSAGE_KEY));
-                    returnValues = responseBody;
+                        returnValues = responseBody;
-                }
+                    }
-                break;
+                    break;
-            case HttpStatus.SC_BAD_REQUEST:
+                case HttpStatus.SC_BAD_REQUEST:
-            case HttpStatus.SC_UNPROCESSABLE_ENTITY:
+                case HttpStatus.SC_UNPROCESSABLE_ENTITY:
-                if (responseBody != null && responseBody.has(EXCEPTION_KEY))
+                    if (responseBody != null  && responseBody.has(EXCEPTION_KEY))
-                {
+                    {
-                    LOGGER.error("Request failed: {}", responseBody.getString(EXCEPTION_KEY));
+                        LOGGER.error("Request failed: {}", responseBody.getString(EXCEPTION_KEY));
-                    returnValues = responseBody;
+                        returnValues = responseBody;
-                }
+                    }
-                break;
+                    break;
 
-            default:
+                default:
-                LOGGER.error("Request returned unexpected HTTP status {}", response.getStatusLine().getStatusCode());
+                    LOGGER.error("Request returned unexpected HTTP status {}", response.getStatusLine().getStatusCode());
-                break;
+                    break;
             }
         }
         catch (JSONException error)
@@ -767,16 +686,40 @@ public abstract class BaseAPI
     }
 
     /**
-     * Used to set RM items properties including records, categories and folders
+     * Used to set RM items properties
+     * including records, categories and folders
      */
     public enum RMProperty
     {
-        NAME, TITLE, CONTENT, DESCRIPTION, AUTHOR, PHYSICAL_SIZE, NUMBER_OF_COPIES, STORAGE_LOCATION, SHELF, BOX, FILE, ORIGINATOR, ORIGINATING_ORGANIZATION, PUBLICATION_DATE
+        NAME,
+        TITLE,
+        CONTENT,
+        DESCRIPTION,
+        AUTHOR,
+        PHYSICAL_SIZE,
+        NUMBER_OF_COPIES,
+        STORAGE_LOCATION,
+        SHELF,
+        BOX,
+        FILE,
+        ORIGINATOR,
+        ORIGINATING_ORGANIZATION,
+        PUBLICATION_DATE
     }
 
     public enum RETENTION_SCHEDULE
     {
-        NAME, DESCRIPTION, RETENTION_AUTHORITY, RETENTION_INSTRUCTIONS, RETENTION_PERIOD, RETENTION_LOCATION, RETENTION_PERIOD_PROPERTY, RETENTION_GHOST, RETENTION_ELIGIBLE_FIRST_EVENT, RETENTION_EVENTS, COMBINE_DISPOSITION_STEP_CONDITIONS
+        NAME,
+        DESCRIPTION,
+        RETENTION_AUTHORITY,
+        RETENTION_INSTRUCTIONS,
+        RETENTION_PERIOD,
+        RETENTION_LOCATION,
+        RETENTION_PERIOD_PROPERTY,
+        RETENTION_GHOST,
+        RETENTION_ELIGIBLE_FIRST_EVENT,
+        RETENTION_EVENTS,
+        COMBINE_DISPOSITION_STEP_CONDITIONS
     }
 
     /**
@@ -784,8 +727,14 @@ public abstract class BaseAPI
      */
     public enum RM_ACTIONS
     {
-        EDIT_DISPOSITION_DATE("editDispositionActionAsOfDate"), END_RETENTION("retain"), CUT_OFF("cutoff"), UNDO_CUT_OFF("undoCutoff"), TRANSFER("transfer"), COMPLETE_EVENT("completeEvent"), UNDO_EVENT("undoEvent"), DESTROY("destroy");
+        EDIT_DISPOSITION_DATE("editDispositionActionAsOfDate"),
-
+        END_RETENTION("retain"),
+        CUT_OFF("cutoff"),
+        UNDO_CUT_OFF("undoCutoff"),
+        TRANSFER("transfer"),
+        COMPLETE_EVENT("completeEvent"),
+        UNDO_EVENT("undoEvent"),
+        DESTROY("destroy");
         String action;
 
         private RM_ACTIONS(String action)
@@ -801,16 +750,17 @@ public abstract class BaseAPI
 
     public enum PermissionType
     {
-        SET_READ, REMOVE_READ, SET_READ_AND_FILE, REMOVE_READ_AND_FILE,
+        SET_READ,
+        REMOVE_READ,
+        SET_READ_AND_FILE,
+        REMOVE_READ_AND_FILE,
     }
 
     /**
      * Util to return the property value from a map
      *
-     * @param properties
+     * @param properties the map containing properties
-     *            the map containing properties
+     * @param property   to get value for
-     * @param property
-     *            to get value for
      * @return the property value
      */
     public <K extends Enum<?>> String getPropertyValue(Map<K, String> properties, Enum<?> property)
@@ -826,14 +776,10 @@ public abstract class BaseAPI
     /**
      * Retrieves the property value and decides if that gets to be added to the request
      *
-     * @param requestParams
+     * @param requestParams        the request parameters
-     *            the request parameters
+     * @param propertyRequestValue the property name in the request, eg. "prop_cm_name"
-     * @param propertyRequestValue
+     * @param itemProperties       map of item's properties values
-     *            the property name in the request, eg. "prop_cm_name"
+     * @param property             the property in the property map to check value for
-     * @param itemProperties
-     *            map of item's properties values
-     * @param property
-     *            the property in the property map to check value for
      * @return the json object used in request with the property with its value added if that is not null or empty
      */
     protected <K extends Enum<?>> JSONObject addPropertyToRequest(JSONObject requestParams, String propertyRequestValue, Map<K, String> itemProperties, Enum<?> property) throws JSONException
@@ -850,14 +796,12 @@ public abstract class BaseAPI
     /**
      * Deletes the category, folder or record given as parameter
      *
-     * @param username
+     * @param username the username with whom the delete is performed
-     *            the username with whom the delete is performed
+     * @param password the user's password
-     * @param password
+     * @param itemPath the path to the item eg. in case of a category it would be the "/" + category name,
-     *            the user's password
+     *                 in case of a folder or subCategory it would be /categoryName/folderName or /categoryName/subCategoryName/
-     * @param itemPath
+     *                 in case of a record /categoryName/folderName/recordName
-     *            the path to the item eg. in case of a category it would be the "/" + category name, in case of a folder or subCategory it would be /categoryName/folderName or /categoryName/subCategoryName/ in case of a record /categoryName/folderName/recordName
+     * @throws AssertionError if the delete was not successful.
-     * @throws AssertionError
-     *             if the delete was not successful.
      */
     protected void deleteItem(String username, String password, String itemPath)
     {

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/core/v0/RMEvents.java

@@ -28,8 +28,13 @@ package org.alfresco.rest.core.v0;
 
 public enum RMEvents
 {
-    ABOLISHED("abolished"), ALL_ALLOWANCES_GRANTED_ARE_TERMINATED("all_allowances_granted_are_terminated"), CASE_CLOSED("case_closed"), DECLASSIFICATION_REVIEW("declassification_review"), OBSOLETE("obsolete"), NO_LONGER_NEEDED("no_longer_needed"), STUDY_COMPLETE("study_complete");
+    ABOLISHED("abolished"),
-
+    ALL_ALLOWANCES_GRANTED_ARE_TERMINATED("all_allowances_granted_are_terminated"),
+    CASE_CLOSED("case_closed"),
+    DECLASSIFICATION_REVIEW("declassification_review"),
+    OBSOLETE("obsolete"),
+    NO_LONGER_NEEDED("no_longer_needed"),
+    STUDY_COMPLETE("study_complete");
     private String eventName;
 
     RMEvents(String eventName)

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/audit/AuditEntry.java

@@ -30,11 +30,11 @@ import java.util.List;
 
 import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
 import com.fasterxml.jackson.annotation.JsonProperty;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.NoArgsConstructor;
-
 import org.alfresco.utility.model.TestModel;
 
 /**
@@ -47,43 +47,43 @@ import org.alfresco.utility.model.TestModel;
 @Data
 @NoArgsConstructor
 @AllArgsConstructor
-@JsonIgnoreProperties(ignoreUnknown = true)
+@JsonIgnoreProperties (ignoreUnknown = true)
 public class AuditEntry extends TestModel
 {
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String nodeName;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private List<Object> changedValues;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String identifier;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String path;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String nodeRef;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String fullName;
 
     @JsonProperty
     private String createPerson;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String userName;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String userRole;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String nodeType;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String event;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String timestamp;
 
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/audit/AuditEvents.java

@@ -29,13 +29,24 @@ package org.alfresco.rest.rm.community.model.audit;
 /**
  * Enumerates the list of events audited
  *
- * @author Rodica Sutu
+ * @author  Rodica Sutu
  * @since 2.7
  *
  */
 public enum AuditEvents
 {
-    CREATE_PERSON("Create Person", "Create User"), DELETE_PERSON("Delete Person", "Delete User"), CREATE_USER_GROUP("Create User Group", "Create User Group"), DELETE_USER_GROUP("Delete User Group", "Delete User Group"), ADD_TO_USER_GROUP("Add To User Group", "Add To User Group"), REMOVE_FROM_USER_GROUP("Remove From User Group", "Remove From User Group"), LOGIN_UNSUCCESSFUL("Login.Failure", "Login Unsuccessful"), LOGIN_SUCCESSFUL("Login.Success", "Login Successful"), CREATE_HOLD("Create Hold", "Create Hold"), DELETE_HOLD("Delete Hold", "Delete Hold"), ADD_TO_HOLD("Add To Hold", "Add To Hold"), REMOVE_FROM_HOLD("Remove From Hold", "Remove From Hold");
+    CREATE_PERSON("Create Person", "Create User"),
+    DELETE_PERSON("Delete Person", "Delete User"),
+    CREATE_USER_GROUP("Create User Group", "Create User Group"),
+    DELETE_USER_GROUP("Delete User Group", "Delete User Group"),
+    ADD_TO_USER_GROUP("Add To User Group", "Add To User Group"),
+    REMOVE_FROM_USER_GROUP("Remove From User Group", "Remove From User Group"),
+    LOGIN_UNSUCCESSFUL("Login.Failure", "Login Unsuccessful"),
+    LOGIN_SUCCESSFUL("Login.Success", "Login Successful"),
+    CREATE_HOLD("Create Hold", "Create Hold"),
+    DELETE_HOLD("Delete Hold", "Delete Hold"),
+    ADD_TO_HOLD("Add To Hold", "Add To Hold"),
+    REMOVE_FROM_HOLD("Remove From Hold", "Remove From Hold");
 
     /** event audited */
     public final String event;

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/common/IdNamePair.java

@@ -29,6 +29,7 @@ package org.alfresco.rest.rm.community.model.common;
 import java.util.List;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
@@ -46,15 +47,15 @@ import lombok.NoArgsConstructor;
 @AllArgsConstructor
 public class IdNamePair
 {
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String id;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String name;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private List<String> aspectNames;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String nodeType;
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/common/Owner.java

@@ -27,10 +27,10 @@
 package org.alfresco.rest.rm.community.model.common;
 
 import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
+
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
-
 import org.alfresco.utility.model.TestModel;
 
 /**
@@ -42,9 +42,9 @@ import org.alfresco.utility.model.TestModel;
 @Builder
 @Data
 @EqualsAndHashCode(callSuper = true)
-// @NoArgsConstructor
+//@NoArgsConstructor
-// @AllArgsConstructor
+//@AllArgsConstructor
-@JsonIgnoreProperties(ignoreUnknown = true)
+@JsonIgnoreProperties (ignoreUnknown = true)
 public class Owner extends TestModel
 {
 

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/common/Path.java

@@ -29,14 +29,15 @@ package org.alfresco.rest.rm.community.model.common;
 import java.util.List;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
+import org.alfresco.utility.model.TestModel;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.utility.model.TestModel;
-
 /**
  * POJO for path parameter
  *
@@ -50,12 +51,12 @@ import org.alfresco.utility.model.TestModel;
 @AllArgsConstructor
 public class Path extends TestModel
 {
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String name;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private Boolean isComplete;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private List<IdNamePair> elements;
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/common/ReviewPeriod.java

@@ -27,6 +27,7 @@
 package org.alfresco.rest.rm.community.model.common;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
@@ -44,9 +45,9 @@ import lombok.NoArgsConstructor;
 @AllArgsConstructor
 public class ReviewPeriod
 {
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String periodType;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String expression;
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/custom/CustomDefinitions.java

@@ -34,8 +34,10 @@ package org.alfresco.rest.rm.community.model.custom;
  */
 public enum CustomDefinitions
 {
-    ATTACHMENT("Attachment"), MESSAGE("Message"), NEXT_VERSION("Next Version"), RENDITION("Rendition");
+    ATTACHMENT("Attachment"),
-
+    MESSAGE("Message"),
+    NEXT_VERSION("Next Version"),
+    RENDITION("Rendition");
     /**
      * The name of custom reference.
      */

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/fileplan/FilePlan.java

@@ -29,16 +29,17 @@ package org.alfresco.rest.rm.community.model.fileplan;
 import java.util.List;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
+import org.alfresco.rest.model.RestByUserModel;
+import org.alfresco.rest.rm.community.model.common.Path;
+import org.alfresco.utility.model.TestModel;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.rest.model.RestByUserModel;
-import org.alfresco.rest.rm.community.model.common.Path;
-import org.alfresco.utility.model.TestModel;
-
 /**
  * POJO for file plan
  *
@@ -57,34 +58,34 @@ public class FilePlan extends TestModel
     /** Mandatory parameters */
     /*************************/
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RestByUserModel createdByUser;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String modifiedAt;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String nodeType;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String parentId;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private List<String> aspectNames;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String createdAt;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RestByUserModel modifiedByUser;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String name;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String id;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private FilePlanProperties properties;
 
     /************************/

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/fileplan/FilePlanProperties.java

@@ -34,14 +34,15 @@ import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanCo
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_TITLE;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
+import org.alfresco.utility.model.TestModel;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.utility.model.TestModel;
-
 /**
  * POJO for file plan properties
  *
@@ -58,24 +59,24 @@ public class FilePlanProperties extends TestModel
     /*************************/
     /** Mandatory parameters */
     /*************************/
-    @JsonProperty(required = true, value = PROPERTIES_IDENTIFIER)
+    @JsonProperty (required = true, value = PROPERTIES_IDENTIFIER)
     private String identifier;
 
-    @JsonProperty(required = true, value = PROPERTIES_COMPONENT_ID)
+    @JsonProperty (required = true, value = PROPERTIES_COMPONENT_ID)
     private String componentd;
 
-    @JsonProperty(required = true, value = PROPERTIES_ROOT_NODE_REF)
+    @JsonProperty (required = true, value = PROPERTIES_ROOT_NODE_REF)
     private String rootNodeRef;
 
     /************************/
     /** Optional parameters */
     /************************/
-    @JsonProperty(PROPERTIES_COUNT)
+    @JsonProperty (PROPERTIES_COUNT)
     private Integer count;
 
-    @JsonProperty(PROPERTIES_TITLE)
+    @JsonProperty (PROPERTIES_TITLE)
     private String title;
 
-    @JsonProperty(PROPERTIES_DESCRIPTION)
+    @JsonProperty (PROPERTIES_DESCRIPTION)
     private String description;
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/fileplancomponents/FilePlanComponentAspects.java

@@ -36,15 +36,16 @@ public class FilePlanComponentAspects
 {
     /** Private constructor to prevent instantiation. */
     private FilePlanComponentAspects()
-    {}
+    {
+    }
 
     // aspect present on completed records
     public static final String ASPECTS_COMPLETED_RECORD = "rma:declaredRecord";
 
-    // aspect present on record folders/categories with vital records
+	// aspect present on record folders/categories with vital records
-    public static final String ASPECTS_VITAL_RECORD_DEFINITION = "rma:vitalRecordDefinition";
+    public static final String ASPECTS_VITAL_RECORD_DEFINITION= "rma:vitalRecordDefinition";
 
-    // aspect present on vital records
+	// aspect present on vital records
     public static final String ASPECTS_VITAL_RECORD = "rma:vitalRecord";
 
     // Frozen aspect

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/fileplancomponents/FilePlanComponentFields.java

@@ -47,7 +47,7 @@ public class FilePlanComponentFields
     public static final String PROPERTIES_VITAL_RECORD_INDICATOR = "rma:vitalRecordIndicator";
     public static final String PROPERTIES_REVIEW_PERIOD = "rma:reviewPeriod";
     public static final String PROPERTIES_OWNER = "cm:owner";
-    public static final String PROPERTIES_AUTHOR = "cm:author";
+    public static final String PROPERTIES_AUTHOR="cm:author";
 
     /** Common properties for record folders and records */
     public static final String PROPERTIES_RECORD_SEARCH_HAS_DISPOSITION_SCHEDULE = "rma:recordSearchHasDispositionSchedule";

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/hold/Hold.java

@@ -29,11 +29,11 @@ package org.alfresco.rest.rm.community.model.hold;
 import java.util.Objects;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.NoArgsConstructor;
-
 import org.alfresco.utility.model.TestModel;
 
 /**
@@ -72,7 +72,7 @@ public class Hold extends TestModel
         }
         Hold hold = (Hold) o;
         return Objects.equals(id, hold.id) && Objects.equals(name, hold.name)
-                && Objects.equals(description, hold.description) && Objects.equals(reason, hold.reason);
+            && Objects.equals(description, hold.description) && Objects.equals(reason, hold.reason);
     }
 
     @Override

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/hold/HoldBulkOperation.java

@@ -27,11 +27,11 @@
 package org.alfresco.rest.rm.community.model.hold;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.NoArgsConstructor;
-
 import org.alfresco.rest.search.RestRequestQueryModel;
 import org.alfresco.utility.model.TestModel;
 

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/hold/HoldBulkOperationEntry.java

@@ -26,6 +26,8 @@
  */
 package org.alfresco.rest.rm.community.model.hold;
 
+import com.fasterxml.jackson.annotation.JsonProperty;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/hold/HoldBulkStatus.java

@@ -30,7 +30,6 @@ import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.NoArgsConstructor;
-
 import org.alfresco.utility.model.TestModel;
 
 /**

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/hold/HoldBulkStatusCollection.java

@@ -34,4 +34,5 @@ import org.alfresco.rest.core.RestModels;
  * @author Damian Ujma
  */
 public class HoldBulkStatusCollection extends RestModels<HoldBulkStatusEntry, HoldBulkStatusCollection>
-{}
+{
+}

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/hold/HoldBulkStatusEntry.java

@@ -26,12 +26,13 @@
  */
 package org.alfresco.rest.rm.community.model.hold;
 
+import com.fasterxml.jackson.annotation.JsonProperty;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
-
 import org.alfresco.rest.core.RestModels;
 
 @Builder

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/hold/HoldChild.java

@@ -27,12 +27,12 @@
 package org.alfresco.rest.rm.community.model.hold;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
-
 import org.alfresco.utility.model.TestModel;
 
 /**

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/hold/HoldChildCollection.java

@@ -34,4 +34,5 @@ import org.alfresco.rest.core.RestModels;
  * @author Damian Ujma
  */
 public class HoldChildCollection extends RestModels<HoldChildEntry, HoldChildCollection>
-{}
+{
+}

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/hold/HoldChildEntry.java

@@ -27,12 +27,12 @@
 package org.alfresco.rest.rm.community.model.hold;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
-
 import org.alfresco.rest.core.RestModels;
 
 /**

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/hold/HoldCollection.java

@@ -34,4 +34,5 @@ import org.alfresco.rest.core.RestModels;
  * @author Damian Ujma
  */
 public class HoldCollection extends RestModels<HoldEntry, HoldCollection>
-{}
+{
+}

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/hold/HoldDeletionReason.java

@@ -27,12 +27,12 @@
 package org.alfresco.rest.rm.community.model.hold;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
-
 import org.alfresco.utility.model.TestModel;
 
 /**

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/hold/HoldDeletionReasonEntry.java

@@ -27,12 +27,12 @@
 package org.alfresco.rest.rm.community.model.hold;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
-
 import org.alfresco.rest.core.RestModels;
 
 /**

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/hold/HoldEntry.java

@@ -27,12 +27,12 @@
 package org.alfresco.rest.rm.community.model.hold;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
-
 import org.alfresco.rest.core.RestModels;
 
 /**

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/hold/v0/HoldEntry.java

@@ -28,11 +28,11 @@ package org.alfresco.rest.rm.community.model.hold.v0;
 
 import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
 import com.fasterxml.jackson.annotation.JsonProperty;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.NoArgsConstructor;
-
 import org.alfresco.utility.model.TestModel;
 
 /**
@@ -45,12 +45,12 @@ import org.alfresco.utility.model.TestModel;
 @Data
 @NoArgsConstructor
 @AllArgsConstructor
-@JsonIgnoreProperties(ignoreUnknown = true)
+@JsonIgnoreProperties (ignoreUnknown = true)
 public class HoldEntry extends TestModel
 {
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String name;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String nodeRef;
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/record/Record.java

@@ -29,11 +29,6 @@ package org.alfresco.rest.rm.community.model.record;
 import java.util.List;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
-import lombok.AllArgsConstructor;
-import lombok.Builder;
-import lombok.Data;
-import lombok.EqualsAndHashCode;
-import lombok.NoArgsConstructor;
 
 import org.alfresco.rest.core.IRestModel;
 import org.alfresco.rest.core.assertion.ModelAssertion;
@@ -42,6 +37,12 @@ import org.alfresco.rest.model.RestNodeModel;
 import org.alfresco.rest.rm.community.model.common.Path;
 import org.alfresco.utility.model.TestModel;
 
+import lombok.AllArgsConstructor;
+import lombok.Builder;
+import lombok.Data;
+import lombok.EqualsAndHashCode;
+import lombok.NoArgsConstructor;
+
 /**
  * POJO for record
  *
@@ -60,28 +61,28 @@ public class Record extends TestModel implements IRestModel<RestNodeModel>
     /*************************/
     /** Mandatory parameters */
     /*************************/
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String createdAt;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RestByUserModel createdByUser;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String modifiedAt;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RestByUserModel modifiedByUser;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String name;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String id;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String nodeType;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String parentId;
 
     /************************/
@@ -117,7 +118,7 @@ public class Record extends TestModel implements IRestModel<RestNodeModel>
         return assertThat();
     }
 
-    @JsonProperty(value = "entry")
+    @JsonProperty (value = "entry")
     RestNodeModel model;
 
     @Override

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/record/RecordBodyFile.java

@@ -27,14 +27,15 @@
 package org.alfresco.rest.rm.community.model.record;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
+import org.alfresco.utility.model.TestModel;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.utility.model.TestModel;
-
 /**
  * POJO for File records
  *

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/record/RecordContent.java

@@ -27,14 +27,15 @@
 package org.alfresco.rest.rm.community.model.record;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
+import org.alfresco.utility.model.TestModel;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.utility.model.TestModel;
-
 /**
  * POJO for record content field
  *
@@ -49,15 +50,15 @@ import org.alfresco.utility.model.TestModel;
 @AllArgsConstructor
 public class RecordContent extends TestModel
 {
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String mimeType;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String mimeTypeName;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private Integer sizeInBytes;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String encoding;
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/record/RecordProperties.java

@@ -64,6 +64,7 @@ import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanCo
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RECORD_SEARCH_VITAL_RECORD_REVIEW_PERIOD;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RECORD_SEARCH_VITAL_RECORD_REVIEW_PERIOD_EXPRESSION;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RESOLUTION_UNIT;
+
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_REVIEW_AS_OF;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RMV_VERSIONED;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_ROOT_NODE_REF;
@@ -84,12 +85,12 @@ import java.util.List;
 
 import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
 import com.fasterxml.jackson.annotation.JsonProperty;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
-
 import org.alfresco.rest.rm.community.model.common.Owner;
 import org.alfresco.utility.model.TestModel;
 
@@ -110,160 +111,160 @@ public class RecordProperties extends TestModel
     /*************************/
     /** Mandatory parameters */
     /*************************/
-    @JsonProperty(required = true, value = PROPERTIES_ROOT_NODE_REF)
+    @JsonProperty (required = true, value = PROPERTIES_ROOT_NODE_REF)
     private String rootNodeRef;
 
-    @JsonProperty(required = true, value = PROPERTIES_DATE_FILED)
+    @JsonProperty (required = true, value = PROPERTIES_DATE_FILED)
     private String dateField;
 
-    @JsonProperty(required = true, value = PROPERTIES_IDENTIFIER)
+    @JsonProperty (required = true, value = PROPERTIES_IDENTIFIER)
     private String identifier;
 
-    @JsonProperty(required = true, value = PROPERTIES_RECORD_SEARCH_HAS_DISPOSITION_SCHEDULE)
+    @JsonProperty (required = true, value = PROPERTIES_RECORD_SEARCH_HAS_DISPOSITION_SCHEDULE)
     private Boolean recordSearchHasDispositionSchedule;
 
-    @JsonProperty(required = true, value = PROPERTIES_ORIGINAL_NAME)
+    @JsonProperty (required = true, value = PROPERTIES_ORIGINAL_NAME)
     private String originalName;
 
-    @JsonProperty(PROPERTIES_CLASSIFICATION)
+    @JsonProperty (PROPERTIES_CLASSIFICATION)
     private List<String> classification;
 
     /*********************************/
     /** Electronic record parameters */
     /*********************************/
-    @JsonProperty(PROPERTIES_VERSION_TYPE)
+    @JsonProperty (PROPERTIES_VERSION_TYPE)
     private String versionType;
 
-    @JsonProperty(PROPERTIES_VERSION_LABEL)
+    @JsonProperty (PROPERTIES_VERSION_LABEL)
     private String versionLabel;
 
     @JsonProperty(PROPERTIES_VERSIONED_NODEREF)
     private String versionedNodeRef;
 
-    @JsonProperty(PROPERTIES_RMV_VERSIONED)
+    @JsonProperty (PROPERTIES_RMV_VERSIONED)
     private String recordVersionLabel;
 
-    @JsonProperty(PROPERTIES_DATE_TIME_ORIGINAL)
+    @JsonProperty (PROPERTIES_DATE_TIME_ORIGINAL)
     private String dateTimeOriginal;
 
-    @JsonProperty(PROPERTIES_EXPOSURE_TIME)
+    @JsonProperty (PROPERTIES_EXPOSURE_TIME)
     private Double exposureTime;
 
-    @JsonProperty(PROPERTIES_FLASH)
+    @JsonProperty (PROPERTIES_FLASH)
     private Boolean flash;
 
-    @JsonProperty(PROPERTIES_F_NUMBER)
+    @JsonProperty (PROPERTIES_F_NUMBER)
     private Double fNumber;
 
-    @JsonProperty(PROPERTIES_FOCAL_LENGTH)
+    @JsonProperty (PROPERTIES_FOCAL_LENGTH)
     private Double focalLength;
 
-    @JsonProperty(PROPERTIES_ISO_SPEED_RATINGS)
+    @JsonProperty (PROPERTIES_ISO_SPEED_RATINGS)
     private Integer isoSpeedRatings;
 
-    @JsonProperty(PROPERTIES_MANUFACTURER)
+    @JsonProperty (PROPERTIES_MANUFACTURER)
     private String manufacturer;
 
-    @JsonProperty(PROPERTIES_MODEL)
+    @JsonProperty (PROPERTIES_MODEL)
     private String model;
 
-    @JsonProperty(PROPERTIES_ORIENTATION)
+    @JsonProperty (PROPERTIES_ORIENTATION)
     private Integer orientation;
 
-    @JsonProperty(PROPERTIES_PIXEL_X_DIMENSION)
+    @JsonProperty (PROPERTIES_PIXEL_X_DIMENSION)
     private Integer pixelXDimension;
 
-    @JsonProperty(PROPERTIES_PIXEL_Y_DIMENSION)
+    @JsonProperty (PROPERTIES_PIXEL_Y_DIMENSION)
     private Integer pixelYDimension;
 
-    @JsonProperty(PROPERTIES_RESOLUTION_UNIT)
+    @JsonProperty (PROPERTIES_RESOLUTION_UNIT)
     private String resolutionUnit;
 
-    @JsonProperty(PROPERTIES_SOFTWARE)
+    @JsonProperty (PROPERTIES_SOFTWARE)
     private String software;
 
-    @JsonProperty(PROPERTIES_X_RESOLUTION)
+    @JsonProperty (PROPERTIES_X_RESOLUTION)
     private Double xResolution;
 
-    @JsonProperty(PROPERTIES_Y_RESOLUTION)
+    @JsonProperty (PROPERTIES_Y_RESOLUTION)
     private Double yResolution;
 
-    @JsonProperty(PROPERTIES_RECORD_ORIGINATING_LOCATION)
+    @JsonProperty (PROPERTIES_RECORD_ORIGINATING_LOCATION)
     private String originatingLocation;
 
-    @JsonProperty(PROPERTIES_RECORD_ORIGINATING_USER_ID)
+    @JsonProperty (PROPERTIES_RECORD_ORIGINATING_USER_ID)
     private String originatingUserId;
 
-    @JsonProperty(PROPERTIES_RECORD_ORIGINATING_CREATION_DATE)
+    @JsonProperty (PROPERTIES_RECORD_ORIGINATING_CREATION_DATE)
     private String originatingCreationDate;
 
     /*************************************/
     /** Non-electronic record parameters */
     /*************************************/
-    @JsonProperty(PROPERTIES_TITLE)
+    @JsonProperty (PROPERTIES_TITLE)
     private String title;
 
-    @JsonProperty(PROPERTIES_SHELF)
+    @JsonProperty (PROPERTIES_SHELF)
     private String shelf;
 
-    @JsonProperty(PROPERTIES_STORAGE_LOCATION)
+    @JsonProperty (PROPERTIES_STORAGE_LOCATION)
     private String storageLocation;
 
-    @JsonProperty(PROPERTIES_FILE)
+    @JsonProperty (PROPERTIES_FILE)
     private String file;
 
-    @JsonProperty(PROPERTIES_BOX)
+    @JsonProperty (PROPERTIES_BOX)
     private String box;
 
-    @JsonProperty(PROPERTIES_DESCRIPTION)
+    @JsonProperty (PROPERTIES_DESCRIPTION)
     private String description;
 
-    @JsonProperty(PROPERTIES_NUMBER_OF_COPIES)
+    @JsonProperty (PROPERTIES_NUMBER_OF_COPIES)
     private Integer numberOfCopies;
 
-    @JsonProperty(PROPERTIES_PHYSICAL_SIZE)
+    @JsonProperty (PROPERTIES_PHYSICAL_SIZE)
     private Integer physicalSize;
 
-    @JsonProperty(PROPERTIES_OWNER)
+    @JsonProperty (PROPERTIES_OWNER)
     private Owner owner;
 
     @JsonProperty(PROPERTIES_AUTHOR)
     private String author;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_DISPOSITION_PERIOD_EXPRESSION)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_DISPOSITION_PERIOD_EXPRESSION)
     private String recordSearchDispositionPeriodExpression;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_DISPOSITION_AUTHORITY)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_DISPOSITION_AUTHORITY)
     private String recordSearchDispositionAuthority;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_DISPOSITION_ACTION_AS_OF)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_DISPOSITION_ACTION_AS_OF)
     private Date recordSearchDispositionActionAsOf;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_DISPOSITION_PERIOD)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_DISPOSITION_PERIOD)
     private String recordSearchDispositionPeriod;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_DISPOSITION_ACTION_NAME)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_DISPOSITION_ACTION_NAME)
     private String recordSearchDispositionActionName;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_DISPOSITION_EVENTS_ELIGIBLE)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_DISPOSITION_EVENTS_ELIGIBLE)
     private Boolean recordSearchDispositionEventsEligible;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_DISPOSITION_EVENTS)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_DISPOSITION_EVENTS)
     private List<String> recordSearchDispositionEvents;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_DISPOSITION_INSTRUCTIONS)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_DISPOSITION_INSTRUCTIONS)
     private String recordSearchDispositionInstructions;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_VITAL_RECORD_REVIEW_PERIOD)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_VITAL_RECORD_REVIEW_PERIOD)
     private String recordSearchVitalRecordReviewPeriod;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_VITAL_RECORD_REVIEW_PERIOD_EXPRESSION)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_VITAL_RECORD_REVIEW_PERIOD_EXPRESSION)
     private String recordSearchVitalRecordReviewPeriodExpression;
 
     @JsonProperty(PROPERTIES_REVIEW_AS_OF)
     private Date reviewAsOf;
 
-    @JsonProperty(PROPERTIES_STORE)
+    @JsonProperty (PROPERTIES_STORE)
     private String store;
 
     @JsonProperty(PROPERTIES_WORM_UNLOCK_DATE)

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/recordcategory/RecordCategory.java

@@ -29,16 +29,17 @@ package org.alfresco.rest.rm.community.model.recordcategory;
 import java.util.List;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
+import org.alfresco.rest.model.RestByUserModel;
+import org.alfresco.rest.rm.community.model.common.Path;
+import org.alfresco.utility.model.TestModel;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.rest.model.RestByUserModel;
-import org.alfresco.rest.rm.community.model.common.Path;
-import org.alfresco.utility.model.TestModel;
-
 /**
  * POJO for record category
  *
@@ -58,34 +59,34 @@ public class RecordCategory extends TestModel
     /** Mandatory parameters */
     /*************************/
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RestByUserModel createdByUser;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String modifiedAt;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String nodeType;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String parentId;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private List<String> aspectNames;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String createdAt;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RestByUserModel modifiedByUser;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String name;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String id;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RecordCategoryProperties properties;
 
     /************************/

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/recordcategory/RecordCategoryChild.java

@@ -29,16 +29,17 @@ package org.alfresco.rest.rm.community.model.recordcategory;
 import java.util.List;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
+import org.alfresco.rest.model.RestByUserModel;
+import org.alfresco.rest.rm.community.model.common.Path;
+import org.alfresco.utility.model.TestModel;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.rest.model.RestByUserModel;
-import org.alfresco.rest.rm.community.model.common.Path;
-import org.alfresco.utility.model.TestModel;
-
 /**
  * POJO for record category child
  *
@@ -57,28 +58,28 @@ public class RecordCategoryChild extends TestModel
     /*************************/
     /** Mandatory parameters */
     /*************************/
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String createdAt;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RestByUserModel createdByUser;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String modifiedAt;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RestByUserModel modifiedByUser;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String name;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String id;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String nodeType;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String parentId;
 
     /************************/

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/recordcategory/RecordCategoryChildEntry.java

@@ -27,11 +27,12 @@
 package org.alfresco.rest.rm.community.model.recordcategory;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
-import lombok.Data;
-import lombok.EqualsAndHashCode;
 
 import org.alfresco.rest.core.RestModels;
 
+import lombok.Data;
+import lombok.EqualsAndHashCode;
+
 /**
  * POJO for record category child entry
  *

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/recordcategory/RecordCategoryChildProperties.java

@@ -32,15 +32,15 @@ import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanCo
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_IS_CLOSED;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_LOCATION;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_OWNER;
-import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RECORD_SEARCH_DISPOSITION_ACTION_AS_OF;
-import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RECORD_SEARCH_DISPOSITION_ACTION_NAME;
-import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RECORD_SEARCH_DISPOSITION_AUTHORITY;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RECORD_SEARCH_DISPOSITION_EVENTS;
+import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RECORD_SEARCH_HAS_DISPOSITION_SCHEDULE;
+import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RECORD_SEARCH_DISPOSITION_PERIOD_EXPRESSION;
+import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RECORD_SEARCH_DISPOSITION_AUTHORITY;
+import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RECORD_SEARCH_DISPOSITION_ACTION_AS_OF;
+import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RECORD_SEARCH_DISPOSITION_PERIOD;
+import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RECORD_SEARCH_DISPOSITION_ACTION_NAME;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RECORD_SEARCH_DISPOSITION_EVENTS_ELIGIBLE;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RECORD_SEARCH_DISPOSITION_INSTRUCTIONS;
-import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RECORD_SEARCH_DISPOSITION_PERIOD;
-import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RECORD_SEARCH_DISPOSITION_PERIOD_EXPRESSION;
-import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RECORD_SEARCH_HAS_DISPOSITION_SCHEDULE;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_REVIEW_PERIOD;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_ROOT_NODE_REF;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_TITLE;
@@ -51,17 +51,18 @@ import java.util.List;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
 import com.fasterxml.jackson.databind.annotation.JsonSerialize;
-import lombok.AllArgsConstructor;
-import lombok.Builder;
-import lombok.Data;
-import lombok.EqualsAndHashCode;
-import lombok.NoArgsConstructor;
 
 import org.alfresco.rest.rm.community.model.common.Owner;
 import org.alfresco.rest.rm.community.model.common.ReviewPeriod;
 import org.alfresco.rest.rm.community.util.ReviewPeriodSerializer;
 import org.alfresco.utility.model.TestModel;
 
+import lombok.AllArgsConstructor;
+import lombok.Builder;
+import lombok.Data;
+import lombok.EqualsAndHashCode;
+import lombok.NoArgsConstructor;
+
 /**
  * POJO for record category child properties
  *
@@ -78,65 +79,65 @@ public class RecordCategoryChildProperties extends TestModel
     /**************************************************************************/
     /** Mandatory parameters - Shared by record categories and record folders */
     /**************************************************************************/
-    @JsonProperty(required = true, value = PROPERTIES_TITLE)
+    @JsonProperty (required = true, value = PROPERTIES_TITLE)
     private String title;
 
-    @JsonProperty(required = true, value = PROPERTIES_VITAL_RECORD_INDICATOR)
+    @JsonProperty (required = true, value = PROPERTIES_VITAL_RECORD_INDICATOR)
     private Boolean vitalRecordIndicator;
 
-    @JsonProperty(required = true, value = PROPERTIES_ROOT_NODE_REF)
+    @JsonProperty (required = true, value = PROPERTIES_ROOT_NODE_REF)
     private String rootNodeRef;
 
-    @JsonProperty(required = true, value = PROPERTIES_IDENTIFIER)
+    @JsonProperty (required = true, value = PROPERTIES_IDENTIFIER)
     private String identifier;
 
-    @JsonProperty(required = true, value = PROPERTIES_REVIEW_PERIOD)
+    @JsonProperty (required = true, value = PROPERTIES_REVIEW_PERIOD)
-    @JsonSerialize(using = ReviewPeriodSerializer.class)
+    @JsonSerialize (using = ReviewPeriodSerializer.class)
     private ReviewPeriod reviewPeriod;
 
-    @JsonProperty(required = true, value = PROPERTIES_DESCRIPTION)
+    @JsonProperty (required = true, value = PROPERTIES_DESCRIPTION)
     private String description;
 
     /*********************************************************/
     /** Optional parameters - Applies only to record folders */
     /*********************************************************/
-    @JsonProperty(PROPERTIES_HELD_CHILDREN_COUNT)
+    @JsonProperty (PROPERTIES_HELD_CHILDREN_COUNT)
     private Integer heldChildrenCount;
 
-    @JsonProperty(PROPERTIES_LOCATION)
+    @JsonProperty (PROPERTIES_LOCATION)
     private String location;
 
-    @JsonProperty(PROPERTIES_IS_CLOSED)
+    @JsonProperty (PROPERTIES_IS_CLOSED)
     private Boolean isClosed;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_HAS_DISPOSITION_SCHEDULE)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_HAS_DISPOSITION_SCHEDULE)
     private Boolean recordSearchHasDispositionSchedule;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_DISPOSITION_PERIOD_EXPRESSION)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_DISPOSITION_PERIOD_EXPRESSION)
     private String recordSearchDispositionPeriodExpression;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_DISPOSITION_AUTHORITY)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_DISPOSITION_AUTHORITY)
     private String recordSearchDispositionAuthority;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_DISPOSITION_ACTION_AS_OF)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_DISPOSITION_ACTION_AS_OF)
     private Date recordSearchDispositionActionAsOf;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_DISPOSITION_PERIOD)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_DISPOSITION_PERIOD)
     private String recordSearchDispositionPeriod;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_DISPOSITION_ACTION_NAME)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_DISPOSITION_ACTION_NAME)
     private String recordSearchDispositionActionName;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_DISPOSITION_EVENTS_ELIGIBLE)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_DISPOSITION_EVENTS_ELIGIBLE)
     private Boolean recordSearchDispositionEventsEligible;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_DISPOSITION_INSTRUCTIONS)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_DISPOSITION_INSTRUCTIONS)
     private String recordSearchDispositionInstructions;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_DISPOSITION_EVENTS)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_DISPOSITION_EVENTS)
     private List<String> recordSearchDispositionEvents;
 
-    @JsonProperty(PROPERTIES_OWNER)
+    @JsonProperty (PROPERTIES_OWNER)
     private Owner owner;
 
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/recordcategory/RecordCategoryEntry.java

@@ -27,12 +27,13 @@
 package org.alfresco.rest.rm.community.model.recordcategory;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
-import lombok.Data;
-import lombok.EqualsAndHashCode;
 
 import org.alfresco.rest.core.RestModels;
 import org.alfresco.rest.rm.community.model.fileplan.FilePlan;
 
+import lombok.Data;
+import lombok.EqualsAndHashCode;
+
 /**
  * POJO for file plan entry
  *

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/recordcategory/RecordCategoryProperties.java

@@ -35,22 +35,24 @@ import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanCo
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_TITLE;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_VITAL_RECORD_INDICATOR;
 
+
 import java.util.List;
 
 import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
 import com.fasterxml.jackson.annotation.JsonProperty;
 import com.fasterxml.jackson.databind.annotation.JsonSerialize;
-import lombok.AllArgsConstructor;
-import lombok.Builder;
-import lombok.Data;
-import lombok.EqualsAndHashCode;
-import lombok.NoArgsConstructor;
 
 import org.alfresco.rest.rm.community.model.common.Owner;
 import org.alfresco.rest.rm.community.model.common.ReviewPeriod;
 import org.alfresco.rest.rm.community.util.ReviewPeriodSerializer;
 import org.alfresco.utility.model.TestModel;
 
+import lombok.AllArgsConstructor;
+import lombok.Builder;
+import lombok.Data;
+import lombok.EqualsAndHashCode;
+import lombok.NoArgsConstructor;
+
 /**
  * POJO for record category properties
  *
@@ -62,37 +64,37 @@ import org.alfresco.utility.model.TestModel;
 @EqualsAndHashCode(callSuper = true)
 @NoArgsConstructor
 @AllArgsConstructor
-@JsonIgnoreProperties(ignoreUnknown = true)
+@JsonIgnoreProperties (ignoreUnknown = true)
 public class RecordCategoryProperties extends TestModel
 {
     /*************************/
     /** Mandatory parameters */
     /*************************/
-    @JsonProperty(required = true, value = PROPERTIES_IDENTIFIER)
+    @JsonProperty (required = true, value = PROPERTIES_IDENTIFIER)
     private String identifier;
 
-    @JsonProperty(required = true, value = PROPERTIES_REVIEW_PERIOD)
+    @JsonProperty (required = true, value = PROPERTIES_REVIEW_PERIOD)
-    @JsonSerialize(using = ReviewPeriodSerializer.class)
+    @JsonSerialize (using = ReviewPeriodSerializer.class)
     private ReviewPeriod reviewPeriod;
 
-    @JsonProperty(required = true, value = PROPERTIES_VITAL_RECORD_INDICATOR)
+    @JsonProperty (required = true, value = PROPERTIES_VITAL_RECORD_INDICATOR)
     private Boolean vitalRecordIndicator;
 
     /************************/
     /** Optional parameters */
     /************************/
-    @JsonProperty(PROPERTIES_TITLE)
+    @JsonProperty (PROPERTIES_TITLE)
     private String title;
 
-    @JsonProperty(PROPERTIES_ROOT_NODE_REF)
+    @JsonProperty (PROPERTIES_ROOT_NODE_REF)
     private String rootNodeRef;
 
-    @JsonProperty(PROPERTIES_DESCRIPTION)
+    @JsonProperty (PROPERTIES_DESCRIPTION)
     private String description;
 
-    @JsonProperty(PROPERTIES_OWNER)
+    @JsonProperty (PROPERTIES_OWNER)
     private Owner owner;
 
-    @JsonProperty(PROPERTIES_CLASSIFICATION)
+    @JsonProperty (PROPERTIES_CLASSIFICATION)
     private List<String> classification;
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/recordfolder/RecordFolder.java

@@ -29,16 +29,17 @@ package org.alfresco.rest.rm.community.model.recordfolder;
 import java.util.List;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
+import org.alfresco.rest.model.RestByUserModel;
+import org.alfresco.rest.rm.community.model.common.Path;
+import org.alfresco.utility.model.TestModel;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.rest.model.RestByUserModel;
-import org.alfresco.rest.rm.community.model.common.Path;
-import org.alfresco.utility.model.TestModel;
-
 /**
  * POJO for record folder
  *
@@ -55,34 +56,34 @@ public class RecordFolder extends TestModel
     /*************************/
     /** Mandatory parameters */
     /*************************/
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RestByUserModel createdByUser;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String modifiedAt;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String nodeType;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String parentId;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private List<String> aspectNames;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String createdAt;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RestByUserModel modifiedByUser;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String name;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String id;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RecordFolderProperties properties;
 
     /************************/

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/recordfolder/RecordFolderEntry.java

@@ -27,15 +27,16 @@
 package org.alfresco.rest.rm.community.model.recordfolder;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
+import org.alfresco.rest.core.RestModels;
+import org.alfresco.rest.rm.community.model.record.Record;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.rest.core.RestModels;
-import org.alfresco.rest.rm.community.model.record.Record;
-
 /**
  * POJO for record folder entry
  *

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/recordfolder/RecordFolderProperties.java

@@ -33,8 +33,10 @@ import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanCo
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_IS_CLOSED;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_LOCATION;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_OWNER;
-import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RECORD_SEARCH_DISPOSITION_AUTHORITY;
+import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields
-import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RECORD_SEARCH_DISPOSITION_INSTRUCTIONS;
+        .PROPERTIES_RECORD_SEARCH_DISPOSITION_AUTHORITY;
+import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields
+        .PROPERTIES_RECORD_SEARCH_DISPOSITION_INSTRUCTIONS;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RECORD_SEARCH_HAS_DISPOSITION_SCHEDULE;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RECORD_SEARCH_VITAL_RECORD_REVIEW_PERIOD;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_RECORD_SEARCH_VITAL_RECORD_REVIEW_PERIOD_EXPRESSION;
@@ -48,17 +50,18 @@ import java.util.List;
 import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
 import com.fasterxml.jackson.annotation.JsonProperty;
 import com.fasterxml.jackson.databind.annotation.JsonSerialize;
-import lombok.AllArgsConstructor;
-import lombok.Builder;
-import lombok.Data;
-import lombok.EqualsAndHashCode;
-import lombok.NoArgsConstructor;
 
 import org.alfresco.rest.rm.community.model.common.Owner;
 import org.alfresco.rest.rm.community.model.common.ReviewPeriod;
 import org.alfresco.rest.rm.community.util.ReviewPeriodSerializer;
 import org.alfresco.utility.model.TestModel;
 
+import lombok.AllArgsConstructor;
+import lombok.Builder;
+import lombok.Data;
+import lombok.EqualsAndHashCode;
+import lombok.NoArgsConstructor;
+
 /**
  * POJO for record folder properties
  *
@@ -70,62 +73,62 @@ import org.alfresco.utility.model.TestModel;
 @EqualsAndHashCode(callSuper = true)
 @NoArgsConstructor
 @AllArgsConstructor
-@JsonIgnoreProperties(ignoreUnknown = true)
+@JsonIgnoreProperties (ignoreUnknown = true)
 public class RecordFolderProperties extends TestModel
 {
     /*************************/
     /** Mandatory parameters */
     /*************************/
-    @JsonProperty(required = true, value = PROPERTIES_IS_CLOSED)
+    @JsonProperty (required = true, value = PROPERTIES_IS_CLOSED)
     private Boolean isClosed;
 
-    @JsonProperty(required = true, value = PROPERTIES_IDENTIFIER)
+    @JsonProperty (required = true, value = PROPERTIES_IDENTIFIER)
     private String identifier;
 
-    @JsonProperty(required = true, value = PROPERTIES_HELD_CHILDREN_COUNT)
+    @JsonProperty (required = true, value = PROPERTIES_HELD_CHILDREN_COUNT)
     private Integer heldChildrenCount;
 
     /************************/
     /** Optional parameters */
     /************************/
-    @JsonProperty(PROPERTIES_TITLE)
+    @JsonProperty (PROPERTIES_TITLE)
     private String title;
 
-    @JsonProperty(PROPERTIES_VITAL_RECORD_INDICATOR)
+    @JsonProperty (PROPERTIES_VITAL_RECORD_INDICATOR)
     private Boolean vitalRecordIndicator;
 
-    @JsonProperty(PROPERTIES_ROOT_NODE_REF)
+    @JsonProperty (PROPERTIES_ROOT_NODE_REF)
     private String rootNodeRef;
 
-    @JsonProperty(PROPERTIES_LOCATION)
+    @JsonProperty (PROPERTIES_LOCATION)
     private String location;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_HAS_DISPOSITION_SCHEDULE)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_HAS_DISPOSITION_SCHEDULE)
     private Boolean recordSearchHasDispositionSchedule;
 
-    @JsonProperty(PROPERTIES_REVIEW_PERIOD)
+    @JsonProperty (PROPERTIES_REVIEW_PERIOD)
-    @JsonSerialize(using = ReviewPeriodSerializer.class)
+    @JsonSerialize (using = ReviewPeriodSerializer.class)
     private ReviewPeriod reviewPeriod;
 
-    @JsonProperty(PROPERTIES_CLASSIFICATION)
+    @JsonProperty (PROPERTIES_CLASSIFICATION)
     private List<String> classification;
 
-    @JsonProperty(PROPERTIES_DESCRIPTION)
+    @JsonProperty (PROPERTIES_DESCRIPTION)
     private String description;
 
-    @JsonProperty(PROPERTIES_OWNER)
+    @JsonProperty (PROPERTIES_OWNER)
     private Owner owner;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_VITAL_RECORD_REVIEW_PERIOD)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_VITAL_RECORD_REVIEW_PERIOD)
     private String recordSearchVitalRecordReviewPeriod;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_VITAL_RECORD_REVIEW_PERIOD_EXPRESSION)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_VITAL_RECORD_REVIEW_PERIOD_EXPRESSION)
     private String recordSearchVitalRecordReviewPeriodExpression;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_DISPOSITION_AUTHORITY)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_DISPOSITION_AUTHORITY)
     private String recordSearchDispositionAuthority;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_DISPOSITION_INSTRUCTIONS)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_DISPOSITION_INSTRUCTIONS)
     private String recordSearchDispositionInstructions;
 
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/retentionschedule/RetentionSchedule.java

@@ -26,12 +26,11 @@
  */
 package org.alfresco.rest.rm.community.model.retentionschedule;
 
-import java.util.List;
-
-import lombok.Data;
 import lombok.EqualsAndHashCode;
-
 import org.alfresco.utility.model.TestModel;
+import lombok.Data;
+
+import java.util.List;
 
 /**
  * retention schedule
@@ -40,7 +39,7 @@ import org.alfresco.utility.model.TestModel;
 @Data
 public class RetentionSchedule extends TestModel
 {
-    private String id;
+    private String id ;
     private String parentId;
     private String authority;
     private String instructions;
@@ -53,8 +52,7 @@ public class RetentionSchedule extends TestModel
         return isRecordLevel;
     }
 
-    public void setIsRecordLevel(boolean recordLevel)
+    public void setIsRecordLevel(boolean recordLevel) {
-    {
         isRecordLevel = recordLevel;
     }
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/retentionschedule/RetentionScheduleActionDefinition.java

@@ -27,7 +27,6 @@
 package org.alfresco.rest.rm.community.model.retentionschedule;
 
 import java.util.List;
-
 import lombok.Data;
 
 /**

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/retentionschedule/RetentionScheduleCollection.java

@@ -27,6 +27,6 @@
 package org.alfresco.rest.rm.community.model.retentionschedule;
 
 import org.alfresco.rest.core.RestModels;
-
 public class RetentionScheduleCollection extends RestModels<RetentionScheduleEntry, RetentionScheduleCollection>
-{}
+{
+}

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/retentionschedule/RetentionScheduleEntry.java

@@ -28,9 +28,7 @@ package org.alfresco.rest.rm.community.model.retentionschedule;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
 import lombok.Data;
-
 import org.alfresco.rest.core.RestModels;
-
 @Data
 public class RetentionScheduleEntry extends RestModels<RetentionSchedule, RetentionScheduleEntry>
 {

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/retentionschedule/RetentionScheduleStepCollection.java

@@ -29,4 +29,5 @@ package org.alfresco.rest.rm.community.model.retentionschedule;
 import org.alfresco.rest.core.RestModels;
 
 public class RetentionScheduleStepCollection extends RestModels<RetentionScheduleStepEntry, RetentionScheduleStepCollection>
-{}
+{
+}

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/retentionschedule/RetentionScheduleStepEntry.java

@@ -28,7 +28,6 @@ package org.alfresco.rest.rm.community.model.retentionschedule;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
 import lombok.Data;
-
 import org.alfresco.rest.core.RestModels;
 
 @Data

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/rules/ActionsOnRule.java

@@ -31,7 +31,26 @@ package org.alfresco.rest.rm.community.model.rules;
  */
 public enum ActionsOnRule
 {
-    COMPLETE_RECORD("declareRecord"), REOPEN_RECORD("undeclareRecord"), OPEN_RECORD_FOLDER("openRecordFolder"), CLOSE_RECORD_FOLDER("closeRecordFolder"), FILE_TO("fileTo"), COPY_TO("copyTo"), MOVE_TO("moveTo"), LINK_TO("linkTo"), REJECT("reject"), REQUEST_INFORMATION("requestInfo"), COMPLETE_EVENT("completeEvent"), ADD_RECORD_TYPES("addRecordTypes"), EXECUTE_SCRIPT("executeScript"), SEND_EMAIL("sendEmail"), SET_PROPERTY_VALUE_COLL_SITE("set-property-value"), SET_PROPERTY_VALUE_RM("setPropertyValue"), HIDE_RECORD("hide-record"), DECLARE_VERSION_AS_RECORD("declare-as-version-record"), DECLARE_AS_RECORD("create-record"), WORM_LOCK("wormLock");
+    COMPLETE_RECORD("declareRecord"),
+    REOPEN_RECORD("undeclareRecord"),
+    OPEN_RECORD_FOLDER("openRecordFolder"),
+    CLOSE_RECORD_FOLDER("closeRecordFolder"),
+    FILE_TO("fileTo"),
+    COPY_TO("copyTo"),
+    MOVE_TO("moveTo"),
+    LINK_TO("linkTo"),
+    REJECT("reject"),
+    REQUEST_INFORMATION("requestInfo"),
+    COMPLETE_EVENT("completeEvent"),
+    ADD_RECORD_TYPES("addRecordTypes"),
+    EXECUTE_SCRIPT("executeScript"),
+    SEND_EMAIL("sendEmail"),
+    SET_PROPERTY_VALUE_COLL_SITE("set-property-value"),
+    SET_PROPERTY_VALUE_RM("setPropertyValue"),
+    HIDE_RECORD("hide-record"),
+    DECLARE_VERSION_AS_RECORD("declare-as-version-record"),
+    DECLARE_AS_RECORD("create-record"),
+    WORM_LOCK("wormLock");
 
     private String actionValue;
 

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/rules/ConditionsOnRule.java

@@ -31,17 +31,18 @@ package org.alfresco.rest.rm.community.model.rules;
  */
 public enum ConditionsOnRule
 {
-    UPDATE("update"), ADDED("inbound"), REMOVED("outbound");
+    UPDATE("update"),
-
+    ADDED("inbound"),
+    REMOVED("outbound");
     private String whenConditionValue;
 
     ConditionsOnRule(String value)
     {
-        this.whenConditionValue = value;
+            this.whenConditionValue = value;
     }
 
     public String getWhenConditionValue()
     {
-        return whenConditionValue;
+            return whenConditionValue;
     }
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/rules/RuleDefinition.java

@@ -200,3 +200,4 @@ public class RuleDefinition
         return this;
     }
 }
+

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/site/RMSite.java

@@ -27,14 +27,15 @@
 package org.alfresco.rest.rm.community.model.site;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
+import org.alfresco.rest.model.RestSiteModel;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.rest.model.RestSiteModel;
-
 /**
  * POJO for RM Site component
  *
@@ -47,7 +48,7 @@ import org.alfresco.rest.model.RestSiteModel;
 @AllArgsConstructor
 public class RMSite extends RestSiteModel
 {
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RMSiteCompliance compliance;
 
     /** Private constructor allowing Lombok to include superclass fields in the builder. */

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/site/RMSiteCompliance.java

@@ -34,5 +34,6 @@ package org.alfresco.rest.rm.community.model.site;
  */
 public enum RMSiteCompliance
 {
-    STANDARD, DOD5015
+    STANDARD,
+    DOD5015
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/site/RMSiteFields.java

@@ -27,8 +27,16 @@
 package org.alfresco.rest.rm.community.model.site;
 
 /**
- * RM Site properties from the RM Model Schema "entry": { "id": "string", "guid": "string", "title": "string", "description": "string", "visibility": "{@link org.springframework.social.alfresco.api.entities.Site.Visibility}", "compliance": "{@link RMSiteCompliance}", "role": "{@link org.alfresco.utility.constants.UserRole}" }
+ *RM Site properties from the RM Model Schema
- * 
+ *"entry": {
+ *      "id": "string",
+ *      "guid": "string",
+ *      "title": "string",
+ *      "description": "string",
+ *      "visibility": "{@link org.springframework.social.alfresco.api.entities.Site.Visibility}",
+ *      "compliance": "{@link RMSiteCompliance}",
+ *      "role": "{@link org.alfresco.utility.constants.UserRole}"
+ *}
  * @author Tuna Aksoy
  * @author Rodica Sutu
  * @since 2.6
@@ -39,6 +47,6 @@ public class RMSiteFields
     public static final String COMPLIANCE = "compliance";
     public static final String TITLE = "title";
     public static final String DESCRIPTION = "description";
-    public static final String VISIBILITY = "visibility";
+    public static final String VISIBILITY ="visibility";
     public static final String ROLE = "role";
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/transfer/Transfer.java

@@ -30,15 +30,16 @@ package org.alfresco.rest.rm.community.model.transfer;
 import java.util.List;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
+import org.alfresco.rest.model.RestByUserModel;
+import org.alfresco.utility.model.TestModel;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.rest.model.RestByUserModel;
-import org.alfresco.utility.model.TestModel;
-
 /**
  * POJO for Transfer
  *
@@ -55,28 +56,28 @@ public class Transfer extends TestModel
     /*************************/
     /** Mandatory parameters */
     /*************************/
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RestByUserModel createdByUser;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String nodeType;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String parentId;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private List<String> aspectNames;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String createdAt;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String name;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String id;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private TransferProperties properties;
 
     /************************/

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/transfer/TransferChild.java

@@ -30,16 +30,17 @@ package org.alfresco.rest.rm.community.model.transfer;
 import java.util.List;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
+import org.alfresco.rest.model.RestByUserModel;
+import org.alfresco.rest.rm.community.model.common.Path;
+import org.alfresco.utility.model.TestModel;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.rest.model.RestByUserModel;
-import org.alfresco.rest.rm.community.model.common.Path;
-import org.alfresco.utility.model.TestModel;
-
 /**
  * POJO for transfer child
  *
@@ -56,28 +57,28 @@ public class TransferChild extends TestModel
     /*************************/
     /** Mandatory parameters */
     /*************************/
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String createdAt;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RestByUserModel createdByUser;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String modifiedAt;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RestByUserModel modifiedByUser;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String name;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String id;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String nodeType;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String parentId;
 
     /************************/

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/transfer/TransferChildCollection.java

@@ -31,7 +31,6 @@ import org.alfresco.rest.core.RestModels;
 
 /**
  * Handle collection of {@link TransferChildEntry}
- * 
  * @author Silviu Dinuta
  * @since 2.6
  */

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/transfer/TransferChildEntry.java

@@ -27,15 +27,16 @@
 
 package org.alfresco.rest.rm.community.model.transfer;
 
+import org.alfresco.rest.core.RestModels;
+
 import com.fasterxml.jackson.annotation.JsonProperty;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.rest.core.RestModels;
-
 /**
  * POJO for transfer child entry
  *

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/transfer/TransferChildProperties.java

@@ -64,19 +64,20 @@ import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanCo
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_X_RESOLUTION;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_Y_RESOLUTION;
 
+import org.alfresco.rest.rm.community.model.common.Owner;
+import org.alfresco.rest.rm.community.model.common.ReviewPeriod;
+import org.alfresco.rest.rm.community.util.ReviewPeriodSerializer;
+import org.alfresco.utility.model.TestModel;
+
 import com.fasterxml.jackson.annotation.JsonProperty;
 import com.fasterxml.jackson.databind.annotation.JsonSerialize;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.rest.rm.community.model.common.Owner;
-import org.alfresco.rest.rm.community.model.common.ReviewPeriod;
-import org.alfresco.rest.rm.community.util.ReviewPeriodSerializer;
-import org.alfresco.utility.model.TestModel;
-
 /**
  * POJO for transfer child properties
  *
@@ -91,127 +92,128 @@ import org.alfresco.utility.model.TestModel;
 public class TransferChildProperties extends TestModel
 {
     /**************************************************************************/
-    /** Mandatory parameters - Shared by record folders and records */
+    /** Mandatory parameters - Shared by record folders and records*/
     /**************************************************************************/
-    @JsonProperty(PROPERTIES_TITLE)
+    @JsonProperty (PROPERTIES_TITLE)
     private String title;
 
-    @JsonProperty(required = true, value = PROPERTIES_ROOT_NODE_REF)
+    @JsonProperty (required = true, value = PROPERTIES_ROOT_NODE_REF)
     private String rootNodeRef;
 
-    @JsonProperty(required = true, value = PROPERTIES_IDENTIFIER)
+    @JsonProperty (required = true, value = PROPERTIES_IDENTIFIER)
     private String identifier;
 
-    @JsonProperty(PROPERTIES_DESCRIPTION)
+    @JsonProperty (PROPERTIES_DESCRIPTION)
     private String description;
 
     /*********************************************************/
     /** Optional parameters - Applies only to record folders */
     /*********************************************************/
-    @JsonProperty(PROPERTIES_VITAL_RECORD_INDICATOR)
+    @JsonProperty (PROPERTIES_VITAL_RECORD_INDICATOR)
     private Boolean vitalRecordIndicator;
 
-    @JsonProperty(PROPERTIES_REVIEW_PERIOD)
+    @JsonProperty (PROPERTIES_REVIEW_PERIOD)
-    @JsonSerialize(using = ReviewPeriodSerializer.class)
+    @JsonSerialize (using = ReviewPeriodSerializer.class)
     private ReviewPeriod reviewPeriod;
 
-    @JsonProperty(PROPERTIES_HELD_CHILDREN_COUNT)
+    @JsonProperty (PROPERTIES_HELD_CHILDREN_COUNT)
     private Integer heldChildrenCount;
 
-    @JsonProperty(PROPERTIES_LOCATION)
+    @JsonProperty (PROPERTIES_LOCATION)
     private String location;
 
-    @JsonProperty(PROPERTIES_IS_CLOSED)
+    @JsonProperty (PROPERTIES_IS_CLOSED)
     private Boolean isClosed;
 
     /*********************************************************/
     /** Optional parameters - Applies only to records */
     /*********************************************************/
-    @JsonProperty(PROPERTIES_DATE_FILED)
+    @JsonProperty (PROPERTIES_DATE_FILED)
     private String dateField;
 
-    @JsonProperty(PROPERTIES_RECORD_SEARCH_HAS_DISPOSITION_SCHEDULE)
+    @JsonProperty (PROPERTIES_RECORD_SEARCH_HAS_DISPOSITION_SCHEDULE)
     private Boolean recordSearchHasDispositionSchedule;
 
-    @JsonProperty(PROPERTIES_ORIGINAL_NAME)
+    @JsonProperty (PROPERTIES_ORIGINAL_NAME)
     private String originalName;
 
+
     /*********************************/
     /** Electronic record parameters */
     /*********************************/
-    @JsonProperty(PROPERTIES_VERSION_TYPE)
+    @JsonProperty (PROPERTIES_VERSION_TYPE)
     private String versionType;
 
-    @JsonProperty(PROPERTIES_VERSION_LABEL)
+    @JsonProperty (PROPERTIES_VERSION_LABEL)
     private String versionLabel;
 
-    @JsonProperty(PROPERTIES_DATE_TIME_ORIGINAL)
+    @JsonProperty (PROPERTIES_DATE_TIME_ORIGINAL)
     private String dateTimeOriginal;
 
-    @JsonProperty(PROPERTIES_EXPOSURE_TIME)
+    @JsonProperty (PROPERTIES_EXPOSURE_TIME)
     private Double exposureTime;
 
-    @JsonProperty(PROPERTIES_FLASH)
+    @JsonProperty (PROPERTIES_FLASH)
     private Boolean flash;
 
-    @JsonProperty(PROPERTIES_F_NUMBER)
+    @JsonProperty (PROPERTIES_F_NUMBER)
     private Double fNumber;
 
-    @JsonProperty(PROPERTIES_FOCAL_LENGTH)
+    @JsonProperty (PROPERTIES_FOCAL_LENGTH)
     private Double focalLength;
 
-    @JsonProperty(PROPERTIES_ISO_SPEED_RATINGS)
+    @JsonProperty (PROPERTIES_ISO_SPEED_RATINGS)
     private Integer isoSpeedRatings;
 
-    @JsonProperty(PROPERTIES_MANUFACTURER)
+    @JsonProperty (PROPERTIES_MANUFACTURER)
     private String manufacturer;
 
-    @JsonProperty(PROPERTIES_MODEL)
+    @JsonProperty (PROPERTIES_MODEL)
     private String model;
 
-    @JsonProperty(PROPERTIES_ORIENTATION)
+    @JsonProperty (PROPERTIES_ORIENTATION)
     private Integer orientation;
 
-    @JsonProperty(PROPERTIES_PIXEL_X_DIMENSION)
+    @JsonProperty (PROPERTIES_PIXEL_X_DIMENSION)
     private Integer pixelXDimension;
 
-    @JsonProperty(PROPERTIES_PIXEL_Y_DIMENSION)
+    @JsonProperty (PROPERTIES_PIXEL_Y_DIMENSION)
     private Integer pixelYDimension;
 
-    @JsonProperty(PROPERTIES_RESOLUTION_UNIT)
+    @JsonProperty (PROPERTIES_RESOLUTION_UNIT)
     private String resolutionUnit;
 
-    @JsonProperty(PROPERTIES_SOFTWARE)
+    @JsonProperty (PROPERTIES_SOFTWARE)
     private String software;
 
-    @JsonProperty(PROPERTIES_X_RESOLUTION)
+    @JsonProperty (PROPERTIES_X_RESOLUTION)
     private Double xResolution;
 
-    @JsonProperty(PROPERTIES_Y_RESOLUTION)
+    @JsonProperty (PROPERTIES_Y_RESOLUTION)
     private Double yResolution;
 
     /*************************************/
     /** Non-electronic record parameters */
     /*************************************/
 
-    @JsonProperty(PROPERTIES_SHELF)
+    @JsonProperty (PROPERTIES_SHELF)
     private String shelf;
 
-    @JsonProperty(PROPERTIES_STORAGE_LOCATION)
+    @JsonProperty (PROPERTIES_STORAGE_LOCATION)
     private String storageLocation;
 
-    @JsonProperty(PROPERTIES_FILE)
+    @JsonProperty (PROPERTIES_FILE)
     private String file;
 
-    @JsonProperty(PROPERTIES_BOX)
+    @JsonProperty (PROPERTIES_BOX)
     private String box;
 
-    @JsonProperty(PROPERTIES_NUMBER_OF_COPIES)
+    @JsonProperty (PROPERTIES_NUMBER_OF_COPIES)
     private Integer numberOfCopies;
 
-    @JsonProperty(PROPERTIES_PHYSICAL_SIZE)
+    @JsonProperty (PROPERTIES_PHYSICAL_SIZE)
     private Integer physicalSize;
 
-    @JsonProperty(PROPERTIES_OWNER)
+    @JsonProperty (PROPERTIES_OWNER)
     private Owner owner;
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/transfer/TransferCollection.java

@@ -31,7 +31,6 @@ import org.alfresco.rest.core.RestModels;
 
 /**
  * Handle collection of {@link TransferEntry}
- * 
  * @author Silviu Dinuta
  * @since 2.6
  */

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/transfer/TransferEntry.java

@@ -27,16 +27,17 @@
 
 package org.alfresco.rest.rm.community.model.transfer;
 
+import org.alfresco.rest.core.RestModels;
+import org.alfresco.rest.rm.community.model.transfercontainer.TransferContainer;
+
 import com.fasterxml.jackson.annotation.JsonProperty;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.rest.core.RestModels;
-import org.alfresco.rest.rm.community.model.transfercontainer.TransferContainer;
-
 /**
  * POJO for transfer entry
  *

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/transfer/TransferProperties.java

@@ -27,23 +27,24 @@
 
 package org.alfresco.rest.rm.community.model.transfer;
 
-import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_ACCESSION_INDICATOR;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_IDENTIFIER;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_OWNER;
-import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_PDF_INDICATOR;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_ROOT_NODE_REF;
+import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_PDF_INDICATOR;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_TRANSFER_LOCATION;
+import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_ACCESSION_INDICATOR;
+
+import org.alfresco.rest.rm.community.model.common.Owner;
+import org.alfresco.utility.model.TestModel;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.rest.rm.community.model.common.Owner;
-import org.alfresco.utility.model.TestModel;
-
 /**
  * POJO for Transfer properties
  *
@@ -60,24 +61,24 @@ public class TransferProperties extends TestModel
     /*************************/
     /** Mandatory parameters */
     /*************************/
-    @JsonProperty(required = true, value = PROPERTIES_IDENTIFIER)
+    @JsonProperty (required = true, value = PROPERTIES_IDENTIFIER)
     private String identifier;
 
     /************************/
     /** Optional parameters */
     /************************/
-    @JsonProperty(PROPERTIES_ROOT_NODE_REF)
+    @JsonProperty (PROPERTIES_ROOT_NODE_REF)
     private String rootNodeRef;
 
-    @JsonProperty(PROPERTIES_OWNER)
+    @JsonProperty (PROPERTIES_OWNER)
     private Owner owner;
 
-    @JsonProperty(PROPERTIES_PDF_INDICATOR)
+    @JsonProperty (PROPERTIES_PDF_INDICATOR)
     private Boolean pdfIndicator;
 
-    @JsonProperty(PROPERTIES_TRANSFER_LOCATION)
+    @JsonProperty (PROPERTIES_TRANSFER_LOCATION)
     private String transferLocation;
 
-    @JsonProperty(PROPERTIES_ACCESSION_INDICATOR)
+    @JsonProperty (PROPERTIES_ACCESSION_INDICATOR)
     private Boolean accessionIndicator;
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/transfercontainer/TransferContainer.java

@@ -30,15 +30,16 @@ package org.alfresco.rest.rm.community.model.transfercontainer;
 import java.util.List;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+
+import org.alfresco.rest.model.RestByUserModel;
+import org.alfresco.utility.model.TestModel;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.rest.model.RestByUserModel;
-import org.alfresco.utility.model.TestModel;
-
 /**
  * POJO for Transfer Container
  *
@@ -55,34 +56,34 @@ public class TransferContainer extends TestModel
     /*************************/
     /** Mandatory parameters */
     /*************************/
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RestByUserModel createdByUser;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String modifiedAt;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String nodeType;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String parentId;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private List<String> aspectNames;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String createdAt;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RestByUserModel modifiedByUser;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String name;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String id;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private TransferContainerProperties properties;
 
     /************************/

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/transfercontainer/TransferContainerProperties.java

@@ -31,15 +31,16 @@ import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanCo
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_IDENTIFIER;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentFields.PROPERTIES_ROOT_NODE_REF;
 
+import org.alfresco.utility.model.TestModel;
+
 import com.fasterxml.jackson.annotation.JsonProperty;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.utility.model.TestModel;
-
 /**
  * POJO for Transfer Container properties
  *
@@ -56,16 +57,16 @@ public class TransferContainerProperties extends TestModel
     /*************************/
     /** Mandatory parameters */
     /*************************/
-    @JsonProperty(required = true, value = PROPERTIES_IDENTIFIER)
+    @JsonProperty (required = true, value = PROPERTIES_IDENTIFIER)
     private String identifier;
 
     /************************/
     /** Optional parameters */
     /************************/
-    @JsonProperty(PROPERTIES_ROOT_NODE_REF)
+    @JsonProperty (PROPERTIES_ROOT_NODE_REF)
     private String rootNodeRef;
 
-    @JsonProperty(PROPERTIES_COUNT)
+    @JsonProperty (PROPERTIES_COUNT)
     private Integer count;
 
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/unfiledcontainer/UnfiledContainer.java

@@ -28,17 +28,18 @@ package org.alfresco.rest.rm.community.model.unfiledcontainer;
 
 import java.util.List;
 
+import org.alfresco.rest.model.RestByUserModel;
+import org.alfresco.rest.rm.community.model.common.Path;
+import org.alfresco.utility.model.TestModel;
+
 import com.fasterxml.jackson.annotation.JsonProperty;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.rest.model.RestByUserModel;
-import org.alfresco.rest.rm.community.model.common.Path;
-import org.alfresco.utility.model.TestModel;
-
 /**
  * POJO for unfiled container
  *
@@ -55,34 +56,34 @@ public class UnfiledContainer extends TestModel
     /*************************/
     /** Mandatory parameters */
     /*************************/
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RestByUserModel createdByUser;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String modifiedAt;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String nodeType;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String parentId;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private List<String> aspectNames;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String createdAt;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RestByUserModel modifiedByUser;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String name;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String id;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private UnfiledContainerProperties properties;
 
     /************************/

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/unfiledcontainer/UnfiledContainerChild.java

@@ -28,19 +28,20 @@ package org.alfresco.rest.rm.community.model.unfiledcontainer;
 
 import java.util.List;
 
+import org.alfresco.rest.model.RestByUserModel;
+import org.alfresco.rest.rm.community.model.common.Path;
+import org.alfresco.rest.rm.community.model.record.RecordContent;
+import org.alfresco.utility.model.TestModel;
+
 import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
 import com.fasterxml.jackson.annotation.JsonProperty;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.rest.model.RestByUserModel;
-import org.alfresco.rest.rm.community.model.common.Path;
-import org.alfresco.rest.rm.community.model.record.RecordContent;
-import org.alfresco.utility.model.TestModel;
-
 /**
  * POJO for record category child
  *
@@ -52,40 +53,40 @@ import org.alfresco.utility.model.TestModel;
 @EqualsAndHashCode(callSuper = true)
 @NoArgsConstructor
 @AllArgsConstructor
-@JsonIgnoreProperties(ignoreUnknown = true)
+@JsonIgnoreProperties (ignoreUnknown = true)
 public class UnfiledContainerChild extends TestModel
 {
     /*************************/
     /** Mandatory parameters */
     /*************************/
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String createdAt;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private Boolean isUnfiledRecordFolder;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private Boolean isRecord;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RestByUserModel createdByUser;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String modifiedAt;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RestByUserModel modifiedByUser;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String name;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String id;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String nodeType;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String parentId;
 
     /************************/

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/unfiledcontainer/UnfiledContainerChildEntry.java

@@ -27,11 +27,12 @@
 package org.alfresco.rest.rm.community.model.unfiledcontainer;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
-import lombok.Data;
-import lombok.EqualsAndHashCode;
 
 import org.alfresco.rest.core.RestModels;
 
+import lombok.Data;
+import lombok.EqualsAndHashCode;
+
 /**
  * POJO for record category child entry
  *

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/unfiledcontainer/UnfiledContainerChildProperties.java

@@ -61,15 +61,16 @@ import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanCo
 
 import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
 import com.fasterxml.jackson.annotation.JsonProperty;
+
+import org.alfresco.rest.rm.community.model.common.Owner;
+import org.alfresco.utility.model.TestModel;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.rest.rm.community.model.common.Owner;
-import org.alfresco.utility.model.TestModel;
-
 /**
  * POJO for record category child properties
  *
@@ -82,112 +83,112 @@ import org.alfresco.utility.model.TestModel;
 @EqualsAndHashCode(callSuper = true)
 @NoArgsConstructor
 @AllArgsConstructor
-@JsonIgnoreProperties(ignoreUnknown = true)
+@JsonIgnoreProperties (ignoreUnknown = true)
 public class UnfiledContainerChildProperties extends TestModel
 {
     /**************************************************************************/
     /** Mandatory parameters - Shared by unfiled record folder and records */
     /**************************************************************************/
-    @JsonProperty(required = true, value = PROPERTIES_TITLE)
+    @JsonProperty (required = true, value = PROPERTIES_TITLE)
     private String title;
 
-    @JsonProperty(required = true, value = PROPERTIES_ROOT_NODE_REF)
+    @JsonProperty (required = true, value = PROPERTIES_ROOT_NODE_REF)
     private String rootNodeRef;
 
-    @JsonProperty(required = true, value = PROPERTIES_IDENTIFIER)
+    @JsonProperty (required = true, value = PROPERTIES_IDENTIFIER)
     private String identifier;
 
-    @JsonProperty(required = true, value = PROPERTIES_DESCRIPTION)
+    @JsonProperty (required = true, value = PROPERTIES_DESCRIPTION)
     private String description;
 
-    @JsonProperty(value = PROPERTIES_RECORD_SEARCH_HAS_DISPOSITION_SCHEDULE)
+    @JsonProperty (value = PROPERTIES_RECORD_SEARCH_HAS_DISPOSITION_SCHEDULE)
     private Boolean recordSearchHasDispositionSchedule;
 
     /*********************************/
     /** Electronic record parameters */
     /*********************************/
-    @JsonProperty(PROPERTIES_VERSION_TYPE)
+    @JsonProperty (PROPERTIES_VERSION_TYPE)
     private String versionType;
 
-    @JsonProperty(PROPERTIES_VERSION_LABEL)
+    @JsonProperty (PROPERTIES_VERSION_LABEL)
     private String versionLabel;
 
     @JsonProperty(PROPERTIES_VERSIONED_NODEREF)
     private String versionedNodeRef;
 
-    @JsonProperty(PROPERTIES_RMV_VERSIONED)
+    @JsonProperty (PROPERTIES_RMV_VERSIONED)
     private String recordVersionLabel;
 
-    @JsonProperty(PROPERTIES_DATE_TIME_ORIGINAL)
+    @JsonProperty (PROPERTIES_DATE_TIME_ORIGINAL)
     private String dateTimeOriginal;
 
-    @JsonProperty(PROPERTIES_EXPOSURE_TIME)
+    @JsonProperty (PROPERTIES_EXPOSURE_TIME)
     private Double exposureTime;
 
-    @JsonProperty(PROPERTIES_FLASH)
+    @JsonProperty (PROPERTIES_FLASH)
     private Boolean flash;
 
-    @JsonProperty(PROPERTIES_F_NUMBER)
+    @JsonProperty (PROPERTIES_F_NUMBER)
     private Double fNumber;
 
-    @JsonProperty(PROPERTIES_FOCAL_LENGTH)
+    @JsonProperty (PROPERTIES_FOCAL_LENGTH)
     private Double focalLength;
 
-    @JsonProperty(PROPERTIES_ISO_SPEED_RATINGS)
+    @JsonProperty (PROPERTIES_ISO_SPEED_RATINGS)
     private Integer isoSpeedRatings;
 
-    @JsonProperty(PROPERTIES_MANUFACTURER)
+    @JsonProperty (PROPERTIES_MANUFACTURER)
     private String manufacturer;
 
-    @JsonProperty(PROPERTIES_MODEL)
+    @JsonProperty (PROPERTIES_MODEL)
     private String model;
 
-    @JsonProperty(PROPERTIES_ORIENTATION)
+    @JsonProperty (PROPERTIES_ORIENTATION)
     private Integer orientation;
 
-    @JsonProperty(PROPERTIES_PIXEL_X_DIMENSION)
+    @JsonProperty (PROPERTIES_PIXEL_X_DIMENSION)
     private Integer pixelXDimension;
 
-    @JsonProperty(PROPERTIES_PIXEL_Y_DIMENSION)
+    @JsonProperty (PROPERTIES_PIXEL_Y_DIMENSION)
     private Integer pixelYDimension;
 
-    @JsonProperty(PROPERTIES_RESOLUTION_UNIT)
+    @JsonProperty (PROPERTIES_RESOLUTION_UNIT)
     private String resolutionUnit;
 
-    @JsonProperty(PROPERTIES_SOFTWARE)
+    @JsonProperty (PROPERTIES_SOFTWARE)
     private String software;
 
-    @JsonProperty(PROPERTIES_X_RESOLUTION)
+    @JsonProperty (PROPERTIES_X_RESOLUTION)
     private Double xResolution;
 
-    @JsonProperty(PROPERTIES_Y_RESOLUTION)
+    @JsonProperty (PROPERTIES_Y_RESOLUTION)
     private Double yResolution;
 
-    @JsonProperty(PROPERTIES_ORIGINAL_NAME)
+    @JsonProperty (PROPERTIES_ORIGINAL_NAME)
     private String originalName;
 
     /*************************************/
     /** Non-electronic record parameters */
     /*************************************/
 
-    @JsonProperty(PROPERTIES_SHELF)
+    @JsonProperty (PROPERTIES_SHELF)
     private String shelf;
 
-    @JsonProperty(PROPERTIES_STORAGE_LOCATION)
+    @JsonProperty (PROPERTIES_STORAGE_LOCATION)
     private String storageLocation;
 
-    @JsonProperty(PROPERTIES_FILE)
+    @JsonProperty (PROPERTIES_FILE)
     private String file;
 
-    @JsonProperty(PROPERTIES_BOX)
+    @JsonProperty (PROPERTIES_BOX)
     private String box;
 
-    @JsonProperty(PROPERTIES_NUMBER_OF_COPIES)
+    @JsonProperty (PROPERTIES_NUMBER_OF_COPIES)
     private Integer numberOfCopies;
 
-    @JsonProperty(PROPERTIES_PHYSICAL_SIZE)
+    @JsonProperty (PROPERTIES_PHYSICAL_SIZE)
     private Integer physicalSize;
 
-    @JsonProperty(PROPERTIES_OWNER)
+    @JsonProperty (PROPERTIES_OWNER)
     private Owner owner;
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/unfiledcontainer/UnfiledContainerProperties.java

@@ -31,14 +31,15 @@ import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanCo
 
 import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
 import com.fasterxml.jackson.annotation.JsonProperty;
+
+import org.alfresco.utility.model.TestModel;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.utility.model.TestModel;
-
 /**
  * POJO for file plan properties
  *
@@ -50,15 +51,15 @@ import org.alfresco.utility.model.TestModel;
 @EqualsAndHashCode(callSuper = true)
 @NoArgsConstructor
 @AllArgsConstructor
-@JsonIgnoreProperties(ignoreUnknown = true)
+@JsonIgnoreProperties (ignoreUnknown = true)
 public class UnfiledContainerProperties extends TestModel
 {
     /*************************/
     /** Mandatory parameters */
     /*************************/
-    @JsonProperty(required = true, value = PROPERTIES_IDENTIFIER)
+    @JsonProperty (required = true, value = PROPERTIES_IDENTIFIER)
     private String identifier;
 
-    @JsonProperty(required = true, value = PROPERTIES_ROOT_NODE_REF)
+    @JsonProperty (required = true, value = PROPERTIES_ROOT_NODE_REF)
     private String rootNodeRef;
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/unfiledcontainer/UnfiledRecordFolder.java

@@ -28,18 +28,19 @@ package org.alfresco.rest.rm.community.model.unfiledcontainer;
 
 import java.util.List;
 
+import org.alfresco.rest.model.RestByUserModel;
+import org.alfresco.rest.rm.community.model.common.Path;
+import org.alfresco.rest.rm.community.model.record.RecordContent;
+import org.alfresco.utility.model.TestModel;
+
 import com.fasterxml.jackson.annotation.JsonProperty;
+
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 
-import org.alfresco.rest.model.RestByUserModel;
-import org.alfresco.rest.rm.community.model.common.Path;
-import org.alfresco.rest.rm.community.model.record.RecordContent;
-import org.alfresco.utility.model.TestModel;
-
 /**
  * POJO for unfiled container
  *
@@ -56,28 +57,28 @@ public class UnfiledRecordFolder extends TestModel
     /*************************/
     /** Mandatory parameters */
     /*************************/
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String createdAt;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RestByUserModel createdByUser;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String modifiedAt;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private RestByUserModel modifiedByUser;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String name;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String id;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String nodeType;
 
-    @JsonProperty(required = true)
+    @JsonProperty (required = true)
     private String parentId;
 
     /************************/

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/user/UserPermissions.java

@@ -28,13 +28,15 @@ package org.alfresco.rest.rm.community.model.user;
 
 /**
  * Constants for RM user capabilities
- * 
+ *
  * @author Kristijan Conkas
  * @since 2.6
  */
 public enum UserPermissions
 {
-    PERMISSION_FILING("Filing"), PERMISSION_READ_RECORDS("ReadRecords"), PERMISSION_FILE_RECORDS("FileRecords");
+    PERMISSION_FILING("Filing"),
+    PERMISSION_READ_RECORDS("ReadRecords"),
+    PERMISSION_FILE_RECORDS("FileRecords");
 
     public final String permissionId;
 

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/model/user/UserRoles.java

@@ -35,7 +35,12 @@ package org.alfresco.rest.rm.community.model.user;
  */
 public enum UserRoles
 {
-    IN_PLACE_WRITERS("ExtendedWriters", "In-Place Writers"), ROLE_RM_ADMIN("Administrator", "Records Management Administrator"), ROLE_RM_MANAGER("RecordsManager", "Records Management Manager"), ROLE_RM_POWER_USER("PowerUser", "Records Management Power User"), ROLE_RM_SECURITY_OFFICER("SecurityOfficer", "Records Management Security Officer"), ROLE_RM_USER("User", "Records Management User");
+    IN_PLACE_WRITERS("ExtendedWriters", "In-Place Writers"),
+    ROLE_RM_ADMIN("Administrator", "Records Management Administrator"),
+    ROLE_RM_MANAGER("RecordsManager", "Records Management Manager"),
+    ROLE_RM_POWER_USER("PowerUser", "Records Management Power User"),
+    ROLE_RM_SECURITY_OFFICER("SecurityOfficer", "Records Management Security Officer"),
+    ROLE_RM_USER("User", "Records Management User");
 
     public final String roleId;
     public final String displayName;
@@ -46,4 +51,5 @@ public enum UserRoles
         this.displayName = displayName;
     }
 
+
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/requests/RMModelRequest.java

@@ -29,12 +29,12 @@ package org.alfresco.rest.rm.community.requests;
 import static lombok.AccessLevel.PRIVATE;
 import static lombok.AccessLevel.PROTECTED;
 
-import lombok.Getter;
-import lombok.Setter;
-
 import org.alfresco.rest.core.RMRestWrapper;
 import org.alfresco.rest.requests.ModelRequest;
 
+import lombok.Getter;
+import lombok.Setter;
+
 /**
  * Extends {@link ModelRequest} to set {@link RMRestWrapper}
  *
@@ -43,8 +43,8 @@ import org.alfresco.rest.requests.ModelRequest;
  */
 public abstract class RMModelRequest<Request> extends ModelRequest<Request>
 {
-    @Getter(value = PROTECTED)
+    @Getter (value = PROTECTED)
-    @Setter(value = PRIVATE)
+    @Setter (value = PRIVATE)
     private RMRestWrapper rmRestWrapper;
 
     /**

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/requests/gscore/GSCoreAPI.java

@@ -33,8 +33,8 @@ import io.restassured.RestAssured;
 
 import org.alfresco.rest.core.RMRestProperties;
 import org.alfresco.rest.core.RMRestWrapper;
-import org.alfresco.rest.rm.community.requests.RMModelRequest;
 import org.alfresco.rest.rm.community.requests.gscore.api.ActionsExecutionAPI;
+import org.alfresco.rest.rm.community.requests.RMModelRequest;
 import org.alfresco.rest.rm.community.requests.gscore.api.FilePlanAPI;
 import org.alfresco.rest.rm.community.requests.gscore.api.FilesAPI;
 import org.alfresco.rest.rm.community.requests.gscore.api.HoldsAPI;
@@ -43,14 +43,15 @@ import org.alfresco.rest.rm.community.requests.gscore.api.RMUserAPI;
 import org.alfresco.rest.rm.community.requests.gscore.api.RecordCategoryAPI;
 import org.alfresco.rest.rm.community.requests.gscore.api.RecordFolderAPI;
 import org.alfresco.rest.rm.community.requests.gscore.api.RecordsAPI;
-import org.alfresco.rest.rm.community.requests.gscore.api.RetentionScheduleAPI;
 import org.alfresco.rest.rm.community.requests.gscore.api.TransferAPI;
 import org.alfresco.rest.rm.community.requests.gscore.api.TransferContainerAPI;
 import org.alfresco.rest.rm.community.requests.gscore.api.UnfiledContainerAPI;
 import org.alfresco.rest.rm.community.requests.gscore.api.UnfiledRecordFolderAPI;
+import org.alfresco.rest.rm.community.requests.gscore.api.RetentionScheduleAPI;
 
 /**
- * Defines the entire GS Core API {@link http://host:port/gs-api-explorer} select "GS Core API"
+ * Defines the entire GS Core API
+ * {@link http://host:port/gs-api-explorer} select "GS Core API"
  *
  * @author Tuna Aksoy
  * @since 2.6
@@ -60,10 +61,8 @@ public class GSCoreAPI extends RMModelRequest
     /**
      * Constructor
      *
-     * @param rmRestWrapper
+     * @param rmRestWrapper RM REST Wrapper
-     *            RM REST Wrapper
+     * @param rmRestProperties RM REST Properties
-     * @param rmRestProperties
-     *            RM REST Properties
      */
     public GSCoreAPI(RMRestWrapper rmRestWrapper, RMRestProperties rmRestProperties)
     {
@@ -81,7 +80,7 @@ public class GSCoreAPI extends RMModelRequest
      */
     public RMSiteAPI usingRMSite()
     {
-        return new RMSiteAPI(getRmRestWrapper());
+      return new RMSiteAPI(getRmRestWrapper());
     }
 
     /**
@@ -194,10 +193,7 @@ public class GSCoreAPI extends RMModelRequest
         return new ActionsExecutionAPI(getRmRestWrapper());
     }
 
-    public HoldsAPI usingHoldsAPI()
+    public HoldsAPI usingHoldsAPI() { return new HoldsAPI(getRmRestWrapper()); }
-    {
-        return new HoldsAPI(getRmRestWrapper());
-    }
 
     public RetentionScheduleAPI usingRetentionScheduleAPI()
     {

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/requests/gscore/api/ActionsExecutionAPI.java

@@ -27,13 +27,13 @@
 package org.alfresco.rest.rm.community.requests.gscore.api;
 
 import com.google.common.collect.ImmutableMap;
-import lombok.SneakyThrows;
-import org.json.JSONObject;
 
+import lombok.SneakyThrows;
 import org.alfresco.rest.core.RMRestWrapper;
 import org.alfresco.rest.rm.community.model.rules.ActionsOnRule;
 import org.alfresco.rest.rm.community.requests.RMModelRequest;
 import org.alfresco.utility.model.RepoTestModel;
+import org.json.JSONObject;
 
 /**
  * Produces processed results from Core Actions API calls
@@ -44,8 +44,7 @@ import org.alfresco.utility.model.RepoTestModel;
 public class ActionsExecutionAPI extends RMModelRequest
 {
     /**
-     * @param rmRestWrapper
+     * @param rmRestWrapper RM REST Wrapper
-     *            RM REST Wrapper
      */
     public ActionsExecutionAPI(RMRestWrapper rmRestWrapper)
     {
@@ -55,89 +54,81 @@ public class ActionsExecutionAPI extends RMModelRequest
     /**
      * Declares and files a document as record to a record folder using v1 actions api
      *
-     * @param targetNode
+     * @param targetNode      the node on which the action is executed
-     *            the node on which the action is executed
+     * @param destinationPath the path to the record folder
-     * @param destinationPath
-     *            the path to the record folder
      * @throws Exception
      */
     public JSONObject declareAndFile(RepoTestModel targetNode, String destinationPath) throws Exception
     {
         return getRmRestWrapper().withCoreAPI().usingActions()
-                .executeAction(ActionsOnRule.DECLARE_AS_RECORD.getActionValue(), targetNode,
+                                 .executeAction(ActionsOnRule.DECLARE_AS_RECORD.getActionValue(), targetNode,
-                        ImmutableMap.of("path", destinationPath));
+                                         ImmutableMap.of("path", destinationPath));
     }
 
     /**
      * Declares a document as record using v1 actions api
      *
-     * @param targetNode
+     * @param targetNode the node on which the action is executed
-     *            the node on which the action is executed
      * @throws Exception
      */
     public JSONObject declareAsRecord(RepoTestModel targetNode) throws Exception
     {
         return getRmRestWrapper().withCoreAPI().usingActions()
-                .executeAction(ActionsOnRule.DECLARE_AS_RECORD.getActionValue(), targetNode);
+                                 .executeAction(ActionsOnRule.DECLARE_AS_RECORD.getActionValue(), targetNode);
     }
 
+
     /**
      * Declares and file a document version as record to a record folder using v1 actions api
      *
-     * @param targetNode
+     * @param targetNode      the node on which the action is executed
-     *            the node on which the action is executed
+     * @param destinationPath the path to the record folder
-     * @param destinationPath
-     *            the path to the record folder
      * @throws Exception
      */
     public JSONObject declareAndFileVersionAsRecord(RepoTestModel targetNode, String destinationPath) throws Exception
     {
         return getRmRestWrapper().withCoreAPI().usingActions()
-                .executeAction(ActionsOnRule.DECLARE_VERSION_AS_RECORD.getActionValue(), targetNode,
+                                 .executeAction(ActionsOnRule.DECLARE_VERSION_AS_RECORD.getActionValue(), targetNode,
-                        ImmutableMap.of("path", destinationPath));
+                                         ImmutableMap.of("path", destinationPath));
     }
 
     /**
      * Declares a document version as record using v1 actions api
      *
-     * @param targetNode
+     * @param targetNode the node on which the action is executed
-     *            the node on which the action is executed
      * @throws Exception
      */
     public JSONObject declareVersionAsRecord(RepoTestModel targetNode) throws Exception
     {
         return getRmRestWrapper().withCoreAPI().usingActions()
-                .executeAction(ActionsOnRule.DECLARE_VERSION_AS_RECORD.getActionValue(), targetNode);
+                                 .executeAction(ActionsOnRule.DECLARE_VERSION_AS_RECORD.getActionValue(), targetNode);
     }
 
     /**
      * Add WORM lock to a node using v1 actions api
      *
-     * @param targetNode
+     * @param targetNode the node on which the action is executed
-     *            the node on which the action is executed
      * @throws Exception
      */
     @SneakyThrows
     public JSONObject addWORMLock(RepoTestModel targetNode)
     {
         return getRmRestWrapper().withCoreAPI().usingActions()
-                .executeAction(ActionsOnRule.WORM_LOCK.getActionValue(), targetNode);
+                                 .executeAction(ActionsOnRule.WORM_LOCK.getActionValue(), targetNode);
     }
 
     /**
      * WORM lock a node for a period of days
      *
-     * @param targetNode
+     * @param targetNode      the node on which the action is executed
-     *            the node on which the action is executed
+     * @param retentionPeriod the retention period in days for the WORM lock
-     * @param retentionPeriod
-     *            the retention period in days for the WORM lock
      * @throws Exception
      */
     @SneakyThrows
     public JSONObject addWORMLock(RepoTestModel targetNode, int retentionPeriod)
     {
         return getRmRestWrapper().withCoreAPI().usingActions()
-                .executeAction(ActionsOnRule.WORM_LOCK.getActionValue(), targetNode,
+                                 .executeAction(ActionsOnRule.WORM_LOCK.getActionValue(), targetNode,
-                        ImmutableMap.of("retentionPeriod", String.valueOf(retentionPeriod)));
+                                         ImmutableMap.of("retentionPeriod", String.valueOf(retentionPeriod)));
     }
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/requests/gscore/api/FilePlanAPI.java

@@ -26,16 +26,15 @@
  */
 package org.alfresco.rest.rm.community.requests.gscore.api;
 
-import static org.apache.commons.lang3.StringUtils.EMPTY;
-import static org.springframework.http.HttpMethod.GET;
-import static org.springframework.http.HttpMethod.POST;
-import static org.springframework.http.HttpMethod.PUT;
-
 import static org.alfresco.rest.core.RestRequest.requestWithBody;
 import static org.alfresco.rest.core.RestRequest.simpleRequest;
 import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryObject;
 import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryString;
 import static org.alfresco.rest.rm.community.util.PojoUtility.toJson;
+import static org.apache.commons.lang3.StringUtils.EMPTY;
+import static org.springframework.http.HttpMethod.GET;
+import static org.springframework.http.HttpMethod.POST;
+import static org.springframework.http.HttpMethod.PUT;
 
 import org.alfresco.rest.core.RMRestWrapper;
 import org.alfresco.rest.rm.community.model.fileplan.FilePlan;
@@ -57,8 +56,7 @@ public class FilePlanAPI extends RMModelRequest
     /**
      * Constructor.
      *
-     * @param rmRestWrapper
+     * @param rmRestWrapper RM REST Wrapper
-     *            RM REST Wrapper
      */
     public FilePlanAPI(RMRestWrapper rmRestWrapper)
     {
@@ -78,19 +76,16 @@ public class FilePlanAPI extends RMModelRequest
     /**
      * Gets a file plan.
      *
-     * @param filePlanId
+     * @param filePlanId The identifier of a file plan
-     *            The identifier of a file plan
+     * @param parameters The URL parameters to add
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link FilePlan} for the given {@code filePlanId}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>{@code filePlanId} is not a valid format</li>
-     *             <li>{@code filePlanId} is not a valid format</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to read {@code filePlanId}</li>
-     *             <li>current user does not have permission to read {@code filePlanId}</li>
+     *  <li>{@code filePlanId} does not exist</li>
-     *             <li>{@code filePlanId} does not exist</li>
+     * </ul>
-     *             </ul>
      */
     public FilePlan getFilePlan(String filePlanId, String parameters)
     {
@@ -100,7 +95,8 @@ public class FilePlanAPI extends RMModelRequest
                 GET,
                 "/file-plans/{filePlanId}?{parameters}",
                 filePlanId,
-                parameters));
+                parameters
+        ));
     }
 
     /**
@@ -116,28 +112,26 @@ public class FilePlanAPI extends RMModelRequest
     /**
      * Gets the children (root categories) of a file plan.
      *
-     * @param filePlanId
+     * @param filePlanId The identifier of a file plan
-     *            The identifier of a file plan
+     * @param parameters The URL parameters to add
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link RecordCategoryCollection} for the given {@code filePlanId}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to read {@code filePlanId}</li>
-     *             <li>current user does not have permission to read {@code filePlanId}</li>
+     *  <li>{@code filePlanId} does not exist</li>
-     *             <li>{@code filePlanId} does not exist</li>
+     *</ul>
-     *             </ul>
      */
     public RecordCategoryCollection getRootRecordCategories(String filePlanId, String parameters)
     {
         mandatoryString("filePlanId", filePlanId);
 
         return getRmRestWrapper().processModels(RecordCategoryCollection.class, simpleRequest(
-                GET,
+            GET,
-                "file-plans/{filePlanId}/categories?{parameters}",
+            "file-plans/{filePlanId}/categories?{parameters}",
-                filePlanId,
+            filePlanId,
-                parameters));
+            parameters
+        ));
     }
 
     /**
@@ -154,23 +148,19 @@ public class FilePlanAPI extends RMModelRequest
     /**
      * Creates a root record category.
      *
-     * @param recordCategoryModel
+     * @param recordCategoryModel The record category model which holds the information
-     *            The record category model which holds the information
+     * @param filePlanId The identifier of a file plan
-     * @param filePlanId
+     * @param parameters The URL parameters to add
-     *            The identifier of a file plan
-     * @param parameters
-     *            The URL parameters to add
      * @return The created {@link RecordCategory}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>{@code filePlanId} is not a valid format or {@code filePlanId} is invalid</li>
-     *             <li>{@code filePlanId} is not a valid format or {@code filePlanId} is invalid</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to add children to {@code filePlanId}</li>
-     *             <li>current user does not have permission to add children to {@code filePlanId}</li>
+     *  <li>{@code filePlanIds} does not exist</li>
-     *             <li>{@code filePlanIds} does not exist</li>
+     *  <li>new name clashes with an existing node in the current parent container</li>
-     *             <li>new name clashes with an existing node in the current parent container</li>
+     *  <li>model integrity exception, including node name with invalid characters</li>
-     *             <li>model integrity exception, including node name with invalid characters</li>
+     * </ul>
-     *             </ul>
      */
     public RecordCategory createRootRecordCategory(RecordCategory recordCategoryModel, String filePlanId, String parameters)
     {
@@ -182,7 +172,8 @@ public class FilePlanAPI extends RMModelRequest
                 toJson(recordCategoryModel),
                 "file-plans/{filePlanId}/categories?{parameters}",
                 filePlanId,
-                parameters));
+                parameters
+        ));
     }
 
     /**
@@ -199,21 +190,17 @@ public class FilePlanAPI extends RMModelRequest
     /**
      * Updates a file plan.
      *
-     * @param filePlanModel
+     * @param filePlanModel The file plan  model which holds the information
-     *            The file plan model which holds the information
+     * @param filePlanId    The identifier of the file plan
-     * @param filePlanId
+     * @param parameters          The URL parameters to add
-     *            The identifier of the file plan
+     * @throws RuntimeException for the following cases:
-     * @param parameters
+     *                   <ul>
-     *            The URL parameters to add
+     *                   <li>the update request is invalid or {@code filePlanId} is not a valid format or {@code filePlanModel} is invalid</li>
-     * @throws RuntimeException
+     *                   <li>authentication fails</li>
-     *             for the following cases:
+     *                   <li>current user does not have permission to update {@code filePlanId}</li>
-     *             <ul>
+     *                   <li>{@code filePlanId} does not exist</li>
-     *             <li>the update request is invalid or {@code filePlanId} is not a valid format or {@code filePlanModel} is invalid</li>
+     *                   <li>model integrity exception, including file name with invalid characters</li>
-     *             <li>authentication fails</li>
+     *                   </ul>
-     *             <li>current user does not have permission to update {@code filePlanId}</li>
-     *             <li>{@code filePlanId} does not exist</li>
-     *             <li>model integrity exception, including file name with invalid characters</li>
-     *             </ul>
      */
     public FilePlan updateFilePlan(FilePlan filePlanModel, String filePlanId, String parameters)
     {
@@ -231,22 +218,18 @@ public class FilePlanAPI extends RMModelRequest
     /**
      * Creates a hold.
      *
-     * @param holdModel
+     * @param holdModel The hold model
-     *            The hold model
+     * @param filePlanId The identifier of a file plan
-     * @param filePlanId
+     * @param parameters The URL parameters to add
-     *            The identifier of a file plan
-     * @param parameters
-     *            The URL parameters to add
      * @return The created {@link Hold}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>{@code filePlanId} is not a valid format or {@code filePlanId} is invalid</li>
-     *             <li>{@code filePlanId} is not a valid format or {@code filePlanId} is invalid</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to add children to {@code filePlanId}</li>
-     *             <li>current user does not have permission to add children to {@code filePlanId}</li>
+     *  <li>{@code filePlanIds} does not exist</li>
-     *             <li>{@code filePlanIds} does not exist</li>
+     *  <li>new name clashes with an existing node in the current parent container</li>
-     *             <li>new name clashes with an existing node in the current parent container</li>
+     * </ul>
-     *             </ul>
      */
     public Hold createHold(Hold holdModel, String filePlanId, String parameters)
     {
@@ -254,11 +237,12 @@ public class FilePlanAPI extends RMModelRequest
         mandatoryObject("holdModel", holdModel);
 
         return getRmRestWrapper().processModel(Hold.class, requestWithBody(
-                POST,
+            POST,
-                toJson(holdModel),
+            toJson(holdModel),
-                "file-plans/{filePlanId}/holds",
+            "file-plans/{filePlanId}/holds",
-                filePlanId,
+            filePlanId,
-                parameters));
+            parameters
+                                                                          ));
     }
 
     /**
@@ -272,28 +256,26 @@ public class FilePlanAPI extends RMModelRequest
     /**
      * Gets the holds of a file plan.
      *
-     * @param filePlanId
+     * @param filePlanId The identifier of a file plan
-     *            The identifier of a file plan
+     * @param parameters The URL parameters to add
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link HoldCollection} for the given {@code filePlanId}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to read {@code filePlanId}</li>
-     *             <li>current user does not have permission to read {@code filePlanId}</li>
+     *  <li>{@code filePlanId} does not exist</li>
-     *             <li>{@code filePlanId} does not exist</li>
+     *</ul>
-     *             </ul>
      */
     public HoldCollection getHolds(String filePlanId, String parameters)
     {
         mandatoryString("filePlanId", filePlanId);
 
         return getRmRestWrapper().processModels(HoldCollection.class, simpleRequest(
-                GET,
+            GET,
-                "file-plans/{filePlanId}/holds?{parameters}",
+            "file-plans/{filePlanId}/holds?{parameters}",
-                filePlanId,
+            filePlanId,
-                parameters));
+            parameters
+                                                                                   ));
     }
 
     /**

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/requests/gscore/api/FilesAPI.java

@@ -26,10 +26,9 @@
  */
 package org.alfresco.rest.rm.community.requests.gscore.api;
 
-import static org.springframework.http.HttpMethod.POST;
-
 import static org.alfresco.rest.core.RestRequest.simpleRequest;
 import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryString;
+import static org.springframework.http.HttpMethod.POST;
 
 import org.alfresco.rest.core.RMRestWrapper;
 import org.alfresco.rest.rm.community.model.record.Record;
@@ -46,8 +45,7 @@ public class FilesAPI extends RMModelRequest<FilesAPI>
     public static final String PARENT_ID_PARAM = "parentId";
 
     /**
-     * @param rmRestWrapper
+     * @param rmRestWrapper RM REST Wrapper
-     *            RM REST Wrapper
      */
     public FilesAPI(RMRestWrapper rmRestWrapper)
     {
@@ -57,20 +55,20 @@ public class FilesAPI extends RMModelRequest<FilesAPI>
     /**
      * Declare file as record
      *
-     * @param fileId
+     * @param fileId The Id of a file to declare as record
-     *            The Id of a file to declare as record
      * @return The {@link Record} for created record
-     * @throws RuntimeException
+     * @throws RuntimeException for malformed JSON responses
-     *             for malformed JSON responses
      */
     public Record declareAsRecord(String fileId)
     {
         mandatoryString("fileId", fileId);
 
         return getRmRestWrapper().processModel(Record.class, simpleRequest(
-                POST,
+            POST,
-                "/files/{fileId}/declare?{parameters}",
+            "/files/{fileId}/declare?{parameters}",
-                fileId,
+            fileId,
-                getRmRestWrapper().getParameters()));
+            getRmRestWrapper().getParameters()
+        ));
     }
 }
+

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/requests/gscore/api/HoldsAPI.java

@@ -26,17 +26,16 @@
  */
 package org.alfresco.rest.rm.community.requests.gscore.api;
 
-import static org.apache.commons.lang3.StringUtils.EMPTY;
-import static org.springframework.http.HttpMethod.DELETE;
-import static org.springframework.http.HttpMethod.GET;
-import static org.springframework.http.HttpMethod.POST;
-import static org.springframework.http.HttpMethod.PUT;
-
 import static org.alfresco.rest.core.RestRequest.requestWithBody;
 import static org.alfresco.rest.core.RestRequest.simpleRequest;
 import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryObject;
 import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryString;
 import static org.alfresco.rest.rm.community.util.PojoUtility.toJson;
+import static org.apache.commons.lang3.StringUtils.EMPTY;
+import static org.springframework.http.HttpMethod.DELETE;
+import static org.springframework.http.HttpMethod.GET;
+import static org.springframework.http.HttpMethod.POST;
+import static org.springframework.http.HttpMethod.PUT;
 
 import org.alfresco.rest.core.RMRestWrapper;
 import org.alfresco.rest.rm.community.model.hold.BulkBodyCancel;
@@ -69,29 +68,27 @@ public class HoldsAPI extends RMModelRequest
     /**
      * Gets a hold.
      *
-     * @param holdId
+     * @param holdId The identifier of a hold
-     *            The identifier of a hold
+     * @param parameters The URL parameters to add
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link Hold} for the given {@code holdId}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>{@code holdId} is not a valid format</li>
-     *             <li>{@code holdId} is not a valid format</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to read {@code holdId}</li>
-     *             <li>current user does not have permission to read {@code holdId}</li>
+     *  <li>{@code holdId} does not exist</li>
-     *             <li>{@code holdId} does not exist</li>
+     * </ul>
-     *             </ul>
      */
     public Hold getHold(String holdId, String parameters)
     {
         mandatoryString("holdId", holdId);
 
         return getRmRestWrapper().processModel(Hold.class, simpleRequest(
-                GET,
+            GET,
-                "holds/{holdId}?{parameters}",
+            "holds/{holdId}?{parameters}",
-                holdId,
+            holdId,
-                parameters));
+            parameters
+                                                                        ));
     }
 
     /**
@@ -107,20 +104,16 @@ public class HoldsAPI extends RMModelRequest
     /**
      * Updates a hold.
      *
-     * @param holdModel
+     * @param holdModel     The hold model which holds the information
-     *            The hold model which holds the information
+     * @param holdId        The identifier of the hold
-     * @param holdId
+     * @param parameters          The URL parameters to add
-     *            The identifier of the hold
+     * @throws RuntimeException for the following cases:
-     * @param parameters
+     * <ul>
-     *            The URL parameters to add
+     *  <li>the update request is invalid or {@code holdId} is not a valid format or {@code holdModel} is invalid</li>
-     * @throws RuntimeException
+     *  <li>authentication fails</li>
-     *             for the following cases:
+     *  <li>current user does not have permission to update {@code holdId}</li>
-     *             <ul>
+     *  <li>{@code holdId} does not exist</li>
-     *             <li>the update request is invalid or {@code holdId} is not a valid format or {@code holdModel} is invalid</li>
+     * </ul>
-     *             <li>authentication fails</li>
-     *             <li>current user does not have permission to update {@code holdId}</li>
-     *             <li>{@code holdId} does not exist</li>
-     *             </ul>
      */
     public Hold updateHold(Hold holdModel, String holdId, String parameters)
     {
@@ -128,11 +121,12 @@ public class HoldsAPI extends RMModelRequest
         mandatoryString("holdId", holdId);
 
         return getRmRestWrapper().processModel(Hold.class, requestWithBody(
-                PUT,
+            PUT,
-                toJson(holdModel),
+            toJson(holdModel),
-                "holds/{holdId}?{parameters}",
+            "holds/{holdId}?{parameters}",
-                holdId,
+            holdId,
-                parameters));
+            parameters
+                                                                            ));
     }
 
     /**
@@ -149,42 +143,38 @@ public class HoldsAPI extends RMModelRequest
     /**
      * Deletes a hold.
      *
-     * @param holdId
+     * @param holdId The identifier of a hold
-     *            The identifier of a hold
+     * @throws RuntimeException for the following cases:
-     * @throws RuntimeException
+     * <ul>
-     *             for the following cases:
+     *  <li>{@code holdId} is not a valid format</li>
-     *             <ul>
+     *  <li>authentication fails</li>
-     *             <li>{@code holdId} is not a valid format</li>
+     *  <li>current user does not have permission to delete {@code holdId}</li>
-     *             <li>authentication fails</li>
+     *  <li>{@code holdId} does not exist</li>
-     *             <li>current user does not have permission to delete {@code holdId}</li>
+     * </ul>
-     *             <li>{@code holdId} does not exist</li>
-     *             </ul>
      */
     public void deleteHold(String holdId)
     {
         mandatoryString("holdId", holdId);
 
         getRmRestWrapper().processEmptyModel(simpleRequest(
-                DELETE,
+            DELETE,
-                "holds/{holdId}",
+            "holds/{holdId}",
-                holdId));
+            holdId
+                                                          ));
     }
 
     /**
      * Deletes a hold and stores a reason for deletion in the audit log.
      *
-     * @param reason
+     * @param reason        The reason for hold deletion
-     *            The reason for hold deletion
+     * @param holdId        The identifier of a hold
-     * @param holdId
+     * @throws RuntimeException for the following cases:
-     *            The identifier of a hold
+     * <ul>
-     * @throws RuntimeException
+     *  <li>{@code holdId} is not a valid format or {@code reason} is invalid</li>
-     *             for the following cases:
+     *  <li>authentication fails</li>
-     *             <ul>
+     *  <li>current user does not have permission to delete {@code holdId}</li>
-     *             <li>{@code holdId} is not a valid format or {@code reason} is invalid</li>
+     *  <li>{@code holdId} does not exist</li>
-     *             <li>authentication fails</li>
+     * </ul>
-     *             <li>current user does not have permission to delete {@code holdId}</li>
-     *             <li>{@code holdId} does not exist</li>
-     *             </ul>
      */
     public HoldDeletionReason deleteHoldWithReason(HoldDeletionReason reason, String holdId)
     {
@@ -192,41 +182,38 @@ public class HoldsAPI extends RMModelRequest
         mandatoryString("holdId", holdId);
 
         return getRmRestWrapper().processModel(HoldDeletionReason.class, requestWithBody(
-                POST,
+            POST,
-                toJson(reason),
+            toJson(reason),
-                "holds/{holdId}/delete",
+            "holds/{holdId}/delete",
-                holdId));
+            holdId
+                                                          ));
     }
 
     /**
      * Adds the relationship between a child and a parent hold.
      *
-     * @param holdChild
+     * @param holdChild The hold child model
-     *            The hold child model
+     * @param holdId The identifier of a hold
-     * @param holdId
+     * @param parameters The URL parameters to add
-     *            The identifier of a hold
-     * @param parameters
-     *            The URL parameters to add
      * @return The created {@link Hold}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>{@code holdId} is not a valid format or {@code holdId} is invalid</li>
-     *             <li>{@code holdId} is not a valid format or {@code holdId} is invalid</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to add children to {@code holdId}</li>
-     *             <li>current user does not have permission to add children to {@code holdId}</li>
+     *  <li>{@code holdId} does not exist</li>
-     *             <li>{@code holdId} does not exist</li>
+     * </ul>
-     *             </ul>
      */
     public HoldChild addChildToHold(HoldChild holdChild, String holdId, String parameters)
     {
         mandatoryObject("holdId", holdId);
 
         return getRmRestWrapper().processModel(HoldChild.class, requestWithBody(
-                POST,
+            POST,
-                toJson(holdChild),
+            toJson(holdChild),
-                "holds/{holdId}/children",
+            "holds/{holdId}/children",
-                holdId,
+            holdId,
-                parameters));
+            parameters));
     }
 
     /**
@@ -240,28 +227,26 @@ public class HoldsAPI extends RMModelRequest
     /**
      * Gets the children of a hold.
      *
-     * @param holdId
+     * @param holdId The identifier of a hold
-     *            The identifier of a hold
+     * @param parameters The URL parameters to add
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link HoldChildCollection} for the given {@code holdId}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to read {@code holdId}</li>
-     *             <li>current user does not have permission to read {@code holdId}</li>
+     *  <li>{@code holdId} does not exist</li>
-     *             <li>{@code holdId} does not exist</li>
+     *</ul>
-     *             </ul>
      */
     public HoldChildCollection getChildren(String holdId, String parameters)
     {
         mandatoryString("holdId", holdId);
 
         return getRmRestWrapper().processModels(HoldChildCollection.class, simpleRequest(
-                GET,
+            GET,
-                "holds/{holdId}/children",
+            "holds/{holdId}/children",
-                holdId,
+            holdId,
-                parameters));
+            parameters
+                                                                                   ));
     }
 
     /**
@@ -275,20 +260,16 @@ public class HoldsAPI extends RMModelRequest
     /**
      * Deletes the relationship between a child and a parent hold.
      *
-     * @param holdChildId
+     * @param holdChildId The identifier of hold child
-     *            The identifier of hold child
+     * @param holdId The identifier of a hold
-     * @param holdId
+     * @param parameters The URL parameters to add
-     *            The identifier of a hold
+     * @throws RuntimeException for the following cases:
-     * @param parameters
+     * <ul>
-     *            The URL parameters to add
+     *  <li>{@code holdId} or {@code holdChildId} is invalid</li>
-     * @throws RuntimeException
+     *  <li>authentication fails</li>
-     *             for the following cases:
+     *  <li>current user does not have permission to delete children from {@code holdId}</li>
-     *             <ul>
+     *  <li>{@code holdId} does not exist</li>
-     *             <li>{@code holdId} or {@code holdChildId} is invalid</li>
+     * </ul>
-     *             <li>authentication fails</li>
-     *             <li>current user does not have permission to delete children from {@code holdId}</li>
-     *             <li>{@code holdId} does not exist</li>
-     *             </ul>
      */
     public void deleteHoldChild(String holdId, String holdChildId, String parameters)
     {
@@ -296,11 +277,12 @@ public class HoldsAPI extends RMModelRequest
         mandatoryString("holdChildId", holdChildId);
 
         getRmRestWrapper().processEmptyModel(simpleRequest(
-                DELETE,
+            DELETE,
-                "holds/{holdId}/children/{holdChildId}",
+            "holds/{holdId}/children/{holdChildId}",
-                holdId,
+            holdId,
-                holdChildId,
+            holdChildId,
-                parameters));
+            parameters
+                                                          ));
     }
 
     /**
@@ -314,21 +296,17 @@ public class HoldsAPI extends RMModelRequest
     /**
      * Starts a bulk process for a hold.
      *
-     * @param holdBulkOperation
+     * @param holdBulkOperation The bulk operation details
-     *            The bulk operation details
+     * @param hold The identifier of a hold
-     * @param hold
+     * @param parameters The URL parameters to add
-     *            The identifier of a hold
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link HoldBulkOperationEntry} for the started bulk process
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>{@code hold} or {@code holdBulkOperation} is invalid</li>
-     *             <li>{@code hold} or {@code holdBulkOperation} is invalid</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to start a bulk process for {@code hold}</li>
-     *             <li>current user does not have permission to start a bulk process for {@code hold}</li>
+     *  <li>{@code hold} does not exist</li>
-     *             <li>{@code hold} does not exist</li>
+     * </ul>
-     *             </ul>
      */
     public HoldBulkOperationEntry startBulkProcess(HoldBulkOperation holdBulkOperation, String hold, String parameters)
     {
@@ -336,11 +314,12 @@ public class HoldsAPI extends RMModelRequest
         mandatoryString("hold", hold);
 
         return getRmRestWrapper().processModel(HoldBulkOperationEntry.class, requestWithBody(
-                POST,
+            POST,
-                toJson(holdBulkOperation),
+            toJson(holdBulkOperation),
-                "holds/{hold}/bulk",
+            "holds/{hold}/bulk",
-                hold,
+            hold,
-                parameters));
+            parameters
+                                                                                          ));
     }
 
     /**
@@ -354,21 +333,17 @@ public class HoldsAPI extends RMModelRequest
     /**
      * Gets the status of a bulk process for a hold.
      *
-     * @param holdId
+     * @param holdId The identifier of a hold
-     *            The identifier of a hold
+     * @param holdBulkStatusId The identifier of a bulk status operation
-     * @param holdBulkStatusId
+     * @param parameters The URL parameters to add
-     *            The identifier of a bulk status operation
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link HoldBulkStatus} for the given {@code holdId} and {@code holdBulkStatusId}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>{@code holdId} or {@code holdBulkStatusId} is invalid</li>
-     *             <li>{@code holdId} or {@code holdBulkStatusId} is invalid</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to get the bulk status for {@code holdId}</li>
-     *             <li>current user does not have permission to get the bulk status for {@code holdId}</li>
+     *  <li>{@code holdId} or {@code holdBulkStatusId} does not exist</li>
-     *             <li>{@code holdId} or {@code holdBulkStatusId} does not exist</li>
+     * </ul>
-     *             </ul>
      */
     public HoldBulkStatus getBulkStatus(String holdId, String holdBulkStatusId, String parameters)
     {
@@ -376,11 +351,12 @@ public class HoldsAPI extends RMModelRequest
         mandatoryString("holdBulkStatusId", holdBulkStatusId);
 
         return getRmRestWrapper().processModel(HoldBulkStatus.class, simpleRequest(
-                GET,
+            GET,
-                "holds/{holdId}/bulk-statuses/{holdBulkStatusId}",
+            "holds/{holdId}/bulk-statuses/{holdBulkStatusId}",
-                holdId,
+            holdId,
-                holdBulkStatusId,
+            holdBulkStatusId,
-                parameters));
+            parameters
+                                                                                   ));
     }
 
     /**
@@ -394,29 +370,27 @@ public class HoldsAPI extends RMModelRequest
     /**
      * Gets the statuses of all bulk processes for a hold.
      *
-     * @param holdId
+     * @param holdId The identifier of a hold
-     *            The identifier of a hold
+     * @param parameters The URL parameters to add
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link HoldBulkStatusCollection} for the given {@code holdId}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *     <li>{@code holdId} is invalid</li>
-     *             <li>{@code holdId} is invalid</li>
+     *     <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *     <li>current user does not have permission to get the bulk statuses for {@code holdId}</li>
-     *             <li>current user does not have permission to get the bulk statuses for {@code holdId}</li>
+     *     <li>{@code holdId} does not exist</li>
-     *             <li>{@code holdId} does not exist</li>
+     * </ul>
-     *             </ul>
      */
     public HoldBulkStatusCollection getBulkStatuses(String holdId, String parameters)
     {
         mandatoryString("holdId", holdId);
 
         return getRmRestWrapper().processModels(HoldBulkStatusCollection.class, simpleRequest(
-                GET,
+            GET,
-                "holds/{holdId}/bulk-statuses",
+            "holds/{holdId}/bulk-statuses",
-                holdId,
+            holdId,
-                parameters));
+            parameters
+                                                                                             ));
     }
 
     /**
@@ -430,22 +404,17 @@ public class HoldsAPI extends RMModelRequest
     /**
      * Cancels a bulk operation for a hold.
      *
-     * @param holdId
+     * @param holdId The identifier of a hold
-     *            The identifier of a hold
+     * @param bulkStatusId The identifier of a bulk status operation
-     * @param bulkStatusId
+     * @param bulkBodyCancel The bulk body cancel model
-     *            The identifier of a bulk status operation
+     * @param parameters The URL parameters to add
-     * @param bulkBodyCancel
+     * @throws RuntimeException for the following cases:
-     *            The bulk body cancel model
+     * <ul>
-     * @param parameters
+     *     <li>{@code holdId}, {@code bulkStatusId} or {@code bulkBodyCancel} is invalid</li>
-     *            The URL parameters to add
+     *     <li>authentication fails</li>
-     * @throws RuntimeException
+     *     <li>current user does not have permission to cancel the bulk operation for {@code bulkStatusId}</li>
-     *             for the following cases:
+     *     <li>{@code holdId} or {@code bulkStatusId} does not exist</li>
-     *             <ul>
+     * </ul>
-     *             <li>{@code holdId}, {@code bulkStatusId} or {@code bulkBodyCancel} is invalid</li>
-     *             <li>authentication fails</li>
-     *             <li>current user does not have permission to cancel the bulk operation for {@code bulkStatusId}</li>
-     *             <li>{@code holdId} or {@code bulkStatusId} does not exist</li>
-     *             </ul>
      */
     public void cancelBulkOperation(String holdId, String bulkStatusId, BulkBodyCancel bulkBodyCancel, String parameters)
     {
@@ -454,12 +423,13 @@ public class HoldsAPI extends RMModelRequest
         mandatoryObject("bulkBodyCancel", bulkBodyCancel);
 
         getRmRestWrapper().processEmptyModel(requestWithBody(
-                POST,
+            POST,
-                toJson(bulkBodyCancel),
+            toJson(bulkBodyCancel),
-                "holds/{holdId}/bulk-statuses/{bulkStatusId}/cancel",
+            "holds/{holdId}/bulk-statuses/{bulkStatusId}/cancel",
-                holdId,
+            holdId,
-                bulkStatusId,
+            bulkStatusId,
-                parameters));
+            parameters
+                                                          ));
     }
 
     /**

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/requests/gscore/api/RMSiteAPI.java

@@ -26,17 +26,16 @@
  */
 package org.alfresco.rest.rm.community.requests.gscore.api;
 
+import static org.alfresco.rest.core.RestRequest.requestWithBody;
+import static org.alfresco.rest.core.RestRequest.simpleRequest;
+import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryObject;
+import static org.alfresco.rest.rm.community.util.PojoUtility.toJson;
 import static org.springframework.http.HttpMethod.DELETE;
 import static org.springframework.http.HttpMethod.GET;
 import static org.springframework.http.HttpMethod.POST;
 import static org.springframework.http.HttpMethod.PUT;
 import static org.springframework.http.HttpStatus.OK;
 
-import static org.alfresco.rest.core.RestRequest.requestWithBody;
-import static org.alfresco.rest.core.RestRequest.simpleRequest;
-import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryObject;
-import static org.alfresco.rest.rm.community.util.PojoUtility.toJson;
-
 import org.alfresco.rest.core.RMRestWrapper;
 import org.alfresco.rest.rm.community.model.site.RMSite;
 import org.alfresco.rest.rm.community.requests.RMModelRequest;
@@ -52,8 +51,7 @@ public class RMSiteAPI extends RMModelRequest
     /**
      * Constructor
      *
-     * @param rmRestWrapper
+     * @param rmRestWrapper RM REST Wrapper
-     *            RM REST Wrapper
      */
     public RMSiteAPI(RMRestWrapper rmRestWrapper)
     {
@@ -64,35 +62,34 @@ public class RMSiteAPI extends RMModelRequest
      * Get the RM site
      *
      * @return The {@link RMSite} for the given file plan component id
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>Api Response code 400 Invalid parameter: GET request is supported only for the RM site</li>
-     *             <li>Api Response code 400 Invalid parameter: GET request is supported only for the RM site</li>
+     *  <li>Api Response code 401 If authentication failed</li>
-     *             <li>Api Response code 401 If authentication failed</li>
+     *  <li>Api Response code 409 If RM Site does not exist</li>
-     *             <li>Api Response code 409 If RM Site does not exist</li>
+     *  <li>Api Response code default Unexpected error</li>
-     *             <li>Api Response code default Unexpected error</li>
+     * </ul>
-     *             </ul>
      */
     public RMSite getSite()
     {
         return getRmRestWrapper().processModel(RMSite.class, simpleRequest(
                 GET,
-                "gs-sites/rm"));
+                "gs-sites/rm"
+        ));
     }
 
     /**
      * Create the RM site
      *
-     * @param rmSiteModel
+     * @param rmSiteModel The properties of the rm site to be created
-     *            The properties of the rm site to be created
      * @return The {@link RMSite} with the given properties
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>Api Response code 400 Invalid parameter: title, or description exceed the maximum length; or siteBodyCreate invalid</li>
-     *             <li>Api Response code 400 Invalid parameter: title, or description exceed the maximum length; or siteBodyCreate invalid</li>
+     *  <li>Api Response code 401 If authentication failed</
-     *             <li>Api Response code 401 If authentication failed</<li>Api Response code 409 RM Site already exists</li>
+     *  <li>Api Response code 409 RM Site already exists</li>
-     *             <li>Api Response code default Unexpected error</li>
+     *  <li>Api Response code default Unexpected error</li>
-     *             </ul>
+     * </ul>
      */
     public RMSite createRMSite(RMSite rmSiteModel)
     {
@@ -101,43 +98,42 @@ public class RMSiteAPI extends RMModelRequest
         return getRmRestWrapper().processModel(RMSite.class, requestWithBody(
                 POST,
                 toJson(rmSiteModel),
-                "gs-sites"));
+                "gs-sites"
+        ));
     }
 
     /**
      * Delete RM site
-     * 
+     * @throws RuntimeException for the following cases:
-     * @throws RuntimeException
+     * <ul>
-     *             for the following cases:
+     *  <li>Api Response code 400 Invalid parameter: DELETE request is supported only for the RM site</li>
-     *             <ul>
+     *  <li>Api Response code 401 If authentication failed</
-     *             <li>Api Response code 400 Invalid parameter: DELETE request is supported only for the RM site</li>
+     *  <li>Api Response code 403 Current user does not have permission to delete the site that is visible to them.</li>
-     *             <li>Api Response code 401 If authentication failed</<li>Api Response code 403 Current user does not have permission to delete the site that is visible to them.</li>
+     *  <li>Api Response code 404 RM site does not exist</li>
-     *             <li>Api Response code 404 RM site does not exist</li>
+     *  <li>Api Response code default Unexpected error</li>
-     *             <li>Api Response code default Unexpected error</li>
+     * </ul>
-     *             </ul>
      */
     public void deleteRMSite()
     {
         getRmRestWrapper().processEmptyModel(simpleRequest(
                 DELETE,
-                "gs-sites/rm"));
+                "gs-sites/rm"
+        ));
     }
 
     /**
      * Update RM site
      *
-     * @param rmSiteModel
+     * @param rmSiteModel The properties to be updated
-     *            The properties to be updated
      * @return The updated {@link RMSite}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>Api Response code 400 the update request is invalid {@code rmSiteModel} is invalid</li>
-     *             <li>Api Response code 400 the update request is invalid {@code rmSiteModel} is invalid</li>
+     *  <li>Api Response code 401 If authentication fails</li>
-     *             <li>Api Response code 401 If authentication fails</li>
+     *  <li>Api Response code 403 does not have permission to update {@code RMSite}</li>
-     *             <li>Api Response code 403 does not have permission to update {@code RMSite}</li>
+     *  <li>Api Response code 404 {@code RMSiteModel} does not exist</li>
-     *             <li>Api Response code 404 {@code RMSiteModel} does not exist</li>
+     *  <li>Api Response code default Unexpected error,model integrity exception</li>
-     *             <li>Api Response code default Unexpected error,model integrity exception</li>
+     * </ul>
-     *             </ul>
      */
     public RMSite updateRMSite(RMSite rmSiteModel)
     {
@@ -146,21 +142,21 @@ public class RMSiteAPI extends RMModelRequest
         return getRmRestWrapper().processModel(RMSite.class, requestWithBody(
                 PUT,
                 toJson(rmSiteModel),
-                "gs-sites/rm"));
+                "gs-sites/rm"
+        ));
     }
 
     /**
      * Checks if the RM site exists or not
      *
      * @return <code>true</code> if the RM site exists, <code>false</code> otherwise
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>Api Response code 400 Invalid parameter: GET request is supported only for the RM site</li>
-     *             <li>Api Response code 400 Invalid parameter: GET request is supported only for the RM site</li>
+     *  <li>Api Response code 401 If authentication failed</li>
-     *             <li>Api Response code 401 If authentication failed</li>
+     *  <li>Api Response code 409 If RM Site does not exist</li>
-     *             <li>Api Response code 409 If RM Site does not exist</li>
+     *  <li>Api Response code default Unexpected error</li>
-     *             <li>Api Response code default Unexpected error</li>
+     * </ul>
-     *             </ul>
      */
     public boolean existsRMSite()
     {

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/requests/gscore/api/RMUserAPI.java

@@ -26,16 +26,17 @@
  */
 package org.alfresco.rest.rm.community.requests.gscore.api;
 
-import static io.restassured.RestAssured.basic;
-import static io.restassured.RestAssured.given;
 import static org.jglue.fluentjson.JsonBuilderFactory.buildObject;
 import static org.springframework.http.HttpStatus.OK;
 
 import com.google.gson.JsonObject;
+
 import io.restassured.builder.RequestSpecBuilder;
 import io.restassured.http.ContentType;
 import io.restassured.response.Response;
 import io.restassured.specification.RequestSpecification;
+import static io.restassured.RestAssured.basic;
+import static io.restassured.RestAssured.given;
 
 import org.alfresco.dataprep.AlfrescoHttpClient;
 import org.alfresco.dataprep.AlfrescoHttpClientFactory;
@@ -58,8 +59,7 @@ import org.alfresco.utility.model.UserModel;
 public class RMUserAPI extends RMModelRequest
 {
     /**
-     * @param rmRestWrapper
+     * @param rmRestWrapper RM REST Wrapper
-     *            RM REST Wrapper
      */
     public RMUserAPI(RMRestWrapper rmRestWrapper)
     {
@@ -68,7 +68,6 @@ public class RMUserAPI extends RMModelRequest
 
     /**
      * Helper method to obtain {@link AlfrescoHttpClient}
-     * 
      * @return Initialized {@link AlfrescoHttpClient} instance
      */
     private AlfrescoHttpClient getAlfrescoHttpClient()
@@ -86,12 +85,9 @@ public class RMUserAPI extends RMModelRequest
     /**
      * Assign RM role to user
      *
-     * @param userName
+     * @param userName User's username
-     *            User's username
+     * @param userRole User's RM role, one of {@link UserRoles} roles
-     * @param userRole
+     * @throws RuntimeException for failed requests
-     *            User's RM role, one of {@link UserRoles} roles
-     * @throws RuntimeException
-     *             for failed requests
      */
     public void assignRoleToUser(String userName, String userRole)
     {
@@ -122,13 +118,9 @@ public class RMUserAPI extends RMModelRequest
 
     /**
      * Helper method to add permission on a component to user
-     * 
+     * @param filePlanComponentId The id of the file plan component on which permission should be given
-     * @param filePlanComponentId
+     * @param user {@link UserModel} for a user to be granted permission
-     *            The id of the file plan component on which permission should be given
+     * @param permission {@link UserPermissions} to be granted
-     * @param user
-     *            {@link UserModel} for a user to be granted permission
-     * @param permission
-     *            {@link UserPermissions} to be granted
      */
     public void addUserPermission(String filePlanComponentId, UserModel user, UserPermissions permission)
     {
@@ -169,10 +161,9 @@ public class RMUserAPI extends RMModelRequest
     /**
      * Helper method to set permission inheritance on a file plan component
      *
-     * @param filePlanComponentId
+     * @param filePlanComponentId The id of the file plan component on which inherited permission should be set
-     *            The id of the file plan component on which inherited permission should be set
+     * @param isInherited          true if the permission is inherited
-     * @param isInherited
+     *                             false if the permission inheritance is disabled
-     *            true if the permission is inherited false if the permission inheritance is disabled
      */
     public void setUserPermissionInheritance(String filePlanComponentId, Boolean isInherited)
     {
@@ -208,15 +199,13 @@ public class RMUserAPI extends RMModelRequest
         getRmRestWrapper().setStatusCode(Integer.toString(response.getStatusCode()));
     }
 
+
     /**
      * Creates a user with the given name using the old APIs
      *
-     * @param userName
+     * @param userName The user name
-     *            The user name
+     * @param userPassword The user's password
-     * @param userPassword
+     * @param userEmail The user's e-mail address
-     *            The user's password
-     * @param userEmail
-     *            The user's e-mail address
      * @return <code>true</code> if the user was created successfully, <code>false</code> otherwise.
      */
     public boolean createUser(String userName, String userPassword, String userEmail)
@@ -225,29 +214,29 @@ public class RMUserAPI extends RMModelRequest
         final AlfrescoHttpClient client = getAlfrescoHttpClient();
 
         JsonObject body = buildObject()
-                .add("userName", userName)
+            .add("userName", userName)
-                .add("firstName", userName)
+            .add("firstName", userName)
-                .add("lastName", userName)
+            .add("lastName", userName)
-                .add("password", userPassword)
+            .add("password", userPassword)
-                .add("email", userEmail)
+            .add("email", userEmail)
-                .getJson();
+            .getJson();
 
         final RequestSpecification spec = new RequestSpecBuilder()
-                .setBaseUri(client.getApiUrl())
+            .setBaseUri(client.getApiUrl())
-                .setBasePath("/")
+            .setBasePath("/")
-                .setAuth(basic(adminUser.getUsername(), adminUser.getPassword()))
+            .setAuth(basic(adminUser.getUsername(), adminUser.getPassword()))
-                .setContentType(ContentType.JSON)
+            .setContentType(ContentType.JSON)
-                .setBody(body.toString())
+            .setBody(body.toString())
-                .build();
+            .build();
 
         // create POST request to "people" endpoint
         Response response = given()
-                .spec(spec)
+            .spec(spec)
-                .log().all()
+            .log().all()
-                .when()
+        .when()
-                .post("people")
+            .post("people")
-                .prettyPeek()
+            .prettyPeek()
-                .andReturn();
+            .andReturn();
 
         return (response.getStatusCode() == OK.value());
     }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/requests/gscore/api/RecordCategoryAPI.java

@@ -26,17 +26,16 @@
  */
 package org.alfresco.rest.rm.community.requests.gscore.api;
 
-import static org.apache.commons.lang3.StringUtils.EMPTY;
-import static org.springframework.http.HttpMethod.DELETE;
-import static org.springframework.http.HttpMethod.GET;
-import static org.springframework.http.HttpMethod.POST;
-import static org.springframework.http.HttpMethod.PUT;
-
 import static org.alfresco.rest.core.RestRequest.requestWithBody;
 import static org.alfresco.rest.core.RestRequest.simpleRequest;
 import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryObject;
 import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryString;
 import static org.alfresco.rest.rm.community.util.PojoUtility.toJson;
+import static org.apache.commons.lang3.StringUtils.EMPTY;
+import static org.springframework.http.HttpMethod.DELETE;
+import static org.springframework.http.HttpMethod.GET;
+import static org.springframework.http.HttpMethod.POST;
+import static org.springframework.http.HttpMethod.PUT;
 
 import org.alfresco.rest.core.RMRestWrapper;
 import org.alfresco.rest.rm.community.model.recordcategory.RecordCategory;
@@ -55,8 +54,7 @@ public class RecordCategoryAPI extends RMModelRequest
     /**
      * Constructor.
      *
-     * @param rmRestWrapper
+     * @param rmRestWrapper RM REST Wrapper
-     *            RM REST Wrapper
      */
     public RecordCategoryAPI(RMRestWrapper rmRestWrapper)
     {
@@ -66,17 +64,15 @@ public class RecordCategoryAPI extends RMModelRequest
     /**
      * Deletes a record category.
      *
-     * @param recordCategoryId
+     * @param recordCategoryId The identifier of a record category
-     *            The identifier of a record category
+     * @throws RuntimeException for the following cases:
-     * @throws RuntimeException
+     * <ul>
-     *             for the following cases:
+     *  <li>{@code recordCategoryId} is not a valid format</li>
-     *             <ul>
+     *  <li>authentication fails</li>
-     *             <li>{@code recordCategoryId} is not a valid format</li>
+     *  <li>current user does not have permission to delete {@code recordCategoryId}</li>
-     *             <li>authentication fails</li>
+     *  <li>{@code recordCategoryId} does not exist</li>
-     *             <li>current user does not have permission to delete {@code recordCategoryId}</li>
+     *  <li>{@code recordCategoryId} is locked and cannot be deleted</li>
-     *             <li>{@code recordCategoryId} does not exist</li>
+     * </ul>
-     *             <li>{@code recordCategoryId} is locked and cannot be deleted</li>
-     *             </ul>
      */
     public void deleteRecordCategory(String recordCategoryId)
     {
@@ -85,7 +81,8 @@ public class RecordCategoryAPI extends RMModelRequest
         getRmRestWrapper().processEmptyModel(simpleRequest(
                 DELETE,
                 "record-categories/{recordCategoryId}",
-                recordCategoryId));
+                recordCategoryId
+        ));
     }
 
     /**
@@ -101,19 +98,16 @@ public class RecordCategoryAPI extends RMModelRequest
     /**
      * Gets a record category.
      *
-     * @param recordCategoryId
+     * @param recordCategoryId The identifier of a record category
-     *            The identifier of a record category
+     * @param parameters The URL parameters to add
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link RecordCategory} for the given {@code recordCategoryId}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>{@code recordCategoryId} is not a valid format</li>
-     *             <li>{@code recordCategoryId} is not a valid format</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to read {@code recordCategoryId}</li>
-     *             <li>current user does not have permission to read {@code recordCategoryId}</li>
+     *  <li>{@code recordCategoryId} does not exist</li>
-     *             <li>{@code recordCategoryId} does not exist</li>
+     * </ul>
-     *             </ul>
      */
     public RecordCategory getRecordCategory(String recordCategoryId, String parameters)
     {
@@ -123,7 +117,8 @@ public class RecordCategoryAPI extends RMModelRequest
                 GET,
                 "record-categories/{recordCategoryId}?{parameters}",
                 recordCategoryId,
-                parameters));
+                parameters
+        ));
     }
 
     /**
@@ -140,24 +135,19 @@ public class RecordCategoryAPI extends RMModelRequest
     /**
      * Updates a record category.
      *
-     * @param recordCategoryModel
+     * @param recordCategoryModel The record category model which holds the information
-     *            The record category model which holds the information
+     * @param recordCategoryId The identifier of a record category
-     * @param recordCategoryId
+     * @param parameters The URL parameters to add
-     *            The identifier of a record category
+     * @param returns The updated {@link RecordCategory}
-     * @param parameters
+     * @throws RuntimeException for the following cases:
-     *            The URL parameters to add
+     * <ul>
-     * @param returns
+     *  <li>the update request is invalid or {@code recordCategoryId} is not a valid format or {@code recordCategoryModel} is invalid</li>
-     *            The updated {@link RecordCategory}
+     *  <li>authentication fails</li>
-     * @throws RuntimeException
+     *  <li>current user does not have permission to update {@code recordCategoryId}</li>
-     *             for the following cases:
+     *  <li>{@code recordCategoryId} does not exist</li>
-     *             <ul>
+     *  <li>the updated name clashes with an existing record category in the current parent category</li>
-     *             <li>the update request is invalid or {@code recordCategoryId} is not a valid format or {@code recordCategoryModel} is invalid</li>
+     *  <li>model integrity exception, including file name with invalid characters</li>
-     *             <li>authentication fails</li>
+     * </ul>
-     *             <li>current user does not have permission to update {@code recordCategoryId}</li>
-     *             <li>{@code recordCategoryId} does not exist</li>
-     *             <li>the updated name clashes with an existing record category in the current parent category</li>
-     *             <li>model integrity exception, including file name with invalid characters</li>
-     *             </ul>
      */
     public RecordCategory updateRecordCategory(RecordCategory recordCategoryModel, String recordCategoryId, String parameters)
     {
@@ -169,7 +159,8 @@ public class RecordCategoryAPI extends RMModelRequest
                 toJson(recordCategoryModel),
                 "record-categories/{recordCategoryId}?{parameters}",
                 recordCategoryId,
-                parameters));
+                parameters
+        ));
     }
 
     /**
@@ -185,28 +176,26 @@ public class RecordCategoryAPI extends RMModelRequest
     /**
      * Gets the children of a record category.
      *
-     * @param recordCategoryId
+     * @param recordCategoryId The identifier of a record category
-     *            The identifier of a record category
+     * @param parameters The URL parameters to add
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link RecordCategoryChildCollection} for the given {@code recordCategoryId}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to read {@code recordCategoryId}</li>
-     *             <li>current user does not have permission to read {@code recordCategoryId}</li>
+     *  <li>{@code recordCategoryId} does not exist</li>
-     *             <li>{@code recordCategoryId} does not exist</li>
+     *</ul>
-     *             </ul>
      */
     public RecordCategoryChildCollection getRecordCategoryChildren(String recordCategoryId, String parameters)
     {
         mandatoryString("recordCategoryId", recordCategoryId);
 
         return getRmRestWrapper().processModels(RecordCategoryChildCollection.class, simpleRequest(
-                GET,
+            GET,
-                "record-categories/{recordCategoryId}/children?{parameters}",
+            "record-categories/{recordCategoryId}/children?{parameters}",
-                recordCategoryId,
+            recordCategoryId,
-                parameters));
+            parameters
+        ));
     }
 
     /**
@@ -223,23 +212,19 @@ public class RecordCategoryAPI extends RMModelRequest
     /**
      * Creates a record category child. Can be a record category or a record folder.
      *
-     * @param recordCategoryChildModel
+     * @param recordCategoryChildModel The record category child model which holds the information
-     *            The record category child model which holds the information
+     * @param recordCategoryId The identifier of a record category
-     * @param recordCategoryId
+     * @param parameters The URL parameters to add
-     *            The identifier of a record category
-     * @param parameters
-     *            The URL parameters to add
      * @return The created {@link RecordCategoryChild}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>{@code recordCategoryId} is not a valid format or {@code recordCategoryChildModel} is invalid</li>
-     *             <li>{@code recordCategoryId} is not a valid format or {@code recordCategoryChildModel} is invalid</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to add children to {@code recordCategoryId}</li>
-     *             <li>current user does not have permission to add children to {@code recordCategoryId}</li>
+     *  <li>{@code recordCategoryId} does not exist</li>
-     *             <li>{@code recordCategoryId} does not exist</li>
+     *  <li>new name clashes with an existing node in the current parent container</li>
-     *             <li>new name clashes with an existing node in the current parent container</li>
+     *  <li>model integrity exception, including node name with invalid characters</li>
-     *             <li>model integrity exception, including node name with invalid characters</li>
+     * </ul>
-     *             </ul>
      */
     public RecordCategoryChild createRecordCategoryChild(RecordCategoryChild recordCategoryChildModel, String recordCategoryId, String parameters)
     {
@@ -251,6 +236,7 @@ public class RecordCategoryAPI extends RMModelRequest
                 toJson(recordCategoryChildModel),
                 "record-categories/{recordCategoryId}/children?{parameters}",
                 recordCategoryId,
-                parameters));
+                parameters
+        ));
     }
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/requests/gscore/api/RecordFolderAPI.java

@@ -26,19 +26,18 @@
  */
 package org.alfresco.rest.rm.community.requests.gscore.api;
 
-import static org.apache.commons.lang3.StringUtils.EMPTY;
-import static org.springframework.http.HttpMethod.DELETE;
-import static org.springframework.http.HttpMethod.GET;
-import static org.springframework.http.HttpMethod.POST;
-import static org.springframework.http.HttpMethod.PUT;
-import static org.testng.Assert.fail;
-
 import static org.alfresco.rest.core.RestRequest.requestWithBody;
 import static org.alfresco.rest.core.RestRequest.simpleRequest;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentType.CONTENT_TYPE;
 import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryObject;
 import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryString;
 import static org.alfresco.rest.rm.community.util.PojoUtility.toJson;
+import static org.apache.commons.lang3.StringUtils.EMPTY;
+import static org.springframework.http.HttpMethod.DELETE;
+import static org.springframework.http.HttpMethod.GET;
+import static org.springframework.http.HttpMethod.POST;
+import static org.springframework.http.HttpMethod.PUT;
+import static org.testng.Assert.fail;
 
 import java.io.File;
 import java.io.IOException;
@@ -67,8 +66,7 @@ public class RecordFolderAPI extends RMModelRequest
     /**
      * Constructor.
      *
-     * @param rmRestWrapper
+     * @param rmRestWrapper RM REST Wrapper
-     *            RM REST Wrapper
      */
     public RecordFolderAPI(RMRestWrapper rmRestWrapper)
     {
@@ -78,17 +76,15 @@ public class RecordFolderAPI extends RMModelRequest
     /**
      * Deletes a record folder.
      *
-     * @param recordFolderId
+     * @param recordFolderId The identifier of a record folder
-     *            The identifier of a record folder
+     * @throws RuntimeException for the following cases:
-     * @throws RuntimeException
+     * <ul>
-     *             for the following cases:
+     *  <li>{@code recordFolderId} is not a valid format</li>
-     *             <ul>
+     *  <li>authentication fails</li>
-     *             <li>{@code recordFolderId} is not a valid format</li>
+     *  <li>current user does not have permission to delete {@code recordFolderId}</li>
-     *             <li>authentication fails</li>
+     *  <li>{@code recordFolderId} does not exist</li>
-     *             <li>current user does not have permission to delete {@code recordFolderId}</li>
+     *  <li>{@code recordFolderId} is locked and cannot be deleted</li>
-     *             <li>{@code recordFolderId} does not exist</li>
+     * </ul>
-     *             <li>{@code recordFolderId} is locked and cannot be deleted</li>
-     *             </ul>
      */
     public void deleteRecordFolder(String recordFolderId)
     {
@@ -97,7 +93,8 @@ public class RecordFolderAPI extends RMModelRequest
         getRmRestWrapper().processEmptyModel(simpleRequest(
                 DELETE,
                 "record-folders/{recordFolderId}",
-                recordFolderId));
+                recordFolderId
+        ));
     }
 
     /**
@@ -113,19 +110,16 @@ public class RecordFolderAPI extends RMModelRequest
     /**
      * Gets a record folder.
      *
-     * @param recordFolderId
+     * @param recordFolderId The identifier of a record folder
-     *            The identifier of a record folder
+     * @param parameters The URL parameters to add
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link RecordFolder} for the given {@code recordFolderId}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>{@code recordFolderId} is not a valid format</li>
-     *             <li>{@code recordFolderId} is not a valid format</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to read {@code recordFolderId}</li>
-     *             <li>current user does not have permission to read {@code recordFolderId}</li>
+     *  <li>{@code recordFolderId} does not exist</li>
-     *             <li>{@code recordFolderId} does not exist</li>
+     * </ul>
-     *             </ul>
      */
     public RecordFolder getRecordFolder(String recordFolderId, String parameters)
     {
@@ -135,7 +129,8 @@ public class RecordFolderAPI extends RMModelRequest
                 GET,
                 "record-folders/{recordFolderId}?{parameters}",
                 recordFolderId,
-                parameters));
+                parameters
+        ));
     }
 
     /**
@@ -152,24 +147,19 @@ public class RecordFolderAPI extends RMModelRequest
     /**
      * Updates a record folder.
      *
-     * @param recordFolderModel
+     * @param recordFolderModel The record folder model which holds the information
-     *            The record folder model which holds the information
+     * @param recordFolderId The identifier of a record folder
-     * @param recordFolderId
+     * @param parameters The URL parameters to add
-     *            The identifier of a record folder
+     * @param returns The updated {@link RecordFolder}
-     * @param parameters
+     * @throws RuntimeException for the following cases:
-     *            The URL parameters to add
+     * <ul>
-     * @param returns
+     *  <li>the update request is invalid or {@code recordFolderId} is not a valid format or {@code recordFolderModel} is invalid</li>
-     *            The updated {@link RecordFolder}
+     *  <li>authentication fails</li>
-     * @throws RuntimeException
+     *  <li>current user does not have permission to update {@code recordFolderId}</li>
-     *             for the following cases:
+     *  <li>{@code recordFolderId} does not exist</li>
-     *             <ul>
+     *  <li>the updated name clashes with an existing record folder in the current parent category</li>
-     *             <li>the update request is invalid or {@code recordFolderId} is not a valid format or {@code recordFolderModel} is invalid</li>
+     *  <li>model integrity exception, including file name with invalid characters</li>
-     *             <li>authentication fails</li>
+     * </ul>
-     *             <li>current user does not have permission to update {@code recordFolderId}</li>
-     *             <li>{@code recordFolderId} does not exist</li>
-     *             <li>the updated name clashes with an existing record folder in the current parent category</li>
-     *             <li>model integrity exception, including file name with invalid characters</li>
-     *             </ul>
      */
     public RecordFolder updateRecordFolder(RecordFolder recordFolderModel, String recordFolderId, String parameters)
     {
@@ -181,7 +171,8 @@ public class RecordFolderAPI extends RMModelRequest
                 toJson(recordFolderModel),
                 "record-folders/{recordFolderId}?{parameters}",
                 recordFolderId,
-                parameters));
+                parameters
+        ));
     }
 
     /**
@@ -197,28 +188,26 @@ public class RecordFolderAPI extends RMModelRequest
     /**
      * Gets the children of a record folder.
      *
-     * @param recordFolderId
+     * @param recordFolderId The identifier of a record folder
-     *            The identifier of a record folder
+     * @param parameters The URL parameters to add
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link RecordFolderCollection} for the given {@code recordFolderId}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to read {@code recordFolderId}</li>
-     *             <li>current user does not have permission to read {@code recordFolderId}</li>
+     *  <li>{@code recordFolderId} does not exist</li>
-     *             <li>{@code recordFolderId} does not exist</li>
+     *</ul>
-     *             </ul>
      */
     public RecordFolderCollection getRecordFolderChildren(String recordFolderId, String parameters)
     {
         mandatoryString("recordFolderId", recordFolderId);
 
         return getRmRestWrapper().processModels(RecordFolderCollection.class, simpleRequest(
-                GET,
+            GET,
-                "record-folders/{recordFolderId}/records?{parameters}",
+            "record-folders/{recordFolderId}/records?{parameters}",
-                recordFolderId,
+            recordFolderId,
-                parameters));
+            parameters
+        ));
     }
 
     /**
@@ -235,15 +224,11 @@ public class RecordFolderAPI extends RMModelRequest
     /**
      * Create a record from file resource
      *
-     * @param recordModel
+     * @param recordModel {@link Record} for electronic record to be created
-     *            {@link Record} for electronic record to be created
+     * @param recordContent {@link File} pointing to the content of the electronic record to be created
-     * @param recordContent
+     * @param recordFolderId The identifier of a record folder
-     *            {@link File} pointing to the content of the electronic record to be created
-     * @param recordFolderId
-     *            The identifier of a record folder
      * @return newly created {@link Record}
-     * @throws RuntimeException
+     * @throws RuntimeException for invalid recordModel JSON strings
-     *             for invalid recordModel JSON strings
      */
     public Record createRecord(Record recordModel, String recordFolderId, File recordContent) throws RuntimeException
     {
@@ -256,7 +241,10 @@ public class RecordFolderAPI extends RMModelRequest
             fail("Only electronic records are supported");
         }
 
-        /* For file uploads nodeBodyCreate is ignored hence can't be used. Append all Record fields to the request. */
+        /*
+         * For file uploads nodeBodyCreate is ignored hence can't be used. Append all Record fields
+         * to the request.
+         */
         RequestSpecBuilder builder = getRmRestWrapper().configureRequestSpec();
         JsonNode root;
         try
@@ -283,22 +271,18 @@ public class RecordFolderAPI extends RMModelRequest
     /**
      * Creates a record in a record folder child, i.e. a record.
      *
-     * @param recordModel
+     * @param recordModel The record model which holds the information
-     *            The record model which holds the information
+     * @param recordFolderId The identifier of a record folder
-     * @param recordFolderId
+     * @param parameters The URL parameters to add
-     *            The identifier of a record folder
-     * @param parameters
-     *            The URL parameters to add
      * @return The created {@link Record}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>{@code recordFolderId is not a valid format or {@code recordModel} is invalid</li>
-     *             <li>{@code recordFolderId is not a valid format or {@code recordModel} is invalid</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to add children to {@code recordFolderId}</li>
-     *             <li>current user does not have permission to add children to {@code recordFolderId}</li>
+     *  <li>{@code recordFolderId} does not exist</li>
-     *             <li>{@code recordFolderId} does not exist</li>
+     *  <li>model integrity exception, including node name with invalid characters</li>
-     *             <li>model integrity exception, including node name with invalid characters</li>
+     * </ul>
-     *             </ul>
      */
     public Record createRecord(Record recordModel, String recordFolderId, String parameters)
     {
@@ -310,6 +294,7 @@ public class RecordFolderAPI extends RMModelRequest
                 toJson(recordModel),
                 "record-folders/{recordFolderId}/records?{parameters}",
                 recordFolderId,
-                parameters));
+                parameters
+        ));
     }
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/requests/gscore/api/RecordsAPI.java

@@ -26,25 +26,25 @@
  */
 package org.alfresco.rest.rm.community.requests.gscore.api;
 
+import static org.alfresco.rest.core.RestRequest.requestWithBody;
+import static org.alfresco.rest.core.RestRequest.simpleRequest;
+import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryObject;
+import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryString;
+import static org.alfresco.rest.rm.community.util.PojoUtility.toJson;
 import static org.apache.commons.lang3.StringUtils.EMPTY;
 import static org.springframework.http.HttpMethod.DELETE;
 import static org.springframework.http.HttpMethod.GET;
 import static org.springframework.http.HttpMethod.POST;
 import static org.springframework.http.HttpMethod.PUT;
 
-import static org.alfresco.rest.core.RestRequest.requestWithBody;
-import static org.alfresco.rest.core.RestRequest.simpleRequest;
-import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryObject;
-import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryString;
-import static org.alfresco.rest.rm.community.util.PojoUtility.toJson;
-
-import io.restassured.response.ResponseBody;
 
 import org.alfresco.rest.core.RMRestWrapper;
 import org.alfresco.rest.rm.community.model.record.Record;
 import org.alfresco.rest.rm.community.model.record.RecordBodyFile;
 import org.alfresco.rest.rm.community.requests.RMModelRequest;
 
+import io.restassured.response.ResponseBody;
+
 /**
  * Records REST API Wrapper
  *
@@ -54,8 +54,7 @@ import org.alfresco.rest.rm.community.requests.RMModelRequest;
 public class RecordsAPI extends RMModelRequest
 {
     /**
-     * @param rmRestWrapper
+     * @param rmRestWrapper RM REST Wrapper
-     *            RM REST Wrapper
      */
     public RecordsAPI(RMRestWrapper rmRestWrapper)
     {
@@ -65,46 +64,41 @@ public class RecordsAPI extends RMModelRequest
     /**
      * Get the content for the electronic record
      *
-     * @param recordId
+     * @param recordId The id of the electronic record
-     *            The id of the electronic record
      * @return {@link ResponseBody} representing content for the given record id
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     * <li>{@code recordId} has no content</li>
-     *             <li>{@code recordId} has no content</li>
+     * <li> {@code recordId} is not a valid format, or is not a record</li>
-     *             <li>{@code recordId} is not a valid format, or is not a record</li>
+     * <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     * <li>{@code recordId} does not exist</li>
-     *             <li>{@code recordId} does not exist</li>
+     * </ul>
-     *             </ul>
      */
     public ResponseBody<?> getRecordContent(String recordId)
     {
         mandatoryString("recordId", recordId);
 
         return getRmRestWrapper()
-                .processHtmlResponse(simpleRequest(GET, "records/{recordId}/content", recordId))
+            .processHtmlResponse(simpleRequest(GET,"records/{recordId}/content", recordId))
-                .getBody();
+            .getBody();
 
     }
 
     /**
      * File the record recordId into file plan structure based on the location sent via the request body
      *
-     * @param recordBodyFile
+     * @param recordBodyFile The properties where to file the record
-     *            The properties where to file the record
+     * @param recordId       The id of the record to file
-     * @param recordId
-     *            The id of the record to file
      * @return The {@link Record} with the given properties
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>Invalid parameter: {@code recordBodyFile} is not a valid format,{@code recordId} is not a record</li>
-     *             <li>Invalid parameter: {@code recordBodyFile} is not a valid format,{@code recordId} is not a record</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to file to {@code fileplanComponentId}</li>
-     *             <li>current user does not have permission to file to {@code fileplanComponentId}</li>
+     *  <li>{@code recordId} does not exist</li>
-     *             <li>{@code recordId} does not exist</li>
+     *  <li>targetParentId from recordBodyFile does not exist</li>
-     *             <li>targetParentId from recordBodyFile does not exist</li>
+     *  <li>model integrity exception: the action breaks system's integrity restrictions</li>
-     *             <li>model integrity exception: the action breaks system's integrity restrictions</li>
+     * </ul>
-     *             </ul>
      *
      */
     public Record fileRecord(RecordBodyFile recordBodyFile, String recordId)
@@ -118,21 +112,18 @@ public class RecordsAPI extends RMModelRequest
     /**
      * File the record recordId into file plan structure based on the location sent via the request body
      *
-     * @param recordBodyFile
+     * @param recordBodyFile The properties where to file the record
-     *            The properties where to file the record
+     * @param recordId       The id of the record to file
-     * @param recordId
-     *            The id of the record to file
      * @return The {@link Record} with the given properties
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>Invalid parameter: {@code recordBodyFile} is not a valid format,{@code recordId} is not a record</li>
-     *             <li>Invalid parameter: {@code recordBodyFile} is not a valid format,{@code recordId} is not a record</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to file to {@code fileplanComponentId}</li>
-     *             <li>current user does not have permission to file to {@code fileplanComponentId}</li>
+     *  <li>{@code recordId} does not exist</li>
-     *             <li>{@code recordId} does not exist</li>
+     *  <li>targetParentId from recordBodyFile does not exist</li>
-     *             <li>targetParentId from recordBodyFile does not exist</li>
+     *  <li>model integrity exception: the action breaks system's integrity restrictions</li>
-     *             <li>model integrity exception: the action breaks system's integrity restrictions</li>
+     * </ul>
-     *             </ul>
      *
      */
     public Record fileRecord(RecordBodyFile recordBodyFile, String recordId, String parameters)
@@ -141,11 +132,12 @@ public class RecordsAPI extends RMModelRequest
         mandatoryString("recordId", recordId);
 
         return getRmRestWrapper().processModel(Record.class, requestWithBody(
-                POST,
+            POST,
-                toJson(recordBodyFile),
+            toJson(recordBodyFile),
-                "/records/{recordId}/file?{parameters}",
+            "/records/{recordId}/file?{parameters}",
-                recordId,
+            recordId,
-                parameters));
+            parameters
+        ));
     }
 
     /**
@@ -161,45 +153,41 @@ public class RecordsAPI extends RMModelRequest
     /**
      * Complete the record recordId
      *
-     * @param recordId
+     * @param recordId The id of the record to complete
-     *            The id of the record to complete
      * @return The completed {@link Record} with the given properties
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     *                   <ul>
-     *             <ul>
+     *                   <li>Invalid parameter: {@code recordId} is not a record</li>
-     *             <li>Invalid parameter: {@code recordId} is not a record</li>
+     *                   <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *                   <li>current user does not have permission to complete {@code recordId}</li>
-     *             <li>current user does not have permission to complete {@code recordId}</li>
+     *                   <li>{@code recordId} does not exist or is frozen</li>
-     *             <li>{@code recordId} does not exist or is frozen</li>
+     *                   <li>model integrity exception: the record is already completed</li>
-     *             <li>model integrity exception: the record is already completed</li>
+     *                   <li>model integrity exception: the record has missing meta-data</li>
-     *             <li>model integrity exception: the record has missing meta-data</li>
+     *                   </ul>
-     *             </ul>
      */
     public Record completeRecord(String recordId, String parameters)
     {
         mandatoryString("recordId", recordId);
 
         return getRmRestWrapper().processModel(Record.class, simpleRequest(
-                POST,
+            POST,
-                "/records/{recordId}/complete?{parameters}",
+            "/records/{recordId}/complete?{parameters}",
-                recordId,
+            recordId,
-                parameters));
+            parameters
+        ));
     }
-
     /**
      * Deletes a record.
      *
-     * @param recordId
+     * @param recordId The identifier of a record
-     *            The identifier of a record
+     * @throws RuntimeException for the following cases:
-     * @throws RuntimeException
+     * <ul>
-     *             for the following cases:
+     *  <li>{@code recordId} is not a valid format</li>
-     *             <ul>
+     *  <li>authentication fails</li>
-     *             <li>{@code recordId} is not a valid format</li>
+     *  <li>current user does not have permission to delete {@code recordId}</li>
-     *             <li>authentication fails</li>
+     *  <li>{@code recordId} does not exist</li>
-     *             <li>current user does not have permission to delete {@code recordId}</li>
+     *  <li>{@code recordId} is locked and cannot be deleted</li>
-     *             <li>{@code recordId} does not exist</li>
+     * </ul>
-     *             <li>{@code recordId} is locked and cannot be deleted</li>
-     *             </ul>
      */
     public void deleteRecord(String recordId)
     {
@@ -208,7 +196,8 @@ public class RecordsAPI extends RMModelRequest
         getRmRestWrapper().processEmptyModel(simpleRequest(
                 DELETE,
                 "records/{recordId}",
-                recordId));
+                recordId
+        ));
     }
 
     /**
@@ -224,19 +213,16 @@ public class RecordsAPI extends RMModelRequest
     /**
      * Gets a record.
      *
-     * @param recordId
+     * @param recordId The identifier of a record
-     *            The identifier of a record
+     * @param parameters The URL parameters to add
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link Record} for the given {@code recordId}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>{@code recordId} is not a valid format</li>
-     *             <li>{@code recordId} is not a valid format</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to read {@code recordId}</li>
-     *             <li>current user does not have permission to read {@code recordId}</li>
+     *  <li>{@code recordId} does not exist</li>
-     *             <li>{@code recordId} does not exist</li>
+     * </ul>
-     *             </ul>
      */
     public Record getRecord(String recordId, String parameters)
     {
@@ -246,7 +232,8 @@ public class RecordsAPI extends RMModelRequest
                 GET,
                 "records/{recordId}?{parameters}",
                 recordId,
-                parameters));
+                parameters
+        ));
     }
 
     /**
@@ -263,23 +250,19 @@ public class RecordsAPI extends RMModelRequest
     /**
      * Updates a record.
      *
-     * @param recordModel
+     * @param recordModel The record model which holds the information
-     *            The record model which holds the information
+     * @param recordId The identifier of a record
-     * @param recordId
+     * @param parameters The URL parameters to add
-     *            The identifier of a record
-     * @param parameters
-     *            The URL parameters to add
      * @return The updated {@link Record}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>the update request is invalid or {@code recordId} is not a valid format or {@code recordModel} is invalid</li>
-     *             <li>the update request is invalid or {@code recordId} is not a valid format or {@code recordModel} is invalid</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to update {@code recordId}</li>
-     *             <li>current user does not have permission to update {@code recordId}</li>
+     *  <li>{@code recordId} does not exist</li>
-     *             <li>{@code recordId} does not exist</li>
+     *  <li>the updated name clashes with an existing record in the current parent folder</li>
-     *             <li>the updated name clashes with an existing record in the current parent folder</li>
+     *  <li>model integrity exception, including file name with invalid characters</li>
-     *             <li>model integrity exception, including file name with invalid characters</li>
+     * </ul>
-     *             </ul>
      */
     public Record updateRecord(Record recordModel, String recordId, String parameters)
     {
@@ -291,6 +274,7 @@ public class RecordsAPI extends RMModelRequest
                 toJson(recordModel),
                 "records/{recordId}?{parameters}",
                 recordId,
-                parameters));
+                parameters
+        ));
     }
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/requests/gscore/api/RetentionScheduleAPI.java

@@ -26,16 +26,6 @@
  */
 package org.alfresco.rest.rm.community.requests.gscore.api;
 
-import static org.apache.commons.lang3.StringUtils.EMPTY;
-import static org.springframework.http.HttpMethod.GET;
-import static org.springframework.http.HttpMethod.POST;
-
-import static org.alfresco.rest.core.RestRequest.requestWithBody;
-import static org.alfresco.rest.core.RestRequest.simpleRequest;
-import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryObject;
-import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryString;
-import static org.alfresco.rest.rm.community.util.PojoUtility.toJson;
-
 import org.alfresco.rest.core.RMRestWrapper;
 import org.alfresco.rest.rm.community.model.retentionschedule.RetentionSchedule;
 import org.alfresco.rest.rm.community.model.retentionschedule.RetentionScheduleActionDefinition;
@@ -43,6 +33,15 @@ import org.alfresco.rest.rm.community.model.retentionschedule.RetentionScheduleC
 import org.alfresco.rest.rm.community.model.retentionschedule.RetentionScheduleStepCollection;
 import org.alfresco.rest.rm.community.requests.RMModelRequest;
 
+import static org.alfresco.rest.core.RestRequest.requestWithBody;
+import static org.alfresco.rest.core.RestRequest.simpleRequest;
+import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryObject;
+import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryString;
+import static org.alfresco.rest.rm.community.util.PojoUtility.toJson;
+import static org.apache.commons.lang3.StringUtils.EMPTY;
+import static org.springframework.http.HttpMethod.GET;
+import static org.springframework.http.HttpMethod.POST;
+
 public class RetentionScheduleAPI extends RMModelRequest
 {
 
@@ -54,25 +53,22 @@ public class RetentionScheduleAPI extends RMModelRequest
         super(rmRestWrapper);
     }
 
+
     /**
      * Creates a retention schedule.
      *
-     * @param retentionScheduleModel
+     * @param retentionScheduleModel The retentionSchedule model
-     *            The retentionSchedule model
+     * @param recordCategoryId The identifier of a record category
-     * @param recordCategoryId
+     * @param parameters The URL parameters to add
-     *            The identifier of a record category
-     * @param parameters
-     *            The URL parameters to add
      * @return The created {@link RetentionSchedule}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>{@code recordCategoryId} is not a valid format or {@code recordCategoryId} is invalid</li>
-     *             <li>{@code recordCategoryId} is not a valid format or {@code recordCategoryId} is invalid</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to add children to {@code recordCategoryId}</li>
-     *             <li>current user does not have permission to add children to {@code recordCategoryId}</li>
+     *  <li>{@code recordCategoryId} does not exist</li>
-     *             <li>{@code recordCategoryId} does not exist</li>
+     *  <li>new name clashes with an existing node in the current parent container</li>
-     *             <li>new name clashes with an existing node in the current parent container</li>
+     * </ul>
-     *             </ul>
      */
     public RetentionSchedule createRetentionSchedule(RetentionSchedule retentionScheduleModel, String recordCategoryId, String parameters)
     {
@@ -80,11 +76,12 @@ public class RetentionScheduleAPI extends RMModelRequest
         mandatoryObject("retentionScheduleModel", retentionScheduleModel);
 
         return getRmRestWrapper().processModel(RetentionSchedule.class, requestWithBody(
-                POST,
+            POST,
-                toJson(retentionScheduleModel),
+            toJson(retentionScheduleModel),
-                "record-categories/{recordCategoryId}/retention-schedules",
+            "record-categories/{recordCategoryId}/retention-schedules",
-                recordCategoryId,
+            recordCategoryId,
-                parameters));
+            parameters
+        ));
     }
 
     /**
@@ -98,28 +95,26 @@ public class RetentionScheduleAPI extends RMModelRequest
     /**
      * Gets the retentionSchedule of a record category.
      *
-     * @param recordCategoryId
+     * @param recordCategoryId The identifier of a record category
-     *            The identifier of a record category
+     * @param parameters The URL parameters to add
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link RetentionSchedule} for the given {@code recordCategoryId}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to read {@code recordCategoryId}</li>
-     *             <li>current user does not have permission to read {@code recordCategoryId}</li>
+     *  <li>{@code recordCategoryId} does not exist</li>
-     *             <li>{@code recordCategoryId} does not exist</li>
+     *</ul>
-     *             </ul>
      */
     public RetentionScheduleCollection getRetentionSchedule(String recordCategoryId, String parameters)
     {
         mandatoryString("recordCategoryId", recordCategoryId);
 
         return getRmRestWrapper().processModels(RetentionScheduleCollection.class, simpleRequest(
-                GET,
+            GET,
-                "record-categories/{recordCategoryId}/retention-schedules?{parameters}",
+            "record-categories/{recordCategoryId}/retention-schedules?{parameters}",
-                recordCategoryId,
+            recordCategoryId,
-                parameters));
+            parameters
+        ));
     }
 
     /**
@@ -133,22 +128,18 @@ public class RetentionScheduleAPI extends RMModelRequest
     /**
      * Creates a step in the retention schedule.
      *
-     * @param retentionScheduleActionDefinition
+     * @param retentionScheduleActionDefinition The retentionScheduleActionDefinition model
-     *            The retentionScheduleActionDefinition model
+     * @param retentionScheduleId The identifier of a retention schedule id
-     * @param retentionScheduleId
+     * @param parameters The URL parameters to add
-     *            The identifier of a retention schedule id
-     * @param parameters
-     *            The URL parameters to add
      * @return The created {@link RetentionScheduleActionDefinition}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>{@code retentionScheduleId} is not a valid format or {@code retentionScheduleId} is invalid</li>
-     *             <li>{@code retentionScheduleId} is not a valid format or {@code retentionScheduleId} is invalid</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to add children to {@code retentionScheduleId}</li>
-     *             <li>current user does not have permission to add children to {@code retentionScheduleId}</li>
+     *  <li>{@code retentionScheduleId} does not exist</li>
-     *             <li>{@code retentionScheduleId} does not exist</li>
+     *  <li>new name clashes with an existing node in the current parent container</li>
-     *             <li>new name clashes with an existing node in the current parent container</li>
+     * </ul>
-     *             </ul>
      */
     public RetentionScheduleActionDefinition createRetentionScheduleStep(RetentionScheduleActionDefinition retentionScheduleActionDefinition, String retentionScheduleId, String parameters)
     {
@@ -156,11 +147,12 @@ public class RetentionScheduleAPI extends RMModelRequest
         mandatoryObject("retentionScheduleActionDefinition", retentionScheduleActionDefinition);
 
         return getRmRestWrapper().processModel(RetentionScheduleActionDefinition.class, requestWithBody(
-                POST,
+            POST,
-                toJson(retentionScheduleActionDefinition),
+            toJson(retentionScheduleActionDefinition),
-                "retention-schedules/{retentionScheduleId}/retention-steps",
+            "retention-schedules/{retentionScheduleId}/retention-steps",
-                retentionScheduleId,
+            retentionScheduleId,
-                parameters));
+            parameters
+        ));
     }
 
     /**
@@ -174,28 +166,26 @@ public class RetentionScheduleAPI extends RMModelRequest
     /**
      * Gets the retentionSchedule of a record category.
      *
-     * @param retentionScheduleId
+     * @param retentionScheduleId The identifier of a record category
-     *            The identifier of a record category
+     * @param parameters The URL parameters to add
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link RetentionScheduleActionDefinition} for the given {@code recordCategoryId}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to read {@code recordCategoryId}</li>
-     *             <li>current user does not have permission to read {@code recordCategoryId}</li>
+     *  <li>{@code recordCategoryId} does not exist</li>
-     *             <li>{@code recordCategoryId} does not exist</li>
+     *</ul>
-     *             </ul>
      */
     public RetentionScheduleStepCollection getRetentionScheduleStep(String retentionScheduleId, String parameters)
     {
         mandatoryString("retentionScheduleId", retentionScheduleId);
 
         return getRmRestWrapper().processModels(RetentionScheduleStepCollection.class, simpleRequest(
-                GET,
+            GET,
-                "retention-schedules/{retentionScheduleId}/retention-steps?{parameters}",
+            "retention-schedules/{retentionScheduleId}/retention-steps?{parameters}",
-                retentionScheduleId,
+            retentionScheduleId,
-                parameters));
+            parameters
+        ));
     }
 
     /**

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/requests/gscore/api/TransferAPI.java

@@ -27,11 +27,10 @@
 
 package org.alfresco.rest.rm.community.requests.gscore.api;
 
-import static org.apache.commons.lang3.StringUtils.EMPTY;
-import static org.springframework.http.HttpMethod.GET;
-
 import static org.alfresco.rest.core.RestRequest.simpleRequest;
 import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryString;
+import static org.apache.commons.lang3.StringUtils.EMPTY;
+import static org.springframework.http.HttpMethod.GET;
 
 import org.alfresco.rest.core.RMRestWrapper;
 import org.alfresco.rest.rm.community.model.transfer.Transfer;
@@ -47,8 +46,7 @@ import org.alfresco.rest.rm.community.requests.RMModelRequest;
 public class TransferAPI extends RMModelRequest
 {
     /**
-     * @param rmRestWrapper
+     * @param rmRestWrapper RM REST Wrapper
-     *            RM REST Wrapper
      */
     public TransferAPI(RMRestWrapper rmRestWrapper)
     {
@@ -68,19 +66,16 @@ public class TransferAPI extends RMModelRequest
     /**
      * Gets a transfer.
      *
-     * @param transferId
+     * @param transferId The identifier of a transfer
-     *            The identifier of a transfer
+     * @param parameters The URL parameters to add
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link Transfer} for the given {@code transferId}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>{@code transferId} is not a valid format</li>
-     *             <li>{@code transferId} is not a valid format</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to read {@code transferId}</li>
-     *             <li>current user does not have permission to read {@code transferId}</li>
+     *  <li>{@code transferId} does not exist</li>
-     *             <li>{@code transferId} does not exist</li>
+     * </ul>
-     *             </ul>
      */
     public Transfer getTransfer(String transferId, String parameters)
     {
@@ -90,9 +85,9 @@ public class TransferAPI extends RMModelRequest
                 GET,
                 "/transfers/{transferId}?{parameters}",
                 transferId,
-                parameters));
+                parameters
+        ));
     }
-
     /**
      * see {@link #getTransfersChildren(String, String)}
      */
@@ -106,27 +101,25 @@ public class TransferAPI extends RMModelRequest
     /**
      * Gets the children (record folder or record) of a transfer.
      *
-     * @param transferId
+     * @param transferId The identifier of a transfer
-     *            The identifier of a transfer
+     * @param parameters The URL parameters to add
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link TransferChildCollection} for the given {@code transferId}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to read {@code transferId}</li>
-     *             <li>current user does not have permission to read {@code transferId}</li>
+     *  <li>{@code filePlanId} does not exist</li>
-     *             <li>{@code filePlanId} does not exist</li>
+     *</ul>
-     *             </ul>
      */
     public TransferChildCollection getTransfersChildren(String transferId, String parameters)
     {
         mandatoryString("transferId", transferId);
 
         return getRmRestWrapper().processModels(TransferChildCollection.class, simpleRequest(
-                GET,
+            GET,
-                "transfers/{filePlanId}/children?{parameters}",
+            "transfers/{filePlanId}/children?{parameters}",
-                transferId,
+            transferId,
-                parameters));
+            parameters
+        ));
     }
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/requests/gscore/api/TransferContainerAPI.java

@@ -27,15 +27,14 @@
 
 package org.alfresco.rest.rm.community.requests.gscore.api;
 
-import static org.apache.commons.lang3.StringUtils.EMPTY;
-import static org.springframework.http.HttpMethod.GET;
-import static org.springframework.http.HttpMethod.PUT;
-
 import static org.alfresco.rest.core.RestRequest.requestWithBody;
 import static org.alfresco.rest.core.RestRequest.simpleRequest;
 import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryObject;
 import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryString;
 import static org.alfresco.rest.rm.community.util.PojoUtility.toJson;
+import static org.apache.commons.lang3.StringUtils.EMPTY;
+import static org.springframework.http.HttpMethod.GET;
+import static org.springframework.http.HttpMethod.PUT;
 
 import org.alfresco.rest.core.RMRestWrapper;
 import org.alfresco.rest.rm.community.model.transfer.TransferCollection;
@@ -51,8 +50,7 @@ import org.alfresco.rest.rm.community.requests.RMModelRequest;
 public class TransferContainerAPI extends RMModelRequest
 {
     /**
-     * @param rmRestWrapper
+     * @param rmRestWrapper RM REST Wrapper
-     *            RM REST Wrapper
      */
     public TransferContainerAPI(RMRestWrapper rmRestWrapper)
     {
@@ -72,19 +70,16 @@ public class TransferContainerAPI extends RMModelRequest
     /**
      * Gets a transfer container.
      *
-     * @param transferContainerId
+     * @param transferContainerId The identifier of a transfer container
-     *            The identifier of a transfer container
+     * @param parameters The URL parameters to add
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link TransferContainer} for the given {@code transferContainerId}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>{@code transferContainerId} is not a valid format</li>
-     *             <li>{@code transferContainerId} is not a valid format</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to read {@code transferContainerId}</li>
-     *             <li>current user does not have permission to read {@code transferContainerId}</li>
+     *  <li>{@code transferContainerId} does not exist</li>
-     *             <li>{@code transferContainerId} does not exist</li>
+     * </ul>
-     *             </ul>
      */
     public TransferContainer getTransferContainer(String transferContainerId, String parameters)
     {
@@ -94,7 +89,8 @@ public class TransferContainerAPI extends RMModelRequest
                 GET,
                 "/transfer-containers/{transferContainerId}?{parameters}",
                 transferContainerId,
-                parameters));
+                parameters
+        ));
     }
 
     /**
@@ -111,24 +107,19 @@ public class TransferContainerAPI extends RMModelRequest
     /**
      * Updates a transfer container.
      *
-     * @param transferContainerModel
+     * @param transferContainerModel The transfer container model which holds the information
-     *            The transfer container model which holds the information
+     * @param transferContainerId The identifier of a transfer container
-     * @param transferContainerId
+     * @param parameters The URL parameters to add
-     *            The identifier of a transfer container
+     * @param returns The updated {@link TransferContainer}
-     * @param parameters
+     * @throws RuntimeException for the following cases:
-     *            The URL parameters to add
+     * <ul>
-     * @param returns
+     *  <li>the update request is invalid or {@code transferContainerId} is not a valid format or {@code transferContainerModel} is invalid</li>
-     *            The updated {@link TransferContainer}
+     *  <li>authentication fails</li>
-     * @throws RuntimeException
+     *  <li>current user does not have permission to update {@code transferContainerId}</li>
-     *             for the following cases:
+     *  <li>{@code transferContainerId} does not exist</li>
-     *             <ul>
+     *  <li>the updated name clashes with an existing transfer container in the current file plan</li>
-     *             <li>the update request is invalid or {@code transferContainerId} is not a valid format or {@code transferContainerModel} is invalid</li>
+     *  <li>model integrity exception, including transfer container name with invalid characters</li>
-     *             <li>authentication fails</li>
+     * </ul>
-     *             <li>current user does not have permission to update {@code transferContainerId}</li>
-     *             <li>{@code transferContainerId} does not exist</li>
-     *             <li>the updated name clashes with an existing transfer container in the current file plan</li>
-     *             <li>model integrity exception, including transfer container name with invalid characters</li>
-     *             </ul>
      */
     public TransferContainer updateTransferContainer(TransferContainer transferContainerModel, String transferContainerId, String parameters)
     {
@@ -140,7 +131,8 @@ public class TransferContainerAPI extends RMModelRequest
                 toJson(transferContainerModel),
                 "transfer-containers/{transferContainerId}?{parameters}",
                 transferContainerId,
-                parameters));
+                parameters
+        ));
     }
 
     /**
@@ -156,27 +148,25 @@ public class TransferContainerAPI extends RMModelRequest
     /**
      * Gets the children (transfers) of a transfer container.
      *
-     * @param transferContainerId
+     * @param transferContainerId The identifier of a transfer container
-     *            The identifier of a transfer container
+     * @param parameters The URL parameters to add
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link TransferCollection} for the given {@code transferContainerId}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to read {@code transferContainerId}</li>
-     *             <li>current user does not have permission to read {@code transferContainerId}</li>
+     *  <li>{@code filePlanId} does not exist</li>
-     *             <li>{@code filePlanId} does not exist</li>
+     *</ul>
-     *             </ul>
      */
     public TransferCollection getTransfers(String transferContainerId, String parameters)
     {
         mandatoryString("transferContainerId", transferContainerId);
 
         return getRmRestWrapper().processModels(TransferCollection.class, simpleRequest(
-                GET,
+            GET,
-                "transfer-containers/{filePlanId}/transfers?{parameters}",
+            "transfer-containers/{filePlanId}/transfers?{parameters}",
-                transferContainerId,
+            transferContainerId,
-                parameters));
+            parameters
+        ));
     }
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/requests/gscore/api/UnfiledContainerAPI.java

@@ -26,18 +26,17 @@
  */
 package org.alfresco.rest.rm.community.requests.gscore.api;
 
-import static org.apache.commons.lang3.StringUtils.EMPTY;
-import static org.springframework.http.HttpMethod.GET;
-import static org.springframework.http.HttpMethod.POST;
-import static org.springframework.http.HttpMethod.PUT;
-import static org.testng.Assert.fail;
-
 import static org.alfresco.rest.core.RestRequest.requestWithBody;
 import static org.alfresco.rest.core.RestRequest.simpleRequest;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentType.CONTENT_TYPE;
 import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryObject;
 import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryString;
 import static org.alfresco.rest.rm.community.util.PojoUtility.toJson;
+import static org.apache.commons.lang3.StringUtils.EMPTY;
+import static org.springframework.http.HttpMethod.GET;
+import static org.springframework.http.HttpMethod.POST;
+import static org.springframework.http.HttpMethod.PUT;
+import static org.testng.Assert.fail;
 
 import java.io.File;
 import java.io.IOException;
@@ -45,6 +44,7 @@ import java.util.Iterator;
 
 import com.fasterxml.jackson.databind.JsonNode;
 import com.fasterxml.jackson.databind.ObjectMapper;
+
 import io.restassured.builder.RequestSpecBuilder;
 import io.restassured.http.ContentType;
 
@@ -65,8 +65,7 @@ import org.alfresco.rest.rm.community.util.UnfiledContainerChildMixin;
 public class UnfiledContainerAPI extends RMModelRequest
 {
     /**
-     * @param rmRestWrapper
+     * @param rmRestWrapper RM REST Wrapper
-     *            RM REST Wrapper
      */
     public UnfiledContainerAPI(RMRestWrapper rmRestWrapper)
     {
@@ -86,19 +85,16 @@ public class UnfiledContainerAPI extends RMModelRequest
     /**
      * Gets an unfiled record container.
      *
-     * @param unfiledContainerId
+     * @param unfiledContainerId The identifier of a unfiled record container
-     *            The identifier of a unfiled record container
+     * @param parameters The URL parameters to add
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link UnfiledContainer} for the given {@code unfiledContainerId}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>{@code unfiledContainerId} is not a valid format</li>
-     *             <li>{@code unfiledContainerId} is not a valid format</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to read {@code unfiledContainerId}</li>
-     *             <li>current user does not have permission to read {@code unfiledContainerId}</li>
+     *  <li>{@code unfiledContainerId} does not exist</li>
-     *             <li>{@code unfiledContainerId} does not exist</li>
+     * </ul>
-     *             </ul>
      */
     public UnfiledContainer getUnfiledContainer(String unfiledContainerId, String parameters)
     {
@@ -108,7 +104,8 @@ public class UnfiledContainerAPI extends RMModelRequest
                 GET,
                 "unfiled-containers/{unfiledContainerId}?{parameters}",
                 unfiledContainerId,
-                parameters));
+                parameters
+        ));
     }
 
     /**
@@ -124,28 +121,26 @@ public class UnfiledContainerAPI extends RMModelRequest
     /**
      * Gets the children of an unfiled records container
      *
-     * @param unfiledContainerId
+     * @param unfiledContainerId The identifier of an unfiled records container
-     *            The identifier of an unfiled records container
+     * @param parameters The URL parameters to add
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link UnfiledContainerChildCollection} for the given {@code unfiledContainerId}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to read {@code unfiledContainerId}</li>
-     *             <li>current user does not have permission to read {@code unfiledContainerId}</li>
+     *  <li>{@code unfiledContainerId} does not exist</li>
-     *             <li>{@code unfiledContainerId} does not exist</li>
+     *</ul>
-     *             </ul>
      */
     public UnfiledContainerChildCollection getUnfiledContainerChildren(String unfiledContainerId, String parameters)
     {
         mandatoryString("unfiledContainerId", unfiledContainerId);
 
         return getRmRestWrapper().processModels(UnfiledContainerChildCollection.class, simpleRequest(
-                GET,
+            GET,
-                "unfiled-containers/{unfiledContainerId}/children?{parameters}",
+            "unfiled-containers/{unfiledContainerId}/children?{parameters}",
-                unfiledContainerId,
+            unfiledContainerId,
-                parameters));
+            parameters
+        ));
     }
 
     /**
@@ -162,23 +157,19 @@ public class UnfiledContainerAPI extends RMModelRequest
     /**
      * Creates an unfiled container child. Can be a record or an unfiled record folder.
      *
-     * @param unfiledContainerChildModel
+     * @param unfiledContainerChildModel The unfiled container child model which holds the information
-     *            The unfiled container child model which holds the information
+     * @param unfiledContainerId The identifier of an unfiled container
-     * @param unfiledContainerId
+     * @param parameters The URL parameters to add
-     *            The identifier of an unfiled container
-     * @param parameters
-     *            The URL parameters to add
      * @return The created {@link UnfiledContainerChild}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>{@code unfiledContainerId} is not a valid format or {@code unfiledContainerChildModel} is invalid</li>
-     *             <li>{@code unfiledContainerId} is not a valid format or {@code unfiledContainerChildModel} is invalid</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to add children to {@code unfiledContainerId}</li>
-     *             <li>current user does not have permission to add children to {@code unfiledContainerId}</li>
+     *  <li>{@code unfiledContainerId} does not exist</li>
-     *             <li>{@code unfiledContainerId} does not exist</li>
+     *  <li>new name clashes with an existing node in the current parent container</li>
-     *             <li>new name clashes with an existing node in the current parent container</li>
+     *  <li>model integrity exception, including node name with invalid characters</li>
-     *             <li>model integrity exception, including node name with invalid characters</li>
+     * </ul>
-     *             </ul>
      */
     public UnfiledContainerChild createUnfiledContainerChild(UnfiledContainerChild unfiledContainerChildModel, String unfiledContainerId, String parameters)
     {
@@ -190,21 +181,18 @@ public class UnfiledContainerAPI extends RMModelRequest
                 toJson(unfiledContainerChildModel),
                 "unfiled-containers/{unfiledContainerId}/children?{parameters}",
                 unfiledContainerId,
-                parameters));
+                parameters
+        ));
     }
 
     /**
      * Create a record from file resource
      *
-     * @param unfiledContainerChildModel
+     * @param unfiledContainerChildModel {@link UnfiledContainerChild} for electronic record to be created
-     *            {@link UnfiledContainerChild} for electronic record to be created
+     * @param unfiledContainerChildContent {@link File} pointing to the content of the electronic record to be created
-     * @param unfiledContainerChildContent
+     * @param unfiledContainerId The identifier of a unfiled container
-     *            {@link File} pointing to the content of the electronic record to be created
-     * @param unfiledContainerId
-     *            The identifier of a unfiled container
      * @return newly created {@link UnfiledContainerChild}
-     * @throws RuntimeException
+     * @throws RuntimeException for invalid recordModel JSON strings
-     *             for invalid recordModel JSON strings
      */
     public UnfiledContainerChild uploadRecord(UnfiledContainerChild unfiledContainerChildModel, String unfiledContainerId, File unfiledContainerChildContent)
     {
@@ -217,7 +205,10 @@ public class UnfiledContainerAPI extends RMModelRequest
             fail("Only electronic records are supported");
         }
 
-        /* For file uploads nodeBodyCreate is ignored hence can't be used. Append all Record fields to the request. */
+        /*
+         * For file uploads nodeBodyCreate is ignored hence can't be used. Append all Record fields
+         * to the request.
+         */
         RequestSpecBuilder builder = getRmRestWrapper().configureRequestSpec();
         JsonNode root;
         try
@@ -255,24 +246,19 @@ public class UnfiledContainerAPI extends RMModelRequest
     /**
      * Updates an unfiled record container
      *
-     * @param unfiledContainerModel
+     * @param unfiledContainerModel The unfiled record container model which holds the information
-     *            The unfiled record container model which holds the information
+     * @param unfiledContainerId The identifier of an unfiled record container
-     * @param unfiledContainerId
+     * @param parameters The URL parameters to add
-     *            The identifier of an unfiled record container
+     * @param returns The updated {@link UnfiledContainer}
-     * @param parameters
+     * @throws RuntimeException for the following cases:
-     *            The URL parameters to add
+     * <ul>
-     * @param returns
+     *  <li>the update request is invalid or {@code unfiledContainerId} is not a valid format or {@code unfiledContainerModel} is invalid</li>
-     *            The updated {@link UnfiledContainer}
+     *  <li>authentication fails</li>
-     * @throws RuntimeException
+     *  <li>current user does not have permission to update {@code unfiledContainerId}</li>
-     *             for the following cases:
+     *  <li>{@code unfiledContainerId} does not exist</li>
-     *             <ul>
+     *  <li>the updated name clashes with an existing root category of special container in the current fileplan</li>
-     *             <li>the update request is invalid or {@code unfiledContainerId} is not a valid format or {@code unfiledContainerModel} is invalid</li>
+     *  <li>model integrity exception, including file name with invalid characters</li>
-     *             <li>authentication fails</li>
+     * </ul>
-     *             <li>current user does not have permission to update {@code unfiledContainerId}</li>
-     *             <li>{@code unfiledContainerId} does not exist</li>
-     *             <li>the updated name clashes with an existing root category of special container in the current fileplan</li>
-     *             <li>model integrity exception, including file name with invalid characters</li>
-     *             </ul>
      */
     public UnfiledContainer updateUnfiledContainer(UnfiledContainer unfiledContainerModel, String unfiledContainerId, String parameters)
     {
@@ -284,7 +270,8 @@ public class UnfiledContainerAPI extends RMModelRequest
                 toJson(unfiledContainerModel),
                 "unfiled-containers/{unfiledContainerId}?{parameters}",
                 unfiledContainerId,
-                parameters));
+                parameters
+        ));
     }
 
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/requests/gscore/api/UnfiledRecordFolderAPI.java

@@ -26,19 +26,18 @@
  */
 package org.alfresco.rest.rm.community.requests.gscore.api;
 
-import static org.apache.commons.lang3.StringUtils.EMPTY;
-import static org.springframework.http.HttpMethod.DELETE;
-import static org.springframework.http.HttpMethod.GET;
-import static org.springframework.http.HttpMethod.POST;
-import static org.springframework.http.HttpMethod.PUT;
-import static org.testng.Assert.fail;
-
 import static org.alfresco.rest.core.RestRequest.requestWithBody;
 import static org.alfresco.rest.core.RestRequest.simpleRequest;
 import static org.alfresco.rest.rm.community.model.fileplancomponents.FilePlanComponentType.CONTENT_TYPE;
 import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryObject;
 import static org.alfresco.rest.rm.community.util.ParameterCheck.mandatoryString;
 import static org.alfresco.rest.rm.community.util.PojoUtility.toJson;
+import static org.apache.commons.lang3.StringUtils.EMPTY;
+import static org.springframework.http.HttpMethod.DELETE;
+import static org.springframework.http.HttpMethod.GET;
+import static org.springframework.http.HttpMethod.POST;
+import static org.springframework.http.HttpMethod.PUT;
+import static org.testng.Assert.fail;
 
 import java.io.File;
 import java.io.IOException;
@@ -65,8 +64,7 @@ import org.alfresco.rest.rm.community.util.UnfiledContainerChildMixin;
 public class UnfiledRecordFolderAPI extends RMModelRequest
 {
     /**
-     * @param rmRestWrapper
+     * @param rmRestWrapper RM REST Wrapper
-     *            RM REST Wrapper
      */
     public UnfiledRecordFolderAPI(RMRestWrapper rmRestWrapper)
     {
@@ -86,19 +84,16 @@ public class UnfiledRecordFolderAPI extends RMModelRequest
     /**
      * Gets an unfiled record folder.
      *
-     * @param unfiledRecordFolderId
+     * @param unfiledRecordFolderId The identifier of a unfiled record folder
-     *            The identifier of a unfiled record folder
+     * @param parameters The URL parameters to add
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link UnfiledRecordFolder} for the given {@code unfiledRecordFolderId}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>{@code unfiledRecordFolderId} is not a valid format</li>
-     *             <li>{@code unfiledRecordFolderId} is not a valid format</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to read {@code unfiledRecordFolderId}</li>
-     *             <li>current user does not have permission to read {@code unfiledRecordFolderId}</li>
+     *  <li>{@code unfiledRecordFolderId} does not exist</li>
-     *             <li>{@code unfiledRecordFolderId} does not exist</li>
+     * </ul>
-     *             </ul>
      */
     public UnfiledRecordFolder getUnfiledRecordFolder(String unfiledRecordFolderId, String parameters)
     {
@@ -108,7 +103,8 @@ public class UnfiledRecordFolderAPI extends RMModelRequest
                 GET,
                 "unfiled-record-folders/{unfiledRecordFolderId}?{parameters}",
                 unfiledRecordFolderId,
-                parameters));
+                parameters
+        ));
     }
 
     /**
@@ -124,28 +120,26 @@ public class UnfiledRecordFolderAPI extends RMModelRequest
     /**
      * Gets the children of an unfiled record folder
      *
-     * @param unfiledRecordFolderId
+     * @param unfiledRecordFolderId The identifier of an unfiled records folder
-     *            The identifier of an unfiled records folder
+     * @param parameters The URL parameters to add
-     * @param parameters
-     *            The URL parameters to add
      * @return The {@link UnfiledRecordFolderChildCollection} for the given {@code unfiledRecordFolderId}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to read {@code unfiledRecordFolderId}</li>
-     *             <li>current user does not have permission to read {@code unfiledRecordFolderId}</li>
+     *  <li>{@code unfiledRecordFolderId} does not exist</li>
-     *             <li>{@code unfiledRecordFolderId} does not exist</li>
+     *</ul>
-     *             </ul>
      */
     public UnfiledContainerChildCollection getUnfiledRecordFolderChildren(String unfiledRecordFolderId, String parameters)
     {
         mandatoryString("unfiledRecordFolderId", unfiledRecordFolderId);
 
         return getRmRestWrapper().processModels(UnfiledContainerChildCollection.class, simpleRequest(
-                GET,
+            GET,
-                "unfiled-record-folders/{unfiledRecordFolderId}/children?{parameters}",
+            "unfiled-record-folders/{unfiledRecordFolderId}/children?{parameters}",
-                unfiledRecordFolderId,
+            unfiledRecordFolderId,
-                parameters));
+            parameters
+        ));
     }
 
     /**
@@ -162,23 +156,19 @@ public class UnfiledRecordFolderAPI extends RMModelRequest
     /**
      * Creates an unfiled record folder child. Can be a record or an unfiled record folder.
      *
-     * @param unfiledRecordFolderChildModel
+     * @param unfiledRecordFolderChildModel The unfiled folder child model which holds the information
-     *            The unfiled folder child model which holds the information
+     * @param unfiledRecordFolderId The identifier of an unfiled folder
-     * @param unfiledRecordFolderId
+     * @param parameters The URL parameters to add
-     *            The identifier of an unfiled folder
-     * @param parameters
-     *            The URL parameters to add
      * @return The created {@link UnfiledRecordFolderChild}
-     * @throws RuntimeException
+     * @throws RuntimeException for the following cases:
-     *             for the following cases:
+     * <ul>
-     *             <ul>
+     *  <li>{@code unfiledRecordFolderId} is not a valid format or {@code unfiledRecordFolderChildModel} is invalid</li>
-     *             <li>{@code unfiledRecordFolderId} is not a valid format or {@code unfiledRecordFolderChildModel} is invalid</li>
+     *  <li>authentication fails</li>
-     *             <li>authentication fails</li>
+     *  <li>current user does not have permission to add children to {@code unfiledRecordFolderId}</li>
-     *             <li>current user does not have permission to add children to {@code unfiledRecordFolderId}</li>
+     *  <li>{@code unfiledRecordFolderId} does not exist</li>
-     *             <li>{@code unfiledRecordFolderId} does not exist</li>
+     *  <li>new name clashes with an existing node in the current parent container</li>
-     *             <li>new name clashes with an existing node in the current parent container</li>
+     *  <li>model integrity exception, including node name with invalid characters</li>
-     *             <li>model integrity exception, including node name with invalid characters</li>
+     * </ul>
-     *             </ul>
      */
     public UnfiledContainerChild createUnfiledRecordFolderChild(UnfiledContainerChild unfiledRecordFolderChildModel, String unfiledRecordFolderId, String parameters)
     {
@@ -190,21 +180,18 @@ public class UnfiledRecordFolderAPI extends RMModelRequest
                 toJson(unfiledRecordFolderChildModel),
                 "unfiled-record-folders/{unfiledRecordFolderId}/children?{parameters}",
                 unfiledRecordFolderId,
-                parameters));
+                parameters
+        ));
     }
 
     /**
      * Create a record from file resource
      *
-     * @param unfiledRecordFolderChildModel
+     * @param unfiledRecordFolderChildModel {@link UnfiledContainerChild} for electronic record to be created
-     *            {@link UnfiledContainerChild} for electronic record to be created
+     * @param unfiledRecordFolderChildContent {@link File} pointing to the content of the electronic record to be created
-     * @param unfiledRecordFolderChildContent
+     * @param unfiledRecordFolderId The identifier of a unfiled record folder
-     *            {@link File} pointing to the content of the electronic record to be created
-     * @param unfiledRecordFolderId
-     *            The identifier of a unfiled record folder
      * @return newly created {@link UnfiledContainerChild}
-     * @throws RuntimeException
+     * @throws RuntimeException for invalid recordModel JSON strings
-     *             for invalid recordModel JSON strings
      */
     public UnfiledContainerChild uploadRecord(UnfiledContainerChild unfiledRecordFolderChildModel, String unfiledRecordFolderId, File unfiledRecordFolderChildContent)
     {
@@ -217,7 +204,10 @@ public class UnfiledRecordFolderAPI extends RMModelRequest
             fail("Only electronic records are supported");
         }
 
-        /* For file uploads nodeBodyCreate is ignored hence can't be used. Append all Record fields to the request. */
+        /*
+         * For file uploads nodeBodyCreate is ignored hence can't be used. Append all Record fields
+         * to the request.
+         */
         RequestSpecBuilder builder = getRmRestWrapper().configureRequestSpec();
         JsonNode root;
         try
@@ -255,24 +245,19 @@ public class UnfiledRecordFolderAPI extends RMModelRequest
     /**
      * Updates an unfiled record folder
      *
-     * @param unfiledRecordFolderModel
+     * @param unfiledRecordFolderModel The unfiled record folder model which holds the information
-     *            The unfiled record folder model which holds the information
+     * @param unfiledRecordFolderId The identifier of an unfiled record folder
-     * @param unfiledRecordFolderId
+     * @param parameters The URL parameters to add
-     *            The identifier of an unfiled record folder
+     * @param returns The updated {@link UnfiledRecordFolder}
-     * @param parameters
+     * @throws RuntimeException for the following cases:
-     *            The URL parameters to add
+     * <ul>
-     * @param returns
+     *  <li>the update request is invalid or {@code unfiledRecordFolderId} is not a valid format or {@code unfiledRecordFolderModel} is invalid</li>
-     *            The updated {@link UnfiledRecordFolder}
+     *  <li>authentication fails</li>
-     * @throws RuntimeException
+     *  <li>current user does not have permission to update {@code unfiledRecordFolderId}</li>
-     *             for the following cases:
+     *  <li>{@code unfiledRecordFolderId} does not exist</li>
-     *             <ul>
+     *  <li>the updated name clashes with an existing root category of special container in the current fileplan</li>
-     *             <li>the update request is invalid or {@code unfiledRecordFolderId} is not a valid format or {@code unfiledRecordFolderModel} is invalid</li>
+     *  <li>model integrity exception, including file name with invalid characters</li>
-     *             <li>authentication fails</li>
+     * </ul>
-     *             <li>current user does not have permission to update {@code unfiledRecordFolderId}</li>
-     *             <li>{@code unfiledRecordFolderId} does not exist</li>
-     *             <li>the updated name clashes with an existing root category of special container in the current fileplan</li>
-     *             <li>model integrity exception, including file name with invalid characters</li>
-     *             </ul>
      */
     public UnfiledRecordFolder updateUnfiledRecordFolder(UnfiledRecordFolder unfiledRecordFolderModel, String unfiledRecordFolderId, String parameters)
     {
@@ -284,23 +269,22 @@ public class UnfiledRecordFolderAPI extends RMModelRequest
                 toJson(unfiledRecordFolderModel),
                 "unfiled-record-folders/{unfiledRecordFolderId}?{parameters}",
                 unfiledRecordFolderId,
-                parameters));
+                parameters
+        ));
     }
 
     /**
      * Deletes an unfiled record folder.
      *
-     * @param unfiledRecordFolderId
+     * @param unfiledRecordFolderId The identifier of a unfiled record folder
-     *            The identifier of a unfiled record folder
+     * @throws RuntimeException for the following cases:
-     * @throws RuntimeException
+     * <ul>
-     *             for the following cases:
+     *  <li>{@code unfiledRecordFolderId} is not a valid format</li>
-     *             <ul>
+     *  <li>authentication fails</li>
-     *             <li>{@code unfiledRecordFolderId} is not a valid format</li>
+     *  <li>current user does not have permission to delete {@code unfiledRecordFolderId}</li>
-     *             <li>authentication fails</li>
+     *  <li>{@code unfiledRecordFolderId} does not exist</li>
-     *             <li>current user does not have permission to delete {@code unfiledRecordFolderId}</li>
+     *  <li>{@code unfiledRecordFolderId} is locked and cannot be deleted</li>
-     *             <li>{@code unfiledRecordFolderId} does not exist</li>
+     * </ul>
-     *             <li>{@code unfiledRecordFolderId} is locked and cannot be deleted</li>
-     *             </ul>
      */
     public void deleteUnfiledRecordFolder(String unfiledRecordFolderId)
     {
@@ -309,7 +293,8 @@ public class UnfiledRecordFolderAPI extends RMModelRequest
         getRmRestWrapper().processEmptyModel(simpleRequest(
                 DELETE,
                 "unfiled-record-folders/{recordFolderId}",
-                unfiledRecordFolderId));
+                unfiledRecordFolderId
+        ));
     }
 
 }

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/util/CommonTestUtils.java

@@ -44,10 +44,12 @@ public class CommonTestUtils
 
     /** Private constructor to prevent instantiation. */
     private CommonTestUtils()
-    {}
+    {
+    }
 
     /**
-     * Generate a prefix to namespace the objects in a test class. Note that four random hex digits should be good enough to avoid collisions when running locally and should also be short enough to maintain readability.
+     * Generate a prefix to namespace the objects in a test class. Note that four random hex digits should be good enough to avoid
+     * collisions when running locally and should also be short enough to maintain readability.
      */
     public static String generateTestPrefix(Class<?> clazz)
     {

amps/ags/rm-automation/rm-automation-community-rest-api/src/main/java/org/alfresco/rest/rm/community/util/DockerHelper.java

@@ -41,17 +41,17 @@ import com.github.dockerjava.api.model.Frame;
 import com.github.dockerjava.core.DockerClientBuilder;
 import com.github.dockerjava.core.command.LogContainerResultCallback;
 import com.github.dockerjava.netty.NettyDockerCmdExecFactory;
+
 import lombok.Getter;
 import lombok.Setter;
-import org.apache.commons.lang.SystemUtils;
+import org.alfresco.utility.Utility;
+import org.apache.commons.lang3.SystemUtils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.beans.factory.annotation.Value;
 import org.springframework.stereotype.Service;
 
-import org.alfresco.utility.Utility;
-
 /**
  * Helper class for interaction with docker containers
  *
@@ -68,31 +68,29 @@ public class DockerHelper
     private DockerClient dockerClient;
 
     @Autowired
-    public DockerHelper(@Value("${docker.host}") String dockerHost)
+    public DockerHelper(@Value ("${docker.host}") String dockerHost)
     {
         if (SystemUtils.IS_OS_WINDOWS)
         {
             this.dockerClient = DockerClientBuilder
-                    .getInstance(dockerHost)
+                .getInstance(dockerHost)
-                    .withDockerCmdExecFactory(new NettyDockerCmdExecFactory())
+                .withDockerCmdExecFactory(new NettyDockerCmdExecFactory())
-                    .build();
+                .build();
         }
         else
         {
             this.dockerClient = DockerClientBuilder
-                    .getInstance()
+                .getInstance()
-                    .withDockerCmdExecFactory(new NettyDockerCmdExecFactory())
+                .withDockerCmdExecFactory(new NettyDockerCmdExecFactory())
-                    .build();
+                .build();
         }
     }
 
     /**
      * Method for returning logs of docker container
      *
-     * @param containerId
+     * @param containerId - ID of the container
-     *            - ID of the container
+     * @param timeStamp - get the logs since a specific timestamp
-     * @param timeStamp
-     *            - get the logs since a specific timestamp
      * @return list of strings, where every string is log line
      */
     private List<String> getDockerLogs(String containerId, int timeStamp)
@@ -101,13 +99,14 @@ public class DockerHelper
 
         final LogContainerCmd logContainerCmd = getDockerClient().logContainerCmd(containerId);
         logContainerCmd.withStdOut(true)
-                .withStdErr(true)
+                       .withStdErr(true)
-                .withSince(timeStamp) // UNIX timestamp to filter logs. Output log-entries since that timestamp.
+                       .withSince(timeStamp) // UNIX timestamp to filter logs. Output log-entries since that timestamp.
-                .withTimestamps(true); // print timestamps for every log line
+                       .withTimestamps(true); //print timestamps for every log line
 
         try
         {
-            logContainerCmd.exec(new LogContainerResultCallback() {
+            logContainerCmd.exec(new LogContainerResultCallback()
+            {
                 @Override
                 public void onNext(Frame item)
                 {
@@ -117,7 +116,7 @@ public class DockerHelper
         }
         catch (InterruptedException e)
         {
-            Thread.currentThread().interrupt(); // set interrupt flag
+            Thread.currentThread().interrupt();  // set interrupt flag
             LOGGER.error("Failed to retrieve logs of container " + containerId, e);
         }
 
@@ -149,14 +148,14 @@ public class DockerHelper
     /**
      * Helper method to check if the specified exception is thrown in alfresco logs
      *
-     * @param expectedException
+     * @param expectedException the expected exception to be thrown
-     *            the expected exception to be thrown
      * @throws Exception
      */
     public void checkExceptionIsInAlfrescoLogs(String expectedException) throws Exception
     {
-        // Retry the operation because sometimes it takes few seconds to throw the exception
+        //Retry the operation because sometimes it takes few seconds to throw the exception
-        Utility.sleep(6000, 30000, () -> {
+        Utility.sleep(6000, 30000, () ->
+        {
             List<String> alfrescoLogs = getAlfrescoLogs();
             assertTrue(alfrescoLogs.stream().anyMatch(logLine -> logLine.contains(expectedException)));
         });
@@ -165,8 +164,7 @@ public class DockerHelper
     /**
      * Method for finding docker containers after the image name
      *
-     * @param imageName
+     * @param imageName - the name of the image used by container
-     *            - the name of the image used by container
      * @return the containers
      */
     private List<Container> findContainersByImageName(String imageName)
@@ -174,7 +172,7 @@ public class DockerHelper
         final List<Container> containers = getDockerClient().listContainersCmd().withShowAll(true).exec();
 
         return containers.stream()
-                .filter(container -> container.getImage().contains(imageName))
+                         .filter(container -> container.getImage().contains(imageName))
-                .collect(Collectors.toList());
+                         .collect(Collectors.toList());
     }
 }