Labyricorn/liquid
1
0
Fork 0
mirror of https://github.com/Shopify/liquid.git synced 2026-10-03 09:05:13 -07:00
Code Issues Packages Projects Releases Wiki Activity

Change to named parameter

Browse Source
This commit is contained in:
Mike Angell
2019-10-02 21:33:02 +10:00
parent e3fba19de0
commit 9bd5362ca1
2 changed files with 8 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/liquid/standardfilters.rb
+2 -2
View File
@@ -419,8 +419,8 @@ module Liquid
result.is_a?(BigDecimal) ? result.to_f : result result.is_a?(BigDecimal) ? result.to_f : result
end end
def default(input, default_value = ''.freeze, false_as_missing = true) def default(input, default_value = ''.freeze, allow_false: false)
if (false_as_missing ? !input : input.nil?) || input.respond_to?(:empty?) && input.empty? if (allow_false ? input.nil? : !input) || input.respond_to?(:empty?) && input.empty?
default_value default_value
else else
input input
test/integration/standard_filter_test.rb
+6 -6
View File
@@ -684,12 +684,12 @@ class StandardFiltersTest < Minitest::Test
end end
def test_default_handle_false def test_default_handle_false
assert_equal "foo", @filters.default("foo", "bar", false) assert_equal "foo", @filters.default("foo", "bar", allow_false: true)
assert_equal "bar", @filters.default(nil, "bar", false) assert_equal "bar", @filters.default(nil, "bar", allow_false: true)
assert_equal "bar", @filters.default("", "bar", false) assert_equal "bar", @filters.default("", "bar", allow_false: true)
assert_equal false, @filters.default(false, "bar", false) assert_equal false, @filters.default(false, "bar", allow_false: true)
assert_equal "bar", @filters.default([], "bar", false) assert_equal "bar", @filters.default([], "bar", allow_false: true)
assert_equal "bar", @filters.default({}, "bar", false) assert_equal "bar", @filters.default({}, "bar", allow_false: true)
end end
def test_cannot_access_private_methods def test_cannot_access_private_methods