diff options
author | Eugen Rochko <eugen@zeonfederated.com> | 2016-03-22 21:38:47 +0100 |
---|---|---|
committer | Eugen Rochko <eugen@zeonfederated.com> | 2016-03-22 21:38:47 +0100 |
commit | 12559b01eae408c0f3e820ef9c6bc6e356ab889a (patch) | |
tree | 87d857393ce24b1937c654ce7252b6315b7e77f9 /app/models/feed.rb | |
parent | 02e4fb2e06f424c16ab25ea294a4af6490a5f7e3 (diff) |
Add pagination by max_id instead of offset/limit
Diffstat (limited to 'app/models/feed.rb')
-rw-r--r-- | app/models/feed.rb | 6 |
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 |