Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

remove side effects on :controller option in #url_for #6375

Closed
wants to merge 1 commit into from
Closed

remove side effects on :controller option in #url_for #6375

wants to merge 1 commit into from

Commits on May 18, 2012

  1. remove side effects on :controller option in #url_for 

    Bug Description:

If the :controller option in an url_for call contains a leading slash it
will be removed, which causes side-effects:

my_controller = '/Example'
url_for(:controller => my_controller)
puts my_controller # => 'Example'

Resolution:

The route-set does not directly manipulate the string passed into #url_for.
It creates a copy without the leading slash if necessary.
    Yves Senn committed May 18, 2012
    Configuration menu
    Copy the full SHA
    3edf53f View commit details
    Browse the repository at this point in the history