Pass current line number to expression parsers

liquid/builtin/statement.py

@@ -60,4 +60,6 @@ class Statement(Tag):
     def parse(self, stream: TokenStream) -> StatementNode:
         tok = stream.current
         expect(stream, TOKEN_STATEMENT)
-        return self.node_class(tok, self.env.parse_filtered_expression_value(tok.value))
+        return self.node_class(
+            tok, self.env.parse_filtered_expression_value(tok.value, tok.linenum)
+        )

liquid/builtin/tags/assign_tag.py

@@ -65,8 +65,8 @@ class AssignTag(Tag):
     block = False
     node_class = AssignNode
 
-    def _parse_expression(self, value: str) -> Expression:
-        return self.env.parse_filtered_expression_value(value)
+    def _parse_expression(self, value: str, linenum: int) -> Expression:
+        return self.env.parse_filtered_expression_value(value, linenum)
 
     def parse(self, stream: TokenStream) -> AssignNode:
         expect(stream, TOKEN_TAG, value=TAG_ASSIGN)
@@ -83,5 +83,9 @@ def parse(self, stream: TokenStream) -> AssignNode:
             )
 
         return self.node_class(
-            tok, AssignmentExpression(name, self._parse_expression(right))
+            tok,
+            AssignmentExpression(
+                name,
+                self._parse_expression(right, stream.current.linenum),
+            ),
         )

liquid/builtin/tags/echo_tag.py

@@ -30,8 +30,8 @@ class EchoTag(Tag):
     block = False
     node_class = EchoNode
 
-    def _parse_expression(self, value: str) -> Expression:
-        return self.env.parse_filtered_expression_value(value)
+    def _parse_expression(self, value: str, linenum: int) -> Expression:
+        return self.env.parse_filtered_expression_value(value, linenum)
 
     def parse(self, stream: TokenStream) -> Node:  # noqa: D102
         expect(stream, TOKEN_TAG, value=TAG_ECHO)
@@ -42,5 +42,5 @@ def parse(self, stream: TokenStream) -> Node:  # noqa: D102
             expr: Expression = NIL
         else:
             expect(stream, TOKEN_EXPRESSION)
-            expr = self._parse_expression(stream.current.value)
+            expr = self._parse_expression(stream.current.value, tok.linenum)
         return self.node_class(tok, expression=expr)

liquid/builtin/tags/for_tag.py

@@ -334,7 +334,10 @@ def parse(self, stream: TokenStream) -> Node:
         stream.next_token()
 
         expect(stream, TOKEN_EXPRESSION)
-        expr = self.env.parse_loop_expression_value(stream.current.value)
+        expr = self.env.parse_loop_expression_value(
+            stream.current.value,
+            stream.current.linenum,
+        )
         stream.next_token()
 
         block = parser.parse_block(stream, ENDFORBLOCK)

liquid/builtin/tags/if_tag.py

@@ -185,7 +185,10 @@ def __init__(self, env: Environment):
     def parse_expression(self, stream: TokenStream) -> Expression:
         """Pare a boolean expression from a stream of tokens."""
         expect(stream, TOKEN_EXPRESSION)
-        return self.env.parse_boolean_expression_value(stream.current.value)
+        return self.env.parse_boolean_expression_value(
+            stream.current.value,
+            stream.current.linenum,
+        )
 
     def parse(self, stream: TokenStream) -> Node:
         expect(stream, TOKEN_TAG, value=TAG_IF)

liquid/builtin/tags/tablerow_tag.py

@@ -267,7 +267,9 @@ def parse(self, stream: TokenStream) -> TablerowNode:
         stream.next_token()
 
         expect(stream, TOKEN_EXPRESSION)
-        loop_expression = self.env.parse_loop_expression_value(stream.current.value)
+        loop_expression = self.env.parse_loop_expression_value(
+            stream.current.value, stream.current.linenum
+        )
         stream.next_token()
 
         block = parser.parse_block(stream, END_TAGBLOCK)

liquid/builtin/tags/unless_tag.py

@@ -184,7 +184,9 @@ def __init__(self, env: Environment):
     def parse_expression(self, stream: TokenStream) -> Expression:
         """Parse a boolean expression from a stream of tokens."""
         expect(stream, TOKEN_EXPRESSION)
-        return self.env.parse_boolean_expression_value(stream.current.value)
+        return self.env.parse_boolean_expression_value(
+            stream.current.value, stream.current.linenum
+        )
 
     def parse(self, stream: TokenStream) -> Union[UnlessNode, IllegalNode]:
         expect(stream, TOKEN_TAG, value=TAG_UNLESS)

liquid/environment.py

@@ -529,7 +529,8 @@ async def analyze_tags_async(
         )
 
     def make_globals(
-        self, globals: Optional[Mapping[str, object]] = None  # noqa: A002
+        self,
+        globals: Optional[Mapping[str, object]] = None,  # noqa: A002
     ) -> Dict[str, object]:
         """Combine environment globals with template globals."""
         if globals:
@@ -577,12 +578,12 @@ def set_expression_cache_size(self, maxsize: int = 0) -> None:
     def _get_expression_parsers(
         self, cache_size: int = 0
     ) -> Tuple[
-        Callable[[str], "BooleanExpression"],
-        Callable[[str], "BooleanExpression"],
-        Callable[[str], "FilteredExpression"],
-        Callable[[str], "FilteredExpression"],
-        Callable[[str], "FilteredExpression"],
-        Callable[[str], "LoopExpression"],
+        Callable[[str, int], "BooleanExpression"],
+        Callable[[str, int], "BooleanExpression"],
+        Callable[[str, int], "FilteredExpression"],
+        Callable[[str, int], "FilteredExpression"],
+        Callable[[str, int], "FilteredExpression"],
+        Callable[[str, int], "LoopExpression"],
     ]:
         if cache_size >= 1:
             return (

liquid/extra/tags/if_expressions.py

@@ -26,7 +26,7 @@ def parse(self, stream: TokenStream) -> StatementNode:
         tok = stream.current
         expect(stream, TOKEN_STATEMENT)
         return StatementNode(
-            tok, self.env.parse_conditional_expression_value(tok.value)
+            tok, self.env.parse_conditional_expression_value(tok.value, tok.linenum)
         )
 
 
@@ -35,17 +35,17 @@ class InlineIfAssignTag(AssignTag):
     inline `if` expressions.
     """
 
-    def _parse_expression(self, value: str) -> Expression:
-        return self.env.parse_conditional_expression_value(value)
+    def _parse_expression(self, value: str, linenum: int) -> Expression:
+        return self.env.parse_conditional_expression_value(value, linenum)
 
 
 class InlineIfEchoTag(EchoTag):
     """A drop-in replacement for the standard `echo` tag that supports
     inline `if` expressions.
     """
 
-    def _parse_expression(self, value: str) -> Expression:
-        return self.env.parse_conditional_expression_value(value)
+    def _parse_expression(self, value: str, linenum: int) -> Expression:
+        return self.env.parse_conditional_expression_value(value, linenum)
 
 
 class InlineIfStatementWithParens(Statement):
@@ -58,7 +58,10 @@ def parse(self, stream: TokenStream) -> StatementNode:
         tok = stream.current
         expect(stream, TOKEN_STATEMENT)
         return StatementNode(
-            tok, self.env.parse_conditional_expression_value_with_parens(tok.value)
+            tok,
+            self.env.parse_conditional_expression_value_with_parens(
+                tok.value, tok.linenum
+            ),
         )
 
 
@@ -68,8 +71,8 @@ class InlineIfAssignTagWithParens(AssignTag):
     terms with parentheses.
     """
 
-    def _parse_expression(self, value: str) -> Expression:
-        return self.env.parse_conditional_expression_value_with_parens(value)
+    def _parse_expression(self, value: str, linenum: int) -> Expression:
+        return self.env.parse_conditional_expression_value_with_parens(value, linenum)
 
 
 class InlineIfEchoTagWithParens(EchoTag):
@@ -78,5 +81,5 @@ class InlineIfEchoTagWithParens(EchoTag):
     terms with parentheses.
     """
 
-    def _parse_expression(self, value: str) -> Expression:
-        return self.env.parse_conditional_expression_value_with_parens(value)
+    def _parse_expression(self, value: str, linenum: int) -> Expression:
+        return self.env.parse_conditional_expression_value_with_parens(value, linenum)

pyproject.toml

@@ -3,7 +3,7 @@ build-backend = "hatchling.build"
 requires = ["hatchling"]
 
 [project]
-authors = [{name = "James Prior", email = "[email protected]"}]
+authors = [{ name = "James Prior", email = "[email protected]" }]
 classifiers = [
   "Development Status :: 5 - Production/Stable",
   "Intended Audience :: Developers",
@@ -18,7 +18,11 @@ classifiers = [
   "Programming Language :: Python :: 3.11",
   "Programming Language :: Python :: 3.12",
 ]
-dependencies = ["python-dateutil>=2.8.1", "typing-extensions>=4.2.0", "importlib-resources>=5.10.0"]
+dependencies = [
+  "python-dateutil>=2.8.1",
+  "typing-extensions>=4.2.0",
+  "importlib-resources>=5.10.0",
+]
 description = "A Python engine for the Liquid template language."
 dynamic = ["version"]
 license = "MIT"
@@ -150,7 +154,6 @@ select = [
   "SLF",
   "T10",
   "T20",
-  "TCH",
   "YTT",
 ]
 # TODO: review ignores