Merge pull request #3463 from projectblacklight/3462-empty-query-after-advanced-search-sends-null-params-to-solr

Check for present query when appending boolean params

Author: jcoyne

lib/blacklight/solr/request.rb

@@ -36,7 +36,7 @@ def append_boolean_query(bool_operator, query)
     self[:json][:query] ||= { bool: { bool_operator => [] } }
     self[:json][:query][:bool][bool_operator] ||= []
 
-    if self['q']
+    if self['q'].present?
       self[:json][:query][:bool][:must] ||= []
       self[:json][:query][:bool][:must] << self['q']
       delete 'q'

spec/models/blacklight/solr/request_spec.rb

@@ -64,5 +64,12 @@
 
       expect(subject.dig('json', 'query', 'bool', 'must')).to contain_exactly('some query', 'also required')
     end
+
+    it 'filters out empty query' do
+      subject['q'] = ""
+      subject.append_boolean_query :must, 'present query'
+
+      expect(subject.dig('json', 'query', 'bool', 'must')).to contain_exactly('present query')
+    end
   end
 end