You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by ts...@apache.org on 2024/05/13 08:15:09 UTC

(wicket) 01/01: fix: configurable transformer factory as param

This is an automated email from the ASF dual-hosted git repository.

tsoloschenko pushed a commit to branch feature/transformerfactory
in repository https://gitbox.apache.org/repos/asf/wicket.git

commit 7a518dda8b41efeabdf147463ced9e770e962464
Author: Tobias Soloschenko <ts...@apache.org>
AuthorDate: Mon May 13 10:14:38 2024 +0200

    fix: configurable transformer factory as param
---
 .../wicket/util/resource/XSLTResourceStream.java   | 28 ++++++++++++----------
 1 file changed, 16 insertions(+), 12 deletions(-)

diff --git a/wicket-util/src/main/java/org/apache/wicket/util/resource/XSLTResourceStream.java b/wicket-util/src/main/java/org/apache/wicket/util/resource/XSLTResourceStream.java
index 1ccadcc658..30e49d59ba 100644
--- a/wicket-util/src/main/java/org/apache/wicket/util/resource/XSLTResourceStream.java
+++ b/wicket-util/src/main/java/org/apache/wicket/util/resource/XSLTResourceStream.java
@@ -56,13 +56,27 @@ public class XSLTResourceStream extends AbstractResourceStream
 
 	/**
 	 * Construct.
-	 * 
+	 *
 	 * @param xsltResource
 	 *            the XSL stylesheet as an {@link IResourceStream}
 	 * @param xmlResource
 	 *            the input XML document as an {@link IResourceStream}
 	 */
 	public XSLTResourceStream(final IResourceStream xsltResource, final IResourceStream xmlResource)
+	{
+		this(xsltResource, xmlResource, javax.xml.transform.TransformerFactory.newInstance());
+	}
+	/**
+	 * Construct.
+	 * 
+	 * @param xsltResource
+	 *            the XSL stylesheet as an {@link IResourceStream}
+	 * @param xmlResource
+	 *            the input XML document as an {@link IResourceStream}
+	 * @param transformerFactory
+	 * 			  the transformer factory used to transform the xmlResource
+	 */
+	public XSLTResourceStream(final IResourceStream xsltResource, final IResourceStream xmlResource, javax.xml.transform.TransformerFactory transformerFactory)
 	{
 		try
 		{
@@ -74,7 +88,7 @@ public class XSLTResourceStream extends AbstractResourceStream
 			javax.xml.transform.Result result = new javax.xml.transform.stream.StreamResult(out);
 
 			// create an instance of TransformerFactory
-			javax.xml.transform.TransformerFactory transFact = getTransformerFactory();
+			javax.xml.transform.TransformerFactory transFact = transformerFactory;
 
 			javax.xml.transform.Transformer trans = transFact.newTransformer(xsltSource);
 			Map<Object, Object> parameters = getParameters();
@@ -99,16 +113,6 @@ public class XSLTResourceStream extends AbstractResourceStream
 		}
 	}
 
-	/**
-	 * Used to provide a configured {@link javax.xml.transform.TransformerFactory}.
-	 *
-	 * @return the transformer factory
-	 */
-	protected javax.xml.transform.TransformerFactory getTransformerFactory()
-	{
-		return javax.xml.transform.TransformerFactory.newInstance();
-	}
-
 	/**
 	 * @see org.apache.wicket.util.resource.IResourceStream#close()
 	 */