Define a default value as suggested by ariejan

babb6a1a · Wouter D'Haeseleer · 825d2d69 · babb6a1a
Commit babb6a1a authored May 03, 2012 by Wouter D'Haeseleer
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 2 deletions

app/helpers/application_helper.rb app/helpers/application_helper.rb +1 -2

No files found.
--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
 require 'digest/md5'
 module ApplicationHelper
-  def gravatar_icon(user_email, size = 40)
+  def gravatar_icon(user_email = '', size = 40)
    gravatar_host = request.ssl? ? "https://secure.gravatar.com" :  "http://www.gravatar.com"
-    user_email ||= ''
    user_email.strip!
    "#{gravatar_host}/avatar/#{Digest::MD5.hexdigest(user_email.downcase)}?s=#{size}&d=identicon"
  end