From 634a869588b9eb9e91d540ff18bd113ea6fe387c Mon Sep 17 00:00:00 2001
From: MAEDA Go <maeda@farend.jp>
Date: Wed, 16 Aug 2017 00:50:05 +0900
Subject: [PATCH] Add avatar for anonymous user.

---
 app/helpers/application_helper.rb       |   8 +++++++-
 public/images/anonymous.png             | Bin 0 -> 1072 bytes
 test/helpers/application_helper_test.rb |   7 +++++--
 3 files changed, 12 insertions(+), 3 deletions(-)
 create mode 100644 public/images/anonymous.png

diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb
index cb0c27715..8d45ca7fd 100644
--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
@@ -1404,7 +1404,13 @@ module ApplicationHelper
       elsif user.to_s =~ %r{<(.+?)>}
         email = $1
       end
-      return gravatar(email.to_s.downcase, options) unless email.blank? rescue nil
+      if email.present?
+        gravatar(email.to_s.downcase, options) rescue nil
+      else
+        image_tag 'anonymous.png',
+                  GravatarHelper::DEFAULT_OPTIONS
+                    .except(:default, :rating, :ssl).merge(options)
+      end
     else
       ''
     end
diff --git a/public/images/anonymous.png b/public/images/anonymous.png
new file mode 100644
index 0000000000000000000000000000000000000000..5f00a8362b23277c1da58e0be91251586c129744
GIT binary patch
literal 1072
zcmV-01kd}4P)<h;3K|Lk000e1NJLTq004jh004jp0{{R3^x%>C0000PbVXQnQ*UN;
zcVTj606}DLVr3vnZDD6+Qe|Oed2z{QJOBUyGf+%aMegqIx3{;$!^7Iz+SAk1zrVlY
z;^MozyXxxd%gf8==H|!8$JW-?-rnBN&(HGm^0TwELdmxJ000A%Nkl<Zc-ril+qRq_
z42HuQMcusr)3)6`h#DY(u-jE~H)|a8O-OzqfsZdXm;e)C0{puGtB{WWG`Z=)0Csc!
zW_+{500j4_DFFijKpG!4CZRWgNVxw**t!B(?z5)WegJ}bipgF8izk|c9sr;`*%Z7F
zfcRI36k1EUJijvQ+W-&$m!gM<G%$D<;O^f%v{$RTRe-AelZ0pZn*iH2yt<yU4q(0d
zk0#hH5%g;S*X4~TxV<F`yaaGv0Kc@QN;v@B1vpNiZ%^7+0Ax&Rwfl}{j_?&^)B*rt
zxYYjH8Cmc@O>qeF00I~L{p51dIv`{L6qdlB3=coV0Zco<O4$6OIpF=?3?TFVc}xf>
zX$W;I5K+ACxm2wHkoPanhmjMjZ3NH-{?X+UjR3O1KMH#^0xWd@D@$i+9YFdb5bn&l
zf3F$=(nXN>ABsd305$%Cpi-{`aEkw)0B4Es{wn|i1!nyK60hE9wc4)?b$km7D|G;X
zAZF^@QmjdGy)*(O@_UiLEUA`TBY^ddrW*l1Od~+^iD+r_@0tPjv;#xmu4|9waVg@T
zM_wyaItc9mM>fB$sxCqbfX#OUfz{Rk>2ivvp~+s~Xl)t-fF%eFs|^6DL9yiD&1(Zd
z0zj7-@{G*B1b}J*0f5xGr?u%^%@WWWg){)9QJhUeVzZXmDSe*$b7jbg?P_XI`AHCj
z0J{~7k9PqO0+E^|Q2iGSES?05jn1*i{!2zS>(i0pocsFqFP#|yryW96$qbdyCsgO`
zVWW)A&TVp0q8Z?GflmVKvMsaqB9Er6MWfAQ{-TrX5R7tu*M`y60N~Imn2w%PUEnJ?
zCMCy+iSPDs!C4qe*pM)&%l!WO<GlvpNRC5cmt39IDEI5P2f0@U^peB=$EvlCr%srg
zPzHd{X*o$?@Jflz5rHat7xhiKkELj2nhWGJ*yT17P27Y|;0FXC$NzlM5@A>mQ$ACN
zst!GKQeV?vOP7~vMFSw;hdFmY`V)?G>Ms@wfbbnn<N++}3GlosKv{dd*?0~>_|7KM
z0Kj?!+|>YzhjgtbQvm$~@CX5n4!}JGK<$mc7=k!}@Iy>Q0o)h>7Xtu4$RGkBMgfQj
zfExwiA^>a@0K)-RKg?DXK#T(rD8Mle;6MTRH~@|UsBr)a1@HrH0(?0D{PzGe2PVJ-
qm;e)C0!)AjFaajOUj*<?fB^uT0nBx7R5{cD0000<MNUMnLSTaIg5~-E

literal 0
HcmV?d00001

diff --git a/test/helpers/application_helper_test.rb b/test/helpers/application_helper_test.rb
index 6a461c31a..936a0ba84 100644
--- a/test/helpers/application_helper_test.rb
+++ b/test/helpers/application_helper_test.rb
@@ -1302,6 +1302,7 @@ RAW
   end
 
   def test_avatar_enabled
+    tag_for_anonymous_re = %r{src="/images/anonymous.png(\?\d+)?"}
     with_settings :gravatar_enabled => '1' do
       assert avatar(User.find_by_mail('jsmith@somenet.foo')).include?(Digest::MD5.hexdigest('jsmith@somenet.foo'))
       assert avatar('jsmith <jsmith@somenet.foo>').include?(Digest::MD5.hexdigest('jsmith@somenet.foo'))
@@ -1313,8 +1314,10 @@ RAW
       # The default class of the img tag should be gravatar
       assert avatar('jsmith <jsmith@somenet.foo>').include?('class="gravatar"')
       assert !avatar('jsmith <jsmith@somenet.foo>', :class => 'picture').include?('class="gravatar"')
-      assert_nil avatar('jsmith')
-      assert_nil avatar(nil)
+      assert_match tag_for_anonymous_re, avatar('jsmith')
+      assert_match tag_for_anonymous_re, avatar(nil)
+      # Avatar for anonymous user
+      assert_match tag_for_anonymous_re, avatar(User.anonymous)
     end
   end
 
-- 
2.11.0 (Apple Git-81)