JRuby

<%- -%> in ERB template causes fatal error in JRuby

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Critical Critical
  • Resolution: Not A Bug
  • Affects Version/s: JRuby 1.3.1
  • Fix Version/s: None
  • Component/s: Extensions
  • Labels:
    None
  • Environment:
  • Number of attachments :
    0

Description

People are telling me I can squelch newlines in ERB templates by putting - around things, as in <%- foo -%> instead of <% bar %>

In practice, however, the following code works:

<% @categories.each do |item| %>
<category name="<%= item %>"/>
<% end %>

and the following code does not work:

<%- @categories.each do |item| -%>
<category name="<%= item %>"/>
<% end %>

and it results in:

SyntaxError at /xml
/Users/rjurney/JRuby/views/xml.erb:4: , unexpected ';'

  • file: xml.erb
  • location: render_erb
  • line: 2

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Charles Oliver Nutter added a comment -

Confirmed on 1.4dev as well, with a better error:

~/projects/jruby &#10132; jruby -e "require 'erb'; x = ERB.new(File.read('foo.erb')); p x.run"
(erb):2: (erb):3: , unexpected ';' (SyntaxError)

; - @categories.each do |item| -; _erbout.concat "\n"
                                ^

~/projects/jruby &#10132; cat foo.erb
<% @categories.each do |item| %>
<%- @categories.each do |item| -%>
<% @categories.each do |item| -%>
Show
Charles Oliver Nutter added a comment - Confirmed on 1.4dev as well, with a better error: ~/projects/jruby &#10132; jruby -e "require 'erb'; x = ERB.new(File.read('foo.erb')); p x.run" (erb):2: (erb):3: , unexpected ';' (SyntaxError) ; - @categories.each do |item| -; _erbout.concat "\n" ^ ~/projects/jruby &#10132; cat foo.erb <% @categories.each do |item| %> <%- @categories.each do |item| -%> <% @categories.each do |item| -%>
Hide
Permalink
Nick Sieger added a comment -

See http://gist.github.com/189181.

Show
Nick Sieger added a comment - See http://gist.github.com/189181 .

People

  • Assignee:
    Unassigned
    Reporter:
    Russell Jurney
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: