Fixes the following bugs:

- RM-249 (Disposition steps sequence can be violated)
- RM-790 (Cannot delete disposition schedule step)

git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/modules/recordsmanagement/HEAD@54223 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261

rm-server/config/alfresco/templates/webscripts/org/alfresco/rma/dispositionactiondefinition.delete.json.ftl

@@ -1 +1 @@
-{}
+<#include "dispositionschedule.get.json.ftl">

rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/script/DispositionAbstractBase.java

@@ -28,6 +28,7 @@ import javax.servlet.http.HttpServletResponse;
 import org.alfresco.module.org_alfresco_module_rm.disposition.DispositionActionDefinition;
 import org.alfresco.module.org_alfresco_module_rm.disposition.DispositionSchedule;
 import org.alfresco.module.org_alfresco_module_rm.event.RecordsManagementEvent;
+import org.alfresco.module.org_alfresco_module_rm.model.RecordsManagementModel;
 import org.alfresco.service.cmr.repository.NodeRef;
 import org.springframework.extensions.webscripts.WebScriptException;
 import org.springframework.extensions.webscripts.WebScriptRequest;
@@ -152,4 +153,63 @@ public class DispositionAbstractBase extends AbstractRmWebScript
 
         return model;
     }
+
+    /**
+     * Helper method to parse the request and retrieve the disposition schedule model.
+     *
+     * @param req The webscript request
+     * @return Map representing the model
+     */
+    protected Map<String, Object> getDispositionScheduleModel(WebScriptRequest req)
+    {
+        // parse the request to retrieve the schedule object
+        DispositionSchedule schedule = parseRequestForSchedule(req);
+
+        // add all the schedule data to Map
+        Map<String, Object> scheduleModel = new HashMap<String, Object>(8);
+
+        // build url
+        String serviceUrl = req.getServiceContextPath() + req.getPathInfo();
+        scheduleModel.put("url", serviceUrl);
+        String actionsUrl = serviceUrl + "/dispositionactiondefinitions";
+        scheduleModel.put("actionsUrl", actionsUrl);
+        scheduleModel.put("nodeRef", schedule.getNodeRef().toString());
+        scheduleModel.put("recordLevelDisposition", schedule.isRecordLevelDisposition());
+        scheduleModel.put("canStepsBeRemoved",
+                    !this.dispositionService.hasDisposableItems(schedule));
+
+        if (schedule.getDispositionAuthority() != null)
+        {
+            scheduleModel.put("authority", schedule.getDispositionAuthority());
+        }
+
+        if (schedule.getDispositionInstructions() != null)
+        {
+            scheduleModel.put("instructions", schedule.getDispositionInstructions());
+        }
+
+        boolean unpublishedUpdates = false;
+        boolean publishInProgress = false;
+
+        List<Map<String, Object>> actions = new ArrayList<Map<String, Object>>();
+        for (DispositionActionDefinition actionDef : schedule.getDispositionActionDefinitions())
+        {
+            NodeRef actionDefNodeRef = actionDef.getNodeRef();
+            if (nodeService.hasAspect(actionDefNodeRef, RecordsManagementModel.ASPECT_UNPUBLISHED_UPDATE) == true)
+            {
+                unpublishedUpdates = true;
+                publishInProgress = ((Boolean)nodeService.getProperty(actionDefNodeRef, RecordsManagementModel.PROP_PUBLISH_IN_PROGRESS)).booleanValue();
+            }
+
+            actions.add(createActionDefModel(actionDef, actionsUrl + "/" + actionDef.getId()));
+        }
+        scheduleModel.put("actions", actions);
+        scheduleModel.put("unpublishedUpdates", unpublishedUpdates);
+        scheduleModel.put("publishInProgress", publishInProgress);
+
+        // create model object with just the schedule data
+        Map<String, Object> model = new HashMap<String, Object>(1);
+        model.put("schedule", scheduleModel);
+        return model;
+    }
 }

rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/script/DispositionActionDefinitionDelete.java

@@ -18,7 +18,7 @@
  */
 package org.alfresco.module.org_alfresco_module_rm.script;
 
-import java.util.HashMap;
+import java.util.List;
 import java.util.Map;
 
 import org.alfresco.module.org_alfresco_module_rm.disposition.DispositionActionDefinition;
@@ -47,9 +47,28 @@ public class DispositionActionDefinitionDelete extends DispositionAbstractBase
         DispositionActionDefinition actionDef = parseRequestForActionDefinition(req, schedule);
 
         // remove the action definition from the schedule
-        this.dispositionService.removeDispositionActionDefinition(schedule, actionDef);
+        removeDispositionActionDefinitions(schedule, actionDef);
 
-        // return an empty model
+        // return the disposition schedule model
-        return new HashMap<String, Object>();
+        return getDispositionScheduleModel(req);
+    }
+
+    /**
+     * Helper method to remove a disposition action definition and the following definition(s)
+     *
+     * @param schedule The disposition schedule
+     * @param actionDef The disposition action definition
+     */
+    private void removeDispositionActionDefinitions(DispositionSchedule schedule, DispositionActionDefinition actionDef)
+    {
+        int index = actionDef.getIndex();
+        List<DispositionActionDefinition> dispositionActionDefinitions = schedule.getDispositionActionDefinitions();
+        for (DispositionActionDefinition dispositionActionDefinition : dispositionActionDefinitions)
+        {
+            if (dispositionActionDefinition.getIndex() >= index)
+            {
+                dispositionService.removeDispositionActionDefinition(schedule, dispositionActionDefinition);
+            }
+        }
     }
 }

rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/script/DispositionScheduleGet.java

@@ -18,15 +18,8 @@
  */
 package org.alfresco.module.org_alfresco_module_rm.script;
 
-import java.util.ArrayList;
-import java.util.HashMap;
-import java.util.List;
 import java.util.Map;
 
-import org.alfresco.module.org_alfresco_module_rm.disposition.DispositionActionDefinition;
-import org.alfresco.module.org_alfresco_module_rm.disposition.DispositionSchedule;
-import org.alfresco.module.org_alfresco_module_rm.model.RecordsManagementModel;
-import org.alfresco.service.cmr.repository.NodeRef;
 import org.springframework.extensions.webscripts.Cache;
 import org.springframework.extensions.webscripts.Status;
 import org.springframework.extensions.webscripts.WebScriptRequest;
@@ -45,54 +38,7 @@ public class DispositionScheduleGet extends DispositionAbstractBase
     @Override
     protected Map<String, Object> executeImpl(WebScriptRequest req, Status status, Cache cache)
     {
-        // parse the request to retrieve the schedule object
+        // return the disposition schedule model
-        DispositionSchedule schedule = parseRequestForSchedule(req);
+        return getDispositionScheduleModel(req);
-
-        // add all the schedule data to Map
-        Map<String, Object> scheduleModel = new HashMap<String, Object>(8);
-        
-        // build url
-        String serviceUrl = req.getServiceContextPath() + req.getPathInfo();
-        scheduleModel.put("url", serviceUrl);
-        String actionsUrl = serviceUrl + "/dispositionactiondefinitions";
-        scheduleModel.put("actionsUrl", actionsUrl);
-        scheduleModel.put("nodeRef", schedule.getNodeRef().toString());
-        scheduleModel.put("recordLevelDisposition", schedule.isRecordLevelDisposition());
-        scheduleModel.put("canStepsBeRemoved", 
-                    !this.dispositionService.hasDisposableItems(schedule));
-        
-        if (schedule.getDispositionAuthority() != null)
-        {
-            scheduleModel.put("authority", schedule.getDispositionAuthority());
-        }
-        
-        if (schedule.getDispositionInstructions() != null)
-        {
-            scheduleModel.put("instructions", schedule.getDispositionInstructions());
-        }
-        
-        boolean unpublishedUpdates = false;
-        boolean publishInProgress = false;                
-        
-        List<Map<String, Object>> actions = new ArrayList<Map<String, Object>>();
-        for (DispositionActionDefinition actionDef : schedule.getDispositionActionDefinitions())
-        {
-            NodeRef actionDefNodeRef = actionDef.getNodeRef();
-            if (nodeService.hasAspect(actionDefNodeRef, RecordsManagementModel.ASPECT_UNPUBLISHED_UPDATE) == true)
-            {
-                unpublishedUpdates = true;
-                publishInProgress = ((Boolean)nodeService.getProperty(actionDefNodeRef, RecordsManagementModel.PROP_PUBLISH_IN_PROGRESS)).booleanValue();
-            }
-            
-            actions.add(createActionDefModel(actionDef, actionsUrl + "/" + actionDef.getId()));
-        }
-        scheduleModel.put("actions", actions);                
-        scheduleModel.put("unpublishedUpdates", unpublishedUpdates);
-        scheduleModel.put("publishInProgress", publishInProgress);
-        
-        // create model object with just the schedule data
-        Map<String, Object> model = new HashMap<String, Object>(1);
-        model.put("schedule", scheduleModel);
-        return model;
     }
 }