core
/
oga
3
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fixed XPath operator precedence. 

Apparently using multiple `left` rules with T_AND and T_OR being separate
solves this problem. Riiiiight....
This commit is contained in:
Yorick Peterse 2014-06-23 00:15:43 +02:00
parent 6a2f4fa82d
commit a440d3f003
2 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lib/oga/xpath/parser.y
View File

@ -12,7 +12,8 @@ options no_result_var
prechigh prechigh
left T_PIPE T_MOD T_DIV T_MUL T_SUB T_ADD left T_PIPE T_MOD T_DIV T_MUL T_SUB T_ADD
left T_GT T_GTE T_LT T_LTE T_NEQ T_EQ T_AND T_OR left T_GT T_GTE T_LT T_LTE T_NEQ T_EQ
left T_AND T_OR
preclow preclow
rule rule

6
spec/oga/xpath/parser/operator_precedence_spec.rb
View File

@ -105,9 +105,9 @@ describe Oga::XPath::Parser do
parse_xpath('A and B = C').should == s( parse_xpath('A and B = C').should == s(
:path, :path,
s( s(
:eq, :and,
s(:and, s(:test, nil, 'A'), s(:test, nil, 'B')), s(:test, nil, 'A'),
s(:test, nil, 'C') s(:eq, s(:test, nil, 'B'), s(:test, nil, 'C'))
) )
) )
end end