always stringify properties in all array filters

Author: Maaarcocr

lib/liquid/standardfilters.rb

@@ -387,6 +387,7 @@ def sort(input, property = nil)
         end
       elsif ary.all? { |el| el.respond_to?(:[]) }
         begin
+          property = Utils.to_s(property)
           ary.sort { |a, b| nil_safe_compare(a[property], b[property]) }
         rescue TypeError
           raise_property_error(property)
@@ -416,6 +417,7 @@ def sort_natural(input, property = nil)
         end
       elsif ary.all? { |el| el.respond_to?(:[]) }
         begin
+          property = Utils.to_s(property)
           ary.sort { |a, b| nil_safe_casecmp(a[property], b[property]) }
         rescue TypeError
           raise_property_error(property)
@@ -503,6 +505,7 @@ def uniq(input, property = nil)
       elsif ary.empty? # The next two cases assume a non-empty array.
         []
       else
+        property = Utils.to_s(property)
         ary.uniq do |item|
           item[property]
         rescue TypeError
@@ -534,6 +537,7 @@ def reverse(input)
     # @liquid_syntax array | map: string
     # @liquid_return [array[untyped]]
     def map(input, property)
+      property = Utils.to_s(property)
       InputIterator.new(input, context).map do |e|
         e = e.call if e.is_a?(Proc)
 
@@ -563,6 +567,7 @@ def compact(input, property = nil)
       elsif ary.empty? # The next two cases assume a non-empty array.
         []
       else
+        property = Liquid::Utils.to_s(property)
         ary.reject do |item|
           item[property].nil?
         rescue TypeError
@@ -952,13 +957,13 @@ def default(input, default_value = '', options = {})
     # @liquid_syntax array | sum
     # @liquid_return [number]
     def sum(input, property = nil)
-      property = Liquid::Utils.to_s(property)
+      property = Liquid::Utils.to_s(property) unless property.nil? else property
 
       ary = InputIterator.new(input, context)
       return 0 if ary.empty?
 
       values_for_sum = ary.map do |item|
-        if property.empty?
+        if property.nil?
           item
         elsif item.respond_to?(:[])
           item[property]