delsolr 0.1.4 → 0.1.5

data/lib/delsolr.rb

@@ -166,8 +166,7 @@ module DelSolr
       end
 
       if body.blank? # cache miss (or wasn't enabled)
-        header, body = connection.get(configuration.path + query_builder.request_string)
-
+        header, body = connection.post("#{configuration.path}/select", query_builder.request_string)
         # We get UTF-8 from Solr back, make sure the string knows about it
         # when running on Ruby >= 1.9
         if body.respond_to?(:force_encoding)
@@ -188,7 +187,7 @@ module DelSolr
         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}"
+        logger.info "#{from_cache ? 'C' : 'S'},#{response_stat_string}http://#{configuration.full_path}/select?#{response.request_url}"
       end
       response
     # If we error, just return nil and let the client decide what to do

data/lib/delsolr/query_builder.rb

@@ -105,7 +105,7 @@ module DelSolr
           end
         end
 
-        "/select?#{param_strings.join('&')}"
+        param_strings.join('&')
       end
 
       # returns the query param

data/test/test_client.rb

@@ -183,7 +183,7 @@ class ClientTest < Test::Unit::TestCase
     mock_query_builder = DelSolr::Client::QueryBuilder
     mock_query_builder.stubs(:request_string).returns('/select?some_query') # mock the query builder
     DelSolr::Client::QueryBuilder.stubs(:new).returns(mock_query_builder)
-    c.connection.expects(:get).raises(StandardError) # mock the connection
+    c.connection.expects(:post).raises(StandardError) # mock the connection
     assert_nothing_raised do
       r = c.query('standard', :query => '123')
       assert_nil(r)
@@ -196,7 +196,7 @@ class ClientTest < Test::Unit::TestCase
     mock_query_builder = DelSolr::Client::QueryBuilder
     mock_query_builder.stubs(:request_string).returns('/select?some_query') # mock the query builder
     DelSolr::Client::QueryBuilder.stubs(:new).returns(mock_query_builder)
-    c.connection.expects(:get).with("/abcsolr" + mock_query_builder.request_string).returns([nil, @@response_buffer]) # mock the connection
+    c.connection.expects(:post).with("/abcsolr/select", mock_query_builder.request_string).returns([nil, @@response_buffer]) # mock the connection
     r = c.query('standard', :query => '123')
     assert(r)
     assert_equal([1,3,4,5,7,8,9,10,11,12], r.ids.sort)
@@ -209,7 +209,7 @@ class ClientTest < Test::Unit::TestCase
     mock_query_builder = DelSolr::Client::QueryBuilder
     mock_query_builder.stubs(:request_string).returns('/select?some_query') # mock the query builder
     DelSolr::Client::QueryBuilder.stubs(:new).returns(mock_query_builder)
-    c.connection.expects(:get).with("/solr" + mock_query_builder.request_string).returns([nil, @@response_buffer]) # mock the connection
+    c.connection.expects(:post).with("/solr/select", mock_query_builder.request_string).returns([nil, @@response_buffer]) # mock the connection
     r = c.query('standard', :query => '123')
     assert(r)
     assert_equal([1,3,4,5,7,8,9,10,11,12], r.ids.sort)
@@ -223,7 +223,7 @@ class ClientTest < Test::Unit::TestCase
     mock_query_builder = DelSolr::Client::QueryBuilder
     mock_query_builder.stubs(:request_string).returns('/select?some_query') # mock the query builder
     DelSolr::Client::QueryBuilder.stubs(:new).returns(mock_query_builder)
-    c.connection.expects(:get).with("/solr" + mock_query_builder.request_string).returns([nil, @@response_buffer]) # mock the connection
+    c.connection.expects(:post).with("/solr/select", mock_query_builder.request_string).returns([nil, @@response_buffer]) # mock the connection
     r = c.query('standard', :query => '123', :enable_caching => true)
     assert(r)
     assert_equal([1,3,4,5,7,8,9,10,11,12], r.ids.sort)
@@ -242,7 +242,7 @@ class ClientTest < Test::Unit::TestCase
       mock_query_builder = DelSolr::Client::QueryBuilder
       mock_query_builder.stubs(:request_string).returns('/select?some_query') # mock the query builder
       DelSolr::Client::QueryBuilder.stubs(:new).returns(mock_query_builder)
-      c.connection.expects(:get).with("/solr" + mock_query_builder.request_string).returns([nil, @@response_buffer]) # mock the connection
+      c.connection.expects(:post).with("/solr/select", mock_query_builder.request_string).returns([nil, @@response_buffer]) # mock the connection
       r = c.query('standard', :query => '123')
       assert(r)
 

data/test/test_query_builder.rb

@@ -162,8 +162,7 @@ private
   
   # given a url returns a hash of the query params (for each duplicate key, it returns an array)
   def get_params(url)
-    query = URI.parse(url).query
-    query = query.split('&')
+    query = url.split('&')
     h = {}
     query.each do |p|
       a = p.split('=')

metadata

@@ -1,12 +1,8 @@
 --- !ruby/object:Gem::Specification 
 name: delsolr
 version: !ruby/object:Gem::Version 
-  prerelease: false
-  segments: 
-  - 0
-  - 1
-  - 4
-  version: 0.1.4
+  prerelease: 
+  version: 0.1.5
 platform: ruby
 authors: 
 - Ben VandenBos
@@ -25,10 +21,6 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        segments: 
-        - 0
-        - 9
-        - 0
         version: 0.9.0
   type: :development
   version_requirements: *id001
@@ -67,21 +59,17 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      segments: 
-      - 0
       version: "0"
 required_rubygems_version: !ruby/object:Gem::Requirement 
   none: false
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      segments: 
-      - 0
       version: "0"
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.3.7
+rubygems_version: 1.5.0
 signing_key: 
 specification_version: 3
 summary: DelSolr is a light weight ruby wrapper for solr.  It's intention is to expose the full power of solr queries while keeping the interface as ruby-esque as possible.