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_controller.rb | |
parent | de4681b2be0b0efa1dede092445a53d4a593c140 (diff) |
Spec coverage and refactor for the api/v1/accounts controllers (#3451)
Diffstat (limited to 'app/controllers/api/v1/accounts_controller.rb')
-rw-r--r-- | app/controllers/api/v1/accounts_controller.rb | 94 |
1 files changed, 3 insertions, 91 deletions
diff --git a/app/controllers/api/v1/accounts_controller.rb b/app/controllers/api/v1/accounts_controller.rb index bd6fb6a2a..3b23e996d 100644 --- a/app/controllers/api/v1/accounts_controller.rb +++ b/app/controllers/api/v1/accounts_controller.rb @@ -1,73 +1,15 @@ # frozen_string_literal: true class Api::V1::AccountsController < ApiController - before_action -> { doorkeeper_authorize! :read }, except: [:follow, :unfollow, :block, :unblock, :mute, :unmute, :update_credentials] + before_action -> { doorkeeper_authorize! :read }, except: [:follow, :unfollow, :block, :unblock, :mute, :unmute] before_action -> { doorkeeper_authorize! :follow }, only: [:follow, :unfollow, :block, :unblock, :mute, :unmute] - before_action -> { doorkeeper_authorize! :write }, only: [:update_credentials] - before_action :require_user!, except: [:show, :following, :followers, :statuses] - before_action :set_account, except: [:verify_credentials, :update_credentials, :suggestions, :search] + before_action :require_user!, except: [:show] + before_action :set_account respond_to :json def show; end - def verify_credentials - @account = current_user.account - render :show - end - - def update_credentials - current_account.update!(account_params) - @account = current_account - render :show - end - - def following - @accounts = Account.includes(:passive_relationships) - .references(:passive_relationships) - .merge(Follow.where(account: @account) - .paginate_by_max_id(limit_param(DEFAULT_ACCOUNTS_LIMIT), params[:max_id], params[:since_id])) - .to_a - - next_path = following_api_v1_account_url(pagination_params(max_id: @accounts.last.passive_relationships.first.id)) if @accounts.size == limit_param(DEFAULT_ACCOUNTS_LIMIT) - prev_path = following_api_v1_account_url(pagination_params(since_id: @accounts.first.passive_relationships.first.id)) unless @accounts.empty? - - set_pagination_headers(next_path, prev_path) - - render :index - end - - def followers - @accounts = Account.includes(:active_relationships) - .references(:active_relationships) - .merge(Follow.where(target_account: @account) - .paginate_by_max_id(limit_param(DEFAULT_ACCOUNTS_LIMIT), - params[:max_id], - params[:since_id])) - .to_a - - next_path = followers_api_v1_account_url(pagination_params(max_id: @accounts.last.active_relationships.first.id)) if @accounts.size == limit_param(DEFAULT_ACCOUNTS_LIMIT) - prev_path = followers_api_v1_account_url(pagination_params(since_id: @accounts.first.active_relationships.first.id)) unless @accounts.empty? - - set_pagination_headers(next_path, prev_path) - - render :index - end - - def statuses - @statuses = @account.statuses.permitted_for(@account, current_account).paginate_by_max_id(limit_param(DEFAULT_STATUSES_LIMIT), params[:max_id], params[:since_id]) - @statuses = @statuses.where(id: MediaAttachment.where(account: @account).where.not(status_id: nil).reorder('').select('distinct status_id')) if params[:only_media] - @statuses = @statuses.without_replies if params[:exclude_replies] - @statuses = cache_collection(@statuses, Status) - - set_maps(@statuses) - - next_path = statuses_api_v1_account_url(statuses_pagination_params(max_id: @statuses.last.id)) if @statuses.size == limit_param(DEFAULT_STATUSES_LIMIT) - prev_path = statuses_api_v1_account_url(statuses_pagination_params(since_id: @statuses.first.id)) unless @statuses.empty? - - set_pagination_headers(next_path, prev_path) - end - def follow FollowService.new.call(current_user.account, @account.acct) set_relationship @@ -111,24 +53,6 @@ class Api::V1::AccountsController < ApiController render :relationship end - def relationships - ids = params[:id].is_a?(Enumerable) ? params[:id].map(&:to_i) : [params[:id].to_i] - - @accounts = Account.where(id: ids).select('id') - @following = Account.following_map(ids, current_user.account_id) - @followed_by = Account.followed_by_map(ids, current_user.account_id) - @blocking = Account.blocking_map(ids, current_user.account_id) - @muting = Account.muting_map(ids, current_user.account_id) - @requested = Account.requested_map(ids, current_user.account_id) - @domain_blocking = Account.domain_blocking_map(ids, current_user.account_id) - end - - def search - @accounts = AccountSearchService.new.call(params[:q], limit_param(DEFAULT_ACCOUNTS_LIMIT), params[:resolve] == 'true', current_account) - - render :index - end - private def set_account @@ -143,16 +67,4 @@ class Api::V1::AccountsController < ApiController @requested = Account.requested_map([@account.id], current_user.account_id) @domain_blocking = Account.domain_blocking_map([@account.id], current_user.account_id) end - - def pagination_params(core_params) - params.permit(:limit).merge(core_params) - end - - def statuses_pagination_params(core_params) - params.permit(:limit, :only_media, :exclude_replies).merge(core_params) - end - - def account_params - params.permit(:display_name, :note, :avatar, :header) - end end |