Merge pull request #524 from TAMULib/514-silent_search_failure

Issue 514: Use Default Field 'df' to prevent syntax problems.
TAMULib · Feb 7, 2023 · 5599f29 · 5599f29
2 parents 093cfbf + 3d4bccc
commit 5599f29
Showing 1 changed file with 4 additions and 11 deletions.
diff --git a/src/main/java/edu/tamu/sage/service/SolrDiscoveryService.java b/src/main/java/edu/tamu/sage/service/SolrDiscoveryService.java
@@ -84,18 +84,11 @@ public DiscoveryContext buildDiscoveryContext(DiscoveryView discoveryView, Strin
             });
         }
 
-        String query = "";
-        if (search.getField().isEmpty() && search.getValue().isEmpty()) {
-            query = "*";
-        } else if (search.getField().isEmpty()) {
-            query = search.getValue();
-        } else if (search.getValue().isEmpty()) {
-            query = search.getField() + ":*";
-        } else {
-            query = search.getField() + ":" + search.getValue();
-        }
+        SolrQuery solrQuery = new SolrQuery(StringUtils.isEmpty(search.getValue()) ? "*" : search.getValue());
 
-        SolrQuery solrQuery = new SolrQuery(query);
+        if (StringUtils.isNotEmpty(search.getField())) {
+          solrQuery.add("df", search.getField());
+        }
 
         // Only filter against designated facet fields.
         List<Filter> filters = new ArrayList<Filter>();