Merge pull request #411 from Shopify/to-s-before-split

Cast input to string before spliting
This commit is contained in:
Jean Boussier
2014-08-08 00:13:35 -04:00
2 changed files with 2 additions and 1 deletions
+1 -1
View File
@@ -75,7 +75,7 @@ module Liquid
# <div class="summary">{{ post | split '//' | first }}</div> # <div class="summary">{{ post | split '//' | first }}</div>
# #
def split(input, pattern) def split(input, pattern)
input.split(pattern) input.to_s.split(pattern)
end end
def strip(input) def strip(input)
+1
View File
@@ -106,6 +106,7 @@ class StandardFiltersTest < Minitest::Test
assert_equal ['A?Z'], @filters.split('A?Z', '~') assert_equal ['A?Z'], @filters.split('A?Z', '~')
# Regexp works although Liquid does not support. # Regexp works although Liquid does not support.
assert_equal ['A','Z'], @filters.split('AxZ', /x/) assert_equal ['A','Z'], @filters.split('AxZ', /x/)
assert_equal [], @filters.split(nil, ' ')
end end
def test_escape def test_escape