From a5fd84f00eece323b9b43e70cb8f386cd92b762d Mon Sep 17 00:00:00 2001 From: Jean-Philippe Lang Date: Sat, 12 Dec 2015 12:37:56 +0000 Subject: Don't include locked user in assignable users (#21477). git-svn-id: http://svn.redmine.org/redmine/trunk@14972 e93f8b46-1217-0410-a6f0-8f06a7374b81 --- app/models/issue.rb | 2 +- test/unit/issue_test.rb | 8 ++++++++ 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/app/models/issue.rb b/app/models/issue.rb index 3a063c9fd..ba7a9b159 100644 --- a/app/models/issue.rb +++ b/app/models/issue.rb @@ -810,7 +810,7 @@ class Issue < ActiveRecord::Base # Users the issue can be assigned to def assignable_users users = project.assignable_users.to_a - users << author if (author && !author.anonymous?) + users << author if author && author.active? users << assigned_to if assigned_to users.uniq.sort end diff --git a/test/unit/issue_test.rb b/test/unit/issue_test.rb index 40ee48bc4..852755473 100644 --- a/test/unit/issue_test.rb +++ b/test/unit/issue_test.rb @@ -1935,6 +1935,14 @@ class IssueTest < ActiveSupport::TestCase assert !issue.assignable_users.include?(User.anonymous) end + def test_assignable_users_should_not_include_locked_user + user = User.generate! + issue = Issue.generate!(:author => user) + user.lock! + + assert !issue.assignable_users.include?(user) + end + test "#assignable_users should include the current assignee" do user = User.generate! issue = Issue.generate!(:assigned_to => user) -- cgit v1.2.3