diff --git a/lib/liquid/standardfilters.rb b/lib/liquid/standardfilters.rb index e1867f2..3f53f50 100644 --- a/lib/liquid/standardfilters.rb +++ b/lib/liquid/standardfilters.rb @@ -419,8 +419,8 @@ module Liquid result.is_a?(BigDecimal) ? result.to_f : result end - def default(input, default_value = ''.freeze, false_as_missing = true) - if (false_as_missing ? !input : input.nil?) || input.respond_to?(:empty?) && input.empty? + def default(input, default_value = ''.freeze, allow_false: false) + if (allow_false ? input.nil? : !input) || input.respond_to?(:empty?) && input.empty? default_value else input diff --git a/test/integration/standard_filter_test.rb b/test/integration/standard_filter_test.rb index 7c32627..2e111eb 100644 --- a/test/integration/standard_filter_test.rb +++ b/test/integration/standard_filter_test.rb @@ -684,12 +684,12 @@ class StandardFiltersTest < Minitest::Test end def test_default_handle_false - assert_equal "foo", @filters.default("foo", "bar", false) - assert_equal "bar", @filters.default(nil, "bar", false) - assert_equal "bar", @filters.default("", "bar", false) - assert_equal false, @filters.default(false, "bar", false) - assert_equal "bar", @filters.default([], "bar", false) - assert_equal "bar", @filters.default({}, "bar", false) + assert_equal "foo", @filters.default("foo", "bar", allow_false: true) + assert_equal "bar", @filters.default(nil, "bar", allow_false: true) + assert_equal "bar", @filters.default("", "bar", allow_false: true) + assert_equal false, @filters.default(false, "bar", allow_false: true) + assert_equal "bar", @filters.default([], "bar", allow_false: true) + assert_equal "bar", @filters.default({}, "bar", allow_false: true) end def test_cannot_access_private_methods