returning an array of objects in rails

Question

I need to pass an array of mailobjects from my rails mailer class to the corresponding controller which i thought should work if i just do

class foo < Actionmailer::Base

    def bar(...)
        mails_array = Array.new
        return mails_array
    end

but as the controller gets mails_array via

@mails = Array.new
@mails.concat(foo.bar(...))

i get a:

TypeError in mailsController#index
can't convert Mail::Message into Array

did i miss something?? I would expect to have the mails_array in mails and can't understand why it is not.

Leo · Accepted Answer · 2010-12-10 21:57:16Z

2

You are calling foo.bar, but bar is defined as instance method, not class method. Try

class foo < Actionmailer::Base      

    def self.bar(...)
        mails_array = Array.new
        return mails_array
    end

instead.

answered Dec 10, 2010 at 21:57

Leo

38.3k8 gold badges78 silver badges102 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Jeff Paquette · Accepted Answer · 2010-12-10 21:09:10Z

0

Array#concat expects the argument to be an array but you're passing a Mail::Message. Try the following:

@mails << foo.bar(...)

or

@mails.push(foo.bar(...))

answered Dec 10, 2010 at 21:09

Jeff Paquette

7,1472 gold badges33 silver badges40 bronze badges

unfortunately that doesn't help.. the array will be filled but just with the first element out of mails_array although it should be the whole array