From 3e63485907dd87df0ab1bf3471ae04c21411160c Mon Sep 17 00:00:00 2001
From: Tuna Aksoy <tuna.aksoy@alfresco.com>
Date: Mon, 9 Feb 2015 10:17:39 +0000
Subject: [PATCH] RM-1867 (Reject rule works incorrect if set up in File Plan)

git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/modules/recordsmanagement/BRANCHES/V2.3@96524 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
---
 .../rm-action-context.xml                     |  5 ++--
 .../FilePlanNamePathDataExtractor.java        | 26 +++++++------------
 2 files changed, 13 insertions(+), 18 deletions(-)

diff --git a/rm-server/config/alfresco/module/org_alfresco_module_rm/rm-action-context.xml b/rm-server/config/alfresco/module/org_alfresco_module_rm/rm-action-context.xml
index f75f4a4ca8..482e854cff 100644
--- a/rm-server/config/alfresco/module/org_alfresco_module_rm/rm-action-context.xml
+++ b/rm-server/config/alfresco/module/org_alfresco_module_rm/rm-action-context.xml
@@ -731,6 +731,7 @@
 
    <bean id="reject" class="org.alfresco.module.org_alfresco_module_rm.action.impl.RejectAction" parent="rmAction">
       <property name="publicAction" value="true"/>
+      <property name="auditedImmediately" value="true" />
    </bean>
 
    <!--  File To -->
@@ -844,9 +845,9 @@
       <property name="publicAction" value="true"/>
       <property name="allowParameterSubstitutions" value="true"/>
    </bean>
-   
+
    <!--  Unlink Record -->
-   
+
    <bean id="unlinkFrom_proxy" parent="rmProxyAction">
       <property name="target" ref="unlinkFrom"/>
         <property name="interceptorNames">
diff --git a/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/audit/extractor/FilePlanNamePathDataExtractor.java b/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/audit/extractor/FilePlanNamePathDataExtractor.java
index ca86e30a5b..c38636c8ff 100644
--- a/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/audit/extractor/FilePlanNamePathDataExtractor.java
+++ b/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/audit/extractor/FilePlanNamePathDataExtractor.java
@@ -77,25 +77,19 @@ public final class FilePlanNamePathDataExtractor extends AbstractDataExtractor
      */
     public Serializable extractData(Serializable value)
     {
-        String extractedData = null;
-
         NodeRef nodeRef = (NodeRef) value;
-        if (nodeService.hasAspect(nodeRef, RecordsManagementModel.ASPECT_FILE_PLAN_COMPONENT))
+
+        // Get path from the RM root
+        List<NodeRef> nodeRefPath = filePlanService.getNodeRefPath(nodeRef);
+
+        StringBuilder sb = new StringBuilder(128);
+        for (NodeRef pathNodeRef : nodeRefPath)
         {
-            // Get path from the RM root
-            List<NodeRef> nodeRefPath = filePlanService.getNodeRefPath(nodeRef);
-
-            StringBuilder sb = new StringBuilder(128);
-            for (NodeRef pathNodeRef : nodeRefPath)
-            {
-                String name = (String)nodeService.getProperty(pathNodeRef, ContentModel.PROP_NAME);
-                sb.append("/").append(name);
-            }
-
-            // Done
-            extractedData = sb.toString();
+            String name = (String)nodeService.getProperty(pathNodeRef, ContentModel.PROP_NAME);
+            sb.append("/").append(name);
         }
 
-        return extractedData;
+        // Done
+        return sb.toString();
     }
 }