How do I create a new/create action for items?

Question

I am working on a rails assignment that asks that I create a list of items. I created the controller and model for item, but am still having some trouble. I keep receiving the following error:

undefined method `items_path' for

Here is some of my code:

class ItemsController < ApplicationController
  def new
    @item = Item.new
  end

  def create
    @item = Item.new(params.require(:item).permit(:name))
     if @item.save
       flash[:notice] = "Item was saved."
       redirect_to @item
     else
       flash[:error] = "There was an error saving the item. Please try again."
       render :new
     end
   end
end

Items Model:

class Item < ActiveRecord::Base
  belongs_to :user
end

New.html.erb in items

class Item < ActiveRecord::Base
  belongs_to :user
end

Routes.rb:

Rails.application.routes.draw do
  devise_for :users
  resources :users do
    resources :items, only: [:new, :create]
  end

  get 'welcome/index'
  root :to => 'welcome#index'
end

Item.html.erb

<%= form_for @item do |f| %>
   <%= f.label :name %>
   <%= f.text_field :name %>

   <%= f.submit "Save" %>
<% end %>

You are using `form_for @item` in your views, is that right? Adding the relevant code might improve the question, but the issue seems to be clear. — D-side, May 29 '15 at 15:49
http://stackoverflow.com/questions/2034700/form-for-with-nested-resources — Rajarshi Das, May 29 '15 at 16:01

score 3 · Accepted Answer · answered May 29 '15 at 16:03

resources :users do
    resources :items, only: [:new, :create]
end

This will nest items route inside user. Check rake routes and you will not get this items_path

You have to define resource :items to get items_path

So if you want to use nested routes you have to update your form and controller and if not just routes resource :items

<%= form_for [@user, @item] do |f| %>
   <%= f.label :name %>
   <%= f.text_field :name %>

   <%= f.submit "Save" %>
<% end %>

and controller

def new
    @item = Item.new
    @user = current_user
end

score 0 · Answer 2 · answered May 29 '15 at 15:59

0

You've nested your items resource under users. It should be user_items_path.

answered May 29 '15 at 15:59

nesiseka

1,268
8
22

score 0 · Answer 3 · answered May 29 '15 at 16:02

The items are nested under users, so they always must have a corresponding user. Example /users/1/item/3

If the logged user is the one creating items, your form should look like this:

<%= form_for [current_user, @item] do |f| %>
  <%= f.label :name %>
  <%= f.text_field :name %>

  <%= f.submit "Save" %>
<% end %>

If you can create items for other users, you have to create a @user instance in the new action of the controller and then change current_user for @user

`current_user` is a devise created variable with the logged user. — Puce, May 29 '15 at 16:03

How do I create a new/create action for items?

3 Answers3