如何使重定向包括名称而不是ID？

Question

I'm using Rails 5.1. In my controller, I would like to redirect to my "show" method like so

redirect_to(@organization)

but I would like the URL to appear as

/organization/organization_name

instead of

/organization/primary_key_id

How do I set this up? I already have a field "name" in my Organization model.

As requested, this is the index method of my PagesController ...

class PagesController < ApplicationController

  # Should be the home page
  def index
    worker_id = params[:worker_id]
    worker = Worker.find_by_id(worker_id)
    if worker && worker.organization
      redirect_to(worker.organization)
    else
      render :file => "#{Rails.root}/public/404", layout: false, status: 404
    end
  end

end

My config/routes.rb file

  resources :organizations, :only => [:show] do
    post :update_work
    get :get_work
    get :mine
    get :poll
    post :submit
    get :home
    get :terms_of_use
  end

Here's the app/model/stratum_worker.rb file

class StratumWorker < ApplicationRecord

  has_one :organization_worker
  has_one :organization, :through => :organization_worker

Answer 1

OK, if you are not interested to use any gem then you can without gem like

class Model < ApplicationRecord
   def to_param  # overridden
      organization_name
    end
end

in this case, you need to make sure the organization_name name is unique, for uniqueness the organization_name you can use validation like this

validates_uniqueness_of :organization_name

then the model will look like this

class Model < ApplicationRecord
    validates_uniqueness_of :organization_name

    def to_param  # overridden
      organization_name
    end
end

and now to the controller using find_by_organization_name(params[:organization_name]) instead of find(params[:id]) .

Second Option

You can not change anything to your controller if used like this in just model

class Model < ApplicationRecord
    def to_param  # overridden
      organization_name
      "#{id} #{organization_name}".parameterize
    end
end

then the URL looks like this /10-microsoft .

See this to_param method. The complete reference of with gem or without gem Rails Friendly URLs

RailsCasts.com created an episode for Pretty URLs with FriendlyId, can you check it out for getting the idea.

From Comment

I don't think what's going on but sure something wrong with the relationship, can you check like this

redirect_to(worker.organizations.first)
#=> OR
redirect_to organization_path(worker.organizations.first.id)

Update

I think worker.organization are missing somehow, would you try like this?

if worker && worker.organizations.present?
   redirect_to(worker.organizations.first)
....

the present method making sure worker.organizations not blank.

I don't know about the relationship, you can try like this and let me know what's happening if it's not working then I strongly recommend to post the models with relationship concept.

Update 2 after question update

At first, you don't need the through relationship because it uses too Many To Many relationships. Your relationship is One To One then your model will look like this

class StratumWorker < ApplicationRecord

    has_one :organization_worker
....

has_one :organization, :through => :organization_worker

organization_worker.rb file like this

class OrganizationWorker < ApplicationRecord

    belongs_to :stratum_worker 

    #=> Add code what you need like for URL which was the actual motive in this post
....

Then the action looks like this

def index
    worker_id = params[:worker_id]
    worker = StratumWorker.find_by_id(worker_id)
    if worker && worker.organization_worker.present?
      #redirect_to(worker.organization_worker)
      redirect_to organization_path(worker.organization_worker)
    else
      render :file => "#{Rails.root}/public/404", layout: false, status: 404
    end
end

and the show action

OrganizationWorker.find(params:id)

I think the problem will solve now. If still, you getting errors then please read the One To One relationship again & again until clearing the relationship concept.

Hope it will help.

Answer 2

Method that is called under the hood for id generation is to_param

so in your case to get your desired result you should add this to your Organization class:

class Organization < ApplicationRecord
  ...

  def to_param
    name
  end

  ...
end

!!!WARNING!!! - since Rails is also using the parameter on the other side (eg in show method Organization.find(params[:id]) uses the URL id), now it will be params[:id] == "some_organization_name" so change your instance lookups accordingly - in show action for example use Organization.find_by!(name: params[:id]) and so on

As for your routing error - make sure that worker.organization is not nil .

Answer 3

There is a gem friendly_id that does exactly what you are asking for: https://github.com/norman/friendly_id

You add,

gem 'friendly_id'

Then bundle install and run rails generate friendly_id and rails db:migrate

to your Gemfile and,

class Organization < ApplicationRecord
  extend FriendlyId
  friendly_id :name, use: :slugged
end

to your model then,

class OrganizationController < ApplicationController
  def show
    @user = Organization.friendly.find(params[:id])
  end
end

to your controller.

This prevents the issues you can run into in Kkulikovskis answer where you have to make sure that you are looking things up correctly.

Answer 4

I wrote a post here detailing exactly this a while ago. Most of my answer will be from there. The relevant Rails documentation for this is here .

Quick definitions:

• Slug: part of the URL to identify the record, in your case organization_name
• Primary key: a unique identifier for database records. This usually is and should be id .

Summary

If you type organization_path(@organization) , it'll automatically use the id attribute in the URL. To adjust to using organization_name , you'll need to make 2 changes:

1. Override the route params in your routes.rb file.
2. Override the to_param method in the model

1. Override The Route Params

At the moment, if you run rails routes your routes look like so:

organizations       GET    /organizations(.:format)                     organizations#index
                    POST   /organizations(.:format)                     organizations#create
new_organization    GET    /organizations/new(.:format)                 organizations#new
edit_organization   GET    /organizations/:id/edit(.:format)            organizations#edit
organization        GET    /organizations/:id(.:format)                 organizations#show
                    PATCH  /organizations/:id(.:format)                 organizations#update
                    PUT    /organizations/:id(.:format)                 organizations#update
                    DELETE /organizations/:id(.:format)                 organizations#destroy

The edit_organization and organization paths use id as a parameter to lookup your organization.

Use this to override the route params

Rails.application.routes.draw do
  resources :organizations, param: :organization_name
end

Now rails routes will show that your routes look like so:

organizations       GET    /organizations(.:format)                                    organizations#index
                    POST   /organizations(.:format)                                    organizations#create
new_organization    GET    /organizations/new(.:format)                                organizations#new
edit_organization   GET    /organizations/:organization_name/edit(.:format)            organizations#edit
organization        GET    /organizations/:organization_name(.:format)                 organizations#show
                    PATCH  /organizations/:organization_name(.:format)                 organizations#update
                    PUT    /organizations/:organization_name(.:format)                 organizations#update
                    DELETE /organizations/:organization_name(.:format)                 organizations#destroy

2. Override The Model Params

By default organization.to_param will return the id of the organization. This needs to be overridden, do this by modifying your Model:

class Organization < ApplicationRecord
  def to_param
    organization_name
  end
end

Conclusion & Warning

You can now continue using your redirects and forms as usual, but instead of the route using the id , it'll now use the organization name.

Also, good luck with your mining pool! Lemme know which coin you're mining and I might join!

Also, I didn't cover this because it isn't a part of your original question, but, you should ensure that the organization_name is unique! Not only should you add a uniqueness constraint validates :organization_name, uniqueness: true in the mode, you should also enforce it at the database level in your migration.

Addendum 1: Customizing for routs

When your routes are defined as so:

  resources :organizations, :only => [:show] do
    post  'update_work'
    get   'get_work'
    get   'mine'
    get   'poll'
    post  'submit'
    get   'home'
    get   'terms_of_use'
  end

Your routes will be as so:

 organization_update_work POST   /organizations/:organization_id/update_work(.:format)                                    organizations#update_work
    organization_get_work GET    /organizations/:organization_id/get_work(.:format)                                       organizations#get_work
        organization_mine GET    /organizations/:organization_id/mine(.:format)                                           organizations#mine
        organization_poll GET    /organizations/:organization_id/poll(.:format)                                           organizations#poll
      organization_submit POST   /organizations/:organization_id/submit(.:format)                                         organizations#submit
        organization_home GET    /organizations/:organization_id/home(.:format)                                           organizations#home
organization_terms_of_use GET    /organizations/:organization_id/terms_of_use(.:format)                                   organizations#terms_of_use
             organization GET    /organizations/:id(.:format)                                                             organizations#show

Changing the param like so:

  resources :organizations, :only => [:show], param: :organization_name do
    post  'update_work'
    get   'get_work'
    get   'mine'
    get   'poll'
    post  'submit'
    get   'home'
    get   'terms_of_use'
  end

Will change your routes to

 organization_update_work POST   /organizations/:organization_organization_name/update_work(.:format)                     organizations#update_work
    organization_get_work GET    /organizations/:organization_organization_name/get_work(.:format)                        organizations#get_work
        organization_mine GET    /organizations/:organization_organization_name/mine(.:format)                            organizations#mine
        organization_poll GET    /organizations/:organization_organization_name/poll(.:format)                            organizations#poll
      organization_submit POST   /organizations/:organization_organization_name/submit(.:format)                          organizations#submit
        organization_home GET    /organizations/:organization_organization_name/home(.:format)                            organizations#home
organization_terms_of_use GET    /organizations/:organization_organization_name/terms_of_use(.:format)                    organizations#terms_of_use
             organization GET    /organizations/:organization_name(.:format)                                              organizations#show

Which should work totally fine with your redirect.