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