diff options
author | Matt Jankowski <mjankowski@thoughtbot.com> | 2017-05-31 15:36:24 -0400 |
---|---|---|
committer | Eugen Rochko <eugen@zeonfederated.com> | 2017-05-31 21:36:24 +0200 |
commit | 5c6352397257d82e75b90cc5e7de1a3922bde7bd (patch) | |
tree | 38d617da023ed80e36d1255f8190b22b4a624c75 /app/controllers/api/v1/accounts/statuses_controller.rb | |
parent | de4681b2be0b0efa1dede092445a53d4a593c140 (diff) |
Spec coverage and refactor for the api/v1/accounts controllers (#3451)
Diffstat (limited to 'app/controllers/api/v1/accounts/statuses_controller.rb')
-rw-r--r-- | app/controllers/api/v1/accounts/statuses_controller.rb | 92 |
1 files changed, 92 insertions, 0 deletions
diff --git a/app/controllers/api/v1/accounts/statuses_controller.rb b/app/controllers/api/v1/accounts/statuses_controller.rb new file mode 100644 index 000000000..1e0d2a740 --- /dev/null +++ b/app/controllers/api/v1/accounts/statuses_controller.rb @@ -0,0 +1,92 @@ +# frozen_string_literal: true + +class Api::V1::Accounts::StatusesController < ApiController + before_action -> { doorkeeper_authorize! :read } + before_action :set_account + after_action :insert_pagination_headers + + respond_to :json + + def index + @statuses = load_statuses + end + + private + + def set_account + @account = Account.find(params[:account_id]) + end + + def load_statuses + cached_account_statuses.tap do |statuses| + set_maps(statuses) + end + end + + def cached_account_statuses + cache_collection account_statuses, Status + end + + def account_statuses + default_statuses.tap do |statuses| + statuses.merge!(only_media_scope) if params[:only_media] + statuses.merge!(no_replies_scope) if params[:exclude_replies] + end + end + + def default_statuses + permitted_account_statuses.paginate_by_max_id( + limit_param(DEFAULT_STATUSES_LIMIT), + params[:max_id], + params[:since_id] + ) + end + + def permitted_account_statuses + @account.statuses.permitted_for(@account, current_account) + end + + def only_media_scope + Status.where(id: account_media_status_ids) + end + + def account_media_status_ids + @account.media_attachments.attached.reorder(nil).select(:status_id).distinct + end + + def no_replies_scope + Status.without_replies + end + + def pagination_params(core_params) + params.permit(:limit, :only_media, :exclude_replies).merge(core_params) + end + + def insert_pagination_headers + set_pagination_headers(next_path, prev_path) + end + + def next_path + if records_continue? + api_v1_account_statuses_url pagination_params(max_id: pagination_max_id) + end + end + + def prev_path + unless @statuses.empty? + api_v1_account_statuses_url pagination_params(since_id: pagination_since_id) + end + end + + def records_continue? + @statuses.size == limit_param(DEFAULT_STATUSES_LIMIT) + end + + def pagination_max_id + @statuses.last.id + end + + def pagination_since_id + @statuses.first.id + end +end |