Commit 41b6cfcf authored by Mike Greiling's avatar Mike Greiling

automatically redirect urls containing author_id or assignee_id

parent ff75bd04
...@@ -23,6 +23,8 @@ class Projects::IssuesController < Projects::ApplicationController ...@@ -23,6 +23,8 @@ class Projects::IssuesController < Projects::ApplicationController
respond_to :html respond_to :html
def index def index
return redirect_to_fixed_params if params[:assignee_id].present? || params[:author_id].present?
@issues = issues_collection @issues = issues_collection
@issues = @issues.page(params[:page]) @issues = @issues.page(params[:page])
if @issues.out_of_range? && @issues.total_pages != 0 if @issues.out_of_range? && @issues.total_pages != 0
...@@ -211,6 +213,22 @@ class Projects::IssuesController < Projects::ApplicationController ...@@ -211,6 +213,22 @@ class Projects::IssuesController < Projects::ApplicationController
end end
end end
def redirect_to_fixed_params
fixed_params = params.except(:assignee_id, :author_id)
if params[:assignee_id].present?
assignee = User.find_by_id(params[:assignee_id])
fixed_params.merge!(assignee_username: assignee.username) if assignee
end
if params[:author_id].present?
author = User.find_by_id(params[:author_id])
fixed_params.merge!(author_username: author.username) if author
end
redirect_to url_for(fixed_params)
end
# Since iids are implemented only in 6.1 # Since iids are implemented only in 6.1
# user may navigate to issue page using old global ids. # user may navigate to issue page using old global ids.
# #
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment