You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tika.apache.org by "Chris A. Mattmann (JIRA)" <ji...@apache.org> on 2015/06/29 07:28:04 UTC

[jira] [Resolved] (TIKA-1669) xpath node test ./node() should match all contained nodes

     [ https://issues.apache.org/jira/browse/TIKA-1669?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Chris A. Mattmann resolved TIKA-1669.
-------------------------------------
    Resolution: Fixed

- fixed!

{noformat}
[chipotle:~/tmp/tika1.10] mattmann% svn commit -m "Fix for TIKA-1669: xpath node test ./node() should match all contained nodes contributed by WulfB <wu...@inacta.ch> this closes #52"
Sending        CHANGES.txt
Sending        tika-core/src/main/java/org/apache/tika/sax/xpath/NodeMatcher.java
Sending        tika-core/src/main/java/org/apache/tika/sax/xpath/XPathParser.java
Sending        tika-core/src/test/java/org/apache/tika/sax/xpath/XPathParserTest.java
Transmitting file data ....
Committed revision 1688087.
[chipotle:~/tmp/tika1.10] mattmann% 
{noformat}


> xpath node test ./node() should match all contained nodes
> ---------------------------------------------------------
>
>                 Key: TIKA-1669
>                 URL: https://issues.apache.org/jira/browse/TIKA-1669
>             Project: Tika
>          Issue Type: Bug
>          Components: handler, parser
>         Environment: Github https://github.com/apache/tika/pull/52
>            Reporter: Chris A. Mattmann
>            Assignee: Chris A. Mattmann
>             Fix For: 1.10
>
>
> Per: https://github.com/apache/tika/pull/52 and Wulf Berschin:
> {noformat}
> following the w3c spec the xpath node test ./node() should matches all contained nodes (types) but not attributes since attributes are no children. (=> Changes in NodeMatcher)
> So "/node()" returns a superset of "/*". In XPathParser the latter lead to the construction of "ChildMatcher(parse(xpath.substring(2)));" but the first simply returned a "NodeMatcher.INSTANCE" what is wrong i.m.o.
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)