Merge pull request #2083 from thostetler/stop-empty-id-search

Stop sending extra empty search request
adsabs · Nov 19, 2020 · a42387d · a42387d
2 parents d53e5e7 + 0a5c537
commit a42387d
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/js/widgets/search_bar/search_bar_widget.js b/src/js/widgets/search_bar/search_bar_widget.js
@@ -859,20 +859,22 @@ define([
 
     dispatchRequest: function(apiQuery) {
       var sort = apiQuery.get('sort');
-      var da = this.defaultQueryArguments;
       if (/citation_count_norm/i.test(sort)) {
-        da = _.extend(da, {
+        this.defaultQueryArguments = _.extend(this.defaultQueryArguments, {
           stats: 'true',
           'stats.field': 'citation_count_norm',
         });
       } else if (/citation_count/i.test(sort)) {
-        da = _.extend(da, {
+        this.defaultQueryArguments = _.extend(this.defaultQueryArguments, {
           stats: 'true',
           'stats.field': 'citation_count',
         });
       } else {
         this.model.unset('citationCount');
         this.model.unset('citationLabel');
+
+        // don't bother sending request
+        return;
       }
       BaseWidget.prototype.dispatchRequest.call(this, apiQuery);
     },