Pagy 6.0 upgrade - pagy_url_for - wrong number of arguments (given 3, expected 2) -

[benkoshy]

If you are upgrading to pagy 6 directly from 5.0 - and you are overriding pagy_url_for then you may face some problems:

module QuotesHelper 
  # overrriding this
  # => wrong number of arguments (given 3, expected 2)
  def pagy_url_for(pagy, page)
    page
  end
end

We need to account for the additional arguments in pagy.rb:

class pagy
  def pagy_url_for(pagy, page, absolute: false, html_escaped: false)
    # etc.
  end

So just pass them in as well:

The Solution

module QuotesHelper 
  # overrriding this  
  def pagy_url_for(pagy, page, **)
    page
  end
end

[benkoshy]

Solution is above in the question: #424 (comment)