开发者

Internal messaging in Rails

I'm using this tutorial to get internal messages working on my site: http://www.nov开发者_JAVA技巧awave.net/public/rails_messaging_tutorial.html

But, since my latest upgrade to Rails 3, I'm getting this error:

NoMethodError in MsgController#sendmsg
undefined method `each' for #<String:0xcc8acc0>

Application trace:

app/models/message.rb:16:in `prepare_copies'
app/controllers/msg_controller.rb:140:in `sendmsg'

The Message model:

class Message < ActiveRecord::Base
  belongs_to :author, :class_name => "User"
  has_many :message_copies
  has_many :recipients, :through => :message_copies
  before_create :prepare_copies

  attr_accessor  :to # array of people to send to
  attr_accessible :subject, :body, :to

  def prepare_copies
    return if to.blank?

    to.each do |recipient|
      recipient = User.find(recipient)
      message_copies.build(:recipient_id => recipient.id, :folder_id => recipient.inbox.id)
    end
  end
end


That tutorial seems a bit dated. It uses Rails 2.0 (and probably some equally old Ruby version).

Are you sure that to holds an Array (as indicated in your attr_accessor comment)? The error message seems to indicate that it is a String.

Did you previously have this code running under Ruby 1.8 (and, presumably, a version of Rails 2.3)?

In Ruby 1.8 you could send each to String instances and it would (by default) iterate on the lines of the string (actually it would split on $/ and iterate on the result).

In Ruby 1.9 you need to use each_line to iterate over the lines of a string.

to.each_line do |recipient|
  …
end


Seems you to is no anymore an Array but a String now. You problem semmes becomes from your controller and how you define your to accessor inside it.

0

上一篇:

下一篇:

精彩评论

暂无评论...
验证码 换一张
取 消

最新问答

问答排行榜