Find visible issues only in ContextMenusController#issues.

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

Find visible issues only in ContextMenusController#issues.
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@4580 e93f8b46-1217-0410-a6f0-8f06a7374b81
d2cc2861 · Jean-Philippe Lang · e9a0e6db · d2cc2861 · d2cc2861
Commit d2cc2861 authored Dec 29, 2010 by Jean-Philippe Lang
Hide whitespace changes
Inline Side-by-side

Showing with 8 additions and 1 deletion

context_menus_controller.rb app/controllers/context_menus_controller.rb +2 -1

context_menus_controller_test.rb test/functional/context_menus_controller_test.rb +6 -0

No files found.
--- a/app/controllers/context_menus_controller.rb
+++ b/app/controllers/context_menus_controller.rb
@@ -2,7 +2,8 @@ class ContextMenusController < ApplicationController
  helper :watchers
  
  def issues
-    @issues = Issue.find_all_by_id(params[:ids], :include => :project)
+    @issues = Issue.visible.all(:conditions => {:id => params[:ids]}, :include => :project)
+    
    if (@issues.size == 1)
      @issue = @issues.first
      @allowed_statuses = @issue.new_statuses_allowed_to(User.current)

--- a/test/functional/context_menus_controller_test.rb
+++ b/test/functional/context_menus_controller_test.rb
@@ -102,4 +102,10 @@ class ContextMenusControllerTest < ActionController::TestCase
                                             :class => 'icon-del' }
  end
  
+  def test_context_menu_issue_visibility
+    get :issues, :ids => [1, 4]
+    assert_response :success
+    assert_template 'context_menu'
+    assert_equal [1], assigns(:issues).collect(&:id)
+  end
 end