Use Kaminari configuration

Fixes rubysherpas#505

Author: mauriciopasquier

app/controllers/forem/application_controller.rb

@@ -9,6 +9,18 @@ def current_ability
     Forem::Ability.new(forem_user)
   end
 
+  # Kaminari defaults page_method_name to :page, will_paginate always uses
+  # :page
+  def pagination_method
+    defined?(Kaminari) ? Kaminari.config.page_method_name : :page
+  end
+
+  # Kaminari defaults param_name to :page, will_paginate always uses :page
+  def pagination_param
+    defined?(Kaminari) ? Kaminari.config.param_name : :page
+  end
+  helper_method :pagination_param
+
   private
 
   def authenticate_forem_user

app/controllers/forem/forums_controller.rb

@@ -17,7 +17,10 @@ def show
         @forum.topics.visible.approved_or_pending_review_for(forem_user)
       end
 
-      @topics = @topics.by_pinned_or_most_recent_post.page(params[:page]).per(Forem.per_page)
+      @topics = @topics.by_pinned_or_most_recent_post
+
+      # Kaminari allows to configure the method and param used
+      @topics = @topics.send(pagination_method, params[pagination_param]).per(Forem.per_page)
 
       respond_to do |format|
         format.html

app/controllers/forem/posts_controller.rb

@@ -57,14 +57,14 @@ def authorize_reply_for_topic!
     def authorize_edit_post_for_forum!
       authorize! :edit_post, @topic.forum
     end
-    
+
     def authorize_destroy_post_for_forum!
       authorize! :destroy_post, @topic.forum
     end
 
     def create_successful
       flash[:notice] = t("forem.post.created")
-      redirect_to forum_topic_url(@topic.forum, @topic, :page => @topic.last_page)
+      redirect_to forum_topic_url(@topic.forum, @topic, pagination_param => @topic.last_page)
     end
 
     def create_failed

app/controllers/forem/topics_controller.rb

@@ -8,7 +8,10 @@ class TopicsController < Forem::ApplicationController
     def show
       if find_topic
         register_view(@topic, forem_user)
-        @posts = find_posts(@topic).page(params[:page]).per(Forem.per_page)
+        @posts = find_posts(@topic)
+
+        # Kaminari allows to configure the method and param used
+        @posts = @posts.send(pagination_method, params[pagination_param]).per(Forem.per_page)
       end
     end
 

app/helpers/forem/topics_helper.rb

@@ -3,7 +3,7 @@ module TopicsHelper
     def link_to_latest_post(topic)
       post = relevant_posts(topic).last
       text = "#{time_ago_in_words(post.created_at)} #{t("ago_by")} #{post.user}"
-      link_to text, forem.forum_topic_path(post.topic.forum, post.topic, :anchor => "post-#{post.id}", :page => topic.last_page)
+      link_to text, forem.forum_topic_path(post.topic.forum, post.topic, :anchor => "post-#{post.id}", pagination_param => topic.last_page)
     end
 
     def new_since_last_view_text(topic)