Hide login from point lists

Author: nygrenh

app/views/participants/show.html.erb

@@ -7,7 +7,9 @@
 
   <section class="participant-entries">
     <ul class="nobullets">
-      <li>Username: <%= @user.login %></li>
+      <% if current_user.administrator? %>
+      <li>Username: <%= @user.login %> (shown to you beacause you're an admin)</li>
+      <% end %>
       <li>E-mail: <%= mail_to(@user.email, @user.email) %></li>
       <% for field in UserField.all %>
         <% if field.field_type != :html %>

app/views/points/index.csv.erb

@@ -1,6 +1,6 @@
 <% require 'portable_csv' %>
 <%= PortableCSV.generate(:force_quotes => true) do |csv|
-  arr = ["Username"]
+  arr = []
   arr += @user_fields.map(&:label) if @user_fields
   arr += @summary[:sheets].map {|sheet| sheet[:name] } + ["Total"]
   csv << arr
@@ -10,7 +10,7 @@
       @summary[:awarded_for_user_and_sheet][user.login][sheet[:name]].to_i
     end
     total = @summary[:total_for_user][user.login].to_i
-    arr = [user.login]
+    arr = []
     if @user_fields
       @user_fields.each do |field|
         user_field_value = user.user_field_values.find { |o| o.field_name == field.name }

app/views/points/index.html.erb

@@ -44,14 +44,7 @@
         </tr>
         <tr>
           <th></th>
-          <th>
-            Student<br />
-            <% unless params[:sort_by].blank? %>
-              [<%= link_to 'sort', organization_course_points_path(@organization, @course) %>]
-            <% else %>
-              [sort]
-            <% end %>
-          </th>
+          <th></th>
             <% if @user_fields %>
               <% @user_fields.each do |field| %>
                 <th><%= field.label %></th>
@@ -89,9 +82,9 @@
             <td><%= index + 1 %></td>
             <td>
               <% if current_user.id == user.id || can?(:teach, @course) %>
-                <%= link_to username, participant_path(user) %>
+                <%= link_to 'Profile', participant_path(user) %>
               <% else %>
-                <%= username %>
+                &nbsp;
               <% end  %>
             </td>
             <% if @user_fields %>

app/views/points/show.html.erb

@@ -8,7 +8,7 @@
       <% if params[:sort_by].blank? %>
         <%= link_to('Sort by points', organization_course_point_path(@organization, @course, @sheetname, sort_by: 'points', show_attempted: params[:show_attempted]),  class: "btn btn-primary")%>
       <% else %>
-        <%= link_to('Sort by username', organization_course_point_path(@organization, @course, @sheetname, show_attempted: params[:show_attempted]), class: "btn btn-primary") %>
+        <%= link_to('Sort by default order', organization_course_point_path(@organization, @course, @sheetname, show_attempted: params[:show_attempted]), class: "btn btn-primary") %>
       <% end %>
 
       <% if can? :refresh_gdocs_spreadsheet, @course %>
@@ -26,7 +26,7 @@
         <% if tailoring.show_exercise_names_in_points_table %>
           <tr>
             <th></th>
-            <th>Username</th>
+            <th></th>
             <% if @user_fields %>
               <% @user_fields.each do |field| %>
                 <th><%= field.label %></th>
@@ -60,9 +60,9 @@
               <td><%= index + 1 %></td>
               <td>
                 <% if current_user.id == user.id || can?(:teach, @course) %>
-                  <%= link_to user.login, participant_path(user) %>
+                  <%= link_to 'Profile', participant_path(user) %>
                 <% else %>
-                  <%= user.login %>
+                  &nbsp;
                 <% end %>
               </td>
               <% if @user_fields %>