@@ -151,7 +151,13 class IssuesController < ApplicationController | |||||
151 | return |
|
151 | return | |
152 | else |
|
152 | else | |
153 | respond_to do |format| |
|
153 | respond_to do |format| | |
154 |
format.html { |
|
154 | format.html { | |
|
155 | if @issue.project.nil? | |||
|
156 | render_error :status => 422 | |||
|
157 | else | |||
|
158 | render :action => 'new' | |||
|
159 | end | |||
|
160 | } | |||
155 | format.api { render_validation_errors(@issue) } |
|
161 | format.api { render_validation_errors(@issue) } | |
156 | end |
|
162 | end | |
157 | end |
|
163 | end | |
@@ -484,7 +490,7 class IssuesController < ApplicationController | |||||
484 | render_error l(:error_no_default_issue_status) |
|
490 | render_error l(:error_no_default_issue_status) | |
485 | return false |
|
491 | return false | |
486 | end |
|
492 | end | |
487 | else |
|
493 | elsif request.get? | |
488 | render_error :message => l(:error_no_projects_with_tracker_allowed_for_new_issue), :status => 403 |
|
494 | render_error :message => l(:error_no_projects_with_tracker_allowed_for_new_issue), :status => 403 | |
489 | return false |
|
495 | return false | |
490 | end |
|
496 | end |
@@ -2558,7 +2558,7 class IssuesControllerTest < Redmine::ControllerTest | |||||
2558 | :issue => {:project_id => 3, |
|
2558 | :issue => {:project_id => 3, | |
2559 | :tracker_id => 2, |
|
2559 | :tracker_id => 2, | |
2560 | :subject => 'Foo'} |
|
2560 | :subject => 'Foo'} | |
2561 |
assert_response 4 |
|
2561 | assert_response 422 | |
2562 | end |
|
2562 | end | |
2563 | end |
|
2563 | end | |
2564 |
|
2564 |
General Comments 0
You need to be logged in to leave comments.
Login now