From 2c1033f62d450b7ed82058ee89d99983b7d97829 Mon Sep 17 00:00:00 2001
From: Andy Allan <git@gravitystorm.co.uk>
Date: Wed, 14 Oct 2020 15:04:58 +0200
Subject: [PATCH] Rename Notifier to UserMailer

It's a convention in rails to name your mailers with a Mailer suffix, and is also common to name the class after the recipient (e.g. User, Admin). So UserMailer seems a reasonable choice.
---
 .rubocop_todo.yml                             | 10 +------
 .../api/changeset_comments_controller.rb      |  2 +-
 app/controllers/api/notes_controller.rb       |  2 +-
 app/controllers/diary_entries_controller.rb   |  2 +-
 app/controllers/friendships_controller.rb     |  2 +-
 app/controllers/messages_controller.rb        |  2 +-
 app/controllers/users_controller.rb           |  8 +++---
 ...tifier_helper.rb => user_mailer_helper.rb} |  2 +-
 app/jobs/trace_importer_job.rb                |  6 ++--
 app/mailers/{notifier.rb => user_mailer.rb}   | 28 +++++++++----------
 app/views/notifier/_gpx_description.html.erb  | 12 --------
 app/views/notifier/email_confirm.html.erb     |  7 -----
 app/views/notifier/email_confirm.text.erb     |  7 -----
 app/views/notifier/gpx_failure.html.erb       | 16 -----------
 app/views/notifier/lost_password.html.erb     |  7 -----
 app/views/notifier/lost_password.text.erb     |  7 -----
 .../user_mailer/_gpx_description.html.erb     | 12 ++++++++
 .../_message_body.html.erb                    |  0
 .../changeset_comment_notification.html.erb   |  0
 .../changeset_comment_notification.text.erb   |  0
 .../diary_comment_notification.html.erb       |  0
 .../diary_comment_notification.text.erb       |  0
 app/views/user_mailer/email_confirm.html.erb  |  7 +++++
 app/views/user_mailer/email_confirm.text.erb  |  7 +++++
 .../friendship_notification.html.erb          |  0
 .../friendship_notification.text.erb          |  0
 app/views/user_mailer/gpx_failure.html.erb    | 16 +++++++++++
 .../gpx_success.html.erb                      |  4 +--
 app/views/user_mailer/lost_password.html.erb  |  7 +++++
 app/views/user_mailer/lost_password.text.erb  |  7 +++++
 .../message_notification.html.erb             |  0
 .../message_notification.text.erb             |  0
 .../note_comment_notification.html.erb        |  0
 .../note_comment_notification.text.erb        |  0
 .../signup_confirm.html.erb                   |  0
 .../signup_confirm.text.erb                   |  0
 config/locales/en.yml                         | 14 +++++-----
 37 files changed, 93 insertions(+), 101 deletions(-)
 rename app/helpers/{notifier_helper.rb => user_mailer_helper.rb} (97%)
 rename app/mailers/{notifier.rb => user_mailer.rb} (83%)
 delete mode 100644 app/views/notifier/_gpx_description.html.erb
 delete mode 100644 app/views/notifier/email_confirm.html.erb
 delete mode 100644 app/views/notifier/email_confirm.text.erb
 delete mode 100644 app/views/notifier/gpx_failure.html.erb
 delete mode 100644 app/views/notifier/lost_password.html.erb
 delete mode 100644 app/views/notifier/lost_password.text.erb
 create mode 100644 app/views/user_mailer/_gpx_description.html.erb
 rename app/views/{notifier => user_mailer}/_message_body.html.erb (100%)
 rename app/views/{notifier => user_mailer}/changeset_comment_notification.html.erb (100%)
 rename app/views/{notifier => user_mailer}/changeset_comment_notification.text.erb (100%)
 rename app/views/{notifier => user_mailer}/diary_comment_notification.html.erb (100%)
 rename app/views/{notifier => user_mailer}/diary_comment_notification.text.erb (100%)
 create mode 100644 app/views/user_mailer/email_confirm.html.erb
 create mode 100644 app/views/user_mailer/email_confirm.text.erb
 rename app/views/{notifier => user_mailer}/friendship_notification.html.erb (100%)
 rename app/views/{notifier => user_mailer}/friendship_notification.text.erb (100%)
 create mode 100644 app/views/user_mailer/gpx_failure.html.erb
 rename app/views/{notifier => user_mailer}/gpx_success.html.erb (57%)
 create mode 100644 app/views/user_mailer/lost_password.html.erb
 create mode 100644 app/views/user_mailer/lost_password.text.erb
 rename app/views/{notifier => user_mailer}/message_notification.html.erb (100%)
 rename app/views/{notifier => user_mailer}/message_notification.text.erb (100%)
 rename app/views/{notifier => user_mailer}/note_comment_notification.html.erb (100%)
 rename app/views/{notifier => user_mailer}/note_comment_notification.text.erb (100%)
 rename app/views/{notifier => user_mailer}/signup_confirm.html.erb (100%)
 rename app/views/{notifier => user_mailer}/signup_confirm.text.erb (100%)

diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
index 61109ddc9..7529d6148 100644
--- a/.rubocop_todo.yml
+++ b/.rubocop_todo.yml
@@ -33,7 +33,7 @@ Lint/AssignmentInCondition:
     - 'app/controllers/users_controller.rb'
     - 'app/helpers/application_helper.rb'
     - 'app/helpers/browse_tags_helper.rb'
-    - 'app/mailers/notifier.rb'
+    - 'app/mailers/user_mailer.rb'
     - 'app/models/client_application.rb'
     - 'lib/nominatim.rb'
     - 'lib/osm.rb'
@@ -149,14 +149,6 @@ Rails/HelperInstanceVariable:
   Exclude:
     - 'app/helpers/title_helper.rb'
 
-# Offense count: 1
-# Cop supports --auto-correct.
-# Configuration parameters: Include.
-# Include: app/mailers/**/*.rb
-Rails/MailerName:
-  Exclude:
-    - 'app/mailers/notifier.rb'
-
 # Offense count: 5
 # Configuration parameters: Include.
 # Include: db/migrate/*.rb
diff --git a/app/controllers/api/changeset_comments_controller.rb b/app/controllers/api/changeset_comments_controller.rb
index 21c854139..a3a13b926 100644
--- a/app/controllers/api/changeset_comments_controller.rb
+++ b/app/controllers/api/changeset_comments_controller.rb
@@ -32,7 +32,7 @@ module Api
 
       # Notify current subscribers of the new comment
       changeset.subscribers.visible.each do |user|
-        Notifier.changeset_comment_notification(comment, user).deliver_later if current_user != user
+        UserMailer.changeset_comment_notification(comment, user).deliver_later if current_user != user
       end
 
       # Add the commenter to the subscribers if necessary
diff --git a/app/controllers/api/notes_controller.rb b/app/controllers/api/notes_controller.rb
index cd71b4374..f4d1d0dac 100644
--- a/app/controllers/api/notes_controller.rb
+++ b/app/controllers/api/notes_controller.rb
@@ -381,7 +381,7 @@ module Api
       comment = note.comments.create!(attributes)
 
       note.comments.map(&:author).uniq.each do |user|
-        Notifier.note_comment_notification(comment, user).deliver_later if notify && user && user != current_user && user.visible?
+        UserMailer.note_comment_notification(comment, user).deliver_later if notify && user && user != current_user && user.visible?
       end
     end
   end
diff --git a/app/controllers/diary_entries_controller.rb b/app/controllers/diary_entries_controller.rb
index 5f53e81b6..12f31bad2 100644
--- a/app/controllers/diary_entries_controller.rb
+++ b/app/controllers/diary_entries_controller.rb
@@ -81,7 +81,7 @@ class DiaryEntriesController < ApplicationController
 
       # Notify current subscribers of the new comment
       @entry.subscribers.visible.each do |user|
-        Notifier.diary_comment_notification(@diary_comment, user).deliver_later if current_user != user
+        UserMailer.diary_comment_notification(@diary_comment, user).deliver_later if current_user != user
       end
 
       # Add the commenter to the subscribers if necessary
diff --git a/app/controllers/friendships_controller.rb b/app/controllers/friendships_controller.rb
index 75e53368d..0bff13df6 100644
--- a/app/controllers/friendships_controller.rb
+++ b/app/controllers/friendships_controller.rb
@@ -21,7 +21,7 @@ class FriendshipsController < ApplicationController
           flash[:warning] = t "friendships.make_friend.already_a_friend", :name => @new_friend.display_name
         elsif friendship.save
           flash[:notice] = t "friendships.make_friend.success", :name => @new_friend.display_name
-          Notifier.friendship_notification(friendship).deliver_later
+          UserMailer.friendship_notification(friendship).deliver_later
         else
           friendship.add_error(t("friendships.make_friend.failed", :name => @new_friend.display_name))
         end
diff --git a/app/controllers/messages_controller.rb b/app/controllers/messages_controller.rb
index 53ae1dbd7..2047c4614 100644
--- a/app/controllers/messages_controller.rb
+++ b/app/controllers/messages_controller.rb
@@ -31,7 +31,7 @@ class MessagesController < ApplicationController
       render :action => "new"
     elsif @message.save
       flash[:notice] = t ".message_sent"
-      Notifier.message_notification(@message).deliver_later
+      UserMailer.message_notification(@message).deliver_later
       redirect_to :action => :inbox
     else
       @title = t "messages.new.title"
diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb
index 1cc27d9d9..4c5827558 100644
--- a/app/controllers/users_controller.rb
+++ b/app/controllers/users_controller.rb
@@ -106,7 +106,7 @@ class UsersController < ApplicationController
             successful_login(current_user)
           else
             session[:token] = current_user.tokens.create.token
-            Notifier.signup_confirm(current_user, current_user.tokens.create(:referer => referer)).deliver_later
+            UserMailer.signup_confirm(current_user, current_user.tokens.create(:referer => referer)).deliver_later
             redirect_to :action => "confirm", :display_name => current_user.display_name
           end
         else
@@ -157,7 +157,7 @@ class UsersController < ApplicationController
 
       if user
         token = user.tokens.create
-        Notifier.lost_password(user, token).deliver_later
+        UserMailer.lost_password(user, token).deliver_later
         flash[:notice] = t "users.lost_password.notice email on way"
         redirect_to :action => "login"
       else
@@ -343,7 +343,7 @@ class UsersController < ApplicationController
     if user.nil? || token.nil? || token.user != user
       flash[:error] = t "users.confirm_resend.failure", :name => params[:display_name]
     else
-      Notifier.signup_confirm(user, user.tokens.create).deliver_later
+      UserMailer.signup_confirm(user, user.tokens.create).deliver_later
       flash[:notice] = t("users.confirm_resend.success", :email => user.email, :sender => Settings.support_email).html_safe
     end
 
@@ -659,7 +659,7 @@ class UsersController < ApplicationController
           flash.now[:notice] = t "users.account.flash update success confirm needed"
 
           begin
-            Notifier.email_confirm(user, user.tokens.create).deliver_later
+            UserMailer.email_confirm(user, user.tokens.create).deliver_later
           rescue StandardError
             # Ignore errors sending email
           end
diff --git a/app/helpers/notifier_helper.rb b/app/helpers/user_mailer_helper.rb
similarity index 97%
rename from app/helpers/notifier_helper.rb
rename to app/helpers/user_mailer_helper.rb
index c18f79903..39977f5a7 100644
--- a/app/helpers/notifier_helper.rb
+++ b/app/helpers/user_mailer_helper.rb
@@ -1,4 +1,4 @@
-module NotifierHelper
+module UserMailerHelper
   def fp(text)
     format_paragraph(text, 72, 0)
   end
diff --git a/app/jobs/trace_importer_job.rb b/app/jobs/trace_importer_job.rb
index 2a9a8678a..1eddcb6f8 100644
--- a/app/jobs/trace_importer_job.rb
+++ b/app/jobs/trace_importer_job.rb
@@ -5,15 +5,15 @@ class TraceImporterJob < ApplicationJob
     gpx = trace.import
 
     if gpx.actual_points.positive?
-      Notifier.gpx_success(trace, gpx.actual_points).deliver
+      UserMailer.gpx_success(trace, gpx.actual_points).deliver
     else
-      Notifier.gpx_failure(trace, "0 points parsed ok. Do they all have lat,lng,alt,timestamp?").deliver
+      UserMailer.gpx_failure(trace, "0 points parsed ok. Do they all have lat,lng,alt,timestamp?").deliver
       trace.destroy
     end
   rescue StandardError => e
     logger.info e.to_s
     e.backtrace.each { |l| logger.info l }
-    Notifier.gpx_failure(trace, e.to_s + "\n" + e.backtrace.join("\n")).deliver
+    UserMailer.gpx_failure(trace, e.to_s + "\n" + e.backtrace.join("\n")).deliver
     trace.destroy
   end
 end
diff --git a/app/mailers/notifier.rb b/app/mailers/user_mailer.rb
similarity index 83%
rename from app/mailers/notifier.rb
rename to app/mailers/user_mailer.rb
index 3e0ba446d..24444ce1b 100644
--- a/app/mailers/notifier.rb
+++ b/app/mailers/user_mailer.rb
@@ -1,4 +1,4 @@
-class Notifier < ApplicationMailer
+class UserMailer < ApplicationMailer
   include ActionView::Helpers::AssetUrlHelper
 
   self.delivery_job = ActionMailer::MailDeliveryJob
@@ -17,7 +17,7 @@ class Notifier < ApplicationMailer
                      :confirm_string => token.token)
 
       mail :to => user.email,
-           :subject => I18n.t("notifier.signup_confirm.subject")
+           :subject => I18n.t("user_mailer.signup_confirm.subject")
     end
   end
 
@@ -28,7 +28,7 @@ class Notifier < ApplicationMailer
                      :confirm_string => token.token)
 
       mail :to => user.new_email,
-           :subject => I18n.t("notifier.email_confirm.subject")
+           :subject => I18n.t("user_mailer.email_confirm.subject")
     end
   end
 
@@ -38,7 +38,7 @@ class Notifier < ApplicationMailer
                      :token => token.token)
 
       mail :to => user.email,
-           :subject => I18n.t("notifier.lost_password.subject")
+           :subject => I18n.t("user_mailer.lost_password.subject")
     end
   end
 
@@ -51,7 +51,7 @@ class Notifier < ApplicationMailer
       @possible_points = possible_points
 
       mail :to => trace.user.email,
-           :subject => I18n.t("notifier.gpx_notification.success.subject")
+           :subject => I18n.t("user_mailer.gpx_notification.success.subject")
     end
   end
 
@@ -63,7 +63,7 @@ class Notifier < ApplicationMailer
       @error = error
 
       mail :to => trace.user.email,
-           :subject => I18n.t("notifier.gpx_notification.failure.subject")
+           :subject => I18n.t("user_mailer.gpx_notification.failure.subject")
     end
   end
 
@@ -81,7 +81,7 @@ class Notifier < ApplicationMailer
 
       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("user_mailer.message_notification.subject_header", :subject => message.title)
     end
   end
 
@@ -102,7 +102,7 @@ class Notifier < ApplicationMailer
 
       mail :from => from_address(comment.user.display_name, "c", comment.id, comment.digest, recipient.id),
            :to => recipient.email,
-           :subject => I18n.t("notifier.diary_comment_notification.subject", :user => comment.user.display_name)
+           :subject => I18n.t("user_mailer.diary_comment_notification.subject", :user => comment.user.display_name)
     end
   end
 
@@ -115,7 +115,7 @@ class Notifier < ApplicationMailer
 
       attach_user_avatar(@friendship.befriender)
       mail :to => friendship.befriendee.email,
-           :subject => I18n.t("notifier.friendship_notification.subject", :user => friendship.befriender.display_name)
+           :subject => I18n.t("user_mailer.friendship_notification.subject", :user => friendship.befriender.display_name)
     end
   end
 
@@ -130,7 +130,7 @@ class Notifier < ApplicationMailer
       @commenter = if comment.author
                      comment.author.display_name
                    else
-                     I18n.t("notifier.note_comment_notification.anonymous")
+                     I18n.t("user_mailer.note_comment_notification.anonymous")
                    end
 
       @author = @commenter
@@ -139,9 +139,9 @@ class Notifier < ApplicationMailer
       set_references("note", comment.note)
 
       subject = if @owner
-                  I18n.t("notifier.note_comment_notification.#{@event}.subject_own", :commenter => @commenter)
+                  I18n.t("user_mailer.note_comment_notification.#{@event}.subject_own", :commenter => @commenter)
                 else
-                  I18n.t("notifier.note_comment_notification.#{@event}.subject_other", :commenter => @commenter)
+                  I18n.t("user_mailer.note_comment_notification.#{@event}.subject_other", :commenter => @commenter)
                 end
 
       mail :to => recipient.email, :subject => subject
@@ -161,9 +161,9 @@ class Notifier < ApplicationMailer
       @author = @commenter
 
       subject = if @owner
-                  I18n.t("notifier.changeset_comment_notification.commented.subject_own", :commenter => @commenter)
+                  I18n.t("user_mailer.changeset_comment_notification.commented.subject_own", :commenter => @commenter)
                 else
-                  I18n.t("notifier.changeset_comment_notification.commented.subject_other", :commenter => @commenter)
+                  I18n.t("user_mailer.changeset_comment_notification.commented.subject_other", :commenter => @commenter)
                 end
 
       attach_user_avatar(comment.author)
diff --git a/app/views/notifier/_gpx_description.html.erb b/app/views/notifier/_gpx_description.html.erb
deleted file mode 100644
index de67f6005..000000000
--- a/app/views/notifier/_gpx_description.html.erb
+++ /dev/null
@@ -1,12 +0,0 @@
-<%= t "notifier.gpx_notification.your_gpx_file" %>
-<strong><%= @trace_name %></strong>
-<%= t "notifier.gpx_notification.with_description" %>
-<em><%= @trace_description %></em>
-<% if @trace_tags.length>0 %>
-  <%= t "notifier.gpx_notification.and_the_tags" %>
-  <em><% @trace_tags.each do |tag| %>
-    <%= tag.tag.rstrip %>
-  <% end %></em>
-<% else %>
-  <%= t "notifier.gpx_notification.and_no_tags" %>
-<% end %>
diff --git a/app/views/notifier/email_confirm.html.erb b/app/views/notifier/email_confirm.html.erb
deleted file mode 100644
index 57359ef59..000000000
--- a/app/views/notifier/email_confirm.html.erb
+++ /dev/null
@@ -1,7 +0,0 @@
-<p><%= t "notifier.email_confirm_html.greeting" %></p>
-
-<p><%= t "notifier.email_confirm_html.hopefully_you", :server_url => Settings.server_url, :new_address => @address %></p>
-
-<p><%= t "notifier.email_confirm_html.click_the_link" %></p>
-
-<p><a href="<%= @url %>"><%= @url %></a></p>
diff --git a/app/views/notifier/email_confirm.text.erb b/app/views/notifier/email_confirm.text.erb
deleted file mode 100644
index 7c4f0a5ab..000000000
--- a/app/views/notifier/email_confirm.text.erb
+++ /dev/null
@@ -1,7 +0,0 @@
-<%= t 'notifier.email_confirm_plain.greeting' %>
-
-<%= word_wrap(t 'notifier.email_confirm_plain.hopefully_you', :server_url => Settings.server_url, :new_address => @address) %>
-
-<%= t 'notifier.email_confirm_plain.click_the_link' %>
-
-<%= @url %>
diff --git a/app/views/notifier/gpx_failure.html.erb b/app/views/notifier/gpx_failure.html.erb
deleted file mode 100644
index 3ce26de8c..000000000
--- a/app/views/notifier/gpx_failure.html.erb
+++ /dev/null
@@ -1,16 +0,0 @@
-<p><%= t "notifier.gpx_notification.greeting" %></p>
-
-<p>
-  <%= render :partial => "gpx_description" %>
-  <%= t "notifier.gpx_notification.failure.failed_to_import" %>
-</p>
-
-<blockquote>
-  <%= @error %>
-</blockquote>
-
-<p>
-  <%= t "notifier.gpx_notification.failure.more_info_1" %>
-  <%= t "notifier.gpx_notification.failure.more_info_2" %>
-  <%= t "notifier.gpx_notification.failure.import_failures_url" %>
-</p>
diff --git a/app/views/notifier/lost_password.html.erb b/app/views/notifier/lost_password.html.erb
deleted file mode 100644
index 3f371c80c..000000000
--- a/app/views/notifier/lost_password.html.erb
+++ /dev/null
@@ -1,7 +0,0 @@
-<p><%= t "notifier.lost_password_html.greeting" %></p>
-
-<p><%= t "notifier.lost_password_html.hopefully_you" %></p>
-
-<p><%= t "notifier.lost_password_html.click_the_link" %></p>
-
-<p><a href="<%= @url %>"><%= @url %></a></p>
diff --git a/app/views/notifier/lost_password.text.erb b/app/views/notifier/lost_password.text.erb
deleted file mode 100644
index 2b74e8e80..000000000
--- a/app/views/notifier/lost_password.text.erb
+++ /dev/null
@@ -1,7 +0,0 @@
-<%= t 'notifier.lost_password_plain.greeting' %>
-
-<%= word_wrap(t 'notifier.lost_password_plain.hopefully_you') %>
-
-<%= t 'notifier.lost_password_plain.click_the_link' %>
-
-<%= @url %>
diff --git a/app/views/user_mailer/_gpx_description.html.erb b/app/views/user_mailer/_gpx_description.html.erb
new file mode 100644
index 000000000..4fdf929ac
--- /dev/null
+++ b/app/views/user_mailer/_gpx_description.html.erb
@@ -0,0 +1,12 @@
+<%= t "user_mailer.gpx_notification.your_gpx_file" %>
+<strong><%= @trace_name %></strong>
+<%= t "user_mailer.gpx_notification.with_description" %>
+<em><%= @trace_description %></em>
+<% if @trace_tags.length>0 %>
+  <%= t "user_mailer.gpx_notification.and_the_tags" %>
+  <em><% @trace_tags.each do |tag| %>
+    <%= tag.tag.rstrip %>
+  <% end %></em>
+<% else %>
+  <%= t "user_mailer.gpx_notification.and_no_tags" %>
+<% end %>
diff --git a/app/views/notifier/_message_body.html.erb b/app/views/user_mailer/_message_body.html.erb
similarity index 100%
rename from app/views/notifier/_message_body.html.erb
rename to app/views/user_mailer/_message_body.html.erb
diff --git a/app/views/notifier/changeset_comment_notification.html.erb b/app/views/user_mailer/changeset_comment_notification.html.erb
similarity index 100%
rename from app/views/notifier/changeset_comment_notification.html.erb
rename to app/views/user_mailer/changeset_comment_notification.html.erb
diff --git a/app/views/notifier/changeset_comment_notification.text.erb b/app/views/user_mailer/changeset_comment_notification.text.erb
similarity index 100%
rename from app/views/notifier/changeset_comment_notification.text.erb
rename to app/views/user_mailer/changeset_comment_notification.text.erb
diff --git a/app/views/notifier/diary_comment_notification.html.erb b/app/views/user_mailer/diary_comment_notification.html.erb
similarity index 100%
rename from app/views/notifier/diary_comment_notification.html.erb
rename to app/views/user_mailer/diary_comment_notification.html.erb
diff --git a/app/views/notifier/diary_comment_notification.text.erb b/app/views/user_mailer/diary_comment_notification.text.erb
similarity index 100%
rename from app/views/notifier/diary_comment_notification.text.erb
rename to app/views/user_mailer/diary_comment_notification.text.erb
diff --git a/app/views/user_mailer/email_confirm.html.erb b/app/views/user_mailer/email_confirm.html.erb
new file mode 100644
index 000000000..3848b192b
--- /dev/null
+++ b/app/views/user_mailer/email_confirm.html.erb
@@ -0,0 +1,7 @@
+<p><%= t "user_mailer.email_confirm_html.greeting" %></p>
+
+<p><%= t "user_mailer.email_confirm_html.hopefully_you", :server_url => Settings.server_url, :new_address => @address %></p>
+
+<p><%= t "user_mailer.email_confirm_html.click_the_link" %></p>
+
+<p><a href="<%= @url %>"><%= @url %></a></p>
diff --git a/app/views/user_mailer/email_confirm.text.erb b/app/views/user_mailer/email_confirm.text.erb
new file mode 100644
index 000000000..e4fff2b6d
--- /dev/null
+++ b/app/views/user_mailer/email_confirm.text.erb
@@ -0,0 +1,7 @@
+<%= t 'user_mailer.email_confirm_plain.greeting' %>
+
+<%= word_wrap(t 'user_mailer.email_confirm_plain.hopefully_you', :server_url => Settings.server_url, :new_address => @address) %>
+
+<%= t 'user_mailer.email_confirm_plain.click_the_link' %>
+
+<%= @url %>
diff --git a/app/views/notifier/friendship_notification.html.erb b/app/views/user_mailer/friendship_notification.html.erb
similarity index 100%
rename from app/views/notifier/friendship_notification.html.erb
rename to app/views/user_mailer/friendship_notification.html.erb
diff --git a/app/views/notifier/friendship_notification.text.erb b/app/views/user_mailer/friendship_notification.text.erb
similarity index 100%
rename from app/views/notifier/friendship_notification.text.erb
rename to app/views/user_mailer/friendship_notification.text.erb
diff --git a/app/views/user_mailer/gpx_failure.html.erb b/app/views/user_mailer/gpx_failure.html.erb
new file mode 100644
index 000000000..a398661a6
--- /dev/null
+++ b/app/views/user_mailer/gpx_failure.html.erb
@@ -0,0 +1,16 @@
+<p><%= t "user_mailer.gpx_notification.greeting" %></p>
+
+<p>
+  <%= render :partial => "gpx_description" %>
+  <%= t "user_mailer.gpx_notification.failure.failed_to_import" %>
+</p>
+
+<blockquote>
+  <%= @error %>
+</blockquote>
+
+<p>
+  <%= t "user_mailer.gpx_notification.failure.more_info_1" %>
+  <%= t "user_mailer.gpx_notification.failure.more_info_2" %>
+  <%= t "user_mailer.gpx_notification.failure.import_failures_url" %>
+</p>
diff --git a/app/views/notifier/gpx_success.html.erb b/app/views/user_mailer/gpx_success.html.erb
similarity index 57%
rename from app/views/notifier/gpx_success.html.erb
rename to app/views/user_mailer/gpx_success.html.erb
index 1571f3798..78af1166c 100644
--- a/app/views/notifier/gpx_success.html.erb
+++ b/app/views/user_mailer/gpx_success.html.erb
@@ -1,7 +1,7 @@
-<p><%= t "notifier.gpx_notification.greeting" %></p>
+<p><%= t "user_mailer.gpx_notification.greeting" %></p>
 
 <p>
   <%= render :partial => "gpx_description" %>
-  <%= t("notifier.gpx_notification.success.loaded_successfully",
+  <%= t("user_mailer.gpx_notification.success.loaded_successfully",
         :trace_points => @trace_points, :possible_points => @possible_points, :count => @possible_points) %>
 </p>
diff --git a/app/views/user_mailer/lost_password.html.erb b/app/views/user_mailer/lost_password.html.erb
new file mode 100644
index 000000000..7e61b6c49
--- /dev/null
+++ b/app/views/user_mailer/lost_password.html.erb
@@ -0,0 +1,7 @@
+<p><%= t "user_mailer.lost_password_html.greeting" %></p>
+
+<p><%= t "user_mailer.lost_password_html.hopefully_you" %></p>
+
+<p><%= t "user_mailer.lost_password_html.click_the_link" %></p>
+
+<p><a href="<%= @url %>"><%= @url %></a></p>
diff --git a/app/views/user_mailer/lost_password.text.erb b/app/views/user_mailer/lost_password.text.erb
new file mode 100644
index 000000000..778fa6540
--- /dev/null
+++ b/app/views/user_mailer/lost_password.text.erb
@@ -0,0 +1,7 @@
+<%= t 'user_mailer.lost_password_plain.greeting' %>
+
+<%= word_wrap(t 'user_mailer.lost_password_plain.hopefully_you') %>
+
+<%= t 'user_mailer.lost_password_plain.click_the_link' %>
+
+<%= @url %>
diff --git a/app/views/notifier/message_notification.html.erb b/app/views/user_mailer/message_notification.html.erb
similarity index 100%
rename from app/views/notifier/message_notification.html.erb
rename to app/views/user_mailer/message_notification.html.erb
diff --git a/app/views/notifier/message_notification.text.erb b/app/views/user_mailer/message_notification.text.erb
similarity index 100%
rename from app/views/notifier/message_notification.text.erb
rename to app/views/user_mailer/message_notification.text.erb
diff --git a/app/views/notifier/note_comment_notification.html.erb b/app/views/user_mailer/note_comment_notification.html.erb
similarity index 100%
rename from app/views/notifier/note_comment_notification.html.erb
rename to app/views/user_mailer/note_comment_notification.html.erb
diff --git a/app/views/notifier/note_comment_notification.text.erb b/app/views/user_mailer/note_comment_notification.text.erb
similarity index 100%
rename from app/views/notifier/note_comment_notification.text.erb
rename to app/views/user_mailer/note_comment_notification.text.erb
diff --git a/app/views/notifier/signup_confirm.html.erb b/app/views/user_mailer/signup_confirm.html.erb
similarity index 100%
rename from app/views/notifier/signup_confirm.html.erb
rename to app/views/user_mailer/signup_confirm.html.erb
diff --git a/app/views/notifier/signup_confirm.text.erb b/app/views/user_mailer/signup_confirm.text.erb
similarity index 100%
rename from app/views/notifier/signup_confirm.text.erb
rename to app/views/user_mailer/signup_confirm.text.erb
diff --git a/config/locales/en.yml b/config/locales/en.yml
index ec08012fe..6157fa517 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -1411,7 +1411,7 @@ en:
       text: Make a Donation
     learn_more: "Learn More"
     more: More
-  notifier:
+  user_mailer:
     diary_comment_notification:
       subject: "[OpenStreetMap] %{user} commented on a diary entry"
       hi: "Hi %{to_user},"
@@ -1639,7 +1639,7 @@ en:
           full <a href="https://opendatacommons.org/licenses/odbl/1.0/">legal
           code</a> explains your rights and responsibilities.
         intro_3_1_html: |
-          Our documentation is licensed under the 
+          Our documentation is licensed under the
           <a href="https://creativecommons.org/licenses/by-sa/2.0/">Creative
           Commons Attribution-ShareAlike 2.0</a> license (CC BY-SA 2.0).
         credit_title_html: How to credit OpenStreetMap
@@ -1656,11 +1656,11 @@ en:
           direct your readers to openstreetmap.org (perhaps by expanding
           'OpenStreetMap' to this full address) and to opendatacommons.org.
         credit_3_1_html: |
-          The map tiles in the &ldquo;standard style&rdquo; at www.openstreetmap.org are a 
-          Produced Work by the OpenStreetMap Foundation using OpenStreetMap data 
-          under the Open Database License. If you are using these tiles please use 
-          the following attribution: 
-          &ldquo;Base map and data from OpenStreetMap and OpenStreetMap Foundation&rdquo;. 
+          The map tiles in the &ldquo;standard style&rdquo; at www.openstreetmap.org are a
+          Produced Work by the OpenStreetMap Foundation using OpenStreetMap data
+          under the Open Database License. If you are using these tiles please use
+          the following attribution:
+          &ldquo;Base map and data from OpenStreetMap and OpenStreetMap Foundation&rdquo;.
         credit_4_html: |
           For a browsable electronic map, the credit should appear in the corner of the map.
           For example:
-- 
2.39.5