diff --git a/config/alfresco/action-services-context.xml b/config/alfresco/action-services-context.xml
index 2db58b8536..10ecabd283 100644
--- a/config/alfresco/action-services-context.xml
+++ b/config/alfresco/action-services-context.xml
@@ -209,9 +209,6 @@
-
-
-
{http://www.alfresco.org/model/content/1.0}content
diff --git a/source/java/org/alfresco/repo/action/executer/TransformActionExecuter.java b/source/java/org/alfresco/repo/action/executer/TransformActionExecuter.java
index acb16641cd..5b4c83e4dc 100644
--- a/source/java/org/alfresco/repo/action/executer/TransformActionExecuter.java
+++ b/source/java/org/alfresco/repo/action/executer/TransformActionExecuter.java
@@ -28,8 +28,6 @@ import java.util.List;
import org.alfresco.model.ContentModel;
import org.alfresco.repo.action.ParameterDefinitionImpl;
-import org.alfresco.repo.content.transform.ContentTransformer;
-import org.alfresco.repo.content.transform.ContentTransformerRegistry;
import org.alfresco.service.cmr.action.Action;
import org.alfresco.service.cmr.action.ParameterDefinition;
import org.alfresco.service.cmr.dictionary.DataTypeDefinition;
@@ -55,7 +53,7 @@ import org.apache.commons.logging.LogFactory;
* @author Roy Wetherall
*/
public class TransformActionExecuter extends ActionExecuterAbstractBase
-{
+{
/** Error messages */
public static final String ERR_OVERWRITE = "Unable to overwrite copy because more than one have been found.";
@@ -82,7 +80,6 @@ public class TransformActionExecuter extends ActionExecuterAbstractBase
private ContentService contentService;
private CopyService copyService;
private MimetypeService mimetypeService;
- private ContentTransformerRegistry transformerRegistry;
/**
* Set the mime type service
@@ -134,16 +131,6 @@ public class TransformActionExecuter extends ActionExecuterAbstractBase
this.copyService = copyService;
}
- /**
- * Set the transformer registry
- *
- * @param transformerRegistry the transformer registry
- */
- public void setTransformerRegistry(ContentTransformerRegistry transformerRegistry)
- {
- this.transformerRegistry = transformerRegistry;
- }
-
/**
* Add parameter definitions
*/
@@ -298,15 +285,18 @@ public class TransformActionExecuter extends ActionExecuterAbstractBase
protected void doTransform( Action ruleAction,
NodeRef sourceNodeRef, ContentReader contentReader,
NodeRef destinationNodeRef, ContentWriter contentWriter)
+
{
- // try to pre-empt the lack of a transformer
- if (!this.contentService.isTransformable(contentReader, contentWriter))
+ // Transformation options
+ TransformationOptions options = new TransformationOptions(sourceNodeRef, ContentModel.PROP_NAME, destinationNodeRef, ContentModel.PROP_NAME);
+
+ // try to pre-empt the lack of a transformer
+ if (this.contentService.isTransformable(contentReader, contentWriter, options) == false)
{
throw new NoTransformerException(contentReader.getMimetype(), contentWriter.getMimetype());
}
// transform
- TransformationOptions options = new TransformationOptions(sourceNodeRef, ContentModel.PROP_NAME, destinationNodeRef, ContentModel.PROP_NAME);
this.contentService.transform(contentReader, contentWriter, options);
}
diff --git a/source/java/org/alfresco/service/cmr/repository/TransformationOptions.java b/source/java/org/alfresco/service/cmr/repository/TransformationOptions.java
index f95a950452..6435ccf168 100644
--- a/source/java/org/alfresco/service/cmr/repository/TransformationOptions.java
+++ b/source/java/org/alfresco/service/cmr/repository/TransformationOptions.java
@@ -41,9 +41,9 @@ import org.alfresco.service.namespace.QName;
public class TransformationOptions
{
/** Option map names to preserve backward compatibility */
- public static final String OPT_SOURCE_NODEREF = "sourceNodeRef";
+ public static final String OPT_SOURCE_NODEREF = "contentReaderNodeRef";
public static final String OPT_SOURCE_CONTENT_PROPERTY = "sourceContentProperty";
- public static final String OPT_TARGET_NODEREF = "targetNodeRef";
+ public static final String OPT_TARGET_NODEREF = "contentWriterNodeRef";
public static final String OPT_TARGET_CONTENT_PROPERTY = "targetContentProperty";
/** The source node reference */
@@ -182,6 +182,11 @@ public class TransformationOptions
return new HashMap(10);
}
+ /**
+ * Places the values of the transformation options into a Map
+ *
+ * @param optionsMap the options map
+ */
protected void toMapImpl(Map optionsMap)
{
optionsMap.put(OPT_SOURCE_NODEREF, getSourceNodeRef());
@@ -190,12 +195,16 @@ public class TransformationOptions
optionsMap.put(OPT_TARGET_CONTENT_PROPERTY, getTargetContentProperty());
}
+ /**
+ * Populates the transformation options from a given Map of value.
+ *
+ * @param optionsMap the options map
+ */
protected void fromMapImpl(Map optionsMap)
{
this.sourceNodeRef = (NodeRef)optionsMap.get(OPT_SOURCE_NODEREF);
this.sourceContentProperty = (QName)optionsMap.get(OPT_SOURCE_CONTENT_PROPERTY);
this.targetNodeRef = (NodeRef)optionsMap.get(OPT_TARGET_NODEREF);
this.targetContentProperty = (QName)optionsMap.get(OPT_TARGET_CONTENT_PROPERTY);
- }
-
+ }
}