sequel 4.49.0 → 5.3.0

data/lib/sequel/plugins/csv_serializer.rb

@@ -1,42 +1,35 @@
 # frozen-string-literal: true
 
-if RUBY_VERSION < "1.9"
-  # :nocov:
-  require 'fastercsv'
-  # :nocov:
-else
-  require 'csv'
-end
+require 'csv'
 
 module Sequel
   module Plugins
     # csv_serializer handles serializing entire Sequel::Model objects to CSV,
     # as well as support for deserializing CSV directly into Sequel::Model
-    # objects.  It requires either the csv standard library when usnig ruby 1.9+,
-    # or the fastercsv gem when using ruby 1.8.
+    # objects.  It requires the csv standard library.
     #
     # Basic Example:
     #
     #   album = Album[1]
-    #   album.to_csv(:write_headers=>true)
+    #   album.to_csv(write_headers: true)
     #   # => "id,name,artist_id\n1,RF,2\n"
     #
     # You can provide options to control the CSV output:
     #
-    #   album.to_csv(:only=>:name)
-    #   album.to_csv(:except=>[:id, :artist_id])
+    #   album.to_csv(only: :name)
+    #   album.to_csv(except: [:id, :artist_id])
     #   # => "RF\n"
     #
     # +to_csv+ also exists as a class and dataset method, both of which return
     # all objects in the dataset:
     #
     #   Album.to_csv
-    #   Album.where(:artist_id=>1).to_csv
+    #   Album.where(artist_id: 1).to_csv
     #
-    # If you have an existing array of model instance you want to convert to
+    # If you have an existing array of model instances you want to convert to
     # CSV, you can call the class to_csv method with the :array option:
     #
-    #   Album.to_csv(:array=>[Album[1], Album[2]])
+    #   Album.to_csv(array: [Album[1], Album[2]])
     #
     # In addition to creating CSV, this plugin also enables Sequel::Model
     # classes to create instances directly from CSV using the from_csv class
@@ -48,7 +41,7 @@ module Sequel
     # The array_from_csv class method exists to parse arrays of model instances
     # from CSV:
     #
-    #   csv = Album.where(:artist_id=>1).to_csv
+    #   csv = Album.where(artist_id: 1).to_csv
     #   albums = Album.array_from_csv(csv)
     #
     # These do not necessarily round trip, since doing so would let users
@@ -56,7 +49,7 @@ module Sequel
     # call set with the values in the hash.  If you want to specify the allowed
     # fields, you can use the :headers option.
     #
-    #   Album.from_csv(album.to_csv, :headers=>%w'id name')
+    #   Album.from_csv(album.to_csv, headers: %w'id name')
     #
     # If you want to update an existing instance, you can use the from_csv
     # instance method:
@@ -77,7 +70,7 @@ module Sequel
       # Set up the column readers to do deserialization and the column writers
       # to save the value in deserialized_values
       def self.configure(model, opts = OPTS)
-        model.instance_eval do
+        model.instance_exec do
           @csv_serializer_opts = (@csv_serializer_opts || OPTS).merge(opts)
         end
       end
@@ -155,7 +148,7 @@ module Sequel
           opts = model.process_csv_serializer_opts(opts)
 
           CSV.generate(opts) do |csv|
-            csv << opts[:headers].map{|k| send(k)}
+            csv << opts[:headers].map{|k| public_send(k)}
           end
         end
       end
@@ -174,7 +167,7 @@ module Sequel
 
           CSV.generate(opts) do |csv|
             items.each do |object|
-              csv << opts[:headers].map{|header| object.send(header) }
+              csv << opts[:headers].map{|header| object.public_send(header) }
             end
           end
         end

data/lib/sequel/plugins/dataset_associations.rb

@@ -12,7 +12,7 @@ module Sequel
     #     plugin :dataset_associations
     #     one_to_many :albums
     #   end
-    #   Artist.where(id=>1..100).albums
+    #   Artist.where(id: 1..100).albums
     #   # SELECT * FROM albums
     #   # WHERE (albums.artist_id IN (
     #   #   SELECT id FROM artists
@@ -27,7 +27,7 @@ module Sequel
     # As the dataset methods return datasets, you can easily chain the
     # methods to get associated datasets of associated datasets:
     #
-    #   Artist.where(id=>1..100).albums.where{name < 'M'}.tags
+    #   Artist.where(id: 1..100).albums.where{name < 'M'}.tags
     #   # SELECT tags.* FROM tags
     #   # WHERE (tags.id IN (
     #   #   SELECT albums_tags.tag_id FROM albums

data/lib/sequel/plugins/def_dataset_method.rb

@@ -6,7 +6,7 @@ module Sequel
     # for defining dataset methods:
     #
     #   Album.def_dataset_method(:by_name) do |name|
-    #     where(:name=>name)
+    #     where(name: name)
     #   end
     #
     # Additionally, this adds support for Model.subset, which can also
@@ -41,8 +41,8 @@ module Sequel
         #
         #   # Add new dataset method and class method that calls it
         #   Artist.def_dataset_method(:by_name){order(:name)}
-        #   Artist.where(:name.like('A%')).by_name
-        #   Artist.by_name.where(:name.like('A%'))
+        #   Artist.where(Sequel[:name].like('A%')).by_name
+        #   Artist.by_name.where(Sequel[:name].like('A%'))
         #
         #   # Just add a class method that calls an existing dataset method
         #   Artist.def_dataset_method(:paginate)
@@ -63,7 +63,7 @@ module Sequel
         # they can be chained.
         # For example:
         #
-        #   Topic.subset(:joes, :username.like('%joe%'))
+        #   Topic.subset(:joes, Sequel[:username].like('%joe%'))
         #   Topic.subset(:popular){num_posts > 100}
         #   Topic.subset(:recent){created_on > Date.today - 7}
         #
@@ -75,7 +75,7 @@ module Sequel
         # have more than 100 posts and were created less than
         # 7 days ago.
         #
-        # Both the args given and the block are passed to <tt>Dataset#filter</tt>.
+        # Both the args given and the block are passed to <tt>Dataset#where</tt>.
         #
         # This method creates dataset methods that do not accept arguments.  To create
         # dataset methods that accept arguments, you should use define a

data/lib/sequel/plugins/defaults_setter.rb

@@ -10,7 +10,7 @@ module Sequel
     #   album = Album.new
     #   album.a # => nil
     #   album.b # => 2
-    #   album = Album.new(:a=>1, :b=>3)
+    #   album = Album.new(a: 1, b: 3)
     #   album.a # => 1
     #   album.b # => 3
     #

data/lib/sequel/plugins/delay_add_association.rb

@@ -40,7 +40,7 @@ module Sequel
           if opts.dataset_need_primary_key? && new?
             o = make_add_associated_object(opts, o)
             delay_validate_associated_object(opts, o)
-            send(opts[:name]) << o
+            public_send(opts[:name]) << o
             after_create_hook{super(opts, o, *args)}
             o
           else

data/lib/sequel/plugins/finder.rb

@@ -7,7 +7,7 @@ module Sequel
     # that represents a model class method that returns a dataset:
     #
     #   def Artist.by_name(name)
-    #     where(:name=>name)
+    #     where(name: name)
     #   end
     #
     #   Artist.finder :by_name
@@ -18,7 +18,7 @@ module Sequel
     #
     # The alternative way to use this to pass your own block:
     #
-    #   Artist.finder(:name=>:first_by_name){|pl, ds| ds.where(:name=>pl.arg).limit(1)}
+    #   Artist.finder(name: :first_by_name){|pl, ds| ds.where(name: pl.arg).limit(1)}
     #
     # Additionally, there is a Model.prepared_finder method.  This works similarly
     # to Model.finder, but uses a prepared statement.  This limits the types of
@@ -50,7 +50,7 @@ module Sequel
         # that represents a model class method that returns a dataset:
         #
         #   def Artist.by_name(name)
-        #     where(:name=>name)
+        #     where(name: name)
         #   end
         #
         #   Artist.finder :by_name
@@ -61,7 +61,7 @@ module Sequel
         #
         # The alternative way to use this to pass your own block:
         #
-        #   Artist.finder(:name=>:first_by_name){|pl, ds| ds.where(:name=>pl.arg).limit(1)}
+        #   Artist.finder(name: :first_by_name){|pl, ds| ds.where(name: pl.arg).limit(1)}
         #
         # Note that if you pass your own block, you are responsible for manually setting
         # limits if necessary (as shown above).
@@ -86,7 +86,7 @@ module Sequel
         # This doesn't handle all possible cases.  For example, if you have a method such as:
         #
         #   def Artist.by_name(name)
-        #     name ? where(:name=>name) : exclude(:name=>nil)
+        #     name ? where(name: name) : exclude(name: nil)
         #   end
         #
         # Then calling a finder without an argument will not work as you expect.
@@ -128,7 +128,7 @@ module Sequel
               ds = if block
                 model.instance_exec(*args, &block)
               else
-                model.send(meth, *args)
+                model.public_send(meth, *args)
               end
               ds = ds.limit(1) if limit1
               model_name = model.name
@@ -144,7 +144,7 @@ module Sequel
               n = argn.call(model)
               block ||= lambda do |pl, model2|
                 args = (0...n).map{pl.arg}
-                ds = model2.send(meth, *args)
+                ds = model2.public_send(meth, *args)
                 ds = ds.limit(1) if limit1
                 ds
               end
@@ -154,7 +154,7 @@ module Sequel
           end
 
           @finder_loaders[meth_name] = loader_proc
-          mod = opts[:mod] || (class << self; self; end)
+          mod = opts[:mod] || singleton_class
           if prepare
             def_prepare_method(mod, meth_name)
           else
@@ -190,14 +190,14 @@ module Sequel
           finder(meth, opts, &block)
         end
 
-        Plugins.inherited_instance_variables(self, :@finders=>:dup, :@autoreloading_associations=>:hash_dup, :@default_association_options=>:dup, :@cache_associations=>nil, :@default_eager_limit_strategy=>nil)
+        Plugins.inherited_instance_variables(self, :@finders=>:dup, :@finder_loaders=>:dup)
 
         private
 
         # Define a finder method in the given module with the given method name that
         # load rows using the finder with the given name.
         def def_finder_method(mod, meth, type)
-          mod.send(:define_method, meth){|*args, &block| finder_for(meth).send(type, *args, &block)}
+          mod.send(:define_method, meth){|*args, &block| finder_for(meth).public_send(type, *args, &block)}
         end
 
         # Define a prepared_finder method in the given module that will call the associated prepared
@@ -234,6 +234,12 @@ module Sequel
             s
           end
         end
+
+        # Clear any finders when reseting the instance dataset
+        def reset_instance_dataset
+          Sequel.synchronize{@finders.clear} if @finders && !@finders.frozen?
+          super
+        end
       end
     end
   end

data/lib/sequel/plugins/force_encoding.rb

@@ -1,9 +1,8 @@
 # frozen-string-literal: true
 
-if RUBY_VERSION >= '1.9.0'
 module Sequel
   module Plugins
-    # The ForceEncoding plugin allows you force specific encodings for all
+    # The force_encoding plugin allows you force specific encodings for all
     # strings that are used by the model.  When model instances are loaded
     # from the database, all values in the hash that are strings are
     # forced to the given encoding.  Whenever you update a model column
@@ -77,8 +76,3 @@ module Sequel
     end
   end
 end
-else
-# :nocov:
-  raise LoadError, 'ForceEncoding plugin only works on Ruby 1.9+'
-# :nocov:
-end

data/lib/sequel/plugins/hook_class_methods.rb

@@ -31,14 +31,6 @@ module Sequel
     #   # Allow the use of hook class methods in the Album class
     #   Album.plugin :hook_class_methods
     module HookClassMethods
-      # SEQUEL5: Remove
-      DEPRECATION_REPLACEMENTS = {
-        :after_commit=>"Use after_save{db.after_commit{}} instead",
-        :after_destroy_commit=>"Use after_destroy{db.after_commit{}} instead",
-        :after_destroy_rollback=>"Use before_destroy{db.after_rollback{}} instead",
-        :after_rollback=>"Use before_save{db.after_rollback{}} instead"
-      }.freeze
-
       # Set up the hooks instance variable in the model.
       def self.apply(model)
         hooks = model.instance_variable_set(:@hooks, {})
@@ -49,52 +41,11 @@ module Sequel
         Model::HOOKS.each do |h|
           class_eval(<<-END, __FILE__, __LINE__ + 1)
             def #{h}(method = nil, &block)
-              #{"Sequel::Deprecation.deprecate('Sequel::Model.#{h} in the hook_class_methods plugin', #{DEPRECATION_REPLACEMENTS[h].inspect})" if DEPRECATION_REPLACEMENTS[h]}
               add_hook(:#{h}, method, &block)
             end
           END
         end
 
-        # This adds a new hook type. It will define both a class
-        # method that you can use to add hooks, as well as an instance method
-        # that you can use to call all hooks of that type.  The class method
-        # can be called with a symbol or a block or both.  If a block is given and
-        # and symbol is not, it adds the hook block to the hook type.  If a block
-        # and symbol are both given, it replaces the hook block associated with
-        # that symbol for a given hook type, or adds it if there is no hook block
-        # with that symbol for that hook type.  If no block is given, it assumes
-        # the symbol specifies an instance method to call and adds it to the hook
-        # type.
-        #
-        # If any before hook block returns false, the instance method will return false
-        # immediately without running the rest of the hooks of that type.
-        #
-        # It is recommended that you always provide a symbol to this method,
-        # for descriptive purposes.  It's only necessary to do so when you 
-        # are using a system that reloads code.
-        # 
-        # Example of usage:
-        #
-        #  class MyModel
-        #   add_hook_type :before_move_to
-        #   before_move_to(:check_move_allowed, &:allow_move?)
-        #   def move_to(there)
-        #     return if before_move_to == false
-        #     # move MyModel object to there
-        #   end
-        #  end
-        #
-        # Do not call this method with untrusted input, as that can result in
-        # arbitrary code execution.
-        def add_hook_type(*hooks)
-          Sequel::Deprecation.deprecate("Sequel::Model.add_hook_type", "You should add your own hook types manually")
-          hooks.each do |hook|
-            @hooks[hook] = []
-            instance_eval("def #{hook}(method = nil, &block); add_hook(:#{hook}, method, &block) end", __FILE__, __LINE__)
-            class_eval("def #{hook}; model.hook_blocks(:#{hook}){|b| return false if instance_eval(&b) == false} end", __FILE__, __LINE__)
-          end
-        end
-
         # Freeze hooks when freezing model class.
         def freeze
           @hooks.freeze.each_value(&:freeze)
@@ -121,7 +72,8 @@ module Sequel
         def add_hook(hook, tag, &block)
           unless block
             (raise Error, 'No hook method specified') unless tag
-            block = proc {send tag}
+            # Allow calling private hook methods
+            block = proc {send(tag)}
           end
           h = @hooks[hook]
           if tag && (old = h.find{|x| x[0] == tag})
@@ -137,63 +89,9 @@ module Sequel
       end
 
       module InstanceMethods
-        # SEQUEL5: Make :before_save, :before_destroy, :after_save, :after_destroy hooks use metaprogramming instead of specific definitions
-        [:before_create, :before_update, :before_validation].each do |h|
-          class_eval(<<-END, __FILE__, __LINE__+1)
-            def #{h}
-              model.hook_blocks(:#{h}) do |b|
-                if instance_eval(&b) == false
-                  Sequel::Deprecation.deprecate("Having #{h} hook block return false to stop evaluation of further #{h} hook blocks", "Instead, call cancel_action inside #{h} hook block")
-                  return false
-                end
-              end
-              super
-            end
-          END
-        end
-        [:after_create, :after_update, :after_validation].each{|h| class_eval("def #{h}; super; model.hook_blocks(:#{h}){|b| instance_eval(&b)}; end", __FILE__, __LINE__)}
-
-        def after_destroy
-          super
-          model.hook_blocks(:after_destroy){|b| instance_eval(&b)}
-          if model.has_hooks?(:after_destroy_commit)
-            db.after_commit{model.hook_blocks(:after_destroy_commit){|b| instance_eval(&b)}}
-          end
-        end
-
-        def after_save
-          super
-          model.hook_blocks(:after_save){|b| instance_eval(&b)}
-          if model.has_hooks?(:after_commit)
-            db.after_commit{model.hook_blocks(:after_commit){|b| instance_eval(&b)}}
-          end
-        end
+        [:before_create, :before_update, :before_validation, :before_save, :before_destroy].each{|h| class_eval("def #{h}; model.hook_blocks(:#{h}){|b| instance_exec(&b)}; super end", __FILE__, __LINE__)}
 
-        def before_destroy
-          model.hook_blocks(:before_destroy) do |b|
-            if instance_eval(&b) == false
-              Sequel::Deprecation.deprecate("Having before_destory hook block return false to stop evaluation of further before_destroy hook blocks", "Instead, call cancel_action inside before_destroy hook block")
-              return false
-            end
-          end
-          super
-          if model.has_hooks?(:after_destroy_rollback)
-            db.after_rollback{model.hook_blocks(:after_destroy_rollback){|b| instance_eval(&b)}}
-          end
-        end
-
-        def before_save
-          model.hook_blocks(:before_save) do |b|
-            if instance_eval(&b) == false
-              Sequel::Deprecation.deprecate("Having before_save hook block return false to stop evaluation of further before_save hook blocks", "Instead, call cancel_action inside before_save hook block")
-              return false
-            end
-          end
-          super
-          if model.has_hooks?(:after_rollback)
-            db.after_rollback{model.hook_blocks(:after_rollback){|b| instance_eval(&b)}}
-          end
-        end
+        [:after_create, :after_update, :after_validation, :after_save, :after_destroy].each{|h| class_eval("def #{h}; super; model.hook_blocks(:#{h}){|b| instance_exec(&b)}; end", __FILE__, __LINE__)}
       end
     end
   end

data/lib/sequel/plugins/input_transformer.rb

@@ -7,14 +7,14 @@ module Sequel
     #
     #   Album.plugin :input_transformer
     #   Album.add_input_transformer(:reverser){|v| v.is_a?(String) ? v.reverse : v}
-    #   album = Album.new(:name=>'foo')
+    #   album = Album.new(name: 'foo')
     #   album.name # => 'oof'
     #
     # You can specifically set some columns to skip some input
     # input transformers:
     #
     #   Album.skip_input_transformer(:reverser, :foo)
-    #   Album.new(:foo=>'bar').foo # => 'bar'
+    #   Album.new(foo: 'bar').foo # => 'bar'
     # 
     # Usage:
     #
@@ -25,9 +25,8 @@ module Sequel
     #   Album.plugin :input_transformer
     module InputTransformer
       def self.apply(model, *)
-        model.instance_eval do
+        model.instance_exec do
           @input_transformers = {}
-          @input_transformer_order = []
           @skip_input_transformer_columns = {}
         end
       end
@@ -42,23 +41,23 @@ module Sequel
         # Hash of input transformer name symbols to transformer callables.
         attr_reader :input_transformers
 
-        # The order in which to call the input transformers.
-        attr_reader :input_transformer_order
+        # The order in which to call the input transformers. For backwards compatibility only.
+        def input_transformer_order
+          input_transformers.keys.reverse
+        end
 
-        Plugins.inherited_instance_variables(self, :@skip_input_transformer_columns=>:hash_dup, :@input_transformers=>:dup, :@input_transformer_order=>:dup)
+        Plugins.inherited_instance_variables(self, :@skip_input_transformer_columns=>:hash_dup, :@input_transformers=>:dup)
 
         # Add an input transformer to this model.
         def add_input_transformer(transformer_name, &block)
           raise(Error, 'must provide both transformer name and block when adding input transformer') unless transformer_name && block
           @input_transformers[transformer_name] = block
-          @input_transformer_order.unshift(transformer_name)
           @skip_input_transformer_columns[transformer_name] = []
         end
 
         # Freeze input transformers when freezing model class
         def freeze
           @input_transformers.freeze
-          @input_transformer_order.freeze
           @skip_input_transformer_columns.freeze.each_value(&:freeze)
 
           super
@@ -79,8 +78,8 @@ module Sequel
         # Transform the input using all of the transformers, except those explicitly
         # skipped, before setting the value in the model object.
         def []=(k, v)
-          model.input_transformer_order.each do |transformer_name|
-            v = model.input_transformers[transformer_name].call(v) unless model.skip_input_transformer?(transformer_name, k)
+          model.input_transformers.reverse_each do |name, transformer|
+            v = transformer.call(v) unless model.skip_input_transformer?(name, k)
           end
           super
         end

data/lib/sequel/plugins/insert_returning_select.rb

@@ -21,7 +21,7 @@ module Sequel
       # Modify the current model's dataset selection, if the model
       # has a dataset.
       def self.configure(model)
-        model.instance_eval do
+        model.instance_exec do
           self.dataset = dataset if @dataset && @dataset.opts[:select]
         end
       end
@@ -30,14 +30,6 @@ module Sequel
         # The dataset to use to insert new rows.  For internal use only.
         attr_reader :instance_insert_dataset
 
-        # Freeze instance insert dataset when freezing model class.
-        def freeze
-          super
-
-          @instance_insert_dataset.freeze if @instance_insert_dataset
-          self
-        end
-
         private
 
         # When reseting the instance dataset, also reset the instance_insert_dataset.

data/lib/sequel/plugins/instance_filters.rb

@@ -17,12 +17,12 @@ module Sequel
     #
     #   # These are two separate objects that represent the same
     #   # database row. 
-    #   i1 = Item.first(:id=>1, :delete_allowed=>false)
-    #   i2 = Item.first(:id=>1, :delete_allowed=>false)
+    #   i1 = Item.first(id: 1, delete_allowed: false)
+    #   i2 = Item.first(id: 1, delete_allowed: false)
     #
     #   # Add an instance filter to the object. This filter is in effect
     #   # until the object is successfully updated or deleted.
-    #   i1.instance_filter(:delete_allowed=>true)
+    #   i1.instance_filter(delete_allowed: true)
     #
     #   # Attempting to delete the object where the filter doesn't
     #   # match any rows raises an error.
@@ -30,7 +30,7 @@ module Sequel
     #
     #   # The other object that represents the same row has no
     #   # instance filters, and can be updated normally.
-    #   i2.update(:delete_allowed=>true)
+    #   i2.update(delete_allowed: true)
     #
     #   # Even though the filter is now still in effect, since the
     #   # database row has been updated to allow deleting,
@@ -123,7 +123,7 @@ module Sequel
         # Only use prepared statements for update and delete queries
         # if there are no instance filters.
         def use_prepared_statements_for?(type)
-          if (type == :update || type == :delete) && !instance_filters.empty?
+          if type == :update && !instance_filters.empty?
             false
           else
             super if defined?(super)