| @@ -425,7 +425,7 class IssuesController < ApplicationController | |||||
| 425 | @issue.start_date ||= Date.today if Setting.default_issue_start_date_to_creation_date? |
|
425 | @issue.start_date ||= Date.today if Setting.default_issue_start_date_to_creation_date? | |
| 426 |
|
426 | |||
| 427 | if attrs = params[:issue].deep_dup |
|
427 | if attrs = params[:issue].deep_dup | |
| 428 | if params[:was_default_status] == attrs[:status_id] |
|
428 | if action_name == 'new' && params[:was_default_status] == attrs[:status_id] | |
| 429 | attrs.delete(:status_id) |
|
429 | attrs.delete(:status_id) | |
| 430 | end |
|
430 | end | |
| 431 | @issue.safe_attributes = attrs |
|
431 | @issue.safe_attributes = attrs | |
| @@ -2605,6 +2605,20 class IssuesControllerTest < ActionController::TestCase | |||||
| 2605 | assert_equal 'Copy', issue.subject |
|
2605 | assert_equal 'Copy', issue.subject | |
| 2606 | end |
|
2606 | end | |
| 2607 |
|
2607 | |||
|
|
2608 | def test_create_as_copy_should_allow_status_to_be_set_to_default | |||
|
|
2609 | copied = Issue.generate! :status_id => 2 | |||
|
|
2610 | assert_equal 2, copied.reload.status_id | |||
|
|
2611 | ||||
|
|
2612 | @request.session[:user_id] = 2 | |||
|
|
2613 | assert_difference 'Issue.count' do | |||
|
|
2614 | post :create, :project_id => 1, :copy_from => copied.id, | |||
|
|
2615 | :issue => {:project_id => '1', :tracker_id => '1', :status_id => '1'}, | |||
|
|
2616 | :was_default_status => '1' | |||
|
|
2617 | end | |||
|
|
2618 | issue = Issue.order('id DESC').first | |||
|
|
2619 | assert_equal 1, issue.status_id | |||
|
|
2620 | end | |||
|
|
2621 | ||||
| 2608 | def test_create_as_copy_should_copy_attachments |
|
2622 | def test_create_as_copy_should_copy_attachments | |
| 2609 | @request.session[:user_id] = 2 |
|
2623 | @request.session[:user_id] = 2 | |
| 2610 | issue = Issue.find(3) |
|
2624 | issue = Issue.find(3) | |
General Comments 0
You need to be logged in to leave comments.
Login now