diff options
author | Matt Jankowski <mjankowski@thoughtbot.com> | 2017-05-23 12:11:39 -0400 |
---|---|---|
committer | Eugen Rochko <eugen@zeonfederated.com> | 2017-05-23 18:11:39 +0200 |
commit | 4289ed1d13b9c3b91663581c44635105c4bc0412 (patch) | |
tree | 5f8f006a47676ac15bb0374c21a2dea4a6a40910 /app/controllers/api/v1/timelines/tag_controller.rb | |
parent | 256e3adc1d9508423aab8fcfb13745c9f85ff948 (diff) |
Refactor of API timeline actions (#3263)
- Increase coverage to exercise all parts of each action - Move into namespace to share common code - Misc refactor of each action for smaller methods, simpler code
Diffstat (limited to 'app/controllers/api/v1/timelines/tag_controller.rb')
-rw-r--r-- | app/controllers/api/v1/timelines/tag_controller.rb | 51 |
1 files changed, 51 insertions, 0 deletions
diff --git a/app/controllers/api/v1/timelines/tag_controller.rb b/app/controllers/api/v1/timelines/tag_controller.rb new file mode 100644 index 000000000..818f49d3d --- /dev/null +++ b/app/controllers/api/v1/timelines/tag_controller.rb @@ -0,0 +1,51 @@ +# frozen_string_literal: true + +module Api::V1::Timelines + class TagController < BaseController + before_action :load_tag + + def show + @statuses = load_statuses + end + + private + + def load_tag + @tag = Tag.find_by(name: params[:id].downcase) + end + + def load_statuses + cached_tagged_statuses.tap do |statuses| + set_maps(statuses) + end + end + + def cached_tagged_statuses + cache_collection tagged_statuses + end + + def tagged_statuses + if @tag.nil? + [] + else + tag_timeline_statuses.paginate_by_max_id( + limit_param(DEFAULT_STATUSES_LIMIT), + params[:max_id], + params[:since_id] + ) + end + end + + def tag_timeline_statuses + Status.as_tag_timeline(@tag, current_account, params[:local]) + end + + def next_path + api_v1_timelines_tag_url params[:id], pagination_params(max_id: @statuses.last.id) + end + + def prev_path + api_v1_timelines_tag_url params[:id], pagination_params(since_id: @statuses.first.id) + end + end +end |