Move "per page" value to a const

quirkey · Dec 29, 2014 · 83da351 · 83da351
1 parent 7f72d11
commit 83da351
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/lib/resque/server/views/statuses.erb b/lib/resque/server/views/statuses.erb
@@ -46,7 +46,7 @@
 </table>
 
 <% unless @statuses.empty? %>
-  <%= partial :next_more, :start => @start, :size => @size, :per_page => 50 %>
+  <%= partial :next_more, :start => @start, :size => @size, :per_page => per_page %>
 <% end %>
 
 <%= status_poll(@start) %>

diff --git a/lib/resque/status_server.rb b/lib/resque/status_server.rb
@@ -5,12 +5,13 @@ module Resque
   module StatusServer
 
     VIEW_PATH = File.join(File.dirname(__FILE__), 'server', 'views')
+    PER_PAGE = 50
 
     def self.registered(app)
 
       app.get '/statuses' do
         @start = params[:start].to_i
-        @end = @start + (params[:per_page] || 50) - 1
+        @end = @start + (params[:per_page] || per_page) - 1
         @statuses = Resque::Plugins::Status::Hash.statuses(@start, @end)
         @size = Resque::Plugins::Status::Hash.count
         status_view(:statuses)
@@ -52,13 +53,17 @@ def self.registered(app)
         @polling = true
 
         @start = params[:start].to_i
-        @end = @start + (params[:per_page] || 50) - 1
+        @end = @start + (params[:per_page] || per_page) - 1
         @statuses = Resque::Plugins::Status::Hash.statuses(@start, @end)
         @size = Resque::Plugins::Status::Hash.count
         status_view(:statuses, {:layout => false})
       end
 
       app.helpers do
+        def per_page
+          PER_PAGE
+        end
+
         def status_view(filename, options = {}, locals = {})
           erb(File.read(File.join(::Resque::StatusServer::VIEW_PATH, "#{filename}.erb")), options, locals)
         end