Skip to content
This repository was archived by the owner on Oct 4, 2018. It is now read-only.

Commit d21dac9

Browse files
author
Mark A. Matienzo
committed
Merge branch 'release-v20.1.4'
2 parents 6b34b57 + 621adff commit d21dac9

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

app/controllers/bookshelf_controller.rb

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@ def show
66
respond_to do |format|
77
format.json do
88
# Bookshelf gives a list of books.
9+
raise Errors::PageLimitError if \
10+
params['page'].to_i > Settings.bookshelf.max_pages.to_i
911
@search = Bookshelf.new *permitted_params.search
1012
@items = @search.result permitted_params.args
1113
render json: @items

app/models/permitted_params.rb

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -27,10 +27,7 @@ def filters(options = {})
2727
def args
2828
args = params.map do |key, value|
2929
case key
30-
when 'page' then
31-
raise Errors::PageLimitError \
32-
if value.to_i > Settings.bookshelf.max_pages.to_i
33-
[key, value]
30+
when 'page' then [key, value]
3431
when 'page_size' then [key, value] if %w(10 20 50 100).include?(value)
3532
when 'sort_by' then [key, value] if %w(title created).include?(value)
3633
when 'sort_order' then [key, value] if %w(asc desc).include?(value)

0 commit comments

Comments
 (0)