Don't duplicate users in Issue#assignable_users. From r4240

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@4241 e93f8b46-1217-0410-a6f0-8f06a7374b81

Don't duplicate users in Issue#assignable_users. From r4240
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@4241 e93f8b46-1217-0410-a6f0-8f06a7374b81
73f12765 · Eric Davis · 7798e1b1 · 73f12765 · 73f12765
Commit 73f12765 authored Oct 07, 2010 by Eric Davis
Hide whitespace changes
Inline Side-by-side

Showing with 10 additions and 1 deletion

issue.rb app/models/issue.rb +1 -1

issue_test.rb test/unit/issue_test.rb +9 -0

No files found.
--- a/app/models/issue.rb
+++ b/app/models/issue.rb
@@ -392,7 +392,7 @@ class Issue < ActiveRecord::Base
  def assignable_users
    users = project.assignable_users
    users << author if author
-    users.sort
+    users.uniq.sort
  end
  
  # Versions that the issue can be assigned to

--- a/test/unit/issue_test.rb
+++ b/test/unit/issue_test.rb
@@ -545,6 +545,15 @@ class IssueTest < ActiveSupport::TestCase

      assert issue.assignable_users.include?(non_project_member)
    end
+
+    should "not show the issue author twice" do
+      assignable_user_ids = Issue.find(1).assignable_users.collect(&:id)
+      assert_equal 2, assignable_user_ids.length
+      
+      assignable_user_ids.each do |user_id|
+        assert_equal 1, assignable_user_ids.count(user_id), "User #{user_id} appears more or less than once"
+      end
+    end
  end
  
  def test_create_should_send_email_notification