Skip to content

Commit 9eef166

Browse files
committed
dspace-api: set search fields in Solr query only if we are interested in the actual search results
1 parent bd75300 commit 9eef166

File tree

1 file changed

+14
-10
lines changed

1 file changed

+14
-10
lines changed

dspace-api/src/main/java/org/dspace/discovery/SolrServiceImpl.java

Lines changed: 14 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -861,16 +861,20 @@ protected SolrQuery resolveToSolrQuery(Context context, DiscoverQuery discoveryQ
861861

862862
solrQuery.setQuery(query);
863863

864-
// Add any search fields to our query. This is the limited list
865-
// of fields that will be returned in the solr result
866-
for (String fieldName : discoveryQuery.getSearchFields()) {
867-
solrQuery.addField(fieldName);
868-
}
869-
// Also ensure a few key obj identifier fields are returned with every query
870-
solrQuery.addField(SearchUtils.RESOURCE_TYPE_FIELD);
871-
solrQuery.addField(SearchUtils.RESOURCE_ID_FIELD);
872-
solrQuery.addField(SearchUtils.RESOURCE_UNIQUE_ID);
873-
solrQuery.addField(STATUS_FIELD);
864+
if (discoveryQuery.getMaxResults() != 0) {
865+
// set search fields in Solr query only if we are interested in the actual search results
866+
867+
// Add any search fields to our query. This is the limited list
868+
// of fields that will be returned in the solr result
869+
for (String fieldName : discoveryQuery.getSearchFields()) {
870+
solrQuery.addField(fieldName);
871+
}
872+
// Also ensure a few key obj identifier fields are returned with every query
873+
solrQuery.addField(SearchUtils.RESOURCE_TYPE_FIELD);
874+
solrQuery.addField(SearchUtils.RESOURCE_ID_FIELD);
875+
solrQuery.addField(SearchUtils.RESOURCE_UNIQUE_ID);
876+
solrQuery.addField(STATUS_FIELD);
877+
}
874878

875879
if (discoveryQuery.isSpellCheck()) {
876880
solrQuery.setParam(SpellingParams.SPELLCHECK_Q, query);

0 commit comments

Comments
 (0)