You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jackrabbit.apache.org by re...@apache.org on 2023/10/27 09:52:40 UTC

(jackrabbit-filevault) 01/01: JCRVLT-724: DocViewSaxFormatter: improve diagnostics when generating DocViewNode2 from JCR node

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

reschke pushed a commit to branch issue/JCRVLT-724
in repository https://gitbox.apache.org/repos/asf/jackrabbit-filevault.git

commit 7c2ae896a819ccb5ee33736439c13db03c6706c0
Author: Julian Reschke <re...@apache.org>
AuthorDate: Fri Oct 27 10:52:13 2023 +0100

    JCRVLT-724: DocViewSaxFormatter: improve diagnostics when generating DocViewNode2 from JCR node
---
 .../org/apache/jackrabbit/vault/fs/impl/io/DocViewSAXFormatter.java   | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/vault-core/src/main/java/org/apache/jackrabbit/vault/fs/impl/io/DocViewSAXFormatter.java b/vault-core/src/main/java/org/apache/jackrabbit/vault/fs/impl/io/DocViewSAXFormatter.java
index 16ca0068..631e573a 100644
--- a/vault-core/src/main/java/org/apache/jackrabbit/vault/fs/impl/io/DocViewSAXFormatter.java
+++ b/vault-core/src/main/java/org/apache/jackrabbit/vault/fs/impl/io/DocViewSAXFormatter.java
@@ -35,6 +35,7 @@ import javax.xml.stream.XMLStreamWriter;
 
 import org.apache.jackrabbit.spi.Name;
 import org.apache.jackrabbit.spi.commons.conversion.DefaultNamePathResolver;
+import org.apache.jackrabbit.spi.commons.conversion.IllegalNameException;
 import org.apache.jackrabbit.spi.commons.name.NameConstants;
 import org.apache.jackrabbit.spi.commons.namespace.NamespaceResolver;
 import org.apache.jackrabbit.spi.commons.namespace.SessionNamespaceResolver;
@@ -201,6 +202,9 @@ public class DocViewSAXFormatter implements AggregateWalkListener {
             docViewNode.writeStart(writer, nsResolver, namespacePrefixes);
         } catch (XMLStreamException e) {
             throw new RepositoryException(e);
+        } catch (IllegalNameException e) {
+            // augment exception message with path of node causing the problem
+            throw new IllegalNameException(e.getMessage() + " (on path '" + node.getPath() + "')", e);
         }
     }