| @@ -54,7 +54,7 class IssueMovesController < ApplicationController | |||||
| 54 |
|
54 | |||
| 55 | def extract_changed_attributes_for_move(params) |
|
55 | def extract_changed_attributes_for_move(params) | |
| 56 | changed_attributes = {} |
|
56 | changed_attributes = {} | |
| 57 | [:assigned_to_id, :status_id, :start_date, :due_date].each do |valid_attribute| |
|
57 | [:assigned_to_id, :status_id, :start_date, :due_date, :priority_id].each do |valid_attribute| | |
| 58 | unless params[valid_attribute].blank? |
|
58 | unless params[valid_attribute].blank? | |
| 59 | changed_attributes[valid_attribute] = (params[valid_attribute] == 'none' ? nil : params[valid_attribute]) |
|
59 | changed_attributes[valid_attribute] = (params[valid_attribute] == 'none' ? nil : params[valid_attribute]) | |
| 60 | end |
|
60 | end | |
| @@ -34,6 +34,11 | |||||
| 34 | </p> |
|
34 | </p> | |
| 35 |
|
35 | |||
| 36 | <p> |
|
36 | <p> | |
|
|
37 | <label><%= l(:field_priority) %></label> | |||
|
|
38 | <%= select_tag('priority_id', "<option value=\"\">#{l(:label_no_change_option)}</option>" + options_from_collection_for_select(IssuePriority.all, :id, :name)) %> | |||
|
|
39 | </p> | |||
|
|
40 | ||||
|
|
41 | <p> | |||
| 37 | <label><%= l(:field_start_date) %></label> |
|
42 | <label><%= l(:field_start_date) %></label> | |
| 38 | <%= text_field_tag 'start_date', '', :size => 10 %><%= calendar_for('start_date') %> |
|
43 | <%= text_field_tag 'start_date', '', :size => 10 %><%= calendar_for('start_date') %> | |
| 39 | </p> |
|
44 | </p> | |
| @@ -40,6 +40,19 class IssueMovesControllerTest < ActionController::TestCase | |||||
| 40 | assert_equal 2, Issue.find(2).tracker_id |
|
40 | assert_equal 2, Issue.find(2).tracker_id | |
| 41 | end |
|
41 | end | |
| 42 |
|
42 | |||
|
|
43 | context "#create via bulk move" do | |||
|
|
44 | should "allow changing the issue priority" do | |||
|
|
45 | @request.session[:user_id] = 2 | |||
|
|
46 | post :create, :ids => [1, 2], :priority_id => 6 | |||
|
|
47 | ||||
|
|
48 | assert_redirected_to :controller => 'issues', :action => 'index', :project_id => 'ecookbook' | |||
|
|
49 | assert_equal 6, Issue.find(1).priority_id | |||
|
|
50 | assert_equal 6, Issue.find(2).priority_id | |||
|
|
51 | ||||
|
|
52 | end | |||
|
|
53 | ||||
|
|
54 | end | |||
|
|
55 | ||||
| 43 | def test_bulk_copy_to_another_project |
|
56 | def test_bulk_copy_to_another_project | |
| 44 | @request.session[:user_id] = 2 |
|
57 | @request.session[:user_id] = 2 | |
| 45 | assert_difference 'Issue.count', 2 do |
|
58 | assert_difference 'Issue.count', 2 do | |
General Comments 0
You need to be logged in to leave comments.
Login now