blacklight_advanced_search 5.1.0 → 5.1.1

data/.travis.yml

@@ -2,8 +2,8 @@ notifications:
   email: false
 
 rvm:
-  - 2.1.0
-  - 2.0.0
+  - 2.1
+  - 2.0
   - 1.9.3
   - jruby-19mode
 

data/VERSION

@@ -1 +1 @@
-5.1.0
+5.1.1

data/lib/blacklight_advanced_search/parse_basic_q.rb

@@ -1,3 +1,6 @@
+require 'parslet'
+require 'parsing_nesting/tree'
+
 # A solr search logic mix-in to CatalogController. 
 # If mixed-in, adds Advanced Search parsing behavior
 # to queries entered on basic/standard/simple search
@@ -12,6 +15,14 @@ module BlacklightAdvancedSearch::ParseBasicQ
   included do
     self.solr_search_params_logic += [:add_advanced_parse_q_to_solr]
   end
+
+  # Different versions of Parslet raise different exception classes,
+  # need to figure out which one exists to rescue
+  @@parslet_failed_exceptions = if defined? Parslet::UnconsumedInput
+    [Parslet::UnconsumedInput]
+  else
+    [Parslet::ParseFailed]
+  end
   
   
   # This method can be included in solr_search_params_logic to have us
@@ -41,10 +52,10 @@ module BlacklightAdvancedSearch::ParseBasicQ
       # and just allow basic search, perhaps with a warning.
       begin
         adv_search_params = ParsingNesting::Tree.parse(req_params[:q], blacklight_config.advanced_search[:query_parser]).to_single_query_params( solr_local_params )
-        
+
         BlacklightAdvancedSearch.deep_merge!(solr_parameters, solr_direct_params)
         BlacklightAdvancedSearch.deep_merge!(solr_parameters, adv_search_params)        
-      rescue Parslet::UnconsumedInput => e 
+      rescue *@@parslet_failed_exceptions => e
         # do nothing, don't merge our input in, keep basic search
         # optional TODO, display error message in flash here, but hard to 
         # display a good one. 

data/lib/blacklight_advanced_search/parsing_nesting_parser.rb

@@ -4,7 +4,7 @@ module BlacklightAdvancedSearch::ParsingNestingParser
   def process_query(params,config)
     queries = []
     keyword_queries.each do |field,query| 
-      queries << ParsingNesting::Tree.parse(query).to_query( local_param_hash(field, config)  )            
+      queries << ParsingNesting::Tree.parse(query, config.advanced_search[:query_parser]).to_query( local_param_hash(field, config) )
     end
     queries.join( ' ' + keyword_op + ' ')
   end

data/spec/parsing_nesting/parse_basic_q_spec.rb

@@ -0,0 +1,85 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe "NestingParser" do
+
+  # Our ParseBasicQ mixin assumes a SolrHelper context. 
+  # SolrHelper is a controller layer mixin, which depends
+  # on being mixed into a class which has #params (from Rails)
+  # and #blacklight_config
+  #
+  # This technique of testing is copied from Blacklight solr_helper_spec.rb
+  #
+  # It gets kind of a mess of dependencies, sorry. 
+  class ParseBasicQTestClass
+    cattr_accessor :blacklight_config
+    cattr_accessor :blacklight_solr
+
+    include Blacklight::SolrHelper
+    
+    include BlacklightAdvancedSearch::ParseBasicQ
+
+
+    def initialize blacklight_config, blacklight_solr
+      self.blacklight_config = blacklight_config
+      self.blacklight_solr = blacklight_solr
+    end
+
+    def params
+      {}
+    end
+
+    def logger
+      Rails.logger
+    end
+  end
+
+  describe "basic functionality" do
+    before do 
+      @blacklight_config = Blacklight::Configuration.new do |config|
+        config.advanced_search = {
+
+        }
+
+        config.add_search_field "all_fields" do |field|
+          
+        end
+
+        config.add_search_field "special_field" do |field|
+          field.advanced_parse = false
+        end
+      end
+      @obj = ParseBasicQTestClass.new @blacklight_config, Blacklight.solr 
+    end
+
+    it "catches a simple example" do
+      solr_params = {}
+      @obj.add_advanced_parse_q_to_solr(solr_params, :q => "one two AND three OR four") 
+
+      expect(solr_params[:defType]).to eq("lucene")
+      # We're not testing succesful parsing here, just that it's doing
+      # something that looks like we expect with subqueries. 
+      expect(solr_params[:q]).to start_with("_query_:")
+    end
+
+    it "passes through an unparseable example" do
+      solr_params = {}
+      unparseable_q = "foo bar\'s AND"
+      @obj.add_advanced_parse_q_to_solr(solr_params, :q => unparseable_q)
+
+      expect(solr_params[:q]).to eq(unparseable_q)
+    end
+
+    it "ignores field with advanced_parse=false" do
+      solr_params = {}
+      original_q = "one two AND three OR four"
+      @obj.add_advanced_parse_q_to_solr(solr_params, 
+        :search_field => "special_field",
+        :q => original_q
+      )
+
+      expect(solr_params).not_to have_key(:q)
+    end
+
+  end
+  
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: blacklight_advanced_search
 version: !ruby/object:Gem::Version
-  version: 5.1.0
+  version: 5.1.1
   prerelease: 
 platform: ruby
 authors:
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-06-05 00:00:00.000000000 Z
+date: 2014-08-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: blacklight
@@ -187,6 +187,7 @@ files:
 - spec/lib/filter_parser_spec.rb
 - spec/parsing_nesting/build_tree_spec.rb
 - spec/parsing_nesting/consuming_spec.rb
+- spec/parsing_nesting/parse_basic_q_spec.rb
 - spec/parsing_nesting/to_solr_spec.rb
 - spec/rcov.opts
 - spec/spec.opts
@@ -209,7 +210,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 816155266112980095
+      hash: 378368463660479159
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -218,7 +219,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 816155266112980095
+      hash: 378368463660479159
 requirements: []
 rubyforge_project: blacklight
 rubygems_version: 1.8.23
@@ -232,6 +233,7 @@ test_files:
 - spec/lib/filter_parser_spec.rb
 - spec/parsing_nesting/build_tree_spec.rb
 - spec/parsing_nesting/consuming_spec.rb
+- spec/parsing_nesting/parse_basic_q_spec.rb
 - spec/parsing_nesting/to_solr_spec.rb
 - spec/rcov.opts
 - spec/spec.opts