Syntax error in content type case statement

Question

This is my code in my Padrino application and I can't figure out what line or bug it is. The error message is "syntax error, unexpected keyword_end expecting $end"

get :index, :provides => [:html, :json] do
    @title = "Restaurants"
    @restaurants = Restaurant.all

    case content_type
      when :json
        render @restaurants
      else
        render 'restaurants/index'
      end
    end
  end

Could you please point out my mistake and also suggest how I might debug it in future? Thanks

score 1 · Accepted Answer · answered Apr 24 '13 at 22:17

You have one spare end keyword. You should remove one.

There is a little mess with indentation in your code. Keeping right indentation helps a lot in avoiding such errors. I would suggest to indent your code like this:

get :index, :provides => [:html, :json] do
  @title = "Restaurants"
  @restaurants = Restaurant.all

  case content_type
  when :json
    render @restaurants
  else
    render 'restaurants/index'
  end
end

timss · Answer 2 · 2013-04-24T22:30:47.960

1

There's an end too much.

Be more careful with your code indentation and this will never be a problem. Example on how it would look in Vim under. I just used =G and it aligned it for me. Additionaly, it'll only highlight the correct use of end. Your favorite editor of choice should have this functionality too. If not, switch.

enter image description here

edited Apr 24 '13 at 22:30

answered Apr 24 '13 at 22:23

timss

9,982
4
34
56

Great tip! I use Vim too. Thanks – Peter Apr 24 '13 at 23:00

score 0 · Answer 3 · answered Apr 24 '13 at 22:19

0

try this:

get :index, :provides => [:html, :json] do
    @title = "Restaurants"
    @restaurants = Restaurant.all

    case content_type
      when :json
        render @restaurants
      else
        render 'restaurants/index'
      end

  end

answered Apr 24 '13 at 22:19

Arup Rakshit

116,827
30
260
317

Syntax error in content type case statement

3 Answers3