1

私は Gumtree.com のような Web サイトに取り組んでおり、そこでユーザーは投稿を作成し、さまざまなニーズを強調しています (たとえば、X 日目に写真家が必要です)。

メッセージ機能に問題があります。Simple Private Messaging プラグインを使用しています。

問題は、ユーザーがメッセージを送信したり、投稿に返信したりするために入力するフォームにあります。/messages/new にアクセスしようとすると、次のエラーが表示されます。

ActiveRecord::RecordNotFound in MessagesController#new
Couldn't find User without an ID

以下に私のモデルを添付します - アドバイスに感謝します!

ありがとう、

ファイサル

メッセージコントローラー

class MessagesController < ApplicationController

before_filter :set_user

def index
if params[:mailbox] == "sent"
  @messages = @user.sent_messages
else
  @messages = @user.received_messages
end
end

def show
@message = Message.read_message(params[:id], current_user)
end

def new
@message = Message.new

if params[:reply_to]
  @reply_to = @user.received_messages.find(params[:reply_to])
  unless @reply_to.nil?
    @message.to = @reply_to.sender.login
    @message.subject = "Re: #{@reply_to.subject}"
    @message.body = "\n\n*Original message*\n\n #{@reply_to.body}"
  end
end
end

def create
@message = Message.new(params[:message])
@message.sender = @user
@message.recipient = User.find_by_login(params[:message][:to])

if @message.save
  flash[:notice] = "Message sent"
  redirect_to user_messages_path(@user)
else
  render :action => :new
end
end

def delete_selected
if request.post?
  if params[:delete]
    params[:delete].each { |id|
      @message = Message.find(:first, :conditions => ["messages.id = ? AND (sender_id = ? OR recipient_id = ?)", id, @user, @user])
      @message.mark_deleted(@user) unless @message.nil?
    }
    flash[:notice] = "Messages deleted"
  end
  redirect_to :back
end
end

private
def set_user
  @user = User.find(params[:user_id])
end
end

メッセージモデル

class Message < ActiveRecord::Base

is_private_message

attr_accessor :to

end

ユーザーモデル

class User < ActiveRecord::Base

has_many :posts  
has_one :profile
has_private_messages

attr_accessible :email

validates_presence_of :email
validates_uniqueness_of :email, :message =>"Hmm, that email's already taken"
validates_format_of :email, :with => /^([^\s]+)((?:[-a-z0-9]\.)[a-z]{2,})$/i, :message => "Hi! Please use a valid email"


end

メッセージ>NEW VIEW

<% form_for @message, :url => user_messages_path(@user) do |f| %>
<p>
To:<br />
    <%= f.text_field :to %>
    <%= error_message_on @message, :to %>
</p>
<p>
Subject:<br />
<%= f.text_field :subject %>
<%= error_message_on @message, :subject %>
</p>
<p>
  Message<br />
  <%= f.text_area :body %>
        <%= error_message_on @message, :body %>
</p>
<p>
  <%= submit_tag "Send" %>
</p>
<% end %>
4

1 に答える 1

2

新しいメッセージのページが で受信:user_idされparamsますか? たとえそれが本当だとしても、これがあなたの意図であったとは思えません:user_id。

したがって、一番下の行：

@user = User.find(params[:user_id]) # I think it's the cause of your error -- no :user_id is passed

でなければならないようです：

@user = User.find(session[:user_id])

セッションについて聞いたことがあると思います。

于 2012-04-18T20:21:26.370 に答える