diff --git a/app/models/project.rb b/app/models/project.rb
index 0afb4bdda0..4d5e3c0513 100644
--- a/app/models/project.rb
+++ b/app/models/project.rb
@@ -550,7 +550,7 @@ class Project < ActiveRecord::Base
 
   # Returns a hash of project users/groups grouped by role
   def principals_by_role
-    memberships.includes(:principal, :roles).inject({}) do |h, m|
+    memberships.active.includes(:principal, :roles).inject({}) do |h, m|
       m.roles.each do |r|
         h[r] ||= []
         h[r] << m.principal
diff --git a/test/unit/project_test.rb b/test/unit/project_test.rb
index 2c23feb833..acd7f99677 100644
--- a/test/unit/project_test.rb
+++ b/test/unit/project_test.rb
@@ -475,6 +475,16 @@ class ProjectTest < ActiveSupport::TestCase
     assert principals_by_role[role].include?(group)
   end
 
+  def test_principals_by_role_should_only_return_active_users
+    group = Group.find(10)
+    Member.create!(:principal => group, :project_id => 1, :role_ids => [1])
+
+    principals_by_role = Project.find(1).principals_by_role
+    locked_user = User.find(5)
+    assert Project.find(1).memberships.map(&:principal).include?(locked_user)
+    assert_not principals_by_role.values.flatten.include?(locked_user)
+  end
+
   def test_rolled_up_trackers
     parent = Project.find(1)
     parent.trackers = Tracker.find([1, 2])