ruby on rails: how to change the routing address

Question

I am new to rails and I would like to know that if there is a way to change the name of the routes. I will explain the sentence in a minute. I have a controller named users_controller and below is the code that I have in my controller:

def create
  if @user.save
  redirect_to @user
end

And my routes.rb file is as below:

resources :users

get '/users/:id', to: 'users#show'

Now suppose I have a form and I fill it correctly then I will be redirected to localhost:3000/users/id(id = some value) which I do. Now I want this URL to be like localhost:3000/registered . That is when i click the submit button and if everything is correct I wanted to see the URL as localhost:3000/registered which takes me to my show action or which works just like localhost:3000/users/id(id = some value). I don't want the ID to be visible.

For this I tried get 'registered', to: 'users#show' but still I see the same URL as localhost:3000/users/id(id = some value). Please tell me how to do this.

And here is my show action

def show

@user = User.find(params[:id])

end

I tried another way by making another action into the same controller. I made index action in my users_controller and executed then I got a No method Error

The [Rails guide on routing](http://guides.rubyonrails.org/routing.html) is excellent. — gdpelican, Apr 29 '14 at 04:21
I referred that. and from there I got this idea get 'registered', to: 'users#show' — chaitanya90, Apr 29 '14 at 04:24
shoud I change my controller to something else rather using @user — chaitanya90, Apr 29 '14 at 04:24

score 5 · Answer 1 · edited May 05 '14 at 16:33

5

Just use your routing as:

match "/registered", :to => "users#show", :as => :user_registered, via: :get

to use this path you will use

redirect_to user_registered_path(id: user_id)

in your method.

edited May 05 '14 at 16:33

Uri Agassi

36,848
14
76
93

answered Apr 29 '14 at 05:14

Babar

1,202
1
12
21

but how will it know that its a GET or POST – chaitanya90 Apr 29 '14 at 05:30
sorry my bad!add this in the end: via: method method being get or post – Babar Apr 29 '14 at 05:39
let me check fot this way. :) – chaitanya90 Apr 29 '14 at 07:27
could not find a user without ID it says :( – chaitanya90 Apr 29 '14 at 07:30
are you we can change the route for a "show" action, because it was suggested to me that we cannot – chaitanya90 Apr 29 '14 at 07:32
`localhost:3000/registered` is the one showing this error – chaitanya90 Apr 29 '14 at 07:33
redirect_to user_registered_path(user.id) – Babar Apr 29 '14 at 10:06
sorry don't know where my mind is... you need to pass the user.id as a param – Babar Apr 29 '14 at 10:06
still not able to get it... :( – chaitanya90 Apr 29 '14 at 15:31
now it comes like "registered.ID". But I dont want this ID to be seen – chaitanya90 Apr 29 '14 at 15:32
but its finding the user correctly now? if yes then lets work on the .id problem now – Babar Apr 29 '14 at 17:21
Well the id is required in the url and can not be done without, however there are ways to use other identifiers such as unique username, otherwise it will be best to create a landing method that doesn't require an id. – Babar Apr 29 '14 at 17:36
how to do the second option – chaitanya90 Apr 29 '14 at 17:38
i mean landing method that doesnot require an id – chaitanya90 Apr 29 '14 at 17:39
well create an action with generic info, and show the user that page after they register – Babar Apr 29 '14 at 17:45
you mean a static page – chaitanya90 Apr 29 '14 at 17:47
because we require something from database to find so that we can show the users their name or other info – chaitanya90 Apr 29 '14 at 17:48
do you use any gem for user management? devise being first choice usual – Babar Apr 29 '14 at 17:50
how about adding constraints? – chaitanya90 Apr 29 '14 at 17:50
if your username is unique you can use that to show in the url – Babar Apr 29 '14 at 17:52
or store the id in session to retrieve the info. – Babar Apr 29 '14 at 17:53
Actually i was thinking of either using the constraints on ID or as you said storing it in a seesion – chaitanya90 Apr 29 '14 at 17:57
1

but when i do this `match 'users/:id', to: 'users#show', :constraints => {:id => /[A-Z]+/}, via: :get` it does not work – chaitanya90 Apr 29 '14 at 17:58
I do not understand what you mean by constraints on ID, so I think session is the way to go, and use [Devise](https://github.com/plataformatec/devise) for user management. It will make these things really easy. – Babar Apr 29 '14 at 18:00
I didn't know how constraints work, so thanks for the share I'll read about them, and what error are you getting? – Babar Apr 29 '14 at 18:02
What I understand from url constraints is that it forces the id to be of certain type, I do not know how you are going to use it to hide your id in the request? – Babar Apr 29 '14 at 18:06
atleast i can change it to something else – chaitanya90 Apr 29 '14 at 18:09
I don't think it works like that, let me do a little search :) – Babar Apr 29 '14 at 18:09
but nothing happened. it again shows me the same users/:id url – chaitanya90 Apr 29 '14 at 18:09
well constraint is just a check for sql injection and type checking of the parameter received... – Babar Apr 29 '14 at 18:14
what other thing I unique to the user except for his id? – Babar Apr 29 '14 at 18:15
I have thier email Id – chaitanya90 Apr 29 '14 at 18:17
Lol... it will still be open to attacks of people shuffling through emails of friends or foes, go with the session, that will be the best approach. – Babar Apr 29 '14 at 18:18
i guess that will be the best way. – chaitanya90 Apr 29 '14 at 18:20

score 2 · Accepted Answer · answered Apr 30 '14 at 12:23

At last, from the discussions I had it has been cleared that there is no way to hide your ID, but we can change the way they appear on the browser. One and suggested way is to create a hash for the ID and use it to display the item in show action or the other way is to redirect your action to some static page.

Joe Kennedy · Answer 3 · 2014-04-29T04:50:24.837

1

You need to change your redirect_to in your user controller's create and update actions.

class UsersController < ApplicationController
  def create
     # get user from params
    if @user.save
      redirect_to registered_path
    else
      render :new
    end
  end

  def update
     # get user from params
    if @user.save
      redirect_to registered_path
    else
      render :edit
    end
  end
end

EDIT: As we discussed in the comments, you don't want to use "users#show", since the registered page will look the same for every user. You can make your own registered controller to do this.

rails g controller registered index

edited Apr 29 '14 at 04:50

answered Apr 29 '14 at 04:28

Joe Kennedy

9,365
7
41
55

Then it gives me an error that the page you are looking doesnot exists while running it on heroku – chaitanya90 Apr 29 '14 at 04:34
It says could not find an User without an ID – chaitanya90 Apr 29 '14 at 04:35
I think it has something to do with my Show action – chaitanya90 Apr 29 '14 at 04:37
I see the issue. The problem is that you're mapping `registered` to `"user#show"`, which expects an id for the user. Do you want to make a boiler plate registered page for all users, or are you looking to make a registered page that's unique for each user? – Joe Kennedy Apr 29 '14 at 04:39
I want all users to see the same show page just with their names in it when they log in. You can suppose it as a Thank you page for some site. – chaitanya90 Apr 29 '14 at 04:42
If user#show expects an ID then should I try changing the the name of action from sho to something different? – chaitanya90 Apr 29 '14 at 04:45
Yes. Just make your own action. `rails g controller registered index` should get everything you need to get started on your registered/thank you page. – Joe Kennedy Apr 29 '14 at 04:47
i tried creating a new action called index in my users_controller and then i made a call and I got No method error. Ihave updated my question. please have a look at it – chaitanya90 Apr 29 '14 at 05:51
You need to make the `index` action in the **RegisteredController**, not the UsersController. Or, you need to `get "registered", to "users#index"`. – Joe Kennedy Apr 29 '14 at 05:54
I did both. lets try it step by step. I HAVE MADE INDEX IN REGISTERED_CONTROLLER – chaitanya90 Apr 29 '14 at 05:55
I get an error saying Undefined local variable register_path in UsersController – chaitanya90 Apr 29 '14 at 05:56
let us [continue this discussion in chat](http://chat.stackoverflow.com/rooms/51642/discussion-between-jken13579-and-ror) – Joe Kennedy Apr 29 '14 at 05:58

ruby on rails: how to change the routing address

3 Answers3