Merge pull request #1887 from sul-dlss/corylown-patch-1

Check that search_state.query_param responds to match?

app/models/search_builder.rb

@@ -25,7 +25,8 @@ class SearchBuilder < Blacklight::SearchBuilder
   # more advanced search queries. Setting `mm=1` means only 1 clause needs to match, but the lucene query parser (in
   # edismax) will precompose the query strings so everything works out.
   def min_match_for_boolean(solr_parameters)
-    return unless search_state.query_param&.match?(/\s(AND|OR|NOT)\s/)
+    return unless search_state.query_param.respond_to?(:match?) &&
+                  search_state.query_param&.match?(/\s(AND|OR|NOT)\s/)
 
     solr_parameters[:mm] = '1'
   end

spec/models/search_builder_spec.rb

@@ -28,5 +28,13 @@
         expect(solr_parameters[:mm]).to be_nil
       end
     end
+
+    context "when the query isn't a string" do
+      let(:query) { { q: { a: 1 } } }
+
+      it 'does not set min match' do
+        expect(solr_parameters[:mm]).to be_nil
+      end
+    end
   end
 end