Fix number of results returned from TrendingTags.get

6 years ago · 56fdbc04b3
parent d987567b39
commit 56fdbc04b3
1 changed files with 1 additions and 1 deletions
--- a/app/models/trending_tags.rb
+++ b/app/models/trending_tags.rb
@ -17,7 +17,7 @@ class TrendingTags

    def get(limit)
      key     = "#{KEY}:#{Time.now.utc.beginning_of_day.to_i}"
-      tag_ids = redis.zrevrange(key, 0, limit).map(&:to_i)
+      tag_ids = redis.zrevrange(key, 0, limit - 1).map(&:to_i)
      tags    = Tag.where(id: tag_ids).to_a.map { |tag| [tag.id, tag] }.to_h
      tag_ids.map { |tag_id| tags[tag_id] }.compact
    end