Merge remote-tracking branch 'upstream/pull/2014'

[rails.git] / app / helpers / user_helper.rb

diff --git a/app/helpers/user_helper.rb b/app/helpers/user_helper.rb
index 55b1d21ab63b89839b754e4ab870b11aa5262bf1..1fc8e3ed2f1e0b6f82f6e325d0425c278a3d1e87 100644 (file)
--- a/app/helpers/user_helper.rb
+++ b/app/helpers/user_helper.rb
@@ -3,6 +3,7 @@ module UserHelper
 
   def user_image(user, options = {})
     options[:class] ||= "user_image"
 
   def user_image(user, options = {})
     options[:class] ||= "user_image"

+    options[:alt] ||= ""

 
     if user.image_use_gravatar
       user_gravatar_tag(user, options)
 
     if user.image_use_gravatar
       user_gravatar_tag(user, options)


@@ -13,6 +14,7 @@ module UserHelper
 
   def user_thumbnail(user, options = {})
     options[:class] ||= "user_thumbnail"
 
   def user_thumbnail(user, options = {})
     options[:class] ||= "user_thumbnail"

+    options[:alt] ||= ""

 
     if user.image_use_gravatar
       user_gravatar_tag(user, options)
 
     if user.image_use_gravatar
       user_gravatar_tag(user, options)


@@ -23,6 +25,7 @@ module UserHelper
 
   def user_thumbnail_tiny(user, options = {})
     options[:class] ||= "user_thumbnail_tiny"
 
   def user_thumbnail_tiny(user, options = {})
     options[:class] ||= "user_thumbnail_tiny"

+    options[:alt] ||= ""

 
     if user.image_use_gravatar
       user_gravatar_tag(user, options)
 
     if user.image_use_gravatar
       user_gravatar_tag(user, options)


@@ -35,33 +38,35 @@ module UserHelper
     if user.image_use_gravatar
       user_gravatar_url(user, options)
     else
     if user.image_use_gravatar
       user_gravatar_url(user, options)
     else

-      image_url(user.image.url)
+      image_url(user.image.url(:large))

     end
   end
 
     end
   end
 

-  # OpenID support
+  # External authentication support

 
   def openid_logo
 
   def openid_logo

-    image_tag "openid_small.png", :alt => t('user.login.openid_logo_alt'), :class => "openid_logo"
+    image_tag "openid_small.png", :alt => t("users.login.openid_logo_alt"), :class => "openid_logo"

   end
 
   end
 

-  def openid_button(name, url)
+  def auth_button(name, provider, options = {})

     link_to(
     link_to(

-      image_tag("#{name}.png", :alt => t("user.login.openid_providers.#{name}.alt")),
-      "#",
-      :class => "openid_button", :data => { :url => url },
-      :title => t("user.login.openid_providers.#{name}.title")
+      image_tag("#{name}.png", :alt => t("users.login.auth_providers.#{name}.alt")),
+      auth_path(options.merge(:provider => provider)),
+      :class => "auth_button",
+      :title => t("users.login.auth_providers.#{name}.title")

     )
   end
 
     )
   end
 

+  private
+

   # Gravatar support
 
   # See http://en.gravatar.com/site/implement/images/ for details.
   def user_gravatar_url(user, options = {})
     size = options[:size] || 100
   # Gravatar support
 
   # See http://en.gravatar.com/site/implement/images/ for details.
   def user_gravatar_url(user, options = {})
     size = options[:size] || 100

-    hash = Digest::MD5::hexdigest(user.email.downcase)
+    hash = Digest::MD5.hexdigest(user.email.downcase)

     default_image_url = image_url("users/images/large.png")
     default_image_url = image_url("users/images/large.png")

-    url = "http://www.gravatar.com/avatar/#{hash}.jpg?s=#{size}&d=#{u(default_image_url)}"
+    "#{request.protocol}www.gravatar.com/avatar/#{hash}.jpg?s=#{size}&d=#{u(default_image_url)}"

   end
 
   def user_gravatar_tag(user, options = {})
   end
 
   def user_gravatar_tag(user, options = {})