pg_search 2.3.6 → 2.3.7

data/lib/pg_search/multisearch.rb

@@ -7,9 +7,11 @@ module PgSearch
     class << self
       def rebuild(model, deprecated_clean_up = nil, clean_up: true, transactional: true)
         unless deprecated_clean_up.nil?
-          ActiveSupport::Deprecation.warn(
+          warn(
             "pg_search 3.0 will no longer accept a boolean second argument to PgSearchMultisearch.rebuild, " \
-            "use keyword argument `clean_up:` instead."
+            "use keyword argument `clean_up:` instead.",
+            category: :deprecated,
+            uplevel: 1
           )
           clean_up = deprecated_clean_up
         end

data/lib/pg_search/multisearchable.rb

@@ -7,12 +7,12 @@ module PgSearch
     def self.included(mod)
       mod.class_eval do
         has_one :pg_search_document,
-                as: :searchable,
-                class_name: "PgSearch::Document",
-                dependent: :delete
+          as: :searchable,
+          class_name: "PgSearch::Document",
+          dependent: :delete
 
         after_save :update_pg_search_document,
-                   if: -> { PgSearch.multisearch_enabled? }
+          if: -> { PgSearch.multisearch_enabled? }
       end
     end
 
@@ -39,7 +39,7 @@ module PgSearch
       conditions.all? { |condition| condition.to_proc.call(self) }
     end
 
-    def update_pg_search_document # rubocop:disable Metrics/AbcSize
+    def update_pg_search_document
       if_conditions = Array(pg_search_multisearchable_options[:if])
       unless_conditions = Array(pg_search_multisearchable_options[:unless])
 
@@ -50,7 +50,7 @@ module PgSearch
       if should_have_document
         create_or_update_pg_search_document
       else
-        pg_search_document&.destroy
+        pg_search_document&.destroy # standard:disable Rails/SaveBang
       end
     end
 
@@ -58,7 +58,7 @@ module PgSearch
       if !pg_search_document
         create_pg_search_document(pg_search_document_attrs)
       elsif should_update_pg_search_document?
-        pg_search_document.update(pg_search_document_attrs)
+        pg_search_document.update(pg_search_document_attrs) # standard:disable Rails/SaveBang
       end
     end
   end

data/lib/pg_search/normalizer.rb

@@ -10,11 +10,11 @@ module PgSearch
       return sql_expression unless config.ignore.include?(:accents)
 
       sql_node = case sql_expression
-                 when Arel::Nodes::Node
-                   sql_expression
-                 else
-                   Arel.sql(sql_expression)
-                 end
+      when Arel::Nodes::Node
+        sql_expression
+      else
+        Arel.sql(sql_expression)
+      end
 
       Arel::Nodes::NamedFunction.new(
         PgSearch.unaccent_function,

data/lib/pg_search/scope_options.rb

@@ -90,11 +90,32 @@ module PgSearch
     end
 
     def conditions
-      config.features
-            .reject { |_feature_name, feature_options| feature_options && feature_options[:sort_only] }
-            .map { |feature_name, _feature_options| feature_for(feature_name).conditions }
-            .inject { |accumulator, expression| Arel::Nodes::Or.new(accumulator, expression) }
+      expressions =
+        config.features
+          .reject { |_feature_name, feature_options| feature_options && feature_options[:sort_only] }
+          .map { |feature_name, _feature_options| feature_for(feature_name).conditions }
+
+      or_node(expressions)
+    end
+
+    # https://github.com/rails/rails/pull/51492
+    # :nocov:
+    # standard:disable Lint/DuplicateMethods
+    or_arity = Arel::Nodes::Or.instance_method(:initialize).arity
+    case or_arity
+    when 1
+      def or_node(expressions)
+        Arel::Nodes::Or.new(expressions)
+      end
+    when 2
+      def or_node(expressions)
+        expressions.inject { |accumulator, expression| Arel::Nodes::Or.new(accumulator, expression) }
+      end
+    else
+      raise "Unsupported arity #{or_arity} for Arel::Nodes::Or#initialize"
     end
+    # :nocov:
+    # standard:enable Lint/DuplicateMethods
 
     def order_within_rank
       config.order_within_rank || "#{primary_key} ASC"
@@ -108,7 +129,7 @@ module PgSearch
       if config.associations.any?
         config.associations.map do |association|
           association.join(primary_key)
-        end.join(' ')
+        end.join(" ")
       end
     end
 

data/lib/pg_search/tasks.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'rake'
-require 'pg_search'
+require "rake"
+require "pg_search"
 
 namespace :pg_search do
   namespace :multisearch do

data/lib/pg_search/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module PgSearch
-  VERSION = '2.3.6'
+  VERSION = "2.3.7"
 end

data/lib/pg_search.rb

@@ -18,7 +18,7 @@ module PgSearch
   autoload :Document, "pg_search/document"
 
   def self.included(base)
-    ActiveSupport::Deprecation.warn <<~MESSAGE
+    warn(<<~MESSAGE, category: :deprecated, uplevel: 1)
       Directly including `PgSearch` into an Active Record model is deprecated and will be removed in pg_search 3.0.
 
       Please replace `include PgSearch` with `include PgSearch::Model`.
@@ -34,8 +34,8 @@ module PgSearch
   self.unaccent_function = "unaccent"
 
   class << self
-    def multisearch(*args)
-      PgSearch::Document.search(*args)
+    def multisearch(...)
+      PgSearch::Document.search(...)
     end
 
     def disable_multisearch

data/pg_search.gemspec

@@ -1,40 +1,25 @@
 # frozen_string_literal: true
 
-$LOAD_PATH.push File.expand_path('lib', __dir__)
-require 'pg_search/version'
+$LOAD_PATH.push File.expand_path("lib", __dir__)
+require "pg_search/version"
 
-Gem::Specification.new do |s| # rubocop:disable Metrics/BlockLength
-  s.name        = 'pg_search'
-  s.version     = PgSearch::VERSION
-  s.platform    = Gem::Platform::RUBY
-  s.authors     = ['Grant Hutchins', 'Case Commons, LLC']
-  s.email       = %w[gems@nertzy.com casecommons-dev@googlegroups.com]
-  s.homepage    = 'https://github.com/Casecommons/pg_search'
-  s.summary     = "PgSearch builds Active Record named scopes that take advantage of PostgreSQL's full text search"
+Gem::Specification.new do |s|
+  s.name = "pg_search"
+  s.version = PgSearch::VERSION
+  s.platform = Gem::Platform::RUBY
+  s.authors = ["Grant Hutchins", "Case Commons, LLC"]
+  s.email = %w[gems@nertzy.com casecommons-dev@googlegroups.com]
+  s.homepage = "https://github.com/Casecommons/pg_search"
+  s.summary = "PgSearch builds Active Record named scopes that take advantage of PostgreSQL's full text search"
   s.description = "PgSearch builds Active Record named scopes that take advantage of PostgreSQL's full text search"
-  s.licenses    = ['MIT']
+  s.licenses = ["MIT"]
   s.metadata["rubygems_mfa_required"] = "true"
 
-  s.files         = `git ls-files`.split("\n")
-  s.test_files    = `git ls-files -- spec/*`.split("\n")
-  s.require_paths = ['lib']
+  s.files = `git ls-files -z`.split("\x0")
+  s.require_paths = ["lib"]
 
-  s.add_dependency 'activerecord', '>= 5.2'
-  s.add_dependency 'activesupport', '>= 5.2'
+  s.add_dependency "activerecord", ">= 6.1"
+  s.add_dependency "activesupport", ">= 6.1"
 
-  s.add_development_dependency 'pry'
-  s.add_development_dependency 'rake'
-  s.add_development_dependency 'rspec'
-  s.add_development_dependency 'rubocop'
-  s.add_development_dependency 'rubocop-performance'
-  s.add_development_dependency 'rubocop-rails'
-  s.add_development_dependency 'rubocop-rake'
-  s.add_development_dependency 'rubocop-rspec'
-  s.add_development_dependency 'simplecov'
-  s.add_development_dependency 'simplecov-lcov'
-  s.add_development_dependency 'undercover'
-  s.add_development_dependency 'warning'
-  s.add_development_dependency 'with_model'
-
-  s.required_ruby_version = '>= 2.6'
+  s.required_ruby_version = ">= 3.0"
 end

data/spec/.rubocop.yml

@@ -1,14 +1,27 @@
 inherit_from:
   - ../.rubocop.yml
 
-Layout/LineLength:
-  Enabled: false
+RSpec/ContextWording:
+  Prefixes:
+    - using
+    - via
+    - when
+    - with
+    - without
 
-Lint/SuppressedException:
-  Enabled: false
+RSpec/DescribedClass:
+  Enabled: true
 
-Lint/UselessAssignment:
-  Enabled: false
+RSpec/ExampleLength:
+  Max: 15
 
-Style/BlockDelimiters:
+RSpec/ExpectInHook:
   Enabled: false
+
+RSpec/FilePath:
+  CustomTransform:
+    TSearch: "tsearch"
+    DMetaphone: "dmetaphone"
+
+RSpec/MultipleExpectations:
+  Max: 5

data/spec/integration/.rubocop.yml

@@ -4,8 +4,8 @@ inherit_from:
 RSpec/DescribeClass:
   Enabled: false
 
-RSpec/MultipleExpectations:
+RSpec/ExampleLength:
   Enabled: false
 
-RSpec/ExampleLength:
+RSpec/MultipleExpectations:
   Enabled: false