Rails Routing Error: Uninitialized Constant with Nested Resources

Question

As rake routes gives the exact route I visited, the page shows
Routing Error uninitialized constant TasksController Did you mean? AppsController

In routes.rb I wrote

  resources :users do 
    resources :tasks do
      collection do
        post 'tasks-info', to: 'users/tasks#tasks_info', format: 'json'
      end
    end
  end

And the tasks_controller.rb is under app/controllers/users folder

def show
end

show.html.slim is under app/views/users/tasks foler
The url I visited is http://localhost:3000/users/1/tasks/1 with rake routes including
user_task_path GET /users/:user_id/tasks/:id(.:format) tasks#show

I really don't know where the error comes from, any help is truly appreciated!

Updated: Problem solved and please refer to the comments below if you have the same issue here(I have summarized based on my own understanding, not sure whether it's exactly correct though)! Or see the answers below as engineersmnky's and Scott's answers should both work.

Is your TasksController in the appropriate namespace? Its not just the file location (afaik), but also the content of the file needs to be in the Users namespace. Please try that and get back. — Vijay Raghavan Aravamudhan
– Vijay Raghavan Aravamudhan, Commented Jul 29, 2021 at 3:44
You should not have tasks_controller under app/controllers/users because "users" is not a namespace. More info here, guides.rubyonrails.org/… — razvans
– razvans, Commented Jul 29, 2021 at 6:18
@VijayRaghavanAravamudhan Yes, I did use class Users::TasksController < ApplicationController — cweih
– cweih, Commented Jul 29, 2021 at 8:19
@razvans However, my self-defined action of tasks wouldn't work unless I put tasks_controller.rb under a users folder. What should I do to make both self-defined action and auto-generated action work? — cweih
– cweih, Commented Jul 29, 2021 at 8:22
@razvans oh, I separated the routes of self-defined actions from the nested resouces, and it works. Thank you for your help! — cweih
– cweih, Commented Jul 29, 2021 at 9:06

ScottM · Accepted Answer · 2021-07-29 06:34:09Z

2

The key is what rails routes is telling you:

user_task_path  GET /users/:user_id/tasks/:id(.:format)  tasks#show

That last section tells you that it’s looking for the show method in the tasks controller – that is, TasksController at the top level of the controller hierarchy.

So rather than placing TasksController at app/controllers/users/tasks_controller.rb, you need to place it at app/controllers/tasks_controller.rb.

The “nesting” of the controller is handled by the params object containing both :user_id and :id when TasksController#show is called.

answered Jul 29, 2021 at 6:34

ScottM

10.9k1 gold badge39 silver badges59 bronze badges

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

3 Comments

cweih Over a year ago

I got it, thanks! But how should I write routers for my self-defined action in routes.rb with nested resources?

cweih Over a year ago

I tried to write outside the nested resouces do, and it works fine, i wonder whether it is the correct way to do it?

ScottM Over a year ago

Your nested custom method should define itself a path, but you're overriding it with the to: parameter. I'd remove that, run rails routes to see what it's creating for you and go from there.

engineersmnky · Accepted Answer · 2021-07-29 13:06:25Z

2

Your issue is that resources :tasks infers TasksController but you want Users::TasksController to handle these actions.

Rails provides a method for handling this type of routing ActionDispatch::Routing::Mapper::Scoping#scope

To implement this all you have to do is add another level to your structure using the scope method like so:

  resources :users do 
    scope module: 'users' do 
      resources :tasks do
        collection do
          post 'tasks-info', to: 'users/tasks#tasks_info', format: 'json'
        end
      end
    end
  end

Now all the nested routes to the user tasks will go to Users::TasksController because we specified that Task is in the Users module. Your route in question will now look like this (in rails routes output):

user_task_path  GET /users/:user_id/tasks/:id(.:format)  users/tasks#show

answered Jul 29, 2021 at 13:06

engineersmnky

30.5k2 gold badges42 silver badges64 bronze badges

2 Comments

cweih Over a year ago

Thanks sooo much and this should also be an answer! Feel so pity as only one answer can be accepted.

engineersmnky Over a year ago

@hcww No pity necessary. If this answer helped in anyway you can upvote it or maybe it will help someone else that stumbles across the question in the future (that's what SO is all about)

Yarrabhumi · Accepted Answer · 2021-07-30 05:50:29Z

0

I assume users is folder where you are storing your tasks controller in that scenario you are supposed to use different key rather than using resources. There can be only one resource at a time we can not not resources inside another resource. Technically it indicates we are having controller inside a controller which is not possible.

namespace :users do
  resources :tasks do
    collection do
      post 'tasks-info', to: 'users/tasks#tasks_info', format: 'json'
    end
  end
end

answered Jul 30, 2021 at 5:50

Yarrabhumi

366 bronze badges

1 Comment

cweih Over a year ago

Thanks for answering, but it is ok to use nested resources, please refer to the answers above if you want to know how to solve the problem.:)

Collectives™ on Stack Overflow

Rails Routing Error: Uninitialized Constant with Nested Resources

I really don't know where the error comes from, any help is truly appreciated!

3 Answers 3

3 Comments

2 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

I really don't know where the error comes from, any help is truly appreciated!

3 Answers 3

3 Comments

2 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related