From cc8d9f0c504732c734420e33092dec1a2179343a Mon Sep 17 00:00:00 2001 From: KevinMidboe Date: Sun, 22 Oct 2017 17:03:16 +0200 Subject: [PATCH] Added docstring, removed a unused console.log and added better error handling for empty return statement.s --- .../webserver/controllers/plex/searchMedia.js | 16 +++++++++++++--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/seasoned_api/src/webserver/controllers/plex/searchMedia.js b/seasoned_api/src/webserver/controllers/plex/searchMedia.js index 0199254..bef7f26 100644 --- a/seasoned_api/src/webserver/controllers/plex/searchMedia.js +++ b/seasoned_api/src/webserver/controllers/plex/searchMedia.js @@ -1,13 +1,23 @@ const PlexRepository = require('src/plex/plexRepository'); const plexRepository = new PlexRepository(); +/** + * Controller: Search for media and check existence + * in plex by query and page + * @param {Request} req http request variable + * @param {Response} res + * @returns {Callback} + */ function searchMediaController(req, res) { const { query, page } = req.query; - console.log(query); plexRepository.searchMedia(query) - .then((movies) => { - res.send(movies); + .then((media) => { + if (media !== undefined || media.length > 0) { + res.send(media); + } else { + res.status(404).send({ success: false, error: 'Search query did not return any results.'}) + } }) .catch((error) => { res.status(500).send({success: false, error: error.message });