diff options
author | Eugen Rochko <eugen@zeonfederated.com> | 2017-12-09 01:32:29 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-12-09 01:32:29 +0100 |
commit | 2f4c5f504fa5edc2a826afa9645371f529ae904d (patch) | |
tree | 4123afd91c72f58ef0262ef97ee5e2ff1e6f5f26 /app/controllers | |
parent | f08e6e9ab54a72cc20b33b270789c245b5cfde39 (diff) |
Limit users to 50 lists, remove pagination from lists API (#5933)
Diffstat (limited to 'app/controllers')
-rw-r--r-- | app/controllers/api/v1/lists_controller.rb | 38 |
1 files changed, 1 insertions, 37 deletions
diff --git a/app/controllers/api/v1/lists_controller.rb b/app/controllers/api/v1/lists_controller.rb index 9437373bd..180a91d81 100644 --- a/app/controllers/api/v1/lists_controller.rb +++ b/app/controllers/api/v1/lists_controller.rb @@ -1,18 +1,14 @@ # frozen_string_literal: true class Api::V1::ListsController < Api::BaseController - LISTS_LIMIT = 50 - before_action -> { doorkeeper_authorize! :read }, only: [:index, :show] before_action -> { doorkeeper_authorize! :write }, except: [:index, :show] before_action :require_user! before_action :set_list, except: [:index, :create] - after_action :insert_pagination_headers, only: :index - def index - @lists = List.where(account: current_account).paginate_by_max_id(limit_param(LISTS_LIMIT), params[:max_id], params[:since_id]) + @lists = List.where(account: current_account).all render json: @lists, each_serializer: REST::ListSerializer end @@ -44,36 +40,4 @@ class Api::V1::ListsController < Api::BaseController def list_params params.permit(:title) end - - def insert_pagination_headers - set_pagination_headers(next_path, prev_path) - end - - def next_path - if records_continue? - api_v1_lists_url pagination_params(max_id: pagination_max_id) - end - end - - def prev_path - unless @lists.empty? - api_v1_lists_url pagination_params(since_id: pagination_since_id) - end - end - - def pagination_max_id - @lists.last.id - end - - def pagination_since_id - @lists.first.id - end - - def records_continue? - @lists.size == limit_param(LISTS_LIMIT) - end - - def pagination_params(core_params) - params.permit(:limit).merge(core_params) - end end |