X-Git-Url: https://git.openstreetmap.org/rails.git/blobdiff_plain/2f228437324d80cb8a408d27912cc716a36aa3b0..036519a4485948fe06fad43e30bbbb3e5df7bb6f:/app/models/notifier.rb diff --git a/app/models/notifier.rb b/app/models/notifier.rb index b1a94a77d..018226566 100644 --- a/app/models/notifier.rb +++ b/app/models/notifier.rb @@ -12,7 +12,7 @@ class Notifier < ActionMailer::Base :confirm_string => token.token) mail :to => user.email, - :subject => I18n.t('notifier.signup_confirm.subject') + :subject => I18n.t("notifier.signup_confirm.subject") end end @@ -24,7 +24,7 @@ class Notifier < ActionMailer::Base :confirm_string => token.token) mail :to => user.new_email, - :subject => I18n.t('notifier.email_confirm.subject') + :subject => I18n.t("notifier.email_confirm.subject") end end @@ -35,7 +35,7 @@ class Notifier < ActionMailer::Base :token => token.token) mail :to => user.email, - :subject => I18n.t('notifier.lost_password.subject') + :subject => I18n.t("notifier.lost_password.subject") end end @@ -48,7 +48,7 @@ class Notifier < ActionMailer::Base @possible_points = possible_points mail :to => trace.user.email, - :subject => I18n.t('notifier.gpx_notification.success.subject') + :subject => I18n.t("notifier.gpx_notification.success.subject") end end @@ -60,7 +60,7 @@ class Notifier < ActionMailer::Base @error = error mail :to => trace.user.email, - :subject => I18n.t('notifier.gpx_notification.failure.subject') + :subject => I18n.t("notifier.gpx_notification.failure.subject") end end @@ -79,7 +79,7 @@ class Notifier < ActionMailer::Base mail :from => from_address(message.sender.display_name, "m", message.id, message.digest), :to => message.recipient.email, - :subject => I18n.t('notifier.message_notification.subject_header', :subject => message.title) + :subject => I18n.t("notifier.message_notification.subject_header", :subject => message.title) end end @@ -108,17 +108,23 @@ class Notifier < ActionMailer::Base :title => "Re: #{comment.diary_entry.title}") mail :from => from_address(comment.user.display_name, "c", comment.id, comment.digest), - :to => comment.diary_entry.user.email, - :subject => I18n.t('notifier.diary_comment_notification.subject', :user => comment.user.display_name) + :to => comment.diary_entry.user.email, + :subject => I18n.t("notifier.diary_comment_notification.subject", :user => comment.user.display_name) end end def friend_notification(friend) with_recipient_locale friend.befriendee do @friend = friend + @viewurl = url_for(:host => SERVER_URL, + :controller => "user", :action => "view", + :display_name => @friend.befriender.display_name) + @friendurl = url_for(:host => SERVER_URL, + :controller => "user", :action => "make_friend", + :display_name => @friend.befriender.display_name) mail :to => friend.befriendee.email, - :subject => I18n.t('notifier.friend_notification.subject', :user => friend.befriender.display_name) + :subject => I18n.t("notifier.friend_notification.subject", :user => friend.befriender.display_name) end end @@ -152,7 +158,7 @@ class Notifier < ActionMailer::Base @comment = comment.body @owner = recipient == comment.changeset.user @commenter = comment.author.display_name - @changeset_comment = comment.changeset.tags['comment'].presence + @changeset_comment = comment.changeset.tags["comment"].presence @time = comment.created_at @changeset_author = comment.changeset.user.display_name @@ -166,7 +172,7 @@ class Notifier < ActionMailer::Base end end -private + private def with_recipient_locale(recipient) old_locale = I18n.locale @@ -181,8 +187,8 @@ private end def from_address(name, type, id, digest) - if Object.const_defined?(:MESSAGES_DOMAIN) and domain = MESSAGES_DOMAIN - "#{name} <#{type}-#{id}-#{digest[0,6]}@#{domain}>" + if Object.const_defined?(:MESSAGES_DOMAIN) && domain = MESSAGES_DOMAIN + "#{name} <#{type}-#{id}-#{digest[0, 6]}@#{domain}>" else EMAIL_FROM end