diff --git a/app/views/projects/settings/_members.html.erb b/app/views/projects/settings/_members.html.erb
index 615bcc65a..f69f9edff 100644
--- a/app/views/projects/settings/_members.html.erb
+++ b/app/views/projects/settings/_members.html.erb
@@ -1,9 +1,10 @@
 <%= error_messages_for 'member' %>
 <%
+   roles = Role.givable.all
    if @project.project_type == Project::ProjectType_course
-     roles = Role.givable.all[3..5]
+     roles = roles[3..5]
    else
-     roles = Role.givable.all[0..2]
+     roles = roles[0..2]
    end
    members = @project.member_principals.includes(:roles, :principal).all.sort
 %>