elasticsearch_record 1.5.1 → 1.5.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b6e4ca9acc6ee9abda4a3dacec4a75f9f1af77c361dbff3b53aa30648dfa5a5f
-  data.tar.gz: f6cf2edda611efb7daa6585764c09c2320040c8970555d64dab96ced763d2f19
+  metadata.gz: 238cf0f5e82cb385fc6d0f4e7f0416192f1cf65134b9726de16149be5fe6dc6b
+  data.tar.gz: 002b7d91869f2c0a9f03729c9c2c0db3719abfa9d523925cfeaf5bacf101e563
 SHA512:
-  metadata.gz: 1d9327c0b6500d47ec5459ebd5e6053d8178d59b33cd690d5f9a2752ec22cc52d2728b295dd4644c5f7fa6b794c6a8d69305b372c6c138e2ae8c717500669453
-  data.tar.gz: 050e345874250113d23c69d4de6d3c8f35c7c6b4538fe8d1ee3dbeaf86a64dc7c745175a8301c5d78121481927bff34033a321e390fbaddf3356110951afb3c4
+  metadata.gz: b58fc1dbdd310c1e5c8c47bb54398a4300042c063afd6a49543a5c6b2a270f2ffe7fb168b357ff1da5cb738ab602550bbf8a1297d4464089c5c0d8809a4e3a8c
+  data.tar.gz: 10be3f8fa2ea798bf58f2187ee7ea23099545ddad32036642e3b16d5e48cde7ccbabaff2bc68bc0baa2c71c41409de14abebedd11d6b0bcbe026c0e8200f0db2

data/README.md

@@ -337,6 +337,8 @@ SearchUser.where(name: 'Peter').limit(nil)
 # returns up to 10_000 items ...
 # => [...]
 
+# hint: setting the 'max_result_window' can also be done by providing '__max__' wto the limit method: SearchUser.limit('__max__')
+
 # hint: if you want more than 10_000 use the +#pit_results+ method!
 ```
 

data/docs/CHANGELOG.md

@@ -1,5 +1,12 @@
 # ElasticsearchRecord - CHANGELOG
 
+## [1.5.3] - 2023-07-14
+* [fix] `ElasticsearchRecord::Relation#where!` on nested, provided `:none` key
+* [ref] minor code tweaks and comment updates
+
+## [1.5.2] - 2023-07-12
+* [fix] `ElasticsearchRecord::Relation#limit` setter method `limit_value=` to work with **delegate_query_nil_limit?**
+
 ## [1.5.1] - 2023-07-11
 * [fix] `ElasticsearchRecord::ModelApi` 'drop!' & 'truncate!' methods to support correct parameter 'confirm'
 * [ref] improved yard documentation

data/docs/CODE_OF_CONDUCT.md

@@ -39,7 +39,7 @@ This Code of Conduct applies within all community spaces, and also applies when
 
 ## Enforcement
 
-Instances of abusive, harassing, or otherwise unacceptable behavior may be reported to the community leaders responsible for enforcement at tg@reimbursement.institute. All complaints will be reviewed and investigated promptly and fairly.
+Instances of abusive, harassing, or otherwise unacceptable behavior may be reported to the community leaders responsible for enforcement at info@ruby-smart.org. All complaints will be reviewed and investigated promptly and fairly.
 
 All community leaders are obligated to respect the privacy and security of the reporter of any incident.
 

data/lib/arel/visitors/elasticsearch_query.rb

@@ -138,12 +138,11 @@ module Arel # :nodoc: all
 
       # CUSTOM node by elasticsearch_record
       def visit_Query(o)
-        # in some cases we don't have a kind, but where conditions.
-        # in this case we force the kind as +:bool+.
-        kind = :bool if o.wheres.present? && o.kind.blank?
-
-        # resolve kind, if not already set
-        kind ||= o.kind.present? ? visit(o.kind.expr) : nil
+        # resolves the query kind.
+        # PLEASE NOTE: in some cases there is no kind, but an existing +where+ conditions.
+        # This will then be treat as +:bool+.
+        kind = o.kind.present? ? visit(o.kind.expr).presence : nil
+        kind ||= :bool if o.wheres.present?
 
         # check for existing kind - we cannot create a node if we don't have any kind
         return unless kind
@@ -423,6 +422,7 @@ module Arel # :nodoc: all
         o
       end
 
+      # alias for RAW returns
       alias :visit_Integer :visit_Struct_Raw
       alias :visit_Symbol :visit_Struct_Raw
       alias :visit_Hash :visit_Struct_Raw
@@ -430,7 +430,6 @@ module Arel # :nodoc: all
       alias :visit_String :visit_Struct_Raw
       alias :visit_Arel_Nodes_SqlLiteral :visit_Struct_Raw
 
-
       # used by insert / update statements.
       # does not claim / assign any values!
       # returns a Hash of key => value pairs
@@ -453,6 +452,7 @@ module Arel # :nodoc: all
         o.name
       end
 
+      # alias for ATTRIBUTE returns
       alias :visit_Arel_Attributes_Attribute :visit_Struct_Attribute
       alias :visit_Arel_Nodes_UnqualifiedColumn :visit_Struct_Attribute
       alias :visit_ActiveModel_Attribute_FromUser :visit_Struct_Attribute
@@ -473,6 +473,7 @@ module Arel # :nodoc: all
         o.value
       end
 
+      # alias for BIND returns
       alias :visit_ActiveModel_Attribute :visit_Struct_BindValue
       alias :visit_ActiveRecord_Relation_QueryAttribute :visit_Struct_BindValue
 
@@ -484,6 +485,7 @@ module Arel # :nodoc: all
         collect(o)
       end
 
+      # alias for ARRAY returns
       alias :visit_Set :visit_Array
     end
   end

data/lib/arel/visitors/elasticsearch_schema.rb

@@ -89,8 +89,7 @@ module Arel # :nodoc: all
         # prepare query
         claim(:type, ::ElasticsearchRecord::Query::TYPE_INDEX_UPDATE_SETTING)
 
-        # special overcomplicated blocks to assign a hash of settings directly to the body...
-        # todo: refactor this in future versions
+        # overcomplicated blocks to assign a hash of settings directly to the body
         assign(:__query__, {}) do
           assign(:body, {}) do
             resolve(o.items, :visit_TableSettingDefinition)

data/lib/elasticsearch_record/gem_version.rb

@@ -9,7 +9,7 @@ module ElasticsearchRecord
   module VERSION
     MAJOR = 1
     MINOR = 5
-    TINY  = 1
+    TINY  = 3
     PRE   = nil
 
     STRING = [MAJOR, MINOR, TINY, PRE].compact.join(".")

data/lib/elasticsearch_record/persistence.rb

@@ -61,21 +61,17 @@ module ElasticsearchRecord
 
       private
 
-      # WARNING: BETA!!!
       # Resolves the +auto_increment+ status from the tables +_meta+ attributes.
       def _insert_with_auto_increment(values)
-        # check, if the primary_key's values is provided.
-        # so, no need to resolve a +auto_increment+ value, but provide
-        if values[self.primary_key].present?
-          # resolve id from values
-          id = values[self.primary_key]
-
+        # check, if the primary_key's value is provided.
+        # so, no need to resolve a +auto_increment+ value, but provide the id directly
+        if (id = values[self.primary_key]).present?
           yield({id: id})
         elsif auto_increment?
           ids = [
-            # we try to resolve the current-auto-increment value from the tables meta
+            # try to resolve the current-auto-increment value from the tables meta
             connection.table_metas(self.table_name).dig('auto_increment').to_i + 1,
-            # for secure reasons, we also resolve the current maximum value for the primary key
+            # for secure reasons: also resolve the current maximum value for the primary key
             self.unscoped.all.maximum(self.primary_key).to_i + 1
           ]
 

data/lib/elasticsearch_record/relation/query_methods.rb

@@ -184,35 +184,36 @@ module ElasticsearchRecord
 
       # creates a condition on the relation.
       # There are several possibilities to call this method.
+      #
       # @example
       #   # create a simple 'term' condition on the query[:filter] param
-      #     where({name: 'hans'})
-      #     > query[:filter] << { term: { name: 'hans' } }
+      #   where({name: 'hans'})
+      #   #> query[:filter] << { term: { name: 'hans' } }
       #
       #   # create a simple 'terms' condition on the query[:filter] param
-      #     where({name: ['hans','peter']})
-      #     > query[:filter] << { terms: { name: ['hans','peter'] } }
+      #   where({name: ['hans','peter']})
+      #   #> query[:filter] << { terms: { name: ['hans','peter'] } }
       #
-      #     where(:must_not, term: {name: 'horst'})
-      #     where(:query_string, "(new york OR dublin)", fields: ['name','description'])
+      #   where(:must_not, term: {name: 'horst'})
+      #   where(:query_string, "(new york OR dublin)", fields: ['name','description'])
       #
       #   # nested array
       #   where([ [:filter, {...}], [:must_not, {...}]])
-      def where(*args)
-        return none if args[0] == :none
-
-        super
-      end
-
+      #
+      #   # invalidate query
+      #   where(:none)
+      #
       def where!(opts, *rest)
         # :nodoc:
         case opts
-          # check the first provided parameter +opts+ and validate, if this is an alias for "must, must_not, should or filter"
-          # if true, we expect the rest[0] to be a hash.
-          # For this correlation we forward this as RAW-data without check & manipulation
         when Symbol
           case opts
+          when :none
+            none!
           when :filter, :must, :must_not, :should
+            # check the first provided parameter +opts+ and validate, if this is an alias for "must, must_not, should or filter".
+            # if true, we expect the rest[0] to be a hash.
+            # For this correlation we forward this as RAW-data without check & manipulation
             send("#{opts}!", *rest)
           else
             raise ArgumentError, "Unsupported prefix type '#{opts}'. Allowed types are: :filter, :must, :must_not, :should"
@@ -220,9 +221,7 @@ module ElasticsearchRecord
         when Array
           # check if this is a nested array of multiple [<kind>,<data>]
           if opts[0].is_a?(Array)
-            opts.each { |item|
-              where!(*item)
-            }
+            opts.each { |item| where!(*item) }
           else
             where!(*opts, *rest)
           end

data/lib/elasticsearch_record/relation/value_methods.rb

@@ -44,11 +44,11 @@ module ElasticsearchRecord
       end
 
       # overwrite the limit_value setter, to provide a special behaviour of auto-setting the +max_result_window+.
-      def limit=(limit)
+      def limit_value=(limit)
         if limit == '__max__' || (limit.nil? && delegate_query_nil_limit?)
           super(max_result_window)
         else
-          super
+          super(limit)
         end
       end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: elasticsearch_record
 version: !ruby/object:Gem::Version
-  version: 1.5.1
+  version: 1.5.3
 platform: ruby
 authors:
 - Tobias Gonsior
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-07-11 00:00:00.000000000 Z
+date: 2023-07-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord