Use new ActionMail API to send emails

This commit is contained in:
Tom Hughes 2010-09-02 23:40:49 +01:00
parent b25a468e3c
commit 2e2c95133b
4 changed files with 10 additions and 10 deletions

View file

@ -60,7 +60,7 @@ class DiaryEntryController < ApplicationController
@diary_comment.user = @user @diary_comment.user = @user
if @diary_comment.save if @diary_comment.save
if @diary_comment.user != @entry.user if @diary_comment.user != @entry.user
Notifier::deliver_diary_comment_notification(@diary_comment) Notifier.diary_comment_notification(@diary_comment).deliver
end end
redirect_to :controller => 'diary_entry', :action => 'view', :display_name => @entry.user.display_name, :id => @entry.id redirect_to :controller => 'diary_entry', :action => 'view', :display_name => @entry.user.display_name, :id => @entry.id

View file

@ -25,7 +25,7 @@ class MessageController < ApplicationController
if @message.save if @message.save
flash[:notice] = t 'message.new.message_sent' flash[:notice] = t 'message.new.message_sent'
Notifier::deliver_message_notification(@message) Notifier.message_notification(@message).deliver
redirect_to :controller => 'message', :action => 'inbox', :display_name => @user.display_name redirect_to :controller => 'message', :action => 'inbox', :display_name => @user.display_name
end end
end end

View file

@ -130,7 +130,7 @@ class UserController < ApplicationController
if @user.save if @user.save
flash[:piwik_goal] = PIWIK_SIGNUP_GOAL if defined?(PIWIK_SIGNUP_GOAL) flash[:piwik_goal] = PIWIK_SIGNUP_GOAL if defined?(PIWIK_SIGNUP_GOAL)
flash[:notice] = t 'user.new.flash create success message', :email => @user.email flash[:notice] = t 'user.new.flash create success message', :email => @user.email
Notifier.deliver_signup_confirm(@user, @user.tokens.create(:referer => session.delete(:referer))) Notifier.signup_confirm(@user, @user.tokens.create(:referer => session.delete(:referer))).deliver
session[:token] = @user.tokens.create.token session[:token] = @user.tokens.create.token
redirect_to :action => 'login', :referer => params[:referer] redirect_to :action => 'login', :referer => params[:referer]
else else
@ -214,7 +214,7 @@ class UserController < ApplicationController
if user if user
token = user.tokens.create token = user.tokens.create
Notifier.deliver_lost_password(user, token) Notifier.lost_password(user, token).deliver
flash[:notice] = t 'user.lost_password.notice email on way' flash[:notice] = t 'user.lost_password.notice email on way'
redirect_to :action => 'login' redirect_to :action => 'login'
else else
@ -356,7 +356,7 @@ class UserController < ApplicationController
def confirm_resend def confirm_resend
if user = User.find_by_display_name(params[:display_name]) if user = User.find_by_display_name(params[:display_name])
Notifier.deliver_signup_confirm(user, user.tokens.create) Notifier.signup_confirm(user, user.tokens.create).deliver
flash[:notice] = t 'user.confirm_resend.success', :email => user.email flash[:notice] = t 'user.confirm_resend.success', :email => user.email
else else
flash[:notice] = t 'user.confirm_resend.failure', :name => params[:display_name] flash[:notice] = t 'user.confirm_resend.failure', :name => params[:display_name]
@ -419,7 +419,7 @@ class UserController < ApplicationController
unless @user.is_friends_with?(new_friend) unless @user.is_friends_with?(new_friend)
if friend.save if friend.save
flash[:notice] = t 'user.make_friend.success', :name => name flash[:notice] = t 'user.make_friend.success', :name => name
Notifier.deliver_friend_notification(friend) Notifier.friend_notification(friend).deliver
else else
friend.add_error(t('user.make_friend.failed', :name => name)) friend.add_error(t('user.make_friend.failed', :name => name))
end end
@ -653,7 +653,7 @@ private
flash.now[:notice] = t 'user.account.flash update success confirm needed' flash.now[:notice] = t 'user.account.flash update success confirm needed'
begin begin
Notifier.deliver_email_confirm(user, user.tokens.create) Notifier.email_confirm(user, user.tokens.create).deliver
rescue rescue
# Ignore errors sending email # Ignore errors sending email
end end

View file

@ -21,15 +21,15 @@ while(true) do
gpx = trace.import gpx = trace.import
if gpx.actual_points > 0 if gpx.actual_points > 0
Notifier::deliver_gpx_success(trace, gpx.actual_points) Notifier.gpx_success(trace, gpx.actual_points).deliver
else else
Notifier::deliver_gpx_failure(trace, '0 points parsed ok. Do they all have lat,lng,alt,timestamp?') Notifier.gpx_failure(trace, '0 points parsed ok. Do they all have lat,lng,alt,timestamp?').deliver
trace.destroy trace.destroy
end end
rescue Exception => ex rescue Exception => ex
logger.info ex.to_s logger.info ex.to_s
ex.backtrace.each {|l| logger.info l } ex.backtrace.each {|l| logger.info l }
Notifier::deliver_gpx_failure(trace, ex.to_s + "\n" + ex.backtrace.join("\n")) Notifier.gpx_failure(trace, ex.to_s + "\n" + ex.backtrace.join("\n")).deliver
trace.destroy trace.destroy
end end