Fix JSONPath operator precedence. (#42)

Author: jg-rp

CHANGELOG.md

@@ -1,5 +1,11 @@
 # Python JSONPath Change Log
 
+## Version 0.10.2 (unreleased)
+
+**Fixes**
+
+- Fixed precedence of the logical not operator in JSONPath filter expressions. Previously, logical _or_ and logical _and_ had priority over _not_. See [#41](https://github.com/jg-rp/python-jsonpath/issues/41).
+
 ## Version 0.10.1
 
 **Hot fix**

jsonpath/parse.py

@@ -158,7 +158,7 @@ class Parser:
         TOKEN_LG: PRECEDENCE_RELATIONAL,
         TOKEN_LT: PRECEDENCE_RELATIONAL,
         TOKEN_NE: PRECEDENCE_RELATIONAL,
-        TOKEN_NOT: PRECEDENCE_LOGICALRIGHT,
+        TOKEN_NOT: PRECEDENCE_PREFIX,
         TOKEN_OR: PRECEDENCE_LOGICAL,
         TOKEN_RE: PRECEDENCE_RELATIONAL,
         TOKEN_RPAREN: PRECEDENCE_LOWEST,
@@ -499,9 +499,7 @@ def parse_prefix_expression(self, stream: TokenStream) -> FilterExpression:
         assert tok.kind == TOKEN_NOT
         return PrefixExpression(
             operator="!",
-            right=self.parse_filter_selector(
-                stream, precedence=self.PRECEDENCE_LOGICALRIGHT
-            ),
+            right=self.parse_filter_selector(stream, precedence=self.PRECEDENCE_PREFIX),
         )
 
     def parse_infix_expression(

tests/test_parse.py

@@ -180,6 +180,21 @@ class Case:
         path='$[[email protected] == "ba\\"r"]',
         want='$[?@[\'foo\'] == "ba\\"r"]',
     ),
+    Case(
+        description="not binds more tightly than or",
+        path="$[[email protected] || [email protected]]",
+        want="$[?(!@['a'] || !@['b'])]",
+    ),
+    Case(
+        description="not binds more tightly than and",
+        path="$[[email protected] && [email protected]]",
+        want="$[?(!@['a'] && !@['b'])]",
+    ),
+    Case(
+        description="control precedence with parens",
+        path="$[?!(@.a && [email protected])]",
+        want="$[?!(@['a'] && !@['b'])]",
+    ),
 ]