Weird range handling

Open charlespwd opened this issue 3 years ago • 0 comments

Template.parse("{{ (true..false) }}") throws a NoMethodError .to_i on TrueClass.

I had a partial fix in https://github.com/Shopify/liquid/commit/185ad2e5104e92912e223586b14deeff28878f51, but it looks like that's breaking lax parse with an extra dot by throwing an Liquid::ArgumentError instead of rendering.

Don't really have more time to put on this right now so I'm opening an issue instead :D

Aug 11 '22 15:08 charlespwd