From: aescling Date: Tue, 30 Aug 2022 20:05:55 +0000 (-0400) Subject: /api/v1/trending/tags: Satisfy standardrb X-Git-Url: https://git.xn--scling-oua.cat.family/?a=commitdiff_plain;h=9606c1589b1bd56d28cb6d70b49aa4a6ae5dbe18;p=mastodon.git /api/v1/trending/tags: Satisfy standardrb --- diff --git a/app/controllers/api/v1/trends/tags_controller.rb b/app/controllers/api/v1/trends/tags_controller.rb index 330cb7506..a85ddca58 100644 --- a/app/controllers/api/v1/trends/tags_controller.rb +++ b/app/controllers/api/v1/trends/tags_controller.rb @@ -32,8 +32,8 @@ class Api::V1::Trends::TagsController < Api::BaseController def always_trending # TODO: do we need to sanitize ALWAYS_TRENDING_TAGS? # TODO: should we log when ALWAYS_TRENDING_TAGS includes a tag that does not exist? - ENV['ALWAYS_TRENDING_TAGS'].to_s.split(',') - .reduce(Tag.none) { |relation, tag_name| relation.or(Tag.where(name: tag_name)) } + ENV["ALWAYS_TRENDING_TAGS"].to_s.split(",") + .reduce(Tag.none) { |relation, tag_name| relation.or(Tag.where(name: tag_name)) } end # Determine the tags that will be reported as trending, overriding the @@ -44,14 +44,14 @@ class Api::V1::Trends::TagsController < Api::BaseController # TODO: is that desirable? should we log a warning in that case? def set_tags @tags = if !enabled? - [] - else - guaranteed_tags = always_trending - # TODO: how does the query handle negative limits? is this necessary? - limit_considering_guaranteed = [0, limit_param(DEFAULT_TAGS_LIMIT) - guaranteed_tags.size].max - - guaranteed_tags | tags_from_trends.offset(offset_param).limit(limit_considering_guaranteed) - end + [] + else + guaranteed_tags = always_trending + # TODO: how does the query handle negative limits? is this necessary? + limit_considering_guaranteed = [0, limit_param(DEFAULT_TAGS_LIMIT) - guaranteed_tags.size].max + + guaranteed_tags | tags_from_trends.offset(offset_param).limit(limit_considering_guaranteed) + end end # Retrieve the tags that the trending tags algorithm determines to be