Fixed ComparisonExpression.parse with MethodLiterals

Author: albchu

lib/liquid/condition.rb

@@ -52,6 +52,10 @@ def self.parse_expression(parse_context, markup)
       @@method_literals[markup] || parse_context.parse_expression(markup)
     end
 
+    def self.parse(markup, ss, cache)
+      @@method_literals[markup] || Expression.parse(markup, ss, cache)
+    end
+
     attr_reader :attachment, :child_condition
     attr_accessor :left, :operator, :right
 
@@ -112,19 +116,23 @@ def inspect
     private
 
     def equal_variables(left, right)
+      if left.is_a?(MethodLiteral) && right.is_a?(MethodLiteral)
+        return left.to_s == right.to_s
+      end
+
       if left.is_a?(MethodLiteral)
         if right.respond_to?(left.method_name)
           return right.send(left.method_name)
         else
-          return nil
+          return left.to_s == right
         end
       end
 
       if right.is_a?(MethodLiteral)
         if left.respond_to?(right.method_name)
           return left.send(right.method_name)
         else
-          return nil
+          return right.to_s == left
         end
       end
 

lib/liquid/expression/comparison_expression.rb

@@ -14,10 +14,9 @@ def parse(markup, ss, cache)
           match = comparison?(markup)
 
           if match
-            left = Expression.parse(match[1].strip, ss, cache)
+            left = Condition.parse(match[1].strip, ss, cache)
             operator = match[2].strip
-            right = Expression.parse(match[3].strip, ss, cache)
-
+            right = Condition.parse(match[3].strip, ss, cache)
             return Condition.new(left, operator, right)
           end