Rails: Devise: post-confirmation error

Question

I am trying to send an email to the event administrator, after the user confirms.

See: Rails: Devise: Sending an email after user comfirms registration

I wrote an EventMailer class as follows:

class EventMailer < ActionMailer::Base
  @user = User

  @host_name = "localhost:3000" if Rails.env.development?
  @host_name = "test.com" if Rails.env.test?
  @host_name = "production.com" if Rails.env.production?

 def self.send_email_to_admin
   @@smtp_settings = { #override default smtp settings since I'm using a difference sender
     :address => "smtp.gmail.com",
     :port => 587,
     :domain => @host_name,

     :authentication => "plain",
     :enable_starttls_auto_=> true,
     :user_name => ENV["ORG_FROM"],
     :password => ENV["ORG_FROM_PWD"],
     :to => ADMIN_RECIPIENTS
     #ADMIN_RECIPIENTS is defined elsewhere
   }
   mail(:subject => "Invite request received from " + @user.first.to_s + " " + @user.last.to_s)
 end
end

When users register, they receive a registration link. When they click on that link, they get the following error message:

undefined method `mail' for EventMailer:Class

app/mailers/event_mailer.rb:20:in `send_email_to_admin'
app/controllers/confirmations_controller.rb:11:in `show'

Any ideas?

EDIT:

The solution is to move both the :to and :from inside the mail nethod

score 0 · Accepted Answer · answered Jun 04 '12 at 21:38

0

Try moving :to => DENT_ADMIN_RECIPIENTS into the mail method:

mail(:to => DENT_ADMIN_RECIPIENTS, :subject => "Invite request received from " + @user.first.to_s + " " + @user.last.to_s)

answered Jun 04 '12 at 21:38

flynfish

5,857
3
24
33

I still get the same error. The docs say that :to can be in the smtp_settings hassh – EastsideDev Jun 04 '12 at 21:47
Where are those docs? I didn't see anything [here](http://api.rubyonrails.org/classes/ActionMailer/Base.html#label-Configuration+options) about allowing :to in the smtp_settings hash. – flynfish Jun 04 '12 at 21:58
Sorry. not THE docs, but a QA session on StackOverflow. It turns out that the solution is to move both To and From inside the mail method – EastsideDev Jun 04 '12 at 22:53
Ah I should have noticed that from wasn't defined too. You can also add `default :from => 'no-reply@example.com'` to the top of the class, so you don't have to pass it into every `mail` call you make. – flynfish Jun 04 '12 at 23:27
Isn't it possible to put :to, :subject, etc. in default? – EastsideDev Jun 05 '12 at 04:58
Yes you can add those to default, but the main reason why you want to add to the default is if you have multiple methods that make a call to `mail()` and those values are all the same. Generally if you are calling `mail()` in different methods, your subject would change for each one, so you wouldn't include it in default. – flynfish Jun 05 '12 at 18:06

Rails: Devise: post-confirmation error

EDIT:

1 Answers1