Dressed up note comment messages

diff --git a/app/models/notifier.rb b/app/models/notifier.rb
index d7fa66c4ad431332455b354528cd10cef97a5aaa..73cbbad541c2040fa1f813211152c43335b60e73 100644 (file)
--- a/app/models/notifier.rb
+++ b/app/models/notifier.rb
@@ -151,6 +151,9 @@ class Notifier < ActionMailer::Base
                      I18n.t("notifier.note_comment_notification.anonymous")
                    end
 
+      @user_message_author = @commenter
+      attach_user_avatar(comment.author)
+
       subject = if @owner
                   I18n.t("notifier.note_comment_notification.#{@event}.subject_own", :commenter => @commenter)
                 else
@@ -202,8 +205,8 @@ class Notifier < ActionMailer::Base
   end
 
   def user_avatar_file_path(user)
-    image = user.image
-    if image.file?
+    image = user && user.image
+    if image && image.file?
       return image.path(:small)
     else
       return "#{Rails.root}/app/assets/images/users/images/small.png"

diff --git a/app/views/notifier/note_comment_notification.html.erb b/app/views/notifier/note_comment_notification.html.erb
index d82723bbb86d5dac0c5ca46a6a3a2f19cb08fde2..ab4378fa275f00067570c4a235a7002f90ee683f 100644 (file)
--- a/app/views/notifier/note_comment_notification.html.erb
+++ b/app/views/notifier/note_comment_notification.html.erb
@@ -1,13 +1,15 @@
-<p><%= t 'notifier.note_comment_notification.greeting' %></p>
+<% content_for :body do %>
+  <p style="margin: 0"><%= t 'notifier.note_comment_notification.greeting' %></p>
 
-<% if @owner %>
-<p><%= t "notifier.note_comment_notification.#{@event}.your_note", :commenter => @commenter, :place => @place %></p>
-<% else %>
-<p><%= t "notifier.note_comment_notification.#{@event}.commented_note", :commenter => @commenter, :place => @place %></p>
-<% end %>
+  <% if @owner %>
+    <p><%= raw t "notifier.note_comment_notification.#{@event}.your_note", :commenter => link_to_user(@commenter), :place => @place %></p>
+  <% else %>
+    <p><%= raw t "notifier.note_comment_notification.#{@event}.commented_note", :commenter => link_to_user(@commenter), :place => @place %></p>
+  <% end %>
 
-==
-<%= @comment.to_html %>
-==
+  <%= render "notifier/user_message_table", :captured => capture { %>
+    <%= @comment.to_html %>
+  <% } %>
 
-<p><%= raw t 'notifier.note_comment_notification.details', :url => link_to(@noteurl, @noteurl) %></p>
+  <p><%= raw t 'notifier.note_comment_notification.details', :url => link_to(@noteurl, @noteurl) %></p>
+<% end %>