about summary refs log tree commit diff
path: root/app/models/feed.rb
diff options
context:
space:
mode:
authorEugen Rochko <eugen@zeonfederated.com>2016-03-22 21:38:47 +0100
committerEugen Rochko <eugen@zeonfederated.com>2016-03-22 21:38:47 +0100
commit12559b01eae408c0f3e820ef9c6bc6e356ab889a (patch)
tree87d857393ce24b1937c654ce7252b6315b7e77f9 /app/models/feed.rb
parent02e4fb2e06f424c16ab25ea294a4af6490a5f7e3 (diff)
Add pagination by max_id instead of offset/limit
Diffstat (limited to 'app/models/feed.rb')
-rw-r--r--app/models/feed.rb6
1 files changed, 3 insertions, 3 deletions
diff --git a/app/models/feed.rb b/app/models/feed.rb
index 9f2b34c82..206f287e7 100644
--- a/app/models/feed.rb
+++ b/app/models/feed.rb
@@ -4,12 +4,12 @@ class Feed
     @account = account
   end
 
-  def get(limit, offset = 0)
-    unhydrated = redis.zrevrange(key, offset, limit)
+  def get(limit, max_id = '+inf')
+    unhydrated = redis.zrevrangebyscore(key, "(#{max_id}", '-inf', limit: [0, limit])
     status_map = Hash.new
 
     # If we're after most recent items and none are there, we need to precompute the feed
-    return PrecomputeFeedService.new.(@type, @account).take(limit) if unhydrated.empty? && offset == 0
+    return PrecomputeFeedService.new.(@type, @account).take(limit) if unhydrated.empty? && max_id == '+inf'
 
     Status.where(id: unhydrated).with_includes.with_counters.each { |status| status_map[status.id.to_s] = status }
     return unhydrated.map { |id| status_map[id] }.compact