thinking-sphinx 4.3.0 → 4.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 30fe9d6940dc5f385310346374829bc1ff941332583060ad549ea8675a6a34ce
-  data.tar.gz: 4b5a25fe1120cba82bf9bd6b9f80be7e1d5175dce887d8acaffc3ba1da78e458
+  metadata.gz: 03cdbbdc5e88bbe791ca53d26ec0408fcc0ac0baa9ab03de58d2e8fb821c7cdd
+  data.tar.gz: b6ed1c04b02e2db70606334d7cc5ea715b6540d92d21d9415b47ed0ef02d8a65
 SHA512:
-  metadata.gz: f8f25c99b61c6bf80f9a81eb71ab1c1a6af005009099cd18c7ef1a13e694f4e65b7ddc3d654167033ecf0fac072c91d88864d6ab5cf44c5f00fea69447195ce2
-  data.tar.gz: b78abfa34ebcfb24ed65e99b07dfb04dc545cc2be74c994a9b786a507b8b69bfa46afce05c74830dfde5bebc6b823e0d44b3073637967090b35ffe85c9629b41
+  metadata.gz: 2731b4341a0eb74f3c0312d4535a1cbc1c8b50e4694ce1a61c5c7fc54981b02b3336467779a9beb437013eea812ee6e1eb9c81e9905247f35b094575e6a7de6d
+  data.tar.gz: 0f3b3eb6a3437100e8337465610cf1eca9c6ae7d5159838cfe0011748e87f59bace8e6cd5c4a199f3dd87dd63ce8048bae5aa0ddd88a72e9d95973e18d88204d

data/CHANGELOG.markdown

@@ -2,13 +2,21 @@
 
 All notable changes to this project (at least, from v3.0.0 onwards) are documented in this file.
 
+## 4.3.1 - 2019-06-27
+
+[Release Notes](https://github.com/pat/thinking-sphinx/releases/tag/v4.3.1)
+
+### Fixed
+
+* Fixed loading of index files to work with Rails 6 and Zeitwerk ([#1137](https://github.com/pat/thinking-sphinx/issues/1137)).
+
 ## 4.3.0 - 2019-05-18
 
 [Release Notes](https://github.com/pat/thinking-sphinx/releases/tag/v4.3.0)
 
 ### Added
 
-* Allow overriding of Sphinx's running state, which is useful when Sphinx commands are interacting with a remote Sphinx daemon. As per discussions in [#1131](https://github.com/pat/thinking-sphinx/pull/1124).
+* Allow overriding of Sphinx's running state, which is useful when Sphinx commands are interacting with a remote Sphinx daemon. As per discussions in [#1131](https://github.com/pat/thinking-sphinx/pull/1131).
 * Allow skipping of directory creation, as per discussions in [#1131](https://github.com/pat/thinking-sphinx/pull/1131).
 
 ### Fixed

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 v4.3.0.
+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 v4.3.1.
 
 h2. Upgrading
 

data/lib/thinking_sphinx/configuration.rb

@@ -87,9 +87,7 @@ class ThinkingSphinx::Configuration < Riddle::Configuration
       return if @preloaded_indices
 
       index_paths.each do |path|
-        Dir["#{path}/**/*.rb"].sort.each do |file|
-          ActiveSupport::Dependencies.require_or_load file
-        end
+        Dir["#{path}/**/*.rb"].sort.each { |file| preload_index file }
       end
 
       normalise
@@ -99,6 +97,14 @@ class ThinkingSphinx::Configuration < Riddle::Configuration
     end
   end
 
+  def preload_index(file)
+    if ActiveRecord::VERSION::MAJOR < 5
+      ActiveSupport::Dependencies.require_or_load file
+    else
+      load file
+    end
+  end
+
   def render
     preload_indices
 

data/spec/thinking_sphinx/configuration_spec.rb

@@ -4,11 +4,18 @@ require 'spec_helper'
 
 describe ThinkingSphinx::Configuration do
   let(:config) { ThinkingSphinx::Configuration.instance }
+  let(:use_load?) { ActiveRecord::VERSION::MAJOR >= 5 }
+  let(:loading_object) { use_load? ? config : ActiveSupport::Dependencies }
+  let(:loading_method) { use_load? ? :load : :require_or_load }
 
   after :each do
     ThinkingSphinx::Configuration.reset
   end
 
+  def expect_loading_of(file)
+    expect(loading_object).to receive(loading_method).with(file).once
+  end
+
   describe '.instance' do
     it "returns an instance of ThinkingSphinx::Configuration" do
       expect(ThinkingSphinx::Configuration.instance).
@@ -254,10 +261,8 @@ describe ThinkingSphinx::Configuration do
         '/path/to/indices/bar_index.rb'
       ]
 
-      expect(ActiveSupport::Dependencies).to receive(:require_or_load).
-        with('/path/to/indices/foo_index.rb').once
-      expect(ActiveSupport::Dependencies).to receive(:require_or_load).
-        with('/path/to/indices/bar_index.rb').once
+      expect_loading_of('/path/to/indices/foo_index.rb')
+      expect_loading_of('/path/to/indices/bar_index.rb')
 
       config.preload_indices
     end
@@ -269,10 +274,8 @@ describe ThinkingSphinx::Configuration do
         '/path/to/indices/bar_index.rb'
       ]
 
-      expect(ActiveSupport::Dependencies).to receive(:require_or_load).
-        with('/path/to/indices/foo_index.rb').once
-      expect(ActiveSupport::Dependencies).to receive(:require_or_load).
-        with('/path/to/indices/bar_index.rb').once
+      expect_loading_of('/path/to/indices/foo_index.rb')
+      expect_loading_of('/path/to/indices/bar_index.rb')
 
       config.preload_indices
       config.preload_indices
@@ -316,10 +319,8 @@ describe ThinkingSphinx::Configuration do
         '/path/to/indices/bar_index.rb'
       ]
 
-      expect(ActiveSupport::Dependencies).to receive(:require_or_load).
-        with('/path/to/indices/foo_index.rb').once
-      expect(ActiveSupport::Dependencies).to receive(:require_or_load).
-        with('/path/to/indices/bar_index.rb').once
+      expect_loading_of('/path/to/indices/foo_index.rb')
+      expect_loading_of('/path/to/indices/bar_index.rb')
 
       config.render
     end
@@ -331,10 +332,8 @@ describe ThinkingSphinx::Configuration do
         '/path/to/indices/bar_index.rb'
       ]
 
-      expect(ActiveSupport::Dependencies).to receive(:require_or_load).
-        with('/path/to/indices/foo_index.rb').once
-      expect(ActiveSupport::Dependencies).to receive(:require_or_load).
-        with('/path/to/indices/bar_index.rb').once
+      expect_loading_of('/path/to/indices/foo_index.rb')
+      expect_loading_of('/path/to/indices/bar_index.rb')
 
       config.preload_indices
       config.preload_indices
@@ -372,6 +371,8 @@ describe ThinkingSphinx::Configuration do
     end
 
     it "skips creating a directory when the binlog_path is blank" do
+      allow(loading_object).to receive(loading_method)
+
       allow(FileUtils).to receive_messages :mkdir_p => true
       allow(config).to receive_messages :searchd => double(:binlog_path => '')
 

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     = '4.3.0'
+  s.version     = '4.3.1'
   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: 4.3.0
+  version: 4.3.1
 platform: ruby
 authors:
 - Pat Allan
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-05-18 00:00:00.000000000 Z
+date: 2019-06-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -521,8 +521,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: thinking-sphinx
-rubygems_version: 2.7.6.2
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: A smart wrapper over Sphinx for ActiveRecord