Defect #31552
closed
View switches from gantt to list after editing an issue
Added by Bernhard Rohloff about 5 years ago.
Updated about 5 years ago.
Description
The view is only affected if a custom filter is applied. If there's
no filter applied the view works as expected.
Steps to reproduce
- Open the gantt view.
- Modify and apply the filter.
- Edit an issue with the right click menu
Result
The view switches to the issues list view
Expected result
The view stays on the gantt view
Files
- Status changed from New to Confirmed
I have confirmed that the changes below fix this problem.
diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb
index 06e2d702c1..afbb30f3ee 100644
--- a/app/controllers/application_controller.rb
+++ b/app/controllers/application_controller.rb
@@ -440,7 +440,7 @@ class ApplicationController < ActionController::Base
end
begin
- uri = URI.parse(back_url)
+ uri = URI.parse(URI.encode(back_url))
rescue URI::InvalidURIError
return false
end
However, I am concerned that this change will cause other problems.
ApplicationController#validate_back_url is a method that includes security changes(#19577), so it needs to be corrected carefully.
I think that the patch attached is better than the correction method suggested in #31552#note-2.
The issues/_list also make the back_url parameter in the same way.
The same problem occurred with the calendar, so we fix it together.
- Target version set to 4.0.5
- Status changed from Confirmed to Resolved
- Assignee set to Go MAEDA
Committed the patch. Thank you all for reporting and fixing this issue.
- Status changed from Resolved to Closed
- Resolution set to Fixed
Also available in: Atom
PDF