Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

Should :page be a required parameter?

#38

Is there ever a case where you would call paginate without :page? If not, I say we raise an exception if you don't include it. Too many people getting tripped up.

Reported by Chris Wanstrath · June 21st, 2007 @ 06:33 AM

State: resolved
Milestone: none
Assigned to: Mislav Mislav

Activity

  1. Chris Wanstrath
    Chris Wanstrath
    • State changed from open to resolved

    (from [272]) Will Paginate: the ":page" option is now required for all paginating finders!! Its value can still be nil (it defaults to 1), but its presence is required or an ArgumentError is raised. This is because people forget to write ":page => params[:page]" in their controllers when calling paginating finders, so they get confused why are they stuck on the same page all the time. [#38 state:resolved]

    http://require.errtheblog.com/pl...

    June 21st, 2007 @ 06:33 AM

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.