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

[jira] [Commented] (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:comment-tabpanel&focusedCommentId=14605165#comment-14605165 ] 

Hudson commented on TIKA-1669:
------------------------------

UNSTABLE: Integrated in tika-trunk-jdk1.7 #771 (See [https://builds.apache.org/job/tika-trunk-jdk1.7/771/])
Fix for TIKA-1669: xpath node test ./node() should match all contained nodes contributed by WulfB <wu...@inacta.ch> this closes #52 (mattmann: http://svn.apache.org/viewvc/tika/trunk/?view=rev&rev=1688087)
* /tika/trunk/CHANGES.txt
* /tika/trunk/tika-core/src/main/java/org/apache/tika/sax/xpath/NodeMatcher.java
* /tika/trunk/tika-core/src/main/java/org/apache/tika/sax/xpath/XPathParser.java
* /tika/trunk/tika-core/src/test/java/org/apache/tika/sax/xpath/XPathParserTest.java


> 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)