| @@ -398,7 +398,7 private | |||||
| 398 | end |
|
398 | end | |
| 399 |
|
399 | |||
| 400 | @issue.project = @project |
|
400 | @issue.project = @project | |
| 401 | @issue.author = User.current |
|
401 | @issue.author ||= User.current | |
| 402 | # Tracker must be set before custom field values |
|
402 | # Tracker must be set before custom field values | |
| 403 | @issue.tracker ||= @project.trackers.find((params[:issue] && params[:issue][:tracker_id]) || params[:tracker_id] || :first) |
|
403 | @issue.tracker ||= @project.trackers.find((params[:issue] && params[:issue][:tracker_id]) || params[:tracker_id] || :first) | |
| 404 | if @issue.tracker.nil? |
|
404 | if @issue.tracker.nil? | |
| @@ -2534,6 +2534,18 class IssuesControllerTest < ActionController::TestCase | |||||
| 2534 | assert_equal 'This is the test_new issue', issue.subject |
|
2534 | assert_equal 'This is the test_new issue', issue.subject | |
| 2535 | end |
|
2535 | end | |
| 2536 |
|
2536 | |||
|
|
2537 | def test_update_edit_form_should_keep_issue_author | |||
|
|
2538 | @request.session[:user_id] = 3 | |||
|
|
2539 | xhr :put, :new, :project_id => 1, :id => 1, :issue => {:subject => 'Changed'} | |||
|
|
2540 | assert_response :success | |||
|
|
2541 | assert_equal 'text/javascript', response.content_type | |||
|
|
2542 | ||||
|
|
2543 | issue = assigns(:issue) | |||
|
|
2544 | assert_equal User.find(2), issue.author | |||
|
|
2545 | assert_equal 2, issue.author_id | |||
|
|
2546 | assert_not_equal User.current, issue.author | |||
|
|
2547 | end | |||
|
|
2548 | ||||
| 2537 | def test_update_edit_form_should_propose_transitions_based_on_initial_status |
|
2549 | def test_update_edit_form_should_propose_transitions_based_on_initial_status | |
| 2538 | @request.session[:user_id] = 2 |
|
2550 | @request.session[:user_id] = 2 | |
| 2539 | WorkflowTransition.delete_all |
|
2551 | WorkflowTransition.delete_all | |
General Comments 0
You need to be logged in to leave comments.
Login now