Skip to content

Commit 9c094ff

Browse files
committed
minor refactor
1 parent bca5d7b commit 9c094ff

File tree

2 files changed

+170
-220
lines changed

2 files changed

+170
-220
lines changed

modules/accredited_representative_portal/app/controllers/accredited_representative_portal/v0/power_of_attorney_requests_controller.rb

+14-17
Original file line numberDiff line numberDiff line change
@@ -37,27 +37,13 @@ def index
3737
MSG
3838
end
3939

40-
poa_requests = relation
41-
.includes(scope_includes)
42-
.paginate(page: page_params[:number], per_page: page_params[:size])
43-
44-
# Add pagination headers for API clients
45-
response.headers['X-Total'] = poa_requests.total_entries.to_s
46-
response.headers['X-Total-Pages'] = poa_requests.total_pages.to_s
47-
response.headers['X-Per-Page'] = poa_requests.per_page.to_s
48-
response.headers['X-Page'] = poa_requests.current_page.to_s
40+
poa_requests = relation.includes(scope_includes)
41+
.paginate(page: page_params[:number], per_page: page_params[:size])
4942

5043
serializer = PowerOfAttorneyRequestSerializer.new(poa_requests)
5144
render json: {
5245
data: serializer.serializable_hash,
53-
meta: {
54-
pagination: {
55-
current_page: poa_requests.current_page,
56-
per_page: poa_requests.per_page,
57-
total_pages: poa_requests.total_pages,
58-
total_count: poa_requests.total_entries
59-
}
60-
}
46+
meta: pagination_meta(poa_requests)
6147
}, status: :ok
6248
end
6349
# rubocop:enable Metrics/MethodLength
@@ -97,6 +83,17 @@ def scope_includes
9783
{ resolution: :resolving }
9884
]
9985
end
86+
87+
def pagination_meta(poa_requests)
88+
{
89+
page: {
90+
number: poa_requests.current_page,
91+
size: poa_requests.limit_value,
92+
total: poa_requests.total_entries,
93+
total_pages: poa_requests.total_pages
94+
}
95+
}
96+
end
10097
end
10198
end
10299
end

0 commit comments

Comments
 (0)