Rails - Mar 2005 - Pagination routing syntax

I''ve got a pagination_links call that creates a link like this:

<%= pagination_links(@device_pages) %>

generates

http://URL/device/list?page=1

but

<%= pagination_links(@device_pages, :params => { "sort_by" =>
"foo" } ) %>

generates the same link above.  Instead, I''d like to have:

http://URL/device/list?page=1&sort_by=foo

Am I just misunderstaning how to structure my options to the pagination_links 
call?

Thanks,
Caleb

Hi Caleb,

Caleb Tennis wrote:
><%= pagination_links(@device_pages, :params => { "sort_by"
=> "foo" } ) %>
>
>generates the same link above.  Instead, I''d like to have:
>
>http://URL/device/list?page=1&sort_by=foo
>
>Am I just misunderstaning how to structure my options to the
pagination_links
>call?
>  
>
no, you don''t. It seems somebody just forgot to implement this part :) 
I''ve written a patch and found this ticket afterwards (should have 
checked before): http://dev.rubyonrails.com/ticket/947

However, it was missing one of the link_tos (see my second patch).

Until the next (0.11.1?) release of rails you can fix this yourself by 
putting the attached file into your project''s vendor directory and 
adding a "require ''pagination_fix''" at the bottom of
your
"config/environment.rb".

Sebastian



_______________________________________________
Rails mailing list
Rails-1W37MKcQCpIf0INCOvqR/iCwEArCW2h5@public.gmane.org
http://lists.rubyonrails.org/mailman/listinfo/rails