ConversationsController中的ActiveRecord :: RecordNotFound #index - 找不到具有＆＃39; id＆＃39; =的用户

Question

我在Ruby on Rails应用程序中尝试访问/ conversations URL时收到以下错误：

ActiveRecord::RecordNotFound in ConversationsController#index

Couldn't find User with 'id'=

Extracted source (around line #154):
152    record = s.execute([id], self, connection).first
153    unless record
154      raise RecordNotFound, "Couldn't find #{name} with '#{primary_key}'=#{id}"
155    end
156    record
157  rescue RangeError

Rails.root: /home/ubuntu/workspace

app/controllers/conversations_controller.rb:25:in `correct_user'

此错误表示什么？我已经在routes.rb文件中定义了资源：

resources :conversations, only: [:index, :show, :destroy]

用于访问/ conversations网址的_header.html.erb中的链接是：

<li>
  <%= link_to conversations_path do %>
    <span class="glyphicon glyphicon-envelope"></span>&nbsp;Messages
  <% end %>
</li>

附加信息：

- 对话控制器：

class ConversationsController < ApplicationController
  before_action :logged_in_user, only: [:index, :show, :destroy]
  before_action :correct_user, only: [:index, :show, :destroy]
  before_action :get_mailbox
  before_action :get_conversation, except: [:index]

  def show
  end

  def index
    @conversations = @mailbox.inbox.paginate(page: params[:page], per_page: 10)
  end

  private

  def get_mailbox
    @mailbox ||= current_user.mailbox
  end

  def get_conversation
    @conversation ||= @mailbox.conversations.find(params[:id])
  end

  def correct_user
    @user = User.find(params[:id])
    redirect_to(root_url) unless current_user?(@user)
  end
end

- 用户模型：

class User < ActiveRecord::Base
  acts_as_messageable
  has_many :listings, dependent: :destroy
  attr_accessor :remember_token, :activation_token, :reset_token
  before_save   :downcase_email
  before_create :create_activation_digest
  validates :first_name,  presence: true, length: { maximum: 25 }
  validates :last_name, presence: true, length: { maximum: 50 }
  validates :username, presence: true, uniqueness: true, length: {maximum: 50 }
  VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-]+(\.[a-z\d\-]+)*\.[a-z]+\z/i
  validates :email, presence: true, length: { maximum: 255 },
                                    format: { with: VALID_EMAIL_REGEX },
                                    uniqueness: { case_sensitive: false }
  has_secure_password
  validates :password, length: { minimum: 6 }, allow_blank: true


  class << self
    # Returns the hash digest of the given string.
    def digest(string)
      cost = ActiveModel::SecurePassword.min_cost ? BCrypt::Engine::MIN_COST :
                                                                                                BCrypt::Engine.cost
      BCrypt::Password.create(string, cost: cost)
    end

    # Returns a random token.
    def new_token
      SecureRandom.urlsafe_base64
    end
  end

# Remembers a user in the database for use in persistent sessions.
def remember
  self.remember_token = User.new_token
  update_attribute(:remember_digest, User.digest(remember_token))
end

# Returns true if the given token matches the digest.
def authenticated?(attribute, token)
  digest = send("#{attribute}_digest")
  return false if digest.nil?
  BCrypt::Password.new(digest).is_password?(token)
end

# Forgets a user.
def forget
  update_attribute(:remember_digest, nil)
end

# Activates an account.
def activate
  update_attribute(:activated,  true)
  update_attribute(:activated_at, Time.zone.now)
end

# Sends activation email.
def send_activation_email
  UserMailer.account_activation(self).deliver_now
end

# Sets the password reset attributes.
def create_reset_digest
  self.reset_token = User.new_token
  update_attribute(:reset_digest, User.digest(reset_token))
  update_attribute(:reset_sent_at, Time.zone.now)
end

# Sends password reset email.
def send_password_reset_email
  UserMailer.password_reset(self).deliver_now
end

# Returns true if a password reset has expired.
def password_reset_expired?
  reset_sent_at < 2.hours.ago
end


private

  # Converts email to all lower-case.
  def downcase_email
    self.email = email.downcase
  end

  # Creates and assigns the activation token and digest.
  def create_activation_digest
    self.activation_token  = User.new_token
    self.activation_digest = User.digest(activation_token)
  end
end

Answer 1

如果您的网址为/conversations并且您只显示current_user.mailbox，那么您不需要导致问题的before_action :correct_user，因此此处最快的解决方案是删除它。如果可以看到其他人的对话（@user.mailbox），并且网址包含：user_id - /users/1/conversations，/users/2/conversations，那么它只会有用（并且正常工作）。如果网址中没有用户ID，则@user = User.find(params[:user_id])正在搜索没有ID的用户：）