sunspot_rails 0.10.7 → 0.10.8

data/VERSION.yml

@@ -1,4 +1,4 @@
 --- 
 :major: 0
 :minor: 10
-:patch: 7
+:patch: 8

data/dev_tasks/gemspec.rake

@@ -1,5 +1,4 @@
 begin
-  gem 'technicalpickles-jeweler', '~> 1.0'
   require 'jeweler'
   Jeweler::Tasks.new do |s|
     s.name = 'sunspot_rails'
@@ -38,5 +37,5 @@ end
 
 namespace :release do
   desc "Release gem to RubyForge and GitHub"
-  task :all => [:"rubyforge:release:gem", :"gemcutter:release"]
+  task :all => [:release, :"rubyforge:release:gem", :"gemcutter:release"]
 end

data/lib/sunspot/rails/request_lifecycle.rb

@@ -9,7 +9,7 @@ module Sunspot #:nodoc:
       class <<self
         def included(base) #:nodoc:
           base.after_filter do
-            Sunspot::Rails.session.commit_if_dirty if Sunspot::Rails.configuration.auto_commit_after_request?
+            Sunspot.commit_if_dirty if Sunspot::Rails.configuration.auto_commit_after_request?
           end
         end
       end

data/lib/sunspot/rails/searchable.rb

@@ -104,7 +104,7 @@ module Sunspot #:nodoc:
         # Sunspot::Search:: Object containing results, totals, facets, etc.
         #
         def search(&block)
-          Sunspot::Rails.session.search(self, &block)
+          Sunspot.search(self, &block)
         end
 
         # 
@@ -125,7 +125,7 @@ module Sunspot #:nodoc:
         # Remove instances of this class from the Solr index.
         #
         def remove_all_from_index
-          Sunspot::Rails.session.remove_all(self)
+          Sunspot.remove_all(self)
         end
 
         # 
@@ -136,8 +136,8 @@ module Sunspot #:nodoc:
         # XXX Sunspot should implement remove_all!()
         #
         def remove_all_from_index!
-          remove_all_from_index
-          Sunspot::Rails.session.commit
+          Sunspot.remove_all(self)
+          Sunspot.commit
         end
 
         # 
@@ -180,20 +180,20 @@ module Sunspot #:nodoc:
           options = { :batch_size => 500, :batch_commit => true, :include => []}.merge(opts)
           remove_all_from_index
           unless options[:batch_size]
-            Sunspot::Rails.session.index!(all(:include => options[:include]))
+            Sunspot.index!(all(:include => options[:include]))
           else
             record_count = count
             counter = 1
             offset = 0
             while(offset < record_count)
               benchmark options[:batch_size], counter do
-                Sunspot::Rails.session.index(all(:include => options[:include], :offset => offset, :limit => options[:batch_size], :order => primary_key))
+                Sunspot.index(all(:include => options[:include], :offset => offset, :limit => options[:batch_size], :order => primary_key))
               end
-              Sunspot::Rails.session.commit if options[:batch_commit]
+              Sunspot.commit if options[:batch_commit]
               offset += options[:batch_size]
               counter += 1
             end
-            Sunspot::Rails.session.commit unless options[:batch_commit]
+            Sunspot.commit unless options[:batch_commit]
           end
         end
 
@@ -267,14 +267,14 @@ module Sunspot #:nodoc:
         # manually.
         #
         def index
-          Sunspot::Rails.session.index(self)
+          Sunspot.index(self)
         end
 
         # 
         # Index the model in Solr and immediately commit. See #index
         #
         def index!
-          Sunspot::Rails.session.index!(self)
+          Sunspot.index!(self)
         end
         
         # 
@@ -285,15 +285,19 @@ module Sunspot #:nodoc:
         # manually.
         #
         def remove_from_index
-          Sunspot::Rails.session.remove(self)
+          Sunspot.remove(self)
         end
 
         # 
         # Remove the model from the Solr index and commit immediately. See
         # #remove_from_index
         #
+        #---
+        #FIXME Sunspot should implement remove!()
+        #
         def remove_from_index!
-          Sunspot::Rails.session.remove!(self)
+          Sunspot.remove(self)
+          Sunspot.commit
         end
       end
     end

data/lib/sunspot/rails.rb

@@ -8,19 +8,6 @@ module Sunspot #:nodoc:
       def configuration
         @configuration ||= Sunspot::Rails::Configuration.new
       end
-
-      def session
-        Thread.current[:sunspot_rails_session] ||=
-          begin
-            session = Sunspot::Session.new
-            session.config.solr.url = URI::HTTP.build(
-              :host => configuration.hostname,
-              :port => configuration.port,
-              :path => configuration.path
-            ).to_s
-            session
-          end
-      end
     end
   end
 end

data/rails/init.rb

@@ -1,5 +1,9 @@
 require 'sunspot'
 
+Sunspot.config.solr.url = URI::HTTP.build(:host => Sunspot::Rails.configuration.hostname,
+                                          :port => Sunspot::Rails.configuration.port,
+                                          :path => Sunspot::Rails.configuration.path).to_s
+
 Sunspot::Adapters::InstanceAdapter.register(Sunspot::Rails::Adapters::ActiveRecordInstanceAdapter, ActiveRecord::Base)
 Sunspot::Adapters::DataAccessor.register(Sunspot::Rails::Adapters::ActiveRecordDataAccessor, ActiveRecord::Base)
 ActiveRecord::Base.module_eval { include(Sunspot::Rails::Searchable) }

data/spec/model_lifecycle_spec.rb

@@ -4,7 +4,7 @@ describe 'searchable with lifecycle' do
   describe 'on create' do
     before :each do
       @post = PostWithAuto.create
-      Sunspot::Rails.session.commit
+      Sunspot.commit
     end
 
     it 'should automatically index' do
@@ -16,7 +16,7 @@ describe 'searchable with lifecycle' do
     before :each do
       @post = PostWithAuto.create
       @post.update_attributes(:title => 'Test 1')
-      Sunspot::Rails.session.commit
+      Sunspot.commit
     end
 
     it 'should automatically update index' do
@@ -28,7 +28,7 @@ describe 'searchable with lifecycle' do
     before :each do
       @post = PostWithAuto.create
       @post.destroy
-      Sunspot::Rails.session.commit
+      Sunspot.commit
     end
 
     it 'should automatically remove it from the index' do

data/spec/model_spec.rb

@@ -12,7 +12,7 @@ describe 'ActiveRecord mixin' do
     end
 
     it 'should index the model' do
-      Sunspot::Rails.session.commit
+      Sunspot.commit
       Post.search.results.should == [@post]
     end
   end
@@ -40,7 +40,7 @@ describe 'ActiveRecord mixin' do
     end
 
     it 'should remove the model from the index' do
-      Sunspot::Rails.session.commit
+      Sunspot.commit
       Post.search.results.should be_empty
     end
   end
@@ -59,8 +59,8 @@ describe 'ActiveRecord mixin' do
 
   describe 'remove_all_from_index' do
     before :each do
-      @posts = Array.new(10) { Post.create! }.each { |post| Sunspot::Rails.session.index(post) }
-      Sunspot::Rails.session.commit
+      @posts = Array.new(10) { Post.create! }.each { |post| Sunspot.index(post) }
+      Sunspot.commit
       Post.remove_all_from_index
     end
 
@@ -69,15 +69,15 @@ describe 'ActiveRecord mixin' do
     end
 
     it 'should remove all instances from the index' do
-      Sunspot::Rails.session.commit
+      Sunspot.commit
       Post.search.results.should be_empty
     end
   end
 
   describe 'remove_all_from_index!' do
     before :each do
-      Array.new(10) { Post.create! }.each { |post| Sunspot::Rails.session.index(post) }
-      Sunspot::Rails.session.commit
+      Array.new(10) { Post.create! }.each { |post| Sunspot.index(post) }
+      Sunspot.commit
       Post.remove_all_from_index!
     end
 
@@ -115,7 +115,7 @@ describe 'ActiveRecord mixin' do
   describe 'search_ids()' do
     before :each do
       @posts = Array.new(2) { Post.create! }.each { |post| post.index }
-      Sunspot::Rails.session.commit
+      Sunspot.commit
     end
 
     it 'should return IDs' do
@@ -136,7 +136,7 @@ describe 'ActiveRecord mixin' do
   describe 'index_orphans()' do
     before :each do
       @posts = Array.new(2) { Post.create }.each { |post| post.index }
-      Sunspot::Rails.session.commit
+      Sunspot.commit
       @posts.first.destroy
     end
 
@@ -148,13 +148,13 @@ describe 'ActiveRecord mixin' do
   describe 'clean_index_orphans()' do
     before :each do
       @posts = Array.new(2) { Post.create }.each { |post| post.index }
-      Sunspot::Rails.session.commit
+      Sunspot.commit
       @posts.first.destroy
     end
 
     it 'should remove orphans from the index' do
       Post.clean_index_orphans
-      Sunspot::Rails.session.commit
+      Sunspot.commit
       Post.search.results.should == [@posts.last]
     end
   end
@@ -166,7 +166,7 @@ describe 'ActiveRecord mixin' do
 
     it 'should index all instances' do
       Post.reindex(:batch_size => nil)
-      Sunspot::Rails.session.commit
+      Sunspot.commit
       Post.search.results.to_set.should == @posts.to_set
     end
 
@@ -175,7 +175,7 @@ describe 'ActiveRecord mixin' do
       old_post.index!
       old_post.destroy
       Post.reindex
-      Sunspot::Rails.session.commit
+      Sunspot.commit
       Post.search.results.to_set.should == @posts.to_set
     end
     
@@ -188,7 +188,7 @@ describe 'ActiveRecord mixin' do
 
     it 'should index all instances' do
       Post.reindex(:batch_size => nil)
-      Sunspot::Rails.session.commit
+      Sunspot.commit
       Post.search.results.to_set.should == @posts.to_set
     end
 
@@ -197,14 +197,14 @@ describe 'ActiveRecord mixin' do
       old_post.index!
       old_post.destroy
       Post.reindex
-      Sunspot::Rails.session.commit
+      Sunspot.commit
       Post.search.results.to_set.should == @posts.to_set
     end
     
     describe "using batch sizes" do
       it 'should index with a specified batch size' do
         Post.reindex(:batch_size => 1)
-        Sunspot::Rails.session.commit
+        Sunspot.commit
         Post.search.results.to_set.should == @posts.to_set
       end
     end
@@ -263,12 +263,12 @@ describe 'ActiveRecord mixin' do
       end
 
       it "should commit after indexing each batch" do
-        Sunspot::Rails.session.should_receive(:commit).twice
+        Sunspot.should_receive(:commit).twice
         Post.reindex(:batch_size => 5)
       end
 
       it "should commit after indexing everything" do
-        Sunspot::Rails.session.should_receive(:commit).once
+        Sunspot.should_receive(:commit).once
         Post.reindex(:batch_commit => false)
       end
       

data/spec/spec_helper.rb

@@ -25,8 +25,8 @@ end
 
 Spec::Runner.configure do |config|
   config.before(:each) do
-    Sunspot::Rails.session.remove_all
-    Sunspot::Rails.session.commit
+    Sunspot.remove_all
+    Sunspot.commit
     load_schema
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: sunspot_rails
 version: !ruby/object:Gem::Version 
-  version: 0.10.7
+  version: 0.10.8
 platform: ruby
 authors: 
 - Mat Brown
@@ -12,7 +12,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-10-12 00:00:00 -04:00
+date: 2009-10-17 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -125,12 +125,10 @@ files:
 - spec/mock_app/config/initializers/session_store.rb
 - spec/mock_app/config/routes.rb
 - spec/mock_app/config/sunspot.yml
-- spec/mock_app/db/test.db
 - spec/model_lifecycle_spec.rb
 - spec/model_spec.rb
 - spec/request_lifecycle_spec.rb
 - spec/schema.rb
-- spec/session_spec.rb
 - spec/spec_helper.rb
 has_rdoc: true
 homepage: http://github.com/outoftime/sunspot_rails
@@ -156,28 +154,27 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: sunspot
-rubygems_version: 1.3.2
+rubygems_version: 1.3.5
 signing_key: 
 specification_version: 3
 summary: Rails integration for the Sunspot Solr search library
 test_files: 
-- spec/spec_helper.rb
-- spec/session_spec.rb
 - spec/configuration_spec.rb
-- spec/model_lifecycle_spec.rb
-- spec/schema.rb
-- spec/model_spec.rb
-- spec/mock_app/app/controllers/posts_controller.rb
-- spec/mock_app/app/controllers/application_controller.rb
 - spec/mock_app/app/controllers/application.rb
+- spec/mock_app/app/controllers/application_controller.rb
+- spec/mock_app/app/controllers/posts_controller.rb
 - spec/mock_app/app/models/blog.rb
-- spec/mock_app/app/models/post_with_auto.rb
 - spec/mock_app/app/models/post.rb
+- spec/mock_app/app/models/post_with_auto.rb
+- spec/mock_app/config/boot.rb
 - spec/mock_app/config/environment.rb
-- spec/mock_app/config/initializers/session_store.rb
-- spec/mock_app/config/initializers/new_rails_defaults.rb
 - spec/mock_app/config/environments/development.rb
 - spec/mock_app/config/environments/test.rb
+- spec/mock_app/config/initializers/new_rails_defaults.rb
+- spec/mock_app/config/initializers/session_store.rb
 - spec/mock_app/config/routes.rb
-- spec/mock_app/config/boot.rb
+- spec/model_lifecycle_spec.rb
+- spec/model_spec.rb
 - spec/request_lifecycle_spec.rb
+- spec/schema.rb
+- spec/spec_helper.rb

data/spec/mock_app/db/test.db

@@ -1 +0,0 @@
-S

data/spec/session_spec.rb

@@ -1,11 +0,0 @@
-require File.join(File.dirname(__FILE__), 'spec_helper')
-
-describe 'Sunspot::Rails session' do
-  it 'should be a different object for each thread' do
-    session1 = nil
-    session2 = nil
-    Thread.new { session1 = Sunspot::Rails.session }.join
-    Thread.new { session2 = Sunspot::Rails.session }.join
-    session1.should_not eql(session2)
-  end
-end