When SearchBuilder has a NameError raise it.

Previously the rescue block was hiding the error and saying that SearchBuilder could not be found.
projectblacklight · Aug 6, 2015 · d98bd82 · d98bd82
1 parent e3969b7
commit d98bd82
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/lib/blacklight/configuration.rb b/lib/blacklight/configuration.rb
@@ -225,7 +225,12 @@ def search_builder_class
 
     def locate_search_builder_class
       ::SearchBuilder
-    rescue NameError
+    rescue NameError => e
+      # If the NameError is a result of the SearchBuilder having a
+      # NameError (e.g. NoMethodError) within it then raise the error.
+      raise e if Object.const_defined? "::SearchBuilder"
+
+      # Otherwise the NameError was a result of not being able to find SearchBuilder
       Deprecation.warn(Configuration, "Your application is missing the SearchBuilder. Have you run `rails generate blacklight:search_builder`? Falling back to Blacklight::Solr::SearchBuilder")
       Blacklight::Solr::SearchBuilder
     end