Renamed function name and comment to make for sense. Also deconstruct page from query
This commit is contained in:
		| @@ -2,13 +2,13 @@ const RequestRepository = require('src/request/request'); | |||||||
| const request = new RequestRepository(); | const request = new RequestRepository(); | ||||||
|  |  | ||||||
| /** | /** | ||||||
|  * Controller: Request by id with type param |  * Controller: Fetch all requested items | ||||||
|  * @param {Request} req http request variable |  * @param {Request} req http request variable | ||||||
|  * @param {Response} res |  * @param {Response} res | ||||||
|  * @returns {Callback} |  * @returns {Callback} | ||||||
|  */ |  */ | ||||||
| function requestTmdbIdController(req, res) { | function fetchAllRequests(req, res) { | ||||||
|   let { filter, sort, query } = req.query; |   let { page, filter, sort, query } = req.query; | ||||||
|   let sort_by = sort; |   let sort_by = sort; | ||||||
|   let sort_direction = undefined; |   let sort_direction = undefined; | ||||||
|  |  | ||||||
| @@ -17,11 +17,11 @@ function requestTmdbIdController(req, res) { | |||||||
|   } |   } | ||||||
|  |  | ||||||
|   Promise.resolve() |   Promise.resolve() | ||||||
|   .then(() => request.fetchAll(sort_by, sort_direction, filter, query)) |   .then(() => request.fetchAll(page, sort_by, sort_direction, filter, query)) | ||||||
|   .then((result) => res.send(result)) |   .then((result) => res.send(result)) | ||||||
|   .catch((error) => { |   .catch((error) => { | ||||||
|     res.status(404).send({ success: false, error: error.message }); |     res.status(404).send({ success: false, error: error.message }); | ||||||
|   }); |   }); | ||||||
| } | } | ||||||
|  |  | ||||||
| module.exports = requestTmdbIdController; | module.exports = fetchAllRequests; | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user