From 3c005719aeeb73a8a6ac02aa99eee5426c558294 Mon Sep 17 00:00:00 2001
From: Jean-Philippe Lang <jp_lang@yahoo.fr>
Date: Sat, 7 Jan 2017 10:28:59 +0000
Subject: [PATCH] Sort user memberships by project.

git-svn-id: http://svn.redmine.org/redmine/trunk@16150 e93f8b46-1217-0410-a6f0-8f06a7374b81
---
 app/models/member.rb                            | 4 ++++
 app/views/principal_memberships/_index.html.erb | 6 ++++--
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/app/models/member.rb b/app/models/member.rb
index 0b73def6c..06b90bf55 100644
--- a/app/models/member.rb
+++ b/app/models/member.rb
@@ -37,6 +37,10 @@ class Member < ActiveRecord::Base
       reorder("#{Role.table_name}.position").
       order(Principal.fields_for_order_statement)
   }
+  scope :sorted_by_project, lambda {
+    includes(:project).
+      reorder("#{Project.table_name}.lft")
+  }
 
   alias :base_reload :reload
   def reload(*args)
diff --git a/app/views/principal_memberships/_index.html.erb b/app/views/principal_memberships/_index.html.erb
index b451f536b..e43419072 100644
--- a/app/views/principal_memberships/_index.html.erb
+++ b/app/views/principal_memberships/_index.html.erb
@@ -1,6 +1,8 @@
+<% memberships = principal.memberships.preload(:member_roles => :role).sorted_by_project.to_a %>
+
 <p><%= link_to l(:label_add_projects), new_principal_membership_path(principal), :remote => true, :class => "icon icon-add" %></p>
 
-<% if principal.memberships.any? %>
+<% if memberships.any? %>
 <table class="list memberships">
   <thead><tr>
     <th><%= l(:label_project) %></th>
@@ -9,7 +11,7 @@
       <%= call_table_header_hook principal %>
   </tr></thead>
   <tbody>
-  <% principal.memberships.preload(:member_roles => :role).each do |membership| %>
+  <% memberships.each do |membership| %>
   <% next if membership.new_record? %>
   <tr id="member-<%= membership.id %>" class="<%= cycle 'odd', 'even' %> class">
   <td class="project name">
-- 
2.39.5