delsolr 0.1.0 → 0.1.2

data/lib/delsolr/configuration.rb

@@ -9,6 +9,10 @@ module DelSolr
         @timeout = timeout || 120
         @path = path || '/solr'
       end
+      
+      def full_path
+        "#{self.server}:#{self.port}#{self.path}"
+      end
 
     end
   end

data/lib/delsolr/query_builder.rb

@@ -198,7 +198,7 @@ module DelSolr
           elsif ['localparams', :localparams, 'name', :name].include?(k.to_s)
             # do nothing
           else
-            params << {"f.#{facet_hash[:field]}.facet.#{k}" => "#{facet_local_params}#{v}"}
+            params << {"f.#{facet_hash[:field]}.facet.#{k}" => "#{v}"}
           end
         end
         params

data/lib/delsolr.rb

@@ -189,7 +189,12 @@ module DelSolr
       end
 
       response = DelSolr::Client::Response.new(body, query_builder, :logger => logger, :from_cache => from_cache, :shortcuts => @shortcuts)
-      logger.info "#{from_cache ? 'C' : 'S'},#{from_cache ? cache_time : response.qtime},#{response.total},http://#{configuration.server}:#{configuration.port}#{response.request_url}" if logger && response && response.success?
+      if logger
+        if response && response.success?
+          response_stat_string = "#{from_cache ? cache_time : response.qtime},#{response.total},"
+        end
+        logger.info "#{from_cache ? 'C' : 'S'},#{response_stat_string}http://#{configuration.full_path}#{response.request_url}"
+      end
       response
     end
     

data/test/test_query_builder.rb

@@ -137,7 +137,9 @@ class QueryBuilderTest < Test::Unit::TestCase
 
     assert_equal('true', p['facet'])
     assert_equal('language_idm', p['facet.field'])
-    assert_equal('{!key=seattle ex=exclusion}city_idm:19596', p['facet.query'])
+    assert_match('city_idm:19596', p['facet.query'])
+    assert_match('key=seattle', p['facet.query'])
+    assert_match('ex=exclusion', p['facet.query'])
   end
 
   def test_range

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 1
-  - 0
-  version: 0.1.0
+  - 2
+  version: 0.1.2
 platform: ruby
 authors: 
 - Ben VandenBos