From: Tom Hughes <tom@compton.nu>
Date: Wed, 21 Mar 2012 19:20:40 +0000 (+0000)
Subject: Move generation of role icons into a helper
X-Git-Tag: live~5689
X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/75300566e123c36599398cdd2e33f7b3e4e49808

Move generation of role icons into a helper
---

diff --git a/app/helpers/user_roles_helper.rb b/app/helpers/user_roles_helper.rb
new file mode 100644
index 000000000..369efca5a
--- /dev/null
+++ b/app/helpers/user_roles_helper.rb
@@ -0,0 +1,37 @@
+module UserRolesHelper
+  def role_icons(user)
+    UserRole::ALL_ROLES.reduce("".html_safe) { |s,r| s + " " + role_icon(user, r) }
+  end
+
+  def role_icon(user, role)
+    if @user and @user.administrator?
+      if user.has_role?(role)
+        image = "roles/#{role}.png"
+        alt = t("user.view.role.revoke.#{role}")
+        title = t("user.view.role.revoke.#{role}")
+        url = revoke_role_path(:display_name => user.display_name, :role => role)
+        confirm = t("user_role.revoke.are_you_sure", :name => user.display_name, :role => role)
+      else
+        image = "roles/blank_#{role}.png"
+        alt = t("user.view.role.grant.#{role}")
+        title = t("user.view.role.grant.#{role}")
+        url = grant_role_path(:display_name => user.display_name, :role => role)
+        confirm = t("user_role.grant.are_you_sure", :name => user.display_name, :role => role)
+      end
+    elsif user.has_role?(role) 
+      image = "roles/#{role}.png"
+      alt = t("user.view.role.#{role}")
+      title = t("user.view.role.#{role}")
+    end
+
+    if image
+      icon = image_tag(image, :size => "20x20", :border => 0, :alt => alt, :title => title)
+
+      if url
+        icon = link_to(icon, url, :method => :post, :confirm => confirm)
+      end
+    end
+
+    icon
+  end
+end
diff --git a/app/views/user/view.html.erb b/app/views/user/view.html.erb
index f661c9b85..0369401eb 100644
--- a/app/views/user/view.html.erb
+++ b/app/views/user/view.html.erb
@@ -1,18 +1,6 @@
 <%= user_image @this_user %>
 
-<h2><%= h(@this_user.display_name) %>
-
-<% UserRole::ALL_ROLES.each do |role| %>
-  <% if @user and @user.administrator? %>
-    <% if @this_user.has_role? role %>
-      <%= link_to image_tag("roles/#{role}.png", :size => "20x20", :border => 0, :alt => t("user.view.role.revoke.#{role}"), :title => t("user.view.role.revoke.#{role}")), revoke_role_path(:display_name => @this_user.display_name, :role => role), :method => :post, :confirm => t('user_role.revoke.are_you_sure', :name => @this_user.display_name, :role => role) %>
-    <% else %>
-      <%= link_to image_tag("roles/blank_#{role}.png", :size => "20x20", :border => 0, :alt => t("user.view.role.grant.#{role}"), :title => t("user.view.role.grant.#{role}")), grant_role_path(:display_name => @this_user.display_name, :role => role), :method => :post, :confirm => t('user_role.grant.are_you_sure', :name => @this_user.display_name, :role => role) %>
-    <% end %>
-  <% elsif @this_user.has_role? role %>
-    <%= image_tag("roles/#{role}.png", :size => "20x20", :border => 0, :alt => t("user.view.role.#{role}"), :title => t("user.view.role.#{role}")) %>
-  <% end %>
-<% end %></h2>
+<h2><%= @this_user.display_name %><%= role_icons(@this_user) %></h2>
 
 <div id="userinformation">
   <% if @user and @this_user.id == @user.id %>