Bugfix: Fixes @filter() involving multiple comparison operators (#148)

Author: kdpsingh

NEWS.md

@@ -3,6 +3,7 @@
 ## v.0.17.0 - 2025-03-24
 - Bugfix: `@count()` can now be called multiple times. If column `n` already exists, then the new column containing the count will be `nn` (and so on).
 - Bugfix: `@unnest_wider()` now works on data where keys are missing
+- Bugfix: Fixes `@filter()` involving multiple comparison operators (e.g., `3 <= a < 5`), which have a `:head` of `:comparison` and are parsed differently than `(3 <= a) && (a < 5)`
 - Adds logging ability to track changes to data frames with `TidierData_set("log", true)`
 - Adds docs describing logging and code printing
 

src/parsing.jl

@@ -143,6 +143,12 @@ function parse_function(lhs::Union{Symbol, Expr}, rhs::Expr; autovec::Bool=true,
     if @capture(x, (fn_(args__)) | (fn_.(args__))) && fn != :esc
       args = args[isa.(args, Symbol)]
       push!(src, args...)
+    elseif hasproperty(x, :head) && x.head == :comparison
+      for (index, value) in enumerate(x.args)
+        if index % 2 == 1 && value isa Symbol
+          push!(src, value)
+        end
+      end
     end
     return x
   end
@@ -357,6 +363,15 @@ function parse_autovec(tidy_expr::Union{Expr,Symbol})
     elseif hasproperty(x, :head) && (x.head == :&& || x.head == :||)
       x.head = Symbol("." * string(x.head))
       return x
+    elseif hasproperty(x, :head) && x.head == :comparison
+      for (index, value) in enumerate(x.args)
+        if index % 2 == 0
+          if first(string(value), 1) != "."
+            x.args[index] = Symbol("." * string(value))
+          end
+        end
+      end
+      return x
     end
     return x
   end