| @@ -451,6 +451,7 class Issue < ActiveRecord::Base | |||||
| 451 | def assignable_users |
|
451 | def assignable_users | |
| 452 | users = project.assignable_users |
|
452 | users = project.assignable_users | |
| 453 | users << author if author |
|
453 | users << author if author | |
|
|
454 | users << assigned_to if assigned_to | |||
| 454 | users.uniq.sort |
|
455 | users.uniq.sort | |
| 455 | end |
|
456 | end | |
| 456 |
|
457 | |||
| @@ -693,6 +693,15 class IssueTest < ActiveSupport::TestCase | |||||
| 693 | assert issue.assignable_users.include?(non_project_member) |
|
693 | assert issue.assignable_users.include?(non_project_member) | |
| 694 | end |
|
694 | end | |
| 695 |
|
695 | |||
|
|
696 | should "include the current assignee" do | |||
|
|
697 | project = Project.find(1) | |||
|
|
698 | user = User.generate! | |||
|
|
699 | issue = Issue.generate_for_project!(project, :assigned_to => user) | |||
|
|
700 | user.lock! | |||
|
|
701 | ||||
|
|
702 | assert Issue.find(issue.id).assignable_users.include?(user) | |||
|
|
703 | end | |||
|
|
704 | ||||
| 696 | should "not show the issue author twice" do |
|
705 | should "not show the issue author twice" do | |
| 697 | assignable_user_ids = Issue.find(1).assignable_users.collect(&:id) |
|
706 | assignable_user_ids = Issue.find(1).assignable_users.collect(&:id) | |
| 698 | assert_equal 2, assignable_user_ids.length |
|
707 | assert_equal 2, assignable_user_ids.length | |
General Comments 0
You need to be logged in to leave comments.
Login now