Chapter 6: Story Controller Won't Display

[pdenlinger]

I'm having trouble getting the Story Controller to display correctly; here is the error page I get when I try to get it to run locally:



Here is the code for story_controller.rb:

class StoryController < ApplicationController
  def index
    @story = Story.find(:first, :order=>'RAND()')
    def new
      @story = Story.new(params[:story])
      if request.post?
        @story.save
        redirect_to :action => 'index'
  end
end

And here is the code for new.rhtml

<% form_for :story do |f| %>
<p>
  name:<br />
  <%= f.text_field :name %>
</p>
<p>
  link:<br />
  <%=f.text_field :link %>
</p>
<p>
  <%= submit_tag %>
</p>
<% end %>

Thanks for your assistance.

[skrolikowski]

You're forgetting to "end" something somewhere.

[gregor002]

[QUOTE=pdenlinger;3314169]

class StoryController < ApplicationController
  def index
    @story = Story.find(:first, :order=>'RAND()')
    def new
      @story = Story.new(params[:story])
      if request.post?
        @story.save
        redirect_to :action => 'index'
  end
end

You've got a method nested within a method. "index" and "new" should be individual methods. Here's the fix:

class StoryController < ApplicationController
  def index
    @story = Story.find(:first, :order=>'RAND()')
  end
  
  def new
    @story = Story.new(params[:story])
    if request.post?
      @story.save
       redirect_to :action => 'index'
    end
  end

[gregor002]

class StoryController < ApplicationController
  def index
    @story = Story.find(:first, :order=>'RAND()')
    def new
      @story = Story.new(params[:story])
      if request.post?
        @story.save
        redirect_to :action => 'index'
  end
end

You've got a method nested within a method. "index" and "new" should be individual methods. Here's the fix:

class StoryController < ApplicationController
  
  def index
    @story = Story.find(:first, :order=>'RAND()')
  end
  
  def new
    @story = Story.new(params[:story])
    if request.post?
      @story.save
       redirect_to :action => 'index'
    end
  end

end

[pdenlinger]

OK, thank you.

I'm working my way through the Pickaxe book to get a better handle on Ruby.