Question

I find myself frequently writing code like this in the HTML HEAD and other places:

<% if @canonical_url %>
  <link rel="canonical" href="<%= @canonical_url %>"/>
<% end %>

I then set the variable in the controller if it’s appropriate.

Is there any way of writing the equivalent on one line, or maybe a different way of organizing the code?

Answer 1

Ok, I came up with this solution. In the template:

<%= show_if('<link rel="canonical" href="$1"/>', @canonical_url) %>

And then the helper method:

#
# Return the template text if the variable has a value.
#
def show_if(template, variable)
  if variable
    template.gsub('$1', variable)
  else
    ''
  end
end