From 5e9ae68fb223195f67bfb3992f1d7b0116437d71 Mon Sep 17 00:00:00 2001
From: Tom Hughes <tom@compton.nu>
Date: Mon, 1 Mar 2010 12:08:11 +0000
Subject: [PATCH] Make friend notification messages include a link to add a
 reciprocal friendship if it doesn't already exist. Closes #2754.

---
 app/models/notifier.rb                        | 12 +++--------
 .../notifier/friend_notification.html.erb     | 20 +++++++++++++++++--
 config/locales/en.yml                         |  3 ++-
 3 files changed, 23 insertions(+), 12 deletions(-)

diff --git a/app/models/notifier.rb b/app/models/notifier.rb
index 1f3d39808..df30d651b 100644
--- a/app/models/notifier.rb
+++ b/app/models/notifier.rb
@@ -89,15 +89,9 @@ class Notifier < ActionMailer::Base
   end
 
   def friend_notification(friend)
-    befriender = User.find_by_id(friend.user_id)
-    befriendee = User.find_by_id(friend.friend_user_id)
-
-    common_headers befriendee
-    subject I18n.t('notifier.friend_notification.subject', :user => befriender.display_name, :locale => locale)
-    body :user => befriender.display_name,
-         :userurl => url_for(:host => SERVER_URL,
-                             :controller => "user", :action => "view",
-                             :display_name => befriender.display_name)
+    common_headers friend.befriendee
+    subject I18n.t('notifier.friend_notification.subject', :user => friend.befriender.display_name, :locale => locale)
+    body :friend => friend
   end
 
 private
diff --git a/app/views/notifier/friend_notification.html.erb b/app/views/notifier/friend_notification.html.erb
index 8917061f1..ca7754757 100644
--- a/app/views/notifier/friend_notification.html.erb
+++ b/app/views/notifier/friend_notification.html.erb
@@ -1,4 +1,20 @@
-<%= t'notifier.friend_notification.had_added_you', :user => @user %>
+<%=
+  t 'notifier.friend_notification.had_added_you',
+     :user => @friend.befriender.display_name
+%>
 
-<%= t'notifier.friend_notification.see_their_profile', :userurl => @userurl %>
+<%=
+  t 'notifier.friend_notification.see_their_profile',
+     :userurl => url_for(:host => SERVER_URL,
+                         :controller => "user", :action => "view",
+                         :display_name => @friend.befriender.display_name)
+%>
 
+<%=
+  unless @friend.befriendee.is_friends_with?(@friend.befriender)
+    t 'notifier.friend_notification.befriend_them',
+       :befriendurl => url_for(:host => SERVER_URL,
+                               :controller => "user", :action => "make_friend",
+                               :display_name => @friend.befriender.display_name)
+  end
+%>
diff --git a/config/locales/en.yml b/config/locales/en.yml
index 139bf5775..45c21f36b 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -941,7 +941,8 @@ en:
     friend_notification:
       subject: "[OpenStreetMap] {{user}} added you as a friend"
       had_added_you: "{{user}} has added you as a friend on OpenStreetMap."
-      see_their_profile: "You can see their profile at {{userurl}} and add them as a friend too if you wish."
+      see_their_profile: "You can see their profile at {{userurl}}."
+      befriend_them: "You can also add them as a friend at {{befriendurl}}."
     gpx_notification:
       greeting: "Hi,"
       your_gpx_file: "It looks like your GPX file"
-- 
2.43.2