- 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
		
			
				
	
	
		
			41 lines
		
	
	
	
		
			879 B
		
	
	
	
		
			Ruby
		
	
	
	
	
	
			
		
		
	
	
			41 lines
		
	
	
	
		
			879 B
		
	
	
	
		
			Ruby
		
	
	
	
	
	
# frozen_string_literal: true
 | 
						|
 | 
						|
module Api::V1::Timelines
 | 
						|
  class PublicController < BaseController
 | 
						|
    def show
 | 
						|
      @statuses = load_statuses
 | 
						|
    end
 | 
						|
 | 
						|
    private
 | 
						|
 | 
						|
    def load_statuses
 | 
						|
      cached_public_statuses.tap do |statuses|
 | 
						|
        set_maps(statuses)
 | 
						|
      end
 | 
						|
    end
 | 
						|
 | 
						|
    def cached_public_statuses
 | 
						|
      cache_collection public_statuses
 | 
						|
    end
 | 
						|
 | 
						|
    def public_statuses
 | 
						|
      public_timeline_statuses.paginate_by_max_id(
 | 
						|
        limit_param(DEFAULT_STATUSES_LIMIT),
 | 
						|
        params[:max_id],
 | 
						|
        params[:since_id]
 | 
						|
      )
 | 
						|
    end
 | 
						|
 | 
						|
    def public_timeline_statuses
 | 
						|
      Status.as_public_timeline(current_account, params[:local])
 | 
						|
    end
 | 
						|
 | 
						|
    def next_path
 | 
						|
      api_v1_timelines_public_url pagination_params(max_id: @statuses.last.id)
 | 
						|
    end
 | 
						|
 | 
						|
    def prev_path
 | 
						|
      api_v1_timelines_public_url pagination_params(since_id: @statuses.first.id)
 | 
						|
    end
 | 
						|
  end
 | 
						|
end
 |