Return routing redirect :path interpolation #5877

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Contributor

avakhov commented Apr 17, 2012

This feature was implemented at 0bda6f1, documented by 1e26bda and removed at 0809c67.

But the description is still here: https://github.com/rails/rails/blob/master/actionpack/lib/action_dispatch/routing/redirection.rb#L75-79

I suggest to reimplement it (it's simple only 3 lines of code in this PR) or remove it from the documentation.

@joshk, @tenderlove what do yout think about it?

Contributor

jeremyf commented May 2, 2012

@avakhov Failed test rails/actionpack/test/dispatch/routing_test.rb:2500

Contributor

avakhov commented May 2, 2012

@jeremyf Thanks for the issue. I rebased this several times without test executions. But it likes that changes in routing that was recently broke this PR. Moreover this defect was fixed by @pixeltrix 3 days ago independently. 75df4c1

@avakhov avakhov closed this May 2, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment