Weird range handling

[charlespwd]

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

I had a partial fix in 185ad2e, 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

[dylanahsmith]

Seems like we can just rescue a NoMethodError from start_obj.to_i..end_obj.to_i and turn it into a Liquid::SyntaxError with an appropriate message. There is no need to tolerate this, since it already wasn't working and the liquid code is clearly wrong

Liquid::Template.parse('{{ ((1..2)..3) }}') is another way to get that NoMethodError, by causing it to call .to_i on a Range object.

I'll open a PR to fix this.