thinking-sphinx 5.2.1 → 5.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 174a0193bd92fcdca5e1af880b893ddf5ae94e477e53c03f7684d48470042a89
-  data.tar.gz: fe6b9df85b827b0ffec5a60ae9c691b4b50efa3774bd83854124e3639ecea93d
+  metadata.gz: 55eace64045aea4431c8a86005e70485efacfb15cc6768098c19fa89c4098d8c
+  data.tar.gz: e13db8a3a331535dd73aed4ffb965a130fdf8ae57df4241cafb4c6e1b421305f
 SHA512:
-  metadata.gz: 7eac5ba48a5f23b6e142a33065c27490df0036abd94899dba580e2c1f738904f46f203657e2b34549028b9c7f64dda6a5642c80e4bea6acc989a66f6b0bebade
-  data.tar.gz: 9b8a4875deeb1b6a38fe3127162e77e0e8a8879ae6e7aa4da7d457cc00869c0e77d4d2a95c691ff1d12eab0c9f20d06be8abc024b911f15819fef03890d1f5f8
+  metadata.gz: 7a9d79a7a86ca0c6f9968bae9c980049174626d12d44b8cafa73139be5bf24c66d3675a5e16a3a104d5b04ce76b1a0e136e2cc788338c142107af9354eca8e7d
+  data.tar.gz: be890cb3de5d4a5885c6bcc213349bfb656567f2b4cc63decabbd608bf7004126358fd13b42e6b3a6271e25db19f85c2dc061ed2c1b862e7f968276fa7632377

data/CHANGELOG.markdown

@@ -2,6 +2,20 @@
 
 All notable changes to this project (at least, from v3.0.0 onwards) are documented in this file.
 
+## 5.3.0 - 2021-08-19
+
+[Release Notes](https://github.com/pat/thinking-sphinx/releases/tag/v5.3.0)
+
+### Changed
+
+* StaleIdsExceptions now include a URL in their error message with recommendations on how to resolve the problem.
+* Fire real-time callbacks on `after_commit` (including deletions) to ensure data is fully persisted to the database before updating Sphinx. More details in [#1204](https://github.com/pat/thinking-sphinx/pull/1204).
+
+### Fixed
+
+* Ensure Thinking Sphinx's ActiveRecord components are loaded by either Rails' after_initialise hook or ActiveSupport's on_load notification, because the order of these two events are not consistent.
+* Remove `app/indices` from eager_load_paths in Rails 4.2 and 5, to match the behaviour in 6.
+
 ## 5.2.1 - 2021-08-09
 
 [Release Notes](https://github.com/pat/thinking-sphinx/releases/tag/v5.2.1)

data/README.textile

@@ -1,6 +1,6 @@
 h1. Thinking Sphinx
 
-Thinking Sphinx is a library for connecting ActiveRecord to the Sphinx full-text search tool, and integrates closely with Rails (but also works with other Ruby web frameworks). The current release is v5.2.1.
+Thinking Sphinx is a library for connecting ActiveRecord to the Sphinx full-text search tool, and integrates closely with Rails (but also works with other Ruby web frameworks). The current release is v5.3.0.
 
 h2. Upgrading
 
@@ -14,7 +14,7 @@ It's a gem, so install it like you would any other gem. You will also need to sp
 
 <pre><code>gem 'mysql2',          '~> 0.4',    :platform => :ruby
 gem 'jdbc-mysql',      '~> 5.1.35', :platform => :jruby
-gem 'thinking-sphinx', '~> 5.2'</code></pre>
+gem 'thinking-sphinx', '~> 5.3'</code></pre>
 
 The MySQL gems mentioned are required for connecting to Sphinx, so please include it even when you're using PostgreSQL for your database.
 

data/lib/thinking_sphinx.rb

@@ -101,10 +101,6 @@ require 'thinking_sphinx/distributed'
 require 'thinking_sphinx/logger'
 require 'thinking_sphinx/real_time'
 
-if defined?(Rails::Railtie)
-  require 'thinking_sphinx/railtie'
-else
-  require 'thinking_sphinx/active_record'
-end
+require 'thinking_sphinx/railtie' if defined?(Rails::Railtie)
 
 ThinkingSphinx.before_index_hooks << ThinkingSphinx::Hooks::GuardPresence

data/lib/thinking_sphinx/active_record.rb

@@ -40,3 +40,5 @@ require 'thinking_sphinx/active_record/depolymorph/conditions_reflection'
 require 'thinking_sphinx/active_record/depolymorph/overridden_reflection'
 require 'thinking_sphinx/active_record/depolymorph/scoped_reflection'
 require 'thinking_sphinx/active_record/filter_reflection'
+
+ActiveRecord::Base.include ThinkingSphinx::ActiveRecord::Base

data/lib/thinking_sphinx/active_record/callbacks/delete_callbacks.rb

@@ -3,7 +3,11 @@
 class ThinkingSphinx::ActiveRecord::Callbacks::DeleteCallbacks <
   ThinkingSphinx::Callbacks
 
-  callbacks :after_destroy, :after_rollback
+  callbacks :after_commit, :after_destroy, :after_rollback
+
+  def after_commit
+    delete_from_sphinx
+  end
 
   def after_destroy
     delete_from_sphinx

data/lib/thinking_sphinx/callbacks/appender.rb

@@ -24,7 +24,10 @@ class ThinkingSphinx::Callbacks::Appender
   attr_reader :model, :reference, :options, :block
 
   def add_core_callbacks
-    model.after_destroy ThinkingSphinx::ActiveRecord::Callbacks::DeleteCallbacks
+    model.after_commit(
+      ThinkingSphinx::ActiveRecord::Callbacks::DeleteCallbacks,
+      on: :destroy
+    )
   end
 
   def add_delta_callbacks
@@ -40,8 +43,9 @@ class ThinkingSphinx::Callbacks::Appender
   end
 
   def add_real_time_callbacks
-    model.after_save ThinkingSphinx::RealTime.callback_for(
-      reference, path, &block
+    model.after_commit(
+      ThinkingSphinx::RealTime.callback_for(reference, path, &block),
+      on: [:create, :update]
     )
   end
 

data/lib/thinking_sphinx/railtie.rb

@@ -7,21 +7,23 @@ class ThinkingSphinx::Railtie < Rails::Railtie
 
   config.after_initialize do
     require 'thinking_sphinx/active_record'
-    ActiveRecord::Base.include ThinkingSphinx::ActiveRecord::Base
   end
 
   initializer 'thinking_sphinx.initialisation' do
-    if ActiveSupport::VERSION::MAJOR > 5
-      if Rails.application.config.autoloader == :zeitwerk
-        ActiveSupport::Dependencies.autoload_paths.delete(
-          Rails.root.join("app", "indices").to_s
-        )
-      end
+    ActiveSupport.on_load(:active_record) do
+      require 'thinking_sphinx/active_record'
+    end
 
-      Rails.application.config.eager_load_paths -=
-        ThinkingSphinx::Configuration.instance.index_paths
-      Rails.application.config.eager_load_paths.freeze
+    if ActiveSupport::VERSION::MAJOR > 5 &&
+      Rails.application.config.autoloader == :zeitwerk
+      ActiveSupport::Dependencies.autoload_paths.delete(
+        Rails.root.join("app", "indices").to_s
+      )
     end
+
+    Rails.application.config.eager_load_paths -=
+      ThinkingSphinx::Configuration.instance.index_paths
+    Rails.application.config.eager_load_paths.freeze
   end
 
   rake_tasks do

data/lib/thinking_sphinx/search/stale_ids_exception.rb

@@ -9,6 +9,7 @@ class ThinkingSphinx::Search::StaleIdsException < StandardError
   end
 
   def message
-    "Record IDs found by Sphinx but not by ActiveRecord : #{ids.join(', ')}"
+    "Record IDs found by Sphinx but not by ActiveRecord : #{ids.join(', ')}\n" \
+    "https://freelancing-gods.com/thinking-sphinx/v5/common_issues.html#record-ids"
   end
 end

data/lib/thinking_sphinx/sinatra.rb

@@ -3,5 +3,5 @@
 require 'thinking_sphinx'
 
 ActiveSupport.on_load :active_record do
-  include ThinkingSphinx::ActiveRecord::Base
+  require 'thinking_sphinx/active_record'
 end

data/spec/acceptance/sql_deltas_spec.rb

@@ -16,7 +16,7 @@ describe 'SQL delta indexing', :live => true do
     )
     sleep 0.25
 
-    expect(Book.search('Terry Pratchett').to_a).to eq([guards, men])
+    expect(Book.search('Terry Pratchett').to_a).to match_array([guards, men])
   end
 
   it "automatically indexes updated records" do

data/spec/thinking_sphinx/active_record/index_spec.rb

@@ -5,7 +5,9 @@ require 'spec_helper'
 describe ThinkingSphinx::ActiveRecord::Index do
   let(:index)        { ThinkingSphinx::ActiveRecord::Index.new :user }
   let(:config)       { double('config', :settings => {},
-    :indices_location => 'location', :next_offset => 8) }
+    :indices_location => 'location', :next_offset => 8,
+    :index_set_class => index_set_class) }
+  let(:index_set_class) { double :reference_name => :user }
 
   before :each do
     allow(ThinkingSphinx::Configuration).to receive_messages :instance => config

data/thinking-sphinx.gemspec

@@ -5,7 +5,7 @@ $:.push File.expand_path('../lib', __FILE__)
 
 Gem::Specification.new do |s|
   s.name        = 'thinking-sphinx'
-  s.version     = '5.2.1'
+  s.version     = '5.3.0'
   s.platform    = Gem::Platform::RUBY
   s.authors     = ["Pat Allan"]
   s.email       = ["pat@freelancing-gods.com"]

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: thinking-sphinx
 version: !ruby/object:Gem::Version
-  version: 5.2.1
+  version: 5.3.0
 platform: ruby
 authors:
 - Pat Allan
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-08-09 00:00:00.000000000 Z
+date: 2021-08-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord