Merge release/V2.2.1.x into release/V2.2.

Conflicts:
	pom.xml
	rm-server/config/alfresco/module/org_alfresco_module_rm/module.properties
	rm-server/pom.xml
	rm-share/config/alfresco/module/org_alfresco_module_rm_share/module.properties
	rm-share/pom.xml
	rm-ui-tests/pom.xml

.gitignore

@@ -24,6 +24,7 @@ test-output
 /rm-server/alfresco-solr.zip
 /rm-server/solr
 /rm-server/shared
+/rm-server/alf_data
 
 # /rm-server/config/
 /rm-server/config/alfresco-global.properties

rm-server/config/alfresco/module/org_alfresco_module_rm/module-context.xml

@@ -168,7 +168,6 @@
       <!-- init caveatConfig behaviours  -->
       <property name="caveatConfigService" ref="caveatConfigService"/>
       <property name="customEmailMappingService" ref="customEmailMappingService"/>
-      <property name="recordsManagementAdminService" ref="RecordsManagementAdminService"/>
    </bean>
 
    <!-- Java script interface for rm caveat config-->

rm-server/config/alfresco/module/org_alfresco_module_rm/rm-service-context.xml

@@ -770,6 +770,7 @@
         <property name="nodeService" ref="NodeService"/>
         <property name="policyComponent" ref="policyComponent"/>
         <property name="contentService" ref="contentService"/>
+        <property name="transactionService" ref="transactionService"/>
         <property name="dictionaryRepositoryBootstrap" ref="dictionaryRepositoryBootstrap"/>
         <property name="customisableTypes">
            <list>
@@ -813,7 +814,6 @@
         <property name="objectDefinitionSource">
             <value>
                 <![CDATA[
-                org.alfresco.module.org_alfresco_module_rm.admin.RecordsManagementAdminService.initialiseCustomModel=RM_ALLOW
                 org.alfresco.module.org_alfresco_module_rm.admin.RecordsManagementAdminService.getCustomisable=RM_ALLOW
                 org.alfresco.module.org_alfresco_module_rm.admin.RecordsManagementAdminService.isCustomisable=RM_ALLOW
                 org.alfresco.module.org_alfresco_module_rm.admin.RecordsManagementAdminService.makeCustomisable=RM_ALLOW

rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/admin/RecordsManagementAdminService.java

@@ -42,11 +42,6 @@ import org.alfresco.service.namespace.QName;
  */
 public interface RecordsManagementAdminService
 {
-    /**
-     * Initialise the custom model
-     */
-    void initialiseCustomModel();
-
     /**
      * Get a list of all registered customisable types and aspects.
      *

rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/admin/RecordsManagementAdminServiceImpl.java

@@ -63,6 +63,7 @@ import org.alfresco.repo.policy.annotation.BehaviourBean;
 import org.alfresco.repo.policy.annotation.BehaviourKind;
 import org.alfresco.repo.security.authentication.AuthenticationUtil;
 import org.alfresco.repo.security.authentication.AuthenticationUtil.RunAsWork;
+import org.alfresco.repo.transaction.RetryingTransactionHelper.RetryingTransactionCallback;
 import org.alfresco.service.cmr.dictionary.AspectDefinition;
 import org.alfresco.service.cmr.dictionary.AssociationDefinition;
 import org.alfresco.service.cmr.dictionary.Constraint;
@@ -83,9 +84,13 @@ import org.alfresco.service.cmr.repository.StoreRef;
 import org.alfresco.service.namespace.NamespaceService;
 import org.alfresco.service.namespace.QName;
 import org.alfresco.service.namespace.RegexQNamePattern;
+import org.alfresco.service.transaction.TransactionService;
 import org.alfresco.util.GUID;
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
+import org.springframework.context.ApplicationListener;
+import org.springframework.context.event.ContextRefreshedEvent;
+import org.springframework.core.Ordered;
 import org.springframework.extensions.surf.util.I18NUtil;
 import org.springframework.extensions.surf.util.ParameterCheck;
 import org.springframework.extensions.surf.util.URLDecoder;
@@ -100,7 +105,9 @@ public class RecordsManagementAdminServiceImpl implements RecordsManagementAdmin
 														  RecordsManagementCustomModel,
 														  NodeServicePolicies.OnAddAspectPolicy,
 														  NodeServicePolicies.OnRemoveAspectPolicy,
-														  NodeServicePolicies.OnCreateNodePolicy
+														  NodeServicePolicies.OnCreateNodePolicy,
+														  ApplicationListener<ContextRefreshedEvent>, 
+														  Ordered
 {
     /** Logger */
     private static Log logger = LogFactory.getLog(RecordsManagementAdminServiceImpl.class);
@@ -152,6 +159,9 @@ public class RecordsManagementAdminServiceImpl implements RecordsManagementAdmin
 
     /** Policy component */
     private PolicyComponent policyComponent;
+    
+    /** Transaction service */
+    private TransactionService transactionService;
 
     /** Policy delegates */
     private ClassPolicyDelegate<BeforeCreateReference> beforeCreateReferenceDelegate;
@@ -162,6 +172,9 @@ public class RecordsManagementAdminServiceImpl implements RecordsManagementAdmin
     /** List of types that can be customisable */
     private List<QName> pendingCustomisableTypes;
     private Map<QName, QName> customisableTypes;
+    
+    /** indicates whether the custom map has been initialised or not */
+    private boolean isCustomMapInit = false;
 
     /**
      * @param dictionaryService     the dictionary service
@@ -212,6 +225,14 @@ public class RecordsManagementAdminServiceImpl implements RecordsManagementAdmin
     {
         this.dictonaryRepositoryBootstrap = dictonaryRepositoryBootstrap;
     }
+	
+	/**
+	 * @param transactionService   transaction service
+	 */
+	public void setTransactionService(TransactionService transactionService)
+    {
+        this.transactionService = transactionService;
+    }
 
 	/**
 	 * Initialisation method
@@ -223,15 +244,57 @@ public class RecordsManagementAdminServiceImpl implements RecordsManagementAdmin
         onCreateReferenceDelegate = policyComponent.registerClassPolicy(OnCreateReference.class);
         beforeRemoveReferenceDelegate = policyComponent.registerClassPolicy(BeforeRemoveReference.class);
         onRemoveReferenceDelegate = policyComponent.registerClassPolicy(OnRemoveReference.class);
+	}
+
+	/**
+	 * Indicate that this application content listener must be executed with the lowest 
+	 * precedence. (ie last)
+	 * 
+	 * @see Ordered#getOrder()
+	 */
+    @Override
+    public int getOrder()
+    {
+        return Ordered.LOWEST_PRECEDENCE;
+    }
+
+    /**
+     * Load the custom properties map
+     * 
+     * @see ApplicationListener#onApplicationEvent(org.springframework.context.ApplicationEvent)
+     */
+    @Override
+    public void onApplicationEvent(ContextRefreshedEvent event)
+    {
+        transactionService.getRetryingTransactionHelper().doInTransaction(new RetryingTransactionCallback<Void>()
+        {
+            public Void execute() throws Throwable
+            {
+                // initialise custom properties
+                initCustomMap();
+                
+                return null;
+            }
+         });                 
+    }
+    
+    /**
+     * Helper method to indicate whether the custom map is initialised or not.
+     * 
+     * @return  boolean true if initialised, false otherwise
+     */
+    public boolean isCustomMapInit()
+    {
+        return isCustomMapInit;
     }
 
 	/**
-     * Invoke before create reference policy
+	 * Invoke before create reference policy
-     *
+	 *
-     * @param fromNodeRef
+	 * @param fromNodeRef
-     * @param toNodeRef
+	 * @param toNodeRef
-     * @param reference
+	 * @param reference
-     */
+	 */
     protected void invokeBeforeCreateReference(NodeRef fromNodeRef, NodeRef toNodeRef, QName reference)
     {
         // get qnames to invoke against
@@ -301,22 +364,25 @@ public class RecordsManagementAdminServiceImpl implements RecordsManagementAdmin
     )
     public void onAddAspect(final NodeRef nodeRef, final QName aspectTypeQName)
     {
-        AuthenticationUtil.runAs(new RunAsWork<Void>()
+        if (isCustomMapInit)
         {
-            @Override
+            AuthenticationUtil.runAs(new RunAsWork<Void>()
-            public Void doWork()
             {
-                if (nodeService.exists(nodeRef) &&
+                @Override
-                    dictionaryService.getAllModels().contains(RM_CUSTOM_MODEL) &&
+                public Void doWork()
-                    isCustomisable(aspectTypeQName))
                 {
-                    QName customPropertyAspect = getCustomAspect(aspectTypeQName);
+                    if (nodeService.exists(nodeRef) &&
-                    nodeService.addAspect(nodeRef, customPropertyAspect, null);
+                        dictionaryService.getAllModels().contains(RM_CUSTOM_MODEL) &&
+                        isCustomisable(aspectTypeQName))
+                    {
+                        QName customPropertyAspect = getCustomAspect(aspectTypeQName);
+                        nodeService.addAspect(nodeRef, customPropertyAspect, null);
+                    }
+    
+                    return null;
                 }
-
+            }, AuthenticationUtil.getSystemUserName());
-                return null;
+        }
-            }
-        }, AuthenticationUtil.getSystemUserName());
     }
 
     /**
@@ -331,21 +397,24 @@ public class RecordsManagementAdminServiceImpl implements RecordsManagementAdmin
     )
     public void onRemoveAspect(final NodeRef nodeRef, final QName aspectTypeQName)
     {
-        AuthenticationUtil.runAs(new RunAsWork<Void>()
+        if (isCustomMapInit)
         {
-            @Override
+            AuthenticationUtil.runAs(new RunAsWork<Void>()
-            public Void doWork()
             {
-                if (nodeService.exists(nodeRef) &&
+                @Override
-                    isCustomisable(aspectTypeQName))
+                public Void doWork()
                 {
-                    QName customPropertyAspect = getCustomAspect(aspectTypeQName);
+                    if (nodeService.exists(nodeRef) &&
-                    nodeService.removeAspect(nodeRef, customPropertyAspect);
+                        isCustomisable(aspectTypeQName))
+                    {
+                        QName customPropertyAspect = getCustomAspect(aspectTypeQName);
+                        nodeService.removeAspect(nodeRef, customPropertyAspect);
+                    }
+    
+                    return null;
                 }
-
+            }, AuthenticationUtil.getSystemUserName());
-                return null;
+        }
-            }
-        }, AuthenticationUtil.getSystemUserName());
     }
 
     /**
@@ -362,47 +431,41 @@ public class RecordsManagementAdminServiceImpl implements RecordsManagementAdmin
     )
     public void onCreateNode(final ChildAssociationRef childAssocRef)
     {
-        AuthenticationUtil.runAs(new RunAsWork<Void>()
+        if (isCustomMapInit)
-        {
+        {            
-            @Override
+            AuthenticationUtil.runAs(new RunAsWork<Void>()
-            public Void doWork()
             {
-                if (dictionaryService.getAllModels().contains(RecordsManagementCustomModel.RM_CUSTOM_MODEL))
+                @Override
+                public Void doWork()
                 {
-                    NodeRef nodeRef = childAssocRef.getChildRef();
+                    if (dictionaryService.getAllModels().contains(RecordsManagementCustomModel.RM_CUSTOM_MODEL))
-                    QName type = nodeService.getType(nodeRef);
-                    while (type != null && !ContentModel.TYPE_CMOBJECT.equals(type))
                     {
-                        if (isCustomisable(type))
+                        NodeRef nodeRef = childAssocRef.getChildRef();
+                        QName type = nodeService.getType(nodeRef);
+                        while (type != null && !ContentModel.TYPE_CMOBJECT.equals(type))
                         {
-                            QName customPropertyAspect = getCustomAspect(type);
+                            if (isCustomisable(type))
-                            nodeService.addAspect(nodeRef, customPropertyAspect, null);
+                            {
-                        }
+                                QName customPropertyAspect = getCustomAspect(type);
-
+                                nodeService.addAspect(nodeRef, customPropertyAspect, null);
-                        TypeDefinition def = dictionaryService.getType(type);
+                            }
-                        if (def != null)
+    
-                        {
+                            TypeDefinition def = dictionaryService.getType(type);
-                            type = def.getParentName();
+                            if (def != null)
-                        }
+                            {
-                        else
+                                type = def.getParentName();
-                        {
+                            }
-                            type = null;
+                            else
+                            {
+                                type = null;
+                            }
                         }
                     }
+    
+                    return null;
                 }
-
+            }, AuthenticationUtil.getSystemUserName());
-                return null;
+        }
-            }
-        }, AuthenticationUtil.getSystemUserName());
-    }
-
-    /**
-     * @see org.alfresco.module.org_alfresco_module_rm.RecordsManagementAdminService#initialiseCustomModel()
-     */
-    public void initialiseCustomModel()
-    {
-        // Initialise the map
-        getCustomisableMap();
     }
 
     /**
@@ -483,6 +546,92 @@ public class RecordsManagementAdminServiceImpl implements RecordsManagementAdmin
 
         return result;
     }
+    
+    /**
+     * Initialise custom type map
+     */
+    private void initCustomMap()
+    {
+        customisableTypes = new HashMap<QName, QName>(7);
+        Collection<QName> aspects = dictionaryService.getAspects(RM_CUSTOM_MODEL);
+        for (QName aspect : aspects)
+        {
+            AspectDefinition aspectDef = dictionaryService.getAspect(aspect);
+            String name = aspectDef.getName().getLocalName();
+            if (name.endsWith("Properties"))
+            {
+                QName type = null;
+                String prefixString = aspectDef.getDescription(dictionaryService);
+                if (prefixString == null)
+                {
+                    // Backward compatibility from previous RM V1.0 custom models
+                    if (CompatibilityModel.NAME_CUSTOM_RECORD_PROPERTIES.equals(name))
+                    {
+                        type = RecordsManagementModel.ASPECT_RECORD;
+                    }
+                    else if (CompatibilityModel.NAME_CUSTOM_RECORD_FOLDER_PROPERTIES.equals(name))
+                    {
+                        type = RecordsManagementModel.TYPE_RECORD_FOLDER;
+                    }
+                    else if (CompatibilityModel.NAME_CUSTOM_RECORD_CATEGORY_PROPERTIES.equals(name))
+                    {
+                        type = RecordsManagementModel.TYPE_RECORD_CATEGORY;
+                    }
+                    else if (CompatibilityModel.NAME_CUSTOM_RECORD_SERIES_PROPERTIES.equals(name) &&
+                            // Only add the deprecated record series type as customisable if
+                            // a v1.0 installation has added custom properties
+                            aspectDef.getProperties().size() != 0)
+                    {
+                        type = CompatibilityModel.TYPE_RECORD_SERIES;
+                    }
+                }
+                else
+                {
+                    type = QName.createQName(prefixString, namespaceService);
+                }
+
+                // Add the customisable type to the map
+                if (type != null)
+                {
+                    customisableTypes.put(type, aspect);
+
+                    // Remove customisable type from the pending list
+                    if (pendingCustomisableTypes != null && pendingCustomisableTypes.contains(type))
+                    {
+                        pendingCustomisableTypes.remove(type);
+                    }
+                }
+            }
+        }
+
+        // Deal with any pending types left over
+        if (pendingCustomisableTypes != null && pendingCustomisableTypes.size() != 0)
+        {
+            NodeRef modelRef = getCustomModelRef(RecordsManagementModel.RM_CUSTOM_URI);
+            M2Model model = readCustomContentModel(modelRef);
+            try
+            {
+                for (QName customisableType : pendingCustomisableTypes)
+                {
+                    QName customAspect = getCustomAspectImpl(customisableType);
+
+                    // Create the new aspect to hold the custom properties
+                    M2Aspect aspect = model.createAspect(customAspect.toPrefixString(namespaceService));
+                    aspect.setDescription(customisableType.toPrefixString(namespaceService));
+
+                    // Make a record of the customisable type
+                    customisableTypes.put(customisableType, customAspect);
+                }
+            }
+            finally
+            {
+                writeCustomContentModel(modelRef, model);
+            }
+        }
+        
+        // indicate map is initialised
+        isCustomMapInit = true;
+    }
 
     /**
      * Gets a map containing all the customisable types
@@ -493,82 +642,7 @@ public class RecordsManagementAdminServiceImpl implements RecordsManagementAdmin
     {
     	if (customisableTypes == null)
     	{
-    		customisableTypes = new HashMap<QName, QName>(7);
+    		throw AlfrescoRuntimeException.create("Customisable map has not been initialised correctly.");
-	    	Collection<QName> aspects = dictionaryService.getAspects(RM_CUSTOM_MODEL);
-	    	for (QName aspect : aspects)
-	    	{
-	    		AspectDefinition aspectDef = dictionaryService.getAspect(aspect);
-	    		String name = aspectDef.getName().getLocalName();
-	    		if (name.endsWith("Properties"))
-	    		{
-	    			QName type = null;
-	    			String prefixString = aspectDef.getDescription(dictionaryService);
-	    			if (prefixString == null)
-	    			{
-	    				// Backward compatibility from previous RM V1.0 custom models
-	    				if (CompatibilityModel.NAME_CUSTOM_RECORD_PROPERTIES.equals(name))
-	    				{
-	    					type = RecordsManagementModel.ASPECT_RECORD;
-	    				}
-	    				else if (CompatibilityModel.NAME_CUSTOM_RECORD_FOLDER_PROPERTIES.equals(name))
-	    				{
-	    					type = RecordsManagementModel.TYPE_RECORD_FOLDER;
-	    				}
-	    				else if (CompatibilityModel.NAME_CUSTOM_RECORD_CATEGORY_PROPERTIES.equals(name))
-	    				{
-	    					type = RecordsManagementModel.TYPE_RECORD_CATEGORY;
-	    				}
-	    				else if (CompatibilityModel.NAME_CUSTOM_RECORD_SERIES_PROPERTIES.equals(name) &&
-	                            // Only add the deprecated record series type as customisable if
-	                            // a v1.0 installation has added custom properties
-	    				        aspectDef.getProperties().size() != 0)
-	    				{
-	    				    type = CompatibilityModel.TYPE_RECORD_SERIES;
-	    				}
-	    			}
-	    			else
-	    			{
-	    				type = QName.createQName(prefixString, namespaceService);
-	    			}
-
-	    			// Add the customisable type to the map
-	    			if (type != null)
-	    			{
-	    			    customisableTypes.put(type, aspect);
-
-            			// Remove customisable type from the pending list
-            			if (pendingCustomisableTypes != null && pendingCustomisableTypes.contains(type))
-            			{
-            			    pendingCustomisableTypes.remove(type);
-            			}
-	    			}
-	    		}
-			}
-
-	    	// Deal with any pending types left over
-	    	if (pendingCustomisableTypes != null && pendingCustomisableTypes.size() != 0)
-	    	{
-	    	    NodeRef modelRef = getCustomModelRef(RecordsManagementModel.RM_CUSTOM_URI);
-                M2Model model = readCustomContentModel(modelRef);
-	    	    try
-	    	    {
-        	    	for (QName customisableType : pendingCustomisableTypes)
-        	        {
-        	            QName customAspect = getCustomAspectImpl(customisableType);
-
-        	            // Create the new aspect to hold the custom properties
-        	            M2Aspect aspect = model.createAspect(customAspect.toPrefixString(namespaceService));
-        	            aspect.setDescription(customisableType.toPrefixString(namespaceService));
-
-        	            // Make a record of the customisable type
-                        customisableTypes.put(customisableType, customAspect);
-        	        }
-	    	    }
-	    	    finally
-	    	    {
-	    	        writeCustomContentModel(modelRef, model);
-	    	    }
-	    	}
     	}
     	return customisableTypes;
     }

rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/bootstrap/RecordsManagementBootstrap.java

@@ -18,7 +18,6 @@
  */
 package org.alfresco.module.org_alfresco_module_rm.bootstrap;
 
-import org.alfresco.module.org_alfresco_module_rm.admin.RecordsManagementAdminService;
 import org.alfresco.module.org_alfresco_module_rm.action.impl.SplitEmailAction;
 import org.alfresco.module.org_alfresco_module_rm.caveat.RMCaveatConfigService;
 import org.alfresco.module.org_alfresco_module_rm.email.CustomEmailMappingService;
@@ -40,8 +39,7 @@ public class RecordsManagementBootstrap extends AbstractLifecycleBean
     private TransactionService transactionService;
     private RMCaveatConfigService caveatConfigService;
     private CustomEmailMappingService customEmailMappingService;
-    private RecordsManagementAdminService adminService;
+    
-
     public void setTransactionService(TransactionService transactionService)
     {
         this.transactionService = transactionService;
@@ -57,11 +55,6 @@ public class RecordsManagementBootstrap extends AbstractLifecycleBean
         this.customEmailMappingService = customEmailMappingService;
     }
 
-    public void setRecordsManagementAdminService(RecordsManagementAdminService adminService)
-    {
-		this.adminService = adminService;
-	}
-
     public CustomEmailMappingService getCustomEmailMappingService()
     {
         return customEmailMappingService;
@@ -82,9 +75,6 @@ public class RecordsManagementBootstrap extends AbstractLifecycleBean
                         // initialise caveat config
                         caveatConfigService.init();
 
-                        // Initialise the custom model
-                        adminService.initialiseCustomModel();
-
                         // Initialise the SplitEmailAction
                         SplitEmailAction action = (SplitEmailAction)getApplicationContext().getBean("splitEmail");
                         action.bootstrap();

rm-server/test/java/org/alfresco/module/org_alfresco_module_rm/test/integration/issue/IssueTestSuite.java

@@ -18,6 +18,7 @@
  */
 package org.alfresco.module.org_alfresco_module_rm.test.integration.issue;
 
+import org.alfresco.module.org_alfresco_module_rm.test.integration.issue.rm3314.RM3314Test;
 import org.junit.runner.RunWith;
 import org.junit.runners.Suite;
 import org.junit.runners.Suite.SuiteClasses;
@@ -44,7 +45,8 @@ import org.junit.runners.Suite.SuiteClasses;
     RM1039Test.class,
     RM1799Test.class,
     //RM2190Test.class,
-    RM2192Test.class
+    RM2192Test.class,
+    RM3314Test.class
 })
 public class IssueTestSuite
 {

rm-server/test/java/org/alfresco/module/org_alfresco_module_rm/test/integration/issue/rm3314/RM3314Test.java

@@ -0,0 +1,61 @@
+/*
+ * Copyright (C) 2005-2016 Alfresco Software Limited.
+ *
+ * This file is part of Alfresco
+ *
+ * Alfresco is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Lesser General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * Alfresco is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public License
+ * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+package org.alfresco.module.org_alfresco_module_rm.test.integration.issue.rm3314;
+
+import java.util.HashMap;
+import java.util.Map;
+
+import org.alfresco.module.org_alfresco_module_rm.test.util.BaseRMTestCase;
+
+/**
+ * Test for https://issues.alfresco.com/jira/browse/RM-3114
+ *
+ * @author Roy Wetherall
+ * @since 2.2.1.5
+ */
+public class RM3314Test extends BaseRMTestCase
+{
+	/** registry to record callback from test beans "test.rm3114.1" and "test.rm3114.2" */
+    public static Map<String, Boolean> callback = new HashMap<String, Boolean>(2);
+    
+    /**
+     * Given that the custom model hasn't been initialised 
+     * When an aspect is added 
+     * Then nothing happens
+     *
+     * Given that the custom model has been initialised 
+     * When an aspect is added 
+     * Then something happens
+     */
+    public void testListenersExecutedInTheCorrectOrder()
+    {
+    	/**
+    	 * The related test beans will call back into the callback map showing
+    	 * whether at the end of their execution whether the custom model has been
+    	 * initialised or not.  Given the order in which these test beans are executed
+    	 * on spring context load, we would expect that .1 executes with the custom
+    	 * map unloaded, and the .2 with it loaded.
+    	 */
+    	
+        assertFalse(callback.isEmpty());
+        assertFalse(callback.get("test.rm3314.1"));
+        assertTrue(callback.get("test.rm3314.2"));
+    }
+}

rm-server/test/java/org/alfresco/module/org_alfresco_module_rm/test/integration/issue/rm3314/RM3314TestListener.java

@@ -0,0 +1,134 @@
+/*
+ * Copyright (C) 2005-2014 Alfresco Software Limited.
+ *
+ * This file is part of Alfresco
+ *
+ * Alfresco is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Lesser General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * Alfresco is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public License
+ * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+package org.alfresco.module.org_alfresco_module_rm.test.integration.issue.rm3314;
+
+import org.alfresco.model.ContentModel;
+import org.alfresco.module.org_alfresco_module_rm.admin.RecordsManagementAdminServiceImpl;
+import org.alfresco.repo.model.Repository;
+import org.alfresco.repo.security.authentication.AuthenticationUtil;
+import org.alfresco.repo.security.authentication.AuthenticationUtil.RunAsWork;
+import org.alfresco.service.cmr.model.FileFolderService;
+import org.alfresco.service.cmr.repository.NodeRef;
+import org.alfresco.service.cmr.repository.NodeService;
+import org.springframework.beans.factory.BeanNameAware;
+import org.springframework.context.ApplicationListener;
+import org.springframework.context.event.ContextRefreshedEvent;
+import org.springframework.core.Ordered;
+import org.springframework.jdbc.BadSqlGrammarException;
+
+/**
+ * Simple bean used to test RM-3314
+ * 
+ * @author rwetherall
+ * @since 2.2.1.5
+ */
+public class RM3314TestListener implements ApplicationListener<ContextRefreshedEvent>, 
+                                           Ordered,
+                                           BeanNameAware
+{
+    private RecordsManagementAdminServiceImpl recordsManagementAdminService;
+    private NodeService nodeService;
+    private FileFolderService fileFolderService;
+    private Repository repository;
+    
+    private String name;
+    private int order = Ordered.LOWEST_PRECEDENCE;
+
+    public void setRecordsManagementAdminService(RecordsManagementAdminServiceImpl recordsManagementAdminService)
+    {
+        this.recordsManagementAdminService = recordsManagementAdminService;
+    }
+    
+    public void setNodeService(NodeService nodeService)
+    {
+        this.nodeService = nodeService;
+    }
+    
+    public void setFileFolderService(FileFolderService fileFolderService)
+    {
+        this.fileFolderService = fileFolderService;
+    }
+    
+    public void setRepository(Repository repository)
+    {
+        this.repository = repository;
+    }
+    
+    @Override
+    public void setBeanName(String name)
+    {
+        this.name = name;
+    }
+
+    public void setOrder(int order)
+    {
+        this.order = order;
+    }
+
+    @Override
+    public void onApplicationEvent(ContextRefreshedEvent event)
+    {
+        // call back to show whether the custom map is initialised or not
+        RM3314Test.callback.put(name, recordsManagementAdminService.isCustomMapInit());
+        
+        // Do some work on a node to show that reguardless of whether the custom map is
+        // init or not, things still work.
+        // Note: using public services to ensure new transaction for each service call        
+        AuthenticationUtil.runAsSystem(new RunAsWork<Void>()
+        {
+            public Void doWork() throws Exception
+            {
+                try
+                {
+                    // create node
+                    NodeRef folder = fileFolderService.create(
+                                repository.getCompanyHome(), 
+                                name, 
+                                ContentModel.TYPE_FOLDER).getNodeRef();
+                    try
+                    {                
+                        // add aspect
+                        nodeService.addAspect(folder, ContentModel.ASPECT_CLASSIFIABLE, null);
+                        
+                        // remove aspect
+                        nodeService.removeAspect(folder, ContentModel.ASPECT_CLASSIFIABLE);
+                    }
+                    finally
+                    {
+                        // delete node
+                        nodeService.deleteNode(folder);
+                    }
+                }
+                catch (BadSqlGrammarException e)
+                {
+                    // ignore and carry on
+                }
+
+                return null;
+            }
+        });
+    }
+
+    @Override
+    public int getOrder()
+    {
+        return order;
+    }
+}

rm-server/test/resources/test-context.xml

@@ -240,4 +240,25 @@
     	</property>
     </bean>
     
+    <!--  Test listeners -->
+    
+    <!--  Executed with '0' order.  This is default order and as such the application event will be executed relative to the other
+          beans in the order in which they appear in the spring context -->
+    <bean id="test.rm3314.1" class="org.alfresco.module.org_alfresco_module_rm.test.integration.issue.rm3314.RM3314TestListener">
+        <property name="order" value="0"/>
+        <property name="recordsManagementAdminService" ref="recordsManagementAdminService"/>
+        <property name="nodeService" ref="NodeService"/>
+        <property name="fileFolderService" ref="FileFolderService" />
+        <property name="repository" ref="repositoryHelper" />
+    </bean>
+    
+    <!--  The default order has this bean executing its application event with the lowest precendence, ie after all the other
+          beans set to '0' precendence by default -->
+    <bean id="test.rm3314.2" class="org.alfresco.module.org_alfresco_module_rm.test.integration.issue.rm3314.RM3314TestListener">
+        <property name="recordsManagementAdminService" ref="recordsManagementAdminService"/>
+        <property name="nodeService" ref="NodeService"/>
+        <property name="fileFolderService" ref="FileFolderService" />
+        <property name="repository" ref="repositoryHelper" />
+    </bean>
+    
 </beans>