[ 
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 
<[email protected]> 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)

Reply via email to