Fixed: issue move by non-admin broken by r2726 (#3354).

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

Fixed: issue move by non-admin broken by r2726 (#3354).
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@2738 e93f8b46-1217-0410-a6f0-8f06a7374b81
566d0a6c · Jean-Philippe Lang · 09a613b0 · 566d0a6c · 566d0a6c
Commit 566d0a6c authored May 13, 2009 by Jean-Philippe Lang
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 5 deletions

issues_controller.rb app/controllers/issues_controller.rb +1 -1

issues_controller_test.rb test/functional/issues_controller_test.rb +4 -4

No files found.
--- a/app/controllers/issues_controller.rb
+++ b/app/controllers/issues_controller.rb
@@ -284,7 +284,7 @@ class IssuesController < ApplicationController
      # admin is allowed to move issues to any active (visible) project
      @allowed_projects = Project.find(:all, :conditions => Project.visible_by(User.current))
    else
-      User.current.memberships.each {|m| @allowed_projects << m.project if m.role.allowed_to?(:move_issues)}
+      User.current.memberships.each {|m| @allowed_projects << m.project if m.roles.detect {|r| r.allowed_to?(:move_issues)}}
    end
    @target_project = @allowed_projects.detect {|p| p.id.to_s == params[:new_project_id]} if params[:new_project_id]
    @target_project ||= @project    

--- a/test/functional/issues_controller_test.rb
+++ b/test/functional/issues_controller_test.rb
@@ -905,14 +905,14 @@ class IssuesControllerTest < Test::Unit::TestCase
  end
  def test_move_one_issue_to_another_project
-    @request.session[:user_id] = 1
+    @request.session[:user_id] = 2
    post :move, :id => 1, :new_project_id => 2
    assert_redirected_to :action => 'index', :project_id => 'ecookbook'
    assert_equal 2, Issue.find(1).project_id
  end
  def test_bulk_move_to_another_project
-    @request.session[:user_id] = 1
+    @request.session[:user_id] = 2
    post :move, :ids => [1, 2], :new_project_id => 2
    assert_redirected_to :action => 'index', :project_id => 'ecookbook'
    # Issues moved to project 2
@@ -924,7 +924,7 @@ class IssuesControllerTest < Test::Unit::TestCase
  end
  def test_bulk_move_to_another_tracker
-    @request.session[:user_id] = 1
+    @request.session[:user_id] = 2
    post :move, :ids => [1, 2], :new_tracker_id => 2
    assert_redirected_to :action => 'index', :project_id => 'ecookbook'
    assert_equal 2, Issue.find(1).tracker_id
@@ -932,7 +932,7 @@ class IssuesControllerTest < Test::Unit::TestCase
  end
  def test_bulk_copy_to_another_project
-    @request.session[:user_id] = 1
+    @request.session[:user_id] = 2
    assert_difference 'Issue.count', 2 do
      assert_no_difference 'Project.find(1).issues.count' do
        post :move, :ids => [1, 2], :new_project_id => 2, :copy_options => {:copy => '1'}