From 2bf51d65c30271b600835eac5ba5676c6fa336e1 Mon Sep 17 00:00:00 2001
From: Nick Burch <nick.burch@quanticate.com>
Date: Mon, 17 Oct 2011 17:35:54 +0000
Subject: [PATCH] ALF-10618 related - put the metadata extractor action
 executor name into a constant, rather than having it inline as a string in
 various places

git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@31296 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
---
 source/java/org/alfresco/filesys/repo/ContentDiskDriver.java   | 3 ++-
 .../repo/action/executer/ContentMetadataExtracter.java         | 2 ++
 source/java/org/alfresco/repo/imap/ImapContentPolicy.java      | 3 ++-
 3 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/source/java/org/alfresco/filesys/repo/ContentDiskDriver.java b/source/java/org/alfresco/filesys/repo/ContentDiskDriver.java
index 67fcdddef0..97f504c588 100644
--- a/source/java/org/alfresco/filesys/repo/ContentDiskDriver.java
+++ b/source/java/org/alfresco/filesys/repo/ContentDiskDriver.java
@@ -82,6 +82,7 @@ import org.alfresco.jlan.smb.server.SMBSrvSession;
 import org.alfresco.jlan.util.MemorySize;
 import org.alfresco.jlan.util.WildCard;
 import org.alfresco.model.ContentModel;
+import org.alfresco.repo.action.executer.ContentMetadataExtracter;
 import org.alfresco.repo.admin.SysAdminParams;
 import org.alfresco.repo.content.MimetypeMap;
 import org.alfresco.repo.node.archive.NodeArchiveService;
@@ -4267,7 +4268,7 @@ public class ContentDiskDriver extends AlfrescoTxDiskDriver implements DiskInter
         }
 
         // Extract metadata pending change for ALF-5082
-        Action action = getActionService().createAction("extract-metadata");
+        Action action = getActionService().createAction(ContentMetadataExtracter.EXECUTOR_NAME);
         if(action != null)
         {
             getActionService().executeAction(action, toNode);
diff --git a/source/java/org/alfresco/repo/action/executer/ContentMetadataExtracter.java b/source/java/org/alfresco/repo/action/executer/ContentMetadataExtracter.java
index 12910d11f8..023501108b 100644
--- a/source/java/org/alfresco/repo/action/executer/ContentMetadataExtracter.java
+++ b/source/java/org/alfresco/repo/action/executer/ContentMetadataExtracter.java
@@ -55,6 +55,8 @@ public class ContentMetadataExtracter extends ActionExecuterAbstractBase
 {
     private static Log logger = LogFactory.getLog(ContentMetadataExtracter.class);
     
+    public static final String EXECUTOR_NAME = "extract-metadata";
+    
     private NodeService nodeService;
     private ContentService contentService;
     private DictionaryService dictionaryService;
diff --git a/source/java/org/alfresco/repo/imap/ImapContentPolicy.java b/source/java/org/alfresco/repo/imap/ImapContentPolicy.java
index 0e02370dd9..0ddf71aca1 100644
--- a/source/java/org/alfresco/repo/imap/ImapContentPolicy.java
+++ b/source/java/org/alfresco/repo/imap/ImapContentPolicy.java
@@ -23,6 +23,7 @@ import java.util.Collections;
 import java.util.Map;
 
 import org.alfresco.model.ImapModel;
+import org.alfresco.repo.action.executer.ContentMetadataExtracter;
 import org.alfresco.repo.copy.CopyBehaviourCallback;
 import org.alfresco.repo.copy.CopyDetails;
 import org.alfresco.repo.copy.CopyServicePolicies.OnCopyNodePolicy;
@@ -78,7 +79,7 @@ public class ImapContentPolicy
         // ensure the aspect is the one we expect
         if (aspectTypeQName.equals(ImapModel.ASPECT_IMAP_CONTENT))
         {
-            Action action = getActionService().createAction("extract-metadata");
+            Action action = getActionService().createAction(ContentMetadataExtracter.EXECUTOR_NAME);
             if(action != null)
             {
                 getActionService().executeAction(action, nodeRef);