Fix new rubocop warnings

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

diff --git a/app/helpers/user_helper.rb b/app/helpers/user_helper.rb
index 1fc8e3ed2f1e0b6f82f6e325d0425c278a3d1e87..1064e95c7dd7b4bed74a6075b002be28223451aa 100644 (file)
--- a/app/helpers/user_helper.rb
+++ b/app/helpers/user_helper.rb
@@ -2,76 +2,111 @@ module UserHelper
   # User images
 
   def user_image(user, options = {})
   # User images
 
   def user_image(user, options = {})

-    options[:class] ||= "user_image"
+    options[:class] ||= "user_image border border-grey"

     options[:alt] ||= ""
 
     if user.image_use_gravatar
       user_gravatar_tag(user, options)
     options[:alt] ||= ""
 
     if user.image_use_gravatar
       user_gravatar_tag(user, options)

+    elsif user.avatar.attached?
+      user_avatar_variant_tag(user, { :resize_to_limit => [100, 100] }, options)

     else
     else

-      image_tag user.image.url(:large), options
+      image_tag "avatar_large.png", options.merge(:width => 100, :height => 100)

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

-    options[:class] ||= "user_thumbnail"
+    options[:class] ||= "user_thumbnail border border-grey"

     options[:alt] ||= ""
 
     if user.image_use_gravatar
     options[:alt] ||= ""
 
     if user.image_use_gravatar

-      user_gravatar_tag(user, options)
+      user_gravatar_tag(user, options.merge(:size => 50))
+    elsif user.avatar.attached?
+      user_avatar_variant_tag(user, { :resize_to_limit => [50, 50] }, options)

     else
     else

-      image_tag user.image.url(:small), options
+      image_tag "avatar_small.png", options.merge(:width => 50, :height => 50)

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

-    options[:class] ||= "user_thumbnail_tiny"
+    options[:class] ||= "user_thumbnail_tiny border border-grey"

     options[:alt] ||= ""
 
     if user.image_use_gravatar
     options[:alt] ||= ""
 
     if user.image_use_gravatar

-      user_gravatar_tag(user, options)
+      user_gravatar_tag(user, options.merge(:size => 50))
+    elsif user.avatar.attached?
+      user_avatar_variant_tag(user, { :resize_to_limit => [50, 50] }, options)

     else
     else

-      image_tag user.image.url(:small), options
+      image_tag "avatar_small.png", options.merge(:width => 50, :height => 50)

     end
   end
 
   def user_image_url(user, options = {})
     if user.image_use_gravatar
       user_gravatar_url(user, options)
     end
   end
 
   def user_image_url(user, options = {})
     if user.image_use_gravatar
       user_gravatar_url(user, options)

+    elsif user.avatar.attached?
+      polymorphic_url(user_avatar_variant(user, :resize_to_limit => [100, 100]), :host => Settings.server_url)

     else
     else

-      image_url(user.image.url(:large))
+      image_url("avatar_large.png")

     end
   end
 
   # External authentication support
 
   def openid_logo
     end
   end
 
   # External authentication support
 
   def openid_logo

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

   end
 
   def auth_button(name, provider, options = {})
     link_to(
   end
 
   def auth_button(name, provider, options = {})
     link_to(

-      image_tag("#{name}.png", :alt => t("users.login.auth_providers.#{name}.alt")),
+      image_tag("#{name}.svg", :alt => t("sessions.new.auth_providers.#{name}.alt"), :class => "rounded-3"),

       auth_path(options.merge(:provider => provider)),
       auth_path(options.merge(:provider => provider)),

+      :method => :post,

       :class => "auth_button",
       :class => "auth_button",

-      :title => t("users.login.auth_providers.#{name}.title")
+      :title => t("sessions.new.auth_providers.#{name}.title")

     )
   end
 
   private
 
     )
   end
 
   private
 

+  # Local avatar support
+  def user_avatar_variant_tag(user, variant_options, options)
+    if user.avatar.variable?
+      variant = user.avatar.variant(variant_options)
+      # https://stackoverflow.com/questions/61893089/get-metadata-of-active-storage-variant/67228171
+      if variant.processed?
+        metadata = variant.processed.send(:record).image.blob.metadata
+        if metadata["width"]
+          options[:width] = metadata["width"]
+          options[:height] = metadata["height"]
+        end
+      end
+      image_tag variant, options
+    else
+      image_tag user.avatar, options
+    end
+  end
+
+  def user_avatar_variant(user, options)
+    if user.avatar.variable?
+      user.avatar.variant(options)
+    else
+      user.avatar
+    end
+  end
+

   # 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)
   # 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)

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

     "#{request.protocol}www.gravatar.com/avatar/#{hash}.jpg?s=#{size}&d=#{u(default_image_url)}"
   end
 
   def user_gravatar_tag(user, options = {})
     url = user_gravatar_url(user, options)
     "#{request.protocol}www.gravatar.com/avatar/#{hash}.jpg?s=#{size}&d=#{u(default_image_url)}"
   end
 
   def user_gravatar_tag(user, options = {})
     url = user_gravatar_url(user, options)

-    options.delete(:size)
+    options[:height] = options[:width] = options.delete(:size) || 100

     image_tag url, options
   end
 end
     image_tag url, options
   end
 end