bug with url_for #36

Closed
mumualex opened this Issue Nov 6, 2013 · 2 comments

Comments

Projects
None yet
2 participants

mumualex commented Nov 6, 2013

I try to use Kaminari pagination with rel_next_prev_link_tags helper.
But I've a routing error "No route matches {:page=>2, :locale=>nil, :controller=>"properties"}"
See https://github.com/amatsuda/kaminari/blob/master/lib/kaminari/helpers/action_view_extension.rb

This bug appear when url_for(params.merge) is used... and it works when I overwrite with

<link rel="next" href="<%= properties_path(Kaminari.config.param_name => (@properties.current_page + 1)) %> '"/>

Thanks for your help

Collaborator

aimerickdesdoit commented Mar 11, 2014

Hi,

After a quick test, it seems that the problem is fixed because I can not reproduce the bug.
Can you confirm that on your side everything is correct ?

Here is a diff file with a test on url_for http://cl.ly/code/0j3h3v1x2X0q.

It's seem to be ok, thanks for your investigation :)

mumualex closed this Mar 11, 2014

aimerickdesdoit self-assigned this Mar 11, 2014

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