@@ -783,7 +783,7 class Issue < ActiveRecord::Base | |||||
783 | # Users the issue can be assigned to |
|
783 | # Users the issue can be assigned to | |
784 | def assignable_users |
|
784 | def assignable_users | |
785 | users = project.assignable_users.to_a |
|
785 | users = project.assignable_users.to_a | |
786 | users << author if author |
|
786 | users << author if author && author.active? | |
787 | users << assigned_to if assigned_to |
|
787 | users << assigned_to if assigned_to | |
788 | users.uniq.sort |
|
788 | users.uniq.sort | |
789 | end |
|
789 | end |
@@ -1875,6 +1875,20 class IssueTest < ActiveSupport::TestCase | |||||
1875 | assert issue.assignable_users.include?(non_project_member) |
|
1875 | assert issue.assignable_users.include?(non_project_member) | |
1876 | end |
|
1876 | end | |
1877 |
|
1877 | |||
|
1878 | def test_assignable_users_should_not_include_anonymous_user | |||
|
1879 | issue = Issue.generate!(:author => User.anonymous) | |||
|
1880 | ||||
|
1881 | assert !issue.assignable_users.include?(User.anonymous) | |||
|
1882 | end | |||
|
1883 | ||||
|
1884 | def test_assignable_users_should_not_include_locked_user | |||
|
1885 | user = User.generate! | |||
|
1886 | issue = Issue.generate!(:author => user) | |||
|
1887 | user.lock! | |||
|
1888 | ||||
|
1889 | assert !issue.assignable_users.include?(user) | |||
|
1890 | end | |||
|
1891 | ||||
1878 | test "#assignable_users should include the current assignee" do |
|
1892 | test "#assignable_users should include the current assignee" do | |
1879 | user = User.generate! |
|
1893 | user = User.generate! | |
1880 | issue = Issue.generate!(:assigned_to => user) |
|
1894 | issue = Issue.generate!(:assigned_to => user) |
General Comments 0
You need to be logged in to leave comments.
Login now