Skip to content

Commit

Permalink
Added logging for empty search results (URL, page and images) [ closes
Browse files Browse the repository at this point in the history
  • Loading branch information
VascoRatoFCCN committed Dec 22, 2022
1 parent a9e14d8 commit 83b0b0b
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 0 deletions.
4 changes: 4 additions & 0 deletions src/search-images.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ const sanitizeInputs = require('./utils/sanitize-search-params');
const SuggestionApi = require('./apis/suggestion-api');
const ImageSearchApiRequest = require('./apis/image-search-api');
const makeExportObject = require('./export-image-search');
const logger = require('./logger')('ImageSearch');

module.exports = function (req, res) {

Expand All @@ -13,6 +14,9 @@ module.exports = function (req, res) {
(suggestion) => {
apiRequest.get(requestData,
(apiData) => {
if(!apiData.responseItems || apiData.responseItems.length == 0){
logger.info('No results found for the following query: '+JSON.stringify(requestData.get('q')));
}
res.render('partials/images-search-results', {
requestData: requestData,
apiData: apiData,
Expand Down
4 changes: 4 additions & 0 deletions src/search-pages.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ const sanitizeInputs = require('./utils/sanitize-search-params');
const makeExportObject = require('./export-page-search');
const SuggestionApi = require('./apis/suggestion-api');
const PageSearchApiRequest = require('./apis/page-search-api');
const logger = require('./logger')('PageSearch');

module.exports = function (req, res) {
const requestData = sanitizeInputs(req, res);
Expand All @@ -12,6 +13,9 @@ module.exports = function (req, res) {
(suggestion) => {
apiRequest.get(requestData,
(apiData) => {
if(!apiData.response_items || apiData.response_items.length == 0){
logger.info('No results found for the following query: '+JSON.stringify(requestData.get('q')));
}
res.render('partials/pages-search-results', {
requestData: requestData,
apiData: apiData,
Expand Down
4 changes: 4 additions & 0 deletions src/search-url.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ const sanitizeInputs = require('./utils/sanitize-search-params');
const CDXSearchApiRequest = require('./apis/cdx-api')
const cdxFilter = require('./filter-cdx')
const SuggestionApi = require('./apis/suggestion-api')
const logger = require('./logger')('UrlSearch');

module.exports = function (req, res) {
const requestData = sanitizeInputs(req, res);
Expand All @@ -23,6 +24,9 @@ module.exports = function (req, res) {
(suggestion) => {
apiRequest.get(requestData,
(apiData) => {
if(apiData.length == 0) {
logger.info('No results found for the following query: '+JSON.stringify(requestData.get('q')));
}
res.render('partials/url-' + viewMode + '-results', {
requestData: requestData,
apiData: cdxFilter(apiData),
Expand Down

0 comments on commit 83b0b0b

Please sign in to comment.