Couldn't find User with 'id'=sign_out

问题

I'm using devise in rails and I'm unable to logout of my user right now.

When I use the users/log_out page, it gives the following error:

ActiveRecord::RecordNotFound in UsersController#show
Couldn't find User with 'id'=sign_out

Anyway, here is my users controller:

class UsersController < ApplicationController

def new
    @user = User.new
end

def create
    @user = User.new(user_params)
    if @user.save
    redirect_to users_path
    else
    render 'new'
    end
end

  def index
    @users=User.all
  end

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

def destroy
    @user = User.find(params[:id])
    @user.destroy
    redirect_to users_path
end

def update
 @user = User.find(params[:id])
 if @user.update_attributes(user_params)
  redirect_to user_path(@user.id)
 else
  render 'edit'
 end
end

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

private

    def user_params
        params.require(:user).permit(:name, :email, :password)
    end

end

My route.rb :

    Rails.application.routes.draw do
  devise_for :users do
  get "/users/sign_out" => "devise/sessions#destroy", :as => :destroy_user_session
  end

  resources :posts
  resources :users
end

My migration file for devise:

class AddDeviseToUsers < ActiveRecord::Migration
  def self.up
    change_table(:users) do |t|
      ## Database authenticatable
      t.string :email,              null: false, default: ""
      t.string :encrypted_password, null: false, default: ""

      ## Recoverable
      t.string   :reset_password_token
      t.datetime :reset_password_sent_at

      ## Rememberable
      t.datetime :remember_created_at

      ## Trackable
      t.integer  :sign_in_count, default: 0, null: false
      t.datetime :current_sign_in_at
      t.datetime :last_sign_in_at
      t.string   :current_sign_in_ip
      t.string   :last_sign_in_ip

      ## Confirmable
      # t.string   :confirmation_token
      # t.datetime :confirmed_at
      # t.datetime :confirmation_sent_at
      # t.string   :unconfirmed_email # Only if using reconfirmable

      ## Lockable
      # t.integer  :failed_attempts, default: 0, null: false # Only if lock strategy is :failed_attempts
      # t.string   :unlock_token # Only if unlock strategy is :email or :both
      # t.datetime :locked_at


      # Uncomment below if timestamps were not included in your original model.
      # t.timestamps
    end

    add_index :users, :email,                unique: true
    add_index :users, :reset_password_token, unique: true
    # add_index :users, :confirmation_token,   unique: true
    # add_index :users, :unlock_token,         unique: true
  end

  def self.down
    # By default, we don't want to make any assumption about how to roll back a migration when your
    # model already existed. Please edit below which fields you would like to remove in this migration.
    raise ActiveRecord::IrreversibleMigration
  end
end

The application trace is as such:

app/controllers/users_controller.rb:40:in `show'

回答1:

Verify that you have the following in application.js

//= require jquery
//= require jquery_ujs

The jquery_ujs stands for Unobtrusive JavaScript and this ensures many things (like delete method, e.g.) work as expected.

See Rafik's answer here: Couldn't find User with id=sign_out

回答2:

The devise users route is matching before the users/sign_out. This is why it's trying to find a User with id sign_out.

Try changing the logout route, just

get "/logout" => "devise/sessions#destroy", :as => :destroy_user_session
end

should work.

I think POST is more appropriate for a logout action anyway:

post "/logout" => "devise/sessions#destroy", :as => :destroy_user_session
end

回答3:

I think you should define your route using like this:

devise_for :users do
  delete "/logout" => "devise/sessions#destroy", :as => :destroy_user_session
end

来源：https://stackoverflow.com/questions/30054399/couldnt-find-user-with-id-sign-out

标签

ruby-on-rails

devise