about summary refs log tree commit diff
diff options
context:
space:
mode:
authorEugen Rochko <eugen@zeonfederated.com>2016-03-24 12:49:34 +0100
committerEugen Rochko <eugen@zeonfederated.com>2016-03-24 12:49:34 +0100
commit7e58303a8da90b31f4179695283d3ac19dfb09d1 (patch)
tree705a53da3f360379737293d234a52af1748ae312
parentc8999a116ef71bef2cb28a783e4ba3d8e3cea6fa (diff)
Limit Atom feeds to 20 items by default, add pagination by max_id
(But there are no Atom feed pagination elements yet)
-rw-r--r--app/controllers/accounts_controller.rb2
-rw-r--r--app/models/stream_entry.rb3
-rw-r--r--app/views/accounts/show.atom.ruby2
3 files changed, 4 insertions, 3 deletions
diff --git a/app/controllers/accounts_controller.rb b/app/controllers/accounts_controller.rb
index cfccd0f19..53f665238 100644
--- a/app/controllers/accounts_controller.rb
+++ b/app/controllers/accounts_controller.rb
@@ -7,7 +7,7 @@ class AccountsController < ApplicationController
   def show
     respond_to do |format|
       format.html { @statuses = @account.statuses.order('id desc').with_includes.with_counters.paginate(page: params[:page], per_page: 10)}
-      format.atom { @entries  = @account.stream_entries.order('id desc').with_includes }
+      format.atom { @entries  = @account.stream_entries.order('id desc').with_includes.paginate_by_max_id(20, params[:max_id] || nil) }
     end
   end
 
diff --git a/app/models/stream_entry.rb b/app/models/stream_entry.rb
index 68a33ccf7..d64edd62f 100644
--- a/app/models/stream_entry.rb
+++ b/app/models/stream_entry.rb
@@ -4,7 +4,8 @@ class StreamEntry < ActiveRecord::Base
 
   validates :account, :activity, presence: true
 
-  scope :with_includes, -> { includes(:activity) }
+  scope :with_includes,      -> { includes(:activity) }
+  scope :paginate_by_max_id, -> (limit, max_id) { order('id desc').limit(limit).where(max_id.nil? ? '1=1' : ['id < ?', max_id]) }
 
   def object_type
     orphaned? ? :activity : (targeted? ? :activity : self.activity.object_type)
diff --git a/app/views/accounts/show.atom.ruby b/app/views/accounts/show.atom.ruby
index e2ae57cc4..b7e3d2590 100644
--- a/app/views/accounts/show.atom.ruby
+++ b/app/views/accounts/show.atom.ruby
@@ -15,7 +15,7 @@ Nokogiri::XML::Builder.new do |xml|
     link_hub       xml, Rails.configuration.x.hub_url
     link_salmon    xml, api_salmon_url(@account.id)
 
-    @entries.order('id desc').each do |stream_entry|
+    @entries.each do |stream_entry|
       entry(xml, false) do
         include_entry xml, stream_entry
       end