remove side effects on :controller option in #url_for #6375
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Bug Description:
If the
:controller
option in an url_for call contains a leading slash itwill be removed, which causes side-effects:
The side-effects then affect plugins like will_paginate, which store the options to pass to
url_for
in an instance_variable. The functionality then breaks after the first call tourl_for
.Resolution:
The route-set does not directly manipulate the string passed into
#url_for
.It creates a copy without the leading slash if necessary.