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

[jackrabbit-filevault] 01/01: JCRVLT-718 fix DocViewParserHandler.endDocViewNode's nodePath argument

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

kwin pushed a commit to branch bugfix/fix-enddocviewnode-argument
in repository https://gitbox.apache.org/repos/asf/jackrabbit-filevault.git

commit ea4f61285d203e6c989cc825ed8155e962b02b1b
Author: Konrad Windszus <kw...@apache.org>
AuthorDate: Wed Sep 13 17:15:03 2023 +0200

    JCRVLT-718 fix DocViewParserHandler.endDocViewNode's nodePath argument
    
    Must point to currently ended node not to its parent
---
 .../java/org/apache/jackrabbit/vault/fs/impl/io/DocViewSAXHandler.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/vault-core/src/main/java/org/apache/jackrabbit/vault/fs/impl/io/DocViewSAXHandler.java b/vault-core/src/main/java/org/apache/jackrabbit/vault/fs/impl/io/DocViewSAXHandler.java
index b6fd8cbb..80e6385f 100644
--- a/vault-core/src/main/java/org/apache/jackrabbit/vault/fs/impl/io/DocViewSAXHandler.java
+++ b/vault-core/src/main/java/org/apache/jackrabbit/vault/fs/impl/io/DocViewSAXHandler.java
@@ -356,8 +356,8 @@ public class DocViewSAXHandler extends RejectingEntityDefaultHandler implements
     public void endElement(String uri, String localName, String qName) throws SAXException {
         log.trace("<- element {}", qName);
         try {
-            currentPath = Text.getRelativeParent(currentPath, 1);
             handler.endDocViewNode(currentPath, nodeStack.pop(), Optional.ofNullable(nodeStack.peek()), locator.getLineNumber(), locator.getColumnNumber());
+            currentPath = Text.getRelativeParent(currentPath, 1);
         } catch (RepositoryException|IOException e) {
             throw new SAXException(e);
         }