Merge pull request #454 from commercekitchen/users_table

Users table

app/assets/stylesheets/components/_sortable_list.scss

@@ -36,6 +36,7 @@
   display: table-cell;
   width: 15%;
   vertical-align: middle;
+
   @media (max-width: $breakpoint-mobile-large) {
     display: block;
   }

app/assets/stylesheets/components/_users_table.scss

@@ -0,0 +1,13 @@
+.users-table {
+  width: 100%;
+  table-layout: fixed;
+
+  th,
+  td {
+    padding: 10px 10px;
+  }
+
+  td {
+    word-wrap: break-word;
+  }
+}

app/controllers/admin/users_controller.rb

@@ -50,7 +50,7 @@ def export_user_info
 
     def users_csv(users)
       CSV.generate do |csv|
-        csv << ['User Name', 'User Last Name', 'User Email', 'User Role', 'Registration Date', 'Branch', 'Zip Code', 'Courses User has Started', 'Courses User has Completed']
+        csv << ['User Name', 'User Last Name', 'User Email', 'User Role', 'Preferred Language', 'Registration Date', 'Branch', 'Zip Code', 'Courses User has Started', 'Courses User has Completed']
 
         users.each do |user|
           row = []
@@ -60,6 +60,7 @@ def users_csv(users)
           row << profile.try(:last_name)
           row << user.email
           row << user.roles.map(&:name).map(&:capitalize).join(', ')
+          row << user.preferred_language
           row << user.created_at.in_time_zone('Central Time (US & Canada)').strftime('%m-%d-%Y')
           row << profile.try(:library_location).try(:name)
           row << profile.try(:zip_code)

app/models/organization.rb

@@ -79,6 +79,16 @@ def clean_up_paperclip_errors
     errors.delete(:footer_logo_file_size)
   end
 
+  def assignable_roles
+    default_roles = ['Admin', 'User', 'Trainer']
+
+    if student_programs?
+      default_roles + ['Student', 'Parent']
+    else
+      default_roles
+    end
+  end
+
   def self.pla
     find_by(subdomain: 'www')
   end

app/models/user.rb

@@ -44,8 +44,6 @@ class User < ApplicationRecord
   # Expose some information from profile
   delegate :library_location_name, :library_location_zipcode, to: :profile, allow_nil: true
 
-  ROLES = %w[Admin Trainer User Parent Student].freeze
-
   ### Devise overrides to allow library card number login
   # TODO: Pull this into a concern
 

app/views/shared/_users_table.html.erb

@@ -1,8 +1,13 @@
 <table class="users-table">
+  <colgroup>
+    <col span="1" style="width: 40%;">
+    <col span="1" style="width: 40%;">
+    <col span="1" style="width: 20%;">
+  </colgroup>
+
   <tr>
     <th>Name</th>
     <th><%= t('login_signup.email') %></th>
-    <th>Language</th>
     <th>User Role</th>
   </tr>
   <% users.each do |user| %>
@@ -13,12 +18,9 @@
       <td>
         <%= user.email %>
       </td>
-      <td>
-        <%= user.preferred_language %>
-      </td>
       <td>
         <% if current_user.has_role?(:admin, current_organization) %>
-          <%= select_tag "user_#{user.id}", options_for_select(User::ROLES, user.current_roles.capitalize), data: { user_id: user.id, role: user.current_roles }, class: "user_role small narrow", method: :patch %>
+          <%= select_tag "user_#{user.id}", options_for_select(current_organization.assignable_roles, user.current_roles.capitalize), data: { user_id: user.id, role: user.current_roles }, class: "user_role small narrow", method: :patch %>
         <% else %>
           <%= user.current_roles.empty? ? "user" : user.current_roles %>
         <% end %>

spec/models/organization_spec.rb

@@ -62,6 +62,17 @@
     end
   end
 
+  describe '#assignable_roles' do
+    it 'returns correct options for typical organization' do
+      expect(org.assignable_roles).to contain_exactly('Admin', 'User', 'Trainer')
+    end
+
+    it 'returns correct options for program organization with student programs' do
+      FactoryBot.create(:program, :student_program, organization: org)
+      expect(org.assignable_roles).to contain_exactly('Admin', 'User', 'Trainer', 'Parent', 'Student')
+    end
+  end
+
   describe '#pla' do
     it 'returns PLA organization' do
       expect(Organization.pla).to eq(pla)