| @@ -418,7 +418,7 class Issue < ActiveRecord::Base | |||
|
|
418 | 418 | |
|
|
419 | 419 | if attrs['parent_issue_id'].present? |
|
|
420 | 420 | s = attrs['parent_issue_id'].to_s |
|
|
421 | unless (m = s.match(%r{\A#?(\d+)\z})) && Issue.visible(user).exists?(m[1]) | |
|
|
421 | unless (m = s.match(%r{\A#?(\d+)\z})) && (m[1] == parent_id.to_s || Issue.visible(user).exists?(m[1])) | |
|
|
422 | 422 | @invalid_parent_issue_id = attrs.delete('parent_issue_id') |
|
|
423 | 423 | end |
|
|
424 | 424 | end |
| @@ -2914,6 +2914,20 class IssuesControllerTest < ActionController::TestCase | |||
|
|
2914 | 2914 | assert_equal spent_hours_before + 2.5, issue.spent_hours |
|
|
2915 | 2915 | end |
|
|
2916 | 2916 | |
|
|
2917 | def test_put_update_should_preserve_parent_issue_even_if_not_visible | |
|
|
2918 | parent = Issue.generate!(:project_id => 1, :is_private => true) | |
|
|
2919 | issue = Issue.generate!(:parent_issue_id => parent.id) | |
|
|
2920 | assert !parent.visible?(User.find(3)) | |
|
|
2921 | @request.session[:user_id] = 3 | |
|
|
2922 | ||
|
|
2923 | get :edit, :id => issue.id | |
|
|
2924 | assert_select 'input[name=?][value=?]', 'issue[parent_issue_id]', parent.id.to_s | |
|
|
2925 | ||
|
|
2926 | put :update, :id => issue.id, :issue => {:subject => 'New subject', :parent_issue_id => parent.id.to_s} | |
|
|
2927 | assert_response 302 | |
|
|
2928 | assert_equal parent, issue.parent | |
|
|
2929 | end | |
|
|
2930 | ||
|
|
2917 | 2931 | def test_put_update_with_attachment_only |
|
|
2918 | 2932 | set_tmp_attachments_directory |
|
|
2919 | 2933 | |
General Comments 0
You need to be logged in to leave comments.
Login now