RM-4808 (Update Record Folder: Reopen a close folder via PUT request throw Permission denied exception)

rm-automation/rm-automation-community-rest-api/src/test/java/org/alfresco/rest/rm/community/fileplancomponents/RecordFolderTests.java

@@ -441,6 +441,54 @@ public class RecordFolderTests extends BaseRMRestTest
         assertTrue(folder2.getPath().getName().contains(NEW_RELATIVE_PATH));
     }
 
+    /**
+     * Given that a record folder exists
+     * When the record folder is closed
+     * Then a request can be made to reopen it
+     */
+    @Test
+    (
+        description = "A closed record folder can be reopened"
+    )
+    @Bug(id="RM-4808")
+    public void openClosedRecordFolder() throws Exception
+    {
+        // Create a record folder
+        FilePlanComponent recordFolder = createCategoryFolderInFilePlan();
+
+        // Assert that the record folder is not closed
+        assertFalse(recordFolder.getProperties().getIsClosed());
+
+        // Get the file plan component API
+        FilePlanComponentAPI filePlanComponentsAPI = getRestAPIFactory().getFilePlanComponentsAPI();
+
+        // Create a record folder model to closed it
+        FilePlanComponent recordFolderModel = FilePlanComponent.builder()
+                                                .properties(FilePlanComponentProperties.builder()
+                                                        .isClosed(true)
+                                                        .build())
+                                                .build();
+
+        // Make a request to close the record folder
+        FilePlanComponent updatedRecordFolder = filePlanComponentsAPI.updateFilePlanComponent(recordFolderModel, recordFolder.getId());
+
+        // Verify that the record folder is closed now
+        assertTrue(updatedRecordFolder.getProperties().getIsClosed());
+
+        // Create a record folder model to reopen it
+        recordFolderModel = FilePlanComponent.builder()
+                                .properties(FilePlanComponentProperties.builder()
+                                        .isClosed(false)
+                                        .build())
+                                .build();
+
+        // Make a request to reopen the record folder
+        updatedRecordFolder = filePlanComponentsAPI.updateFilePlanComponent(recordFolderModel, recordFolder.getId());
+
+        // Verify that the record folder is open now
+        assertFalse(updatedRecordFolder.getProperties().getIsClosed());
+    }
+
     @AfterClass (alwaysRun = true)
     public void tearDown() throws Exception
     {

rm-community/rm-community-repo/config/alfresco/module/org_alfresco_module_rm/capability/rm-capabilities-group-context.xml

@@ -61,7 +61,6 @@
             <ref bean="rmCreateModifyDestroyFileplanMetadataCapability"/>
             <ref bean="rmEditDeclaredRecordMetadataCapability"/>
             <ref bean="rmEditNonRecordMetadataCapability"/>
-            <ref bean="rmCreateModifyRecordsInCuttoffFoldersCapability"/>
             <ref bean="rmEditRecordMetadataCapability"/>
             <ref bean="rmModifyUnfiledRecordsContainerFolderCapability"/>
             <ref bean="rmEditHoldCapability"/>

rm-community/rm-community-repo/config/alfresco/module/org_alfresco_module_rm/capability/rm-capabilities-recordfolder-context.xml

@@ -94,7 +94,6 @@
             <entry key="capabilityCondition.filling" value="true"/>
             <entry key="capabilityCondition.cutoff" value="false"/>
             <entry key="capabilityCondition.frozen" value="false"/>
-            <entry key="capabilityCondition.closed" value="false"/>
          </map>
       </property>
    </bean>