summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEric Davis <edavis@littlestreamsoftware.com>2010-10-25 18:44:41 +0000
committerEric Davis <edavis@littlestreamsoftware.com>2010-10-25 18:44:41 +0000
commit22e3cba0b712150d7286ff9e641b0c05e028191a (patch)
tree81419a761413cbbcedb1ba14d6c13e165e721861
parent4b045badcfde2eb27b63023ca8f3c30334d7f00f (diff)
downloadredmine-22e3cba0b712150d7286ff9e641b0c05e028191a.tar.gz
redmine-22e3cba0b712150d7286ff9e641b0c05e028191a.zip
Allow changing the Priority when moving issues.
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@4291 e93f8b46-1217-0410-a6f0-8f06a7374b81
-rw-r--r--app/controllers/issue_moves_controller.rb2
-rw-r--r--app/views/issue_moves/new.rhtml5
-rw-r--r--test/functional/issue_moves_controller_test.rb13
3 files changed, 19 insertions, 1 deletions
diff --git a/app/controllers/issue_moves_controller.rb b/app/controllers/issue_moves_controller.rb
index 20da38755..028d4d6d7 100644
--- a/app/controllers/issue_moves_controller.rb
+++ b/app/controllers/issue_moves_controller.rb
@@ -54,7 +54,7 @@ class IssueMovesController < ApplicationController
def extract_changed_attributes_for_move(params)
changed_attributes = {}
- [:assigned_to_id, :status_id, :start_date, :due_date].each do |valid_attribute|
+ [:assigned_to_id, :status_id, :start_date, :due_date, :priority_id].each do |valid_attribute|
unless params[valid_attribute].blank?
changed_attributes[valid_attribute] = (params[valid_attribute] == 'none' ? nil : params[valid_attribute])
end
diff --git a/app/views/issue_moves/new.rhtml b/app/views/issue_moves/new.rhtml
index 2dc971df2..946ddab95 100644
--- a/app/views/issue_moves/new.rhtml
+++ b/app/views/issue_moves/new.rhtml
@@ -34,6 +34,11 @@
</p>
<p>
+ <label><%= l(:field_priority) %></label>
+ <%= select_tag('priority_id', "<option value=\"\">#{l(:label_no_change_option)}</option>" + options_from_collection_for_select(IssuePriority.all, :id, :name)) %>
+</p>
+
+<p>
<label><%= l(:field_start_date) %></label>
<%= text_field_tag 'start_date', '', :size => 10 %><%= calendar_for('start_date') %>
</p>
diff --git a/test/functional/issue_moves_controller_test.rb b/test/functional/issue_moves_controller_test.rb
index 7c4005767..62593ac37 100644
--- a/test/functional/issue_moves_controller_test.rb
+++ b/test/functional/issue_moves_controller_test.rb
@@ -40,6 +40,19 @@ class IssueMovesControllerTest < ActionController::TestCase
assert_equal 2, Issue.find(2).tracker_id
end
+ context "#create via bulk move" do
+ should "allow changing the issue priority" do
+ @request.session[:user_id] = 2
+ post :create, :ids => [1, 2], :priority_id => 6
+
+ assert_redirected_to :controller => 'issues', :action => 'index', :project_id => 'ecookbook'
+ assert_equal 6, Issue.find(1).priority_id
+ assert_equal 6, Issue.find(2).priority_id
+
+ end
+
+ end
+
def test_bulk_copy_to_another_project
@request.session[:user_id] = 2
assert_difference 'Issue.count', 2 do