//root/p[@attr='val'] is not working about xpath HOT 4 CLOSED

It seems caused by the childQuery not checking the predicate on its Input directly and wanting to iterate to children before. I believe this change should fix the issue:

diff --git a/query.go b/query.go
index fe2633f..d3ef543 100644
--- a/query.go
+++ b/query.go
@@ -198,6 +198,9 @@ func (c *childQuery) Select(t iterator) NodeNavigator {
                        node = node.Copy()
                        first := true
                        c.iterator = func() NodeNavigator {
+                               if c.Predicate(node) {
+                                       return node
+                               }
                                for {
                                        if first {
						if !node.MoveToChild() {

from xpath.

Perhaps there are legitimate cases where we want to check for children and not the current node, but // means /descendant-or-self::node()/ (section 2.5 of W3C XPath 1.0 recommendation)

from xpath.

The above patch is creating an infinite loop somewhere in the tests, It should probably be tweaked to behave like this only in some cases.

from xpath.

Its generate XPath query for XPath expression problem.

default generate query look is : filterQuery > childQuery > childQuery > descendantQuery but it should be like this: filterQuery > childQuery > descendantQuery.

from xpath.