Small refactorization of avatar methods (#31391).

* move existing methods from ApplicationHelper to a new helper file (AvatarsHelper)
* change default avatar size from 50 to 24 because most of the avatars are using the  size 24
* class 'gravatar' is always added and all custom classes are appended
* added user name as default title for avatar images
* added two new methods: @assignee_avatar@ and @author_avatar@

Patch by Marius BALTEANU.


git-svn-id: http://svn.redmine.org/redmine/trunk@18175 e93f8b46-1217-0410-a6f0-8f06a7374b81

11 files changed, 89 insertions, 45 deletions

diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb
index 54f1e63d7..082bab939 100644
--- a/app/controllers/application_controller.rb
+++ b/app/controllers/application_controller.rb
@@ -27,7 +27,10 @@ class ApplicationController < ActionController::Base
   include Redmine::Pagination
   include Redmine::Hook::Helper
   include RoutesHelper
+  include AvatarsHelper
+
   helper :routes
+  helper :avatars
 
   class_attribute :accept_api_auth_actions
   class_attribute :accept_rss_auth_actions
diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb
index 09df2656b..7ae2c2d88 100644
--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
@@ -23,7 +23,6 @@ require 'cgi'
 module ApplicationHelper
   include Redmine::WikiFormatting::Macros::Definitions
   include Redmine::I18n
-  include GravatarHelper::PublicMethods
   include Redmine::Pagination::Helper
   include Redmine::SudoMode::Helper
   include Redmine::Themes::Helper
@@ -1484,39 +1483,6 @@ module ApplicationHelper
     !!ActionMailer::Base.perform_deliveries
   end
 
-  # Returns the avatar image tag for the given +user+ if avatars are enabled
-  # +user+ can be a User or a string that will be scanned for an email address (eg. 'joe <joe@foo.bar>')
-  def avatar(user, options = { })
-    if Setting.gravatar_enabled?
-      options.merge!(:default => Setting.gravatar_default)
-      email = nil
-      if user.respond_to?(:mail)
-        email = user.mail
-      elsif user.to_s =~ %r{<(.+?)>}
-        email = $1
-      end
-      if email.present?
-        gravatar(email.to_s.downcase, options) rescue nil
-      elsif user.is_a?(AnonymousUser)
-        image_tag 'anonymous.png',
-                  GravatarHelper::DEFAULT_OPTIONS
-                    .except(:default, :rating, :ssl).merge(options)
-      else
-        nil
-      end
-    else
-      ''
-    end
-  end
-
-  # Returns a link to edit user's avatar if avatars are enabled
-  def avatar_edit_link(user, options={})
-    if Setting.gravatar_enabled?
-      url = Redmine::Configuration['avatar_server_url']
-      link_to avatar(user, {:title => l(:button_edit)}.merge(options)), url, :target => '_blank'
-    end
-  end
-
   def sanitize_anchor_name(anchor)
     anchor.gsub(%r{[^\s\-\p{Word}]}, '').gsub(%r{\s+(\-+\s*)?}, '-')
   end
diff --git a/app/helpers/avatars_helper.rb b/app/helpers/avatars_helper.rb
new file mode 100644
index 000000000..71898a4d4
--- /dev/null
+++ b/app/helpers/avatars_helper.rb
@@ -0,0 +1,75 @@
+# frozen_string_literal: true
+
+# Redmine - project management software
+# Copyright (C) 2006-2019  Jean-Philippe Lang
+#
+# This program is free software; you can redistribute it and/or
+# modify it under the terms of the GNU General Public License
+# as published by the Free Software Foundation; either version 2
+# of the License, or (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write to the Free Software
+# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
+
+module AvatarsHelper
+  include GravatarHelper::PublicMethods
+
+  def assignee_avatar(user, options={})
+    return '' unless user
+
+    options.merge!(:title => l(:field_assigned_to) + ": " + user.name)
+    avatar(user, options).html_safe
+  end
+
+  def author_avatar(user, options={})
+    return '' unless user
+
+    options.merge!(:title => l(:field_author) + ": " + user.name)
+    avatar(user, options).html_safe
+  end
+
+  # Returns the avatar image tag for the given +user+ if avatars are enabled
+  # +user+ can be a User or a string that will be scanned for an email address (eg. 'joe <joe@foo.bar>')
+  def avatar(user, options = { })
+    if Setting.gravatar_enabled?
+      options.merge!(:default => Setting.gravatar_default)
+      options[:class] = GravatarHelper::DEFAULT_OPTIONS[:class] + " " + options[:class] if options[:class]
+      email = nil
+      if user.respond_to?(:mail)
+        email = user.mail
+        options[:title] = user.name unless options[:title]
+      elsif user.to_s =~ %r{<(.+?)>}
+        email = $1
+      end
+      if email.present?
+        gravatar(email.to_s.downcase, options) rescue nil
+      elsif user.is_a?(AnonymousUser)
+        anonymous_avatar(options)
+      else
+        nil
+      end
+    else
+      ''
+    end
+  end
+
+  # Returns a link to edit user's avatar if avatars are enabled
+  def avatar_edit_link(user, options={})
+    if Setting.gravatar_enabled?
+      url = Redmine::Configuration['avatar_server_url']
+      link_to avatar(user, {:title => l(:button_edit)}.merge(options)), url, :target => '_blank'
+    end
+  end
+
+  private
+
+  def anonymous_avatar(options={})
+    image_tag 'anonymous.png', GravatarHelper::DEFAULT_OPTIONS.except(:default, :rating, :ssl).merge(options)
+  end
+end
diff --git a/app/views/activities/_activities.html.erb b/app/views/activities/_activities.html.erb
index 2a1f46cd7..aaeea76bc 100644
--- a/app/views/activities/_activities.html.erb
+++ b/app/views/activities/_activities.html.erb
@@ -4,7 +4,7 @@
 <dl>
 <% sort_activity_events(events_by_day[day]).each do |e, in_group| -%>
   <dt class="<%= e.event_type %> icon icon-<%= e.event_type %> <%= "grouped" if in_group %> <%= User.current.logged? && e.respond_to?(:event_author) && User.current == e.event_author ? 'me' : nil %>">
-  <%= avatar(e.event_author, :size => "24") if e.respond_to?(:event_author) %>
+  <%= avatar(e.event_author) if e.respond_to?(:event_author) %>
   <span class="time"><%= format_time(e.event_datetime, false) %></span>
   <%= content_tag('span', e.project, :class => 'project') if @project.nil? || @project != e.project %>
   <%= link_to format_activity_title(e.event_title), e.event_url %>
diff --git a/app/views/issues/_history.html.erb b/app/views/issues/_history.html.erb
index aa8ecbf80..297771d7d 100644
--- a/app/views/issues/_history.html.erb
+++ b/app/views/issues/_history.html.erb
@@ -7,7 +7,7 @@
       <a href="#note-<%= journal.indice %>" class="journal-link">#<%= journal.indice %></a>
     </div>
     <h4>
-      <%= avatar(journal.user, :size => "24") %>
+      <%= avatar(journal.user) %>
       <%= authoring journal.created_on, journal.user, :label => :label_updated_time_by %>
       <%= render_private_notes_indicator(journal) %>
     </h4>
diff --git a/app/views/issues/show.html.erb b/app/views/issues/show.html.erb
index 5a1cc8599..fc93afd9d 100644
--- a/app/views/issues/show.html.erb
+++ b/app/views/issues/show.html.erb
@@ -26,8 +26,8 @@
   <% end %>
 
   <div class="gravatar-with-child">
-    <%= avatar(@issue.author, :size => "50", :title => l(:field_author)) %>
-    <%= avatar(@issue.assigned_to, :size => "22", :class => "gravatar gravatar-child", :title => l(:field_assigned_to)) if @issue.assigned_to %>
+    <%= author_avatar(@issue.author, :size => "50") %>
+    <%= assignee_avatar(@issue.assigned_to, :size => "22", :class => "gravatar-child") if @issue.assigned_to %>
   </div>
 
 <div class="subject">
diff --git a/app/views/messages/edit.html.erb b/app/views/messages/edit.html.erb
index 063676197..559afc6bc 100644
--- a/app/views/messages/edit.html.erb
+++ b/app/views/messages/edit.html.erb
@@ -1,6 +1,6 @@
 <%= board_breadcrumb(@message) %>
 
-<h2><%= avatar(@topic.author, :size => "24") %><%= @topic.subject %></h2>
+<h2><%= avatar(@topic.author) %><%= @topic.subject %></h2>
 
 <%= form_for @message, {
                :as => :message,
diff --git a/app/views/messages/show.html.erb b/app/views/messages/show.html.erb
index 5756a9267..893659d6f 100644
--- a/app/views/messages/show.html.erb
+++ b/app/views/messages/show.html.erb
@@ -22,7 +22,7 @@
          ) if @message.destroyable_by?(User.current) %>
 </div>
 
-<h2><%= avatar(@topic.author, :size => "24") %><%= @topic.subject %></h2>
+<h2><%= avatar(@topic.author) %><%= @topic.subject %></h2>
 
 <div class="message">
 <p><span class="author"><%= authoring @topic.created_on, @topic.author %></span></p>
@@ -66,7 +66,7 @@
           ) if message.destroyable_by?(User.current) %>
     </div>
   <h4>
-    <%= avatar(message.author, :size => "24") %>
+    <%= avatar(message.author) %>
     <%= link_to message.subject, { :controller => 'messages', :action => 'show', :board_id => @board, :id => @topic, :r => message, :anchor => "message-#{message.id}" } %>
     -
     <%= authoring message.created_on, message.author %>
diff --git a/app/views/news/index.html.erb b/app/views/news/index.html.erb
index cd51aa950..073099f56 100644
--- a/app/views/news/index.html.erb
+++ b/app/views/news/index.html.erb
@@ -24,7 +24,7 @@
 <% @newss.each do |news| %>
     <article class="news-article">
       <header>
-        <h3><%= avatar(news.author, :size => "24") %><%= link_to_project(news.project) + ': ' unless news.project == @project %>
+        <h3><%= avatar(news.author) %><%= link_to_project(news.project) + ': ' unless news.project == @project %>
         <%= link_to h(news.title), news_path(news) %>
         <%= "(#{l(:label_x_comments, :count => news.comments_count)})" if news.comments_count > 0 %></h3>
         <p class="author"><%= authoring news.created_on, news.author %></p>
diff --git a/app/views/news/show.html.erb b/app/views/news/show.html.erb
index d8d098cce..932bf0083 100644
--- a/app/views/news/show.html.erb
+++ b/app/views/news/show.html.erb
@@ -8,7 +8,7 @@
 <%= delete_link news_path(@news) if User.current.allowed_to?(:manage_news, @project) %>
 </div>
 
-<h2><%= avatar(@news.author, :size => "24") %> <%=h @news.title %></h2>
+<h2><%= avatar(@news.author) %> <%=h @news.title %></h2>
 
 <% if authorize_for('news', 'edit') %>
 <div id="edit-news" style="display:none;">
@@ -42,7 +42,7 @@
                               :title => l(:button_delete),
                               :class => 'icon-only icon-del' %>
     </div>
-    <h4><%= avatar(comment.author, :size => "24") %><%= authoring comment.created_on, comment.author %></h4>
+    <h4><%= avatar(comment.author) %><%= authoring comment.created_on, comment.author %></h4>
     <div class="wiki">
     <%= textilizable(comment.comments) %>
     </div>
diff --git a/app/views/repositories/_changeset.html.erb b/app/views/repositories/_changeset.html.erb
index 5809be241..393e03e56 100644
--- a/app/views/repositories/_changeset.html.erb
+++ b/app/views/repositories/_changeset.html.erb
@@ -2,7 +2,7 @@
 
 <div class="details">
   <h4>
-    <%= avatar(@changeset.user, :size => "24") %>
+    <%= avatar(@changeset.user) %>
     <%= authoring(@changeset.committed_on, @changeset.author) %>
   </h4>
   <% if @changeset.scmid.present? || @changeset.parents.present? || @changeset.children.present? %>