Defect #31552
View switches from gantt to list after editing an issue
Status: | Closed | Start date: | ||
---|---|---|---|---|
Priority: | Normal | Due date: | ||
Assignee: | % Done: | 0% | ||
Category: | Gantt | |||
Target version: | 4.0.5 | |||
Resolution: | Fixed | Affected version: |
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
Related issues
Associated revisions
View switches from gantt to list after editing an issue (#31552).
Patch by Mizuki ISHIKAWA.
History
#1
Updated by Go MAEDA over 1 year ago
- Status changed from New to Confirmed
#2
Updated by Mizuki ISHIKAWA over 1 year ago
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.
#3
Updated by Mizuki ISHIKAWA over 1 year ago
- File add-back-url-parameter.patch
added
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.
#4
Updated by Go MAEDA over 1 year ago
- Target version set to 4.0.5
Mizuki ISHIKAWA wrote:
The issues/_list also make the back_url parameter in the same way.
source:tags/4.0.4/app/views/issues/_list.html.erb#L5 and source:tags/4.0.4/app/views/timelog/_list.html.erb#L2.
Setting the target version to 4.0.5.
#5
Updated by Go MAEDA over 1 year ago
- Status changed from Confirmed to Resolved
- Assignee set to Go MAEDA
Committed the patch. Thank you all for reporting and fixing this issue.
#6
Updated by Go MAEDA over 1 year ago
- Status changed from Resolved to Closed
- Resolution set to Fixed
#7
Updated by Go MAEDA over 1 year ago
- Related to Defect #31831: Back url parse in validation added