Use of "blank?" (Rails code) in Liquid

[fw42]

See

liquid/lib/liquid/context.rb

Line 139 in 5c5e7de

'blank' => :blank?,

This doesn't work outside of Rails (Jekyll for instance).

[fw42]

See #223.

[phoet]

we could add a default Object#blank? implementation and include it when there is none, i.e. in non rails projects.

[mikebridge]

I'm confused about blank---is it supposed to work by default? In my tests it's always evaluating to false:

{% assign space = " " %}{% if space == blank %}TRUTHY{% else %}NOT TRUTHY{% endif %}
  --> NOT TRUTHY

{% if " " == blank %}TRUTHY{% else %}NOT TRUTHY{% endif %}
  --> NOT TRUTHY

{% if "notblank" == blank %}TRUTHY{% else %}NOT TRUTHY{% endif %}
  --> NOT TRUTHY

{% if myundefined == blank %}TRUTHY{% else %}NOT TRUTHY{% endif %}
  --> NOT TRUTHY

{% assign myarray = "1,2" | split: "," %}{% if myarray == blank %}TRUTHY{% else %}NOT TRUTHY{% endif %}
  --> NOT TRUTHY

{% assign myarray = "" | split: "," %}{% if myarray == blank %}TRUTHY{% else %}NOT TRUTHY{% endif %}
  --> NOT TRUTHY

[fw42]

That's a bug, see #589. It works if activesupport (e.g. Rails) is present.

[mikebridge]

@fw42 Ok got that working, thanks!

[louwers]

After installing activesupport by adding gem "activesupport" to my Gemfile, it still didn't work. Is there any way to trim whitespace so that I can compare with the empty string?

[pushrax]

Since this is a language feature we may as well include it in Liquid.

[fw42]

👎 to adding activesupport as a dependency. But I would be okay with reimplementing this method.

[pushrax]

Right, I agree.