From 2ed4c3b2673dc70ebaeb1888aa6e3c8ba34e0604 Mon Sep 17 00:00:00 2001 From: Paul Pound Date: Wed, 28 May 2014 11:59:54 -0300 Subject: [PATCH] added try catch when checking for number of results --- targets/cufts/js/cufts_results.js | 8 +++++++- targets/eds/js/eds_results.js | 10 ++++++++-- targets/evergreen/js/evergreen_results.js | 8 +++++++- targets/solr_site/js/solr_site_results.js | 10 +++++++++- 4 files changed, 31 insertions(+), 5 deletions(-) diff --git a/targets/cufts/js/cufts_results.js b/targets/cufts/js/cufts_results.js index 6fa598c..de048df 100644 --- a/targets/cufts/js/cufts_results.js +++ b/targets/cufts/js/cufts_results.js @@ -3,7 +3,13 @@ Drupal.behaviors.roblib_search_cufts = { url = settings.roblib_search_cufts.search_url; jQuery.getJSON(url, function(data) { var items = []; - if (data.journals.length < 1) { + var numberOfDocs = 0; + try { + numberOfDocs = data.journals.length; + } catch (err) { + // do nothing leave docLength at 0 + } + if (numberOfDocs < 1) { jQuery('#' + 'roblib-search-content-cufts').empty().append('No Results'); jQuery('.' + 'pane-roblib-search-cufts-roblib-search-cufts-results').hide(); } else { diff --git a/targets/eds/js/eds_results.js b/targets/eds/js/eds_results.js index 36fff7a..7d06d5c 100644 --- a/targets/eds/js/eds_results.js +++ b/targets/eds/js/eds_results.js @@ -2,8 +2,14 @@ Drupal.behaviors.roblib_search_eds = { attach: function(context, settings) { $url = settings.roblib_search_eds.search_url; jQuery.getJSON($url, function(data) { - var items = []; - if(data.records.length < 1){ + var items = []; + var numberOfDocs = 0; + try { + numberOfDocs = data.records.length; + } catch (err) { + // do nothing leave docLength at 0 + } + if(numberOfDocs < 1){ jQuery('#' + 'roblib-search-content-eds').empty().append('No Results'); jQuery('.' + 'pane-roblib-search-eds-roblib-search-eds-results').hide(); } else { diff --git a/targets/evergreen/js/evergreen_results.js b/targets/evergreen/js/evergreen_results.js index c3f7a83..78f3363 100644 --- a/targets/evergreen/js/evergreen_results.js +++ b/targets/evergreen/js/evergreen_results.js @@ -3,7 +3,13 @@ Drupal.behaviors.roblib_search_evergreen = { url = settings.roblib_search_evergreen.search_url; jQuery.getJSON(url, function(data) { var items = []; - if(data.numberOfRecords < 1){ + var numberOfDocs = 0; + try { + numberOfDocs = data.numberOfRecords; + } catch (err) { + // do nothing leave docLength at 0 + } + if(numberOfDocs < 1){ jQuery('#' + 'roblib-search-content-evergreen').empty().append('No Results'); jQuery('.pane-roblib-search-evergreen-roblib-search-evergreen-results').hide(); } else { diff --git a/targets/solr_site/js/solr_site_results.js b/targets/solr_site/js/solr_site_results.js index f5b62ee..26438cf 100644 --- a/targets/solr_site/js/solr_site_results.js +++ b/targets/solr_site/js/solr_site_results.js @@ -3,8 +3,16 @@ Drupal.behaviors.roblib_search_solr_site = { url = settings.roblib_search_solr_site.search_url; jQuery.getJSON(url, function(data) { var items = []; - if (data.response.docs.length < 1) { + var numberOfDocs = 0; + try { + numberOfDocs = data.response.docs.length; + } catch (err) { + // do nothing leave docLength at 0 + } + if (numberOfDocs < 1) { jQuery('#' + 'roblib-search-content-solr-site').empty().append('No Results'); + //
+ jQuery('.' + 'pane-roblib-search-solr-site-roblib-search-solr-site-results').hide(); } else { jQuery.each(data.response.docs, function(key, val) {