Rails uninitialized constant error in controller

Question

Here is my controller directory structure

controllers/
.........../dashboard
...................../admin
.........................../admin_controller.rb
.........................../item_controller.rb
........../items_controller.rb

I expect that manager can add new item, and users can see all items.

I typed rails g scaffold dashboard/admin/items to quickly generate controller with namespace.

And I modified @dashboard_admin_items = Dashboard::Admin::Item.all

to @dashboard_admin_items = Item.all

controllers/dashboard/admin/items_controller.rb

class Dashboard::Admin::ItemsController < Dashboard::Admin::AdminController
  before_action :set_dashboard_admin_item, only: [:show, :edit, :update, :destroy]

  # GET /dashboard/admin/items
  def index
    @dashboard_admin_items = Item.all

  end
...

But it threw this error

Also, I've put this project on github.

Update

view/dashboard/admin/items/index.html.erb

<tbody>
    <% @Items.each do |i| %>
      <tr>
        <th><%= i.id %></th>
        <th><%= i.name %></th>
        <th><%= i.price %></th>
        <!-- <td><%#= link_to 'Show', dashboard_admin_item %></td> -->
        <td>
          <%= link_to 'Edit', edit_dashboard_admin_item_path(dashboard_admin_item) %>
          <%= link_to 'Destroy', dashboard_admin_item, method: :delete, data: { confirm: 'Are you sure?' } %>
        </td>
      </tr>
    <% end %>
  </tbody>

controllers/dashboard/admin/items_controller.rb

def index
    @Items = Item.all
end

cuz I have already a Model Item? I thought manager has to do CRUD with the same model which users can see. — rj487
– rj487, Commented Oct 1, 2015 at 4:20
hey I made it working by making the following change. Look at my answer. try that and let me know. — K M Rakibul Islam
– K M Rakibul Islam, Commented Oct 1, 2015 at 4:36

K M Rakibul Islam · Accepted Answer · 2015-10-01 13:08:01Z

3

The issue is you have two item.rb file, one under app/model/item.rb and the other under app/models/dashboard/admin/item.rb. So, there is a conflict.

To use app/model/item.rb, use it this way: ::Item.all.

::Item means you are referring to the Item class from the top level namespace i.e. app/model/item.rb.

I ran your code locally. It works fine if you change:

  def index
    @dashboard_admin_items = Item.all
  end

to:

  def index
    @Items = ::Item.all
  end

And, you are already using @Items in your app/views/dashboard/admin/items/index.html.erb

edited Oct 1, 2015 at 13:08

answered Oct 1, 2015 at 4:32

K M Rakibul Islam

34.4k13 gold badges95 silver badges113 bronze badges

Sign up to request clarification or add additional context in comments.

6 Comments

rj487 Over a year ago

It's weird, I have update it to what you type. But I still have the same error, I update some part of my code on question.

K M Rakibul Islam Over a year ago

see my updated answer. Do it this way: @Items = ::Item.all. This will work.

K M Rakibul Islam Over a year ago

::Item means you are referring to the Item class from the top level namespace i.e. app/model/item.rb.

K M Rakibul Islam Over a year ago

@CodaChang Let me know if you have any other question. If my answer solved your problem, consider accept/upvoting the answer. Thanks.

rj487 Over a year ago

Thanks, you have been so patient and great help :)

|

Collectives™ on Stack Overflow

Rails uninitialized constant error in controller

Update

1 Answer 1

6 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

Update

1 Answer 1

6 Comments

Your Answer

Sign up or log in

Post as a guest

Related