X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/39b60219fe37d1e3f94555a053895adc1eee179c..bb3b0acc57fbaa33c6a83562d9b90286ff9a1fd3:/app/helpers/user_roles_helper.rb

diff --git a/app/helpers/user_roles_helper.rb b/app/helpers/user_roles_helper.rb
index 438076592..a8bc06b6e 100644
--- a/app/helpers/user_roles_helper.rb
+++ b/app/helpers/user_roles_helper.rb
@@ -1,8 +1,6 @@
 module UserRolesHelper
   def role_icons(user)
-    UserRole::ALL_ROLES.reduce("".html_safe) do |acc, elem|
-      acc + " " + role_icon(user, elem)
-    end
+    safe_join(UserRole::ALL_ROLES.filter_map { |role| role_icon(user, role) }, " ")
   end
 
   def role_icon(user, role)
@@ -27,10 +25,10 @@ module UserRolesHelper
     end
 
     if image
-      svg_icon = tag("source", :srcset => image_path("#{image}.svg"), :type => "image/svg+xml")
+      svg_icon = tag.source(:srcset => image_path("#{image}.svg"), :type => "image/svg+xml")
       png_icon = image_tag("#{image}.png", :srcset => image_path("#{image}.svg"), :size => "20x20", :border => 0, :alt => alt, :title => title)
       icon = tag.picture(svg_icon + png_icon)
-      icon = link_to(icon, url, :method => :post, :confirm => confirm) if url
+      icon = link_to(icon, url, :method => :post, :data => { :confirm => confirm }) if url
     end
 
     icon