* Add tests * Fix serialization of followers/following counts when user hides their network Fixes #16382 Signed-off-by: Claire <claire.github-309c@sitedethib.com>
		
			
				
	
	
		
			91 lines
		
	
	
	
		
			2.5 KiB
		
	
	
	
		
			Ruby
		
	
	
	
	
	
			
		
		
	
	
			91 lines
		
	
	
	
		
			2.5 KiB
		
	
	
	
		
			Ruby
		
	
	
	
	
	
| # frozen_string_literal: true
 | |
| 
 | |
| class FollowerAccountsController < ApplicationController
 | |
|   include AccountControllerConcern
 | |
|   include SignatureVerification
 | |
| 
 | |
|   before_action :require_signature!, if: -> { request.format == :json && authorized_fetch_mode? }
 | |
|   before_action :set_cache_headers
 | |
| 
 | |
|   skip_around_action :set_locale, if: -> { request.format == :json }
 | |
|   skip_before_action :require_functional!, unless: :whitelist_mode?
 | |
| 
 | |
|   def index
 | |
|     respond_to do |format|
 | |
|       format.html do
 | |
|         expires_in 0, public: true unless user_signed_in?
 | |
| 
 | |
|         next if @account.user_hides_network?
 | |
| 
 | |
|         follows
 | |
|       end
 | |
| 
 | |
|       format.json do
 | |
|         raise Mastodon::NotPermittedError if page_requested? && @account.user_hides_network?
 | |
| 
 | |
|         expires_in(page_requested? ? 0 : 3.minutes, public: public_fetch_mode?)
 | |
| 
 | |
|         render json: collection_presenter,
 | |
|                serializer: ActivityPub::CollectionSerializer,
 | |
|                adapter: ActivityPub::Adapter,
 | |
|                content_type: 'application/activity+json',
 | |
|                fields: restrict_fields_to
 | |
|       end
 | |
|     end
 | |
|   end
 | |
| 
 | |
|   private
 | |
| 
 | |
|   def follows
 | |
|     return @follows if defined?(@follows)
 | |
| 
 | |
|     scope = Follow.where(target_account: @account)
 | |
|     scope = scope.where.not(account_id: current_account.excluded_from_timeline_account_ids) if user_signed_in?
 | |
|     @follows = scope.recent.page(params[:page]).per(FOLLOW_PER_PAGE).preload(:account)
 | |
|   end
 | |
| 
 | |
|   def page_requested?
 | |
|     params[:page].present?
 | |
|   end
 | |
| 
 | |
|   def page_url(page)
 | |
|     account_followers_url(@account, page: page) unless page.nil?
 | |
|   end
 | |
| 
 | |
|   def next_page_url
 | |
|     page_url(follows.next_page) if follows.respond_to?(:next_page)
 | |
|   end
 | |
| 
 | |
|   def prev_page_url
 | |
|     page_url(follows.prev_page) if follows.respond_to?(:prev_page)
 | |
|   end
 | |
| 
 | |
|   def collection_presenter
 | |
|     if page_requested?
 | |
|       ActivityPub::CollectionPresenter.new(
 | |
|         id: account_followers_url(@account, page: params.fetch(:page, 1)),
 | |
|         type: :ordered,
 | |
|         size: @account.followers_count,
 | |
|         items: follows.map { |f| ActivityPub::TagManager.instance.uri_for(f.account) },
 | |
|         part_of: account_followers_url(@account),
 | |
|         next: next_page_url,
 | |
|         prev: prev_page_url
 | |
|       )
 | |
|     else
 | |
|       ActivityPub::CollectionPresenter.new(
 | |
|         id: account_followers_url(@account),
 | |
|         type: :ordered,
 | |
|         size: @account.followers_count,
 | |
|         first: page_url(1)
 | |
|       )
 | |
|     end
 | |
|   end
 | |
| 
 | |
|   def restrict_fields_to
 | |
|     if page_requested? || !@account.user_hides_network?
 | |
|       # Return all fields
 | |
|     else
 | |
|       %i(id type total_items)
 | |
|     end
 | |
|   end
 | |
| end
 |