X-Git-Url: http://git.argeo.org/?a=blobdiff_plain;f=org.argeo.cms%2Fsrc%2Forg%2Fargeo%2Fcms%2Facr%2Fxml%2FDomContentProvider.java;h=845a6ab4c22a23629194b6bab0e3a0609d8ed2d9;hb=9e6715d360fb29bb817ecaf6d49f0c8a3cc014f5;hp=d6e246df53fb9282c9171cc8e2fd977b12a07b4b;hpb=f3ea14abccc33b1c3326417a87c91145be776c72;p=lgpl%2Fargeo-commons.git

diff --git a/org.argeo.cms/src/org/argeo/cms/acr/xml/DomContentProvider.java b/org.argeo.cms/src/org/argeo/cms/acr/xml/DomContentProvider.java
index d6e246df5..845a6ab4c 100644
--- a/org.argeo.cms/src/org/argeo/cms/acr/xml/DomContentProvider.java
+++ b/org.argeo.cms/src/org/argeo/cms/acr/xml/DomContentProvider.java
@@ -8,6 +8,7 @@ import java.util.Iterator;
 import java.util.List;
 
 import javax.xml.namespace.NamespaceContext;
+import javax.xml.transform.TransformerFactory;
 import javax.xml.xpath.XPath;
 import javax.xml.xpath.XPathConstants;
 import javax.xml.xpath.XPathExpressionException;
@@ -32,12 +33,17 @@ public class DomContentProvider implements ContentProvider, NamespaceContext {
 	// TODO centralise in some executor?
 	private final ThreadLocal<XPath> xPath;
 
+	private TransformerFactory transformerFactory;
+
 	private String mountPath;
 
 	public DomContentProvider(String mountPath, Document document) {
 		this.mountPath = mountPath;
 		this.document = document;
 		this.document.normalizeDocument();
+
+		transformerFactory = TransformerFactory.newInstance();
+
 		XPathFactory xPathFactory = XPathFactory.newInstance();
 		xPath = new ThreadLocal<>() {
 
@@ -51,16 +57,6 @@ public class DomContentProvider implements ContentProvider, NamespaceContext {
 		};
 	}
 
-//	@Override
-//	public Content get() {
-//		return new DomContent(this, document.getDocumentElement());
-//	}
-
-//	public Element createElement(String name) {
-//		return document.createElementNS(null, name);
-//
-//	}
-
 	@Override
 	public ProvidedContent get(ProvidedSession session, String relativePath) {
 		if ("".equals(relativePath))
@@ -70,7 +66,8 @@ public class DomContentProvider implements ContentProvider, NamespaceContext {
 		if (nodes.getLength() > 1)
 			throw new IllegalArgumentException("Multiple content found for " + relativePath + " under " + mountPath);
 		if (nodes.getLength() == 0)
-			throw new ContentNotFoundException("Path " + relativePath + " under " + mountPath + " was not found");
+			throw new ContentNotFoundException(session, mountPath + "/" + relativePath,
+					"Path " + relativePath + " under " + mountPath + " was not found");
 		Element element = (Element) nodes.item(0);
 		return new DomContent(session, this, element);
 	}
@@ -79,8 +76,8 @@ public class DomContentProvider implements ContentProvider, NamespaceContext {
 		if (relativePath.startsWith("/"))
 			throw new IllegalArgumentException("Relative path cannot start with /");
 		String xPathExpression = '/' + relativePath;
-		if ("/".equals(mountPath))
-			xPathExpression = "/" + CrName.root.qName() + xPathExpression;
+		if (Content.ROOT_PATH.equals(mountPath)) // repository root
+			xPathExpression = "/" + CrName.root.get() + xPathExpression;
 		try {
 			NodeList nodes = (NodeList) xPath.get().evaluate(xPathExpression, document, XPathConstants.NODESET);
 			return nodes;
@@ -98,6 +95,7 @@ public class DomContentProvider implements ContentProvider, NamespaceContext {
 		return nodes.getLength() != 0;
 	}
 
+	@Override
 	public void persist(ProvidedSession session) {
 		if (mountPath != null) {
 			Content mountPoint = session.getMountPoint(mountPath);
@@ -145,4 +143,8 @@ public class DomContentProvider implements ContentProvider, NamespaceContext {
 		return Collections.unmodifiableList(res).iterator();
 	}
 
+	TransformerFactory getTransformerFactory() {
+		return transformerFactory;
+	}
+
 }