clowne 1.0.0 → 1.4.0

data/lib/clowne/adapters/active_record/associations/belongs_to.rb

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+module Clowne
+  module Adapters # :nodoc: all
+    class ActiveRecord
+      module Associations
+        class BelongsTo < Base
+          def call(record)
+            child = association
+            return record unless child
+
+            unless declaration.scope.nil?
+              warn(
+                "[Clowne] Belongs to association does not support scopes " \
+                "(#{@association_name} for #{@source.class})"
+              )
+            end
+
+            child_clone = clone_one(child)
+            record.__send__(:"#{association_name}=", child_clone)
+            record
+          end
+          # rubocop: enable Metrics/MethodLength
+        end
+      end
+    end
+  end
+end

data/lib/clowne/adapters/active_record/associations/has_one.rb

@@ -5,14 +5,13 @@ module Clowne
     class ActiveRecord
       module Associations
         class HasOne < Base
-          # rubocop: disable Metrics/MethodLength
           def call(record)
             child = association
             return record unless child
 
             unless declaration.scope.nil?
               warn(
-                '[Clowne] Has one association does not support scopes ' \
+                "[Clowne] Has one association does not support scopes " \
                 "(#{@association_name} for #{@source.class})"
               )
             end

data/lib/clowne/adapters/active_record/associations.rb

@@ -1,16 +1,18 @@
 # frozen_string_literal: true
 
-require 'clowne/adapters/active_record/associations/base'
-require 'clowne/adapters/active_record/associations/noop'
-require 'clowne/adapters/active_record/associations/has_one'
-require 'clowne/adapters/active_record/associations/has_many'
-require 'clowne/adapters/active_record/associations/has_and_belongs_to_many'
+require "clowne/adapters/active_record/associations/base"
+require "clowne/adapters/active_record/associations/noop"
+require "clowne/adapters/active_record/associations/belongs_to"
+require "clowne/adapters/active_record/associations/has_one"
+require "clowne/adapters/active_record/associations/has_many"
+require "clowne/adapters/active_record/associations/has_and_belongs_to_many"
 
 module Clowne
   module Adapters # :nodoc: all
     class ActiveRecord
       module Associations
         AR_2_CLONER = {
+          belongs_to: BelongsTo,
           has_one: HasOne,
           has_many: HasMany,
           has_and_belongs_to_many: HABTM

data/lib/clowne/adapters/active_record/dsl.rb

@@ -6,8 +6,8 @@ module Clowne
     module ActiveRecordDSL
       module InstanceMethods # :nodoc:
         # Shortcut to call class's cloner call with self
-        def clowne(*args, &block)
-          self.class.cloner_class.call(self, *args, &block)
+        def clowne(**args, &block)
+          self.class.cloner_class.call(self, **args, &block)
         end
       end
 

data/lib/clowne/adapters/active_record/resolvers/association.rb

@@ -8,13 +8,12 @@ module Clowne
 
         class Association
           class << self
-            # rubocop: disable Metrics/ParameterLists
             def call(source, record, declaration, adapter:, params:, **_options)
               reflection = source.class.reflections[declaration.name.to_s]
 
               if reflection.nil?
                 raise UnknownAssociation,
-                      "Association #{declaration.name} couldn't be found for #{source.class}"
+                  "Association #{declaration.name} couldn't be found for #{source.class}"
               end
 
               cloner_class = Associations.cloner_for(reflection)

data/lib/clowne/adapters/active_record.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'clowne/ext/orm_ext'
+require "clowne/ext/orm_ext"
 
 module Clowne
   module Adapters
@@ -13,5 +13,5 @@ ActiveSupport.on_load(:active_record) do
   ::ActiveRecord::Base.extend Clowne::Ext::ORMExt
 end
 
-require 'clowne/adapters/active_record/associations'
-require 'clowne/adapters/active_record/resolvers/association'
+require "clowne/adapters/active_record/associations"
+require "clowne/adapters/active_record/resolvers/association"

data/lib/clowne/adapters/base/association.rb

@@ -30,7 +30,7 @@ module Clowne
 
         def clone_one(child)
           cloner = cloner_for(child)
-          cloner ? cloner.call(child, cloner_options) : dup_record(child)
+          cloner ? cloner.call(child, **cloner_options) : dup_record(child)
         end
 
         def with_scope

data/lib/clowne/adapters/base.rb

@@ -1,11 +1,12 @@
 # frozen_string_literal: true
 
-require 'clowne/adapters/registry'
+require "clowne/adapters/registry"
 
-require 'clowne/resolvers/init_as'
-require 'clowne/resolvers/nullify'
-require 'clowne/resolvers/finalize'
-require 'clowne/resolvers/after_persist'
+require "clowne/resolvers/init_as"
+require "clowne/resolvers/nullify"
+require "clowne/resolvers/finalize"
+require "clowne/resolvers/after_persist"
+require "clowne/resolvers/after_clone"
 
 module Clowne
   module Adapters
@@ -16,7 +17,7 @@ module Clowne
 
       class << self
         # Duplicate record and remember record <-> clone relationship in operation
-        # Cab be overrided in special adapter
+        # Can be overrided in special adapter
         # +record+:: Instance of record (ActiveRecord or Sequel)
         def dup_record(record)
           record.dup.tap do |clone|
@@ -26,7 +27,7 @@ module Clowne
         end
 
         # Operation class which  using for cloning
-        # Cab be overrided in special adapter
+        # Can be overrided in special adapter
         def operation_class
           Clowne::Utils::Operation
         end
@@ -70,6 +71,11 @@ Clowne::Adapters::Base.register_resolver(
 )
 
 Clowne::Adapters::Base.register_resolver(
-  :after_persist, Clowne::Resolvers::AfterPersist,
+  :after_clone, Clowne::Resolvers::AfterClone,
   after: :finalize
 )
+
+Clowne::Adapters::Base.register_resolver(
+  :after_persist, Clowne::Resolvers::AfterPersist,
+  after: :after_clone
+)

data/lib/clowne/adapters/sequel/associations/base.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'clowne/adapters/base/association'
+require "clowne/adapters/base/association"
 
 module Clowne
   module Adapters # :nodoc: all
@@ -10,7 +10,7 @@ module Clowne
           private
 
           def init_scope
-            @_init_scope ||= source.__send__([association_name, 'dataset'].join('_'))
+            @_init_scope ||= source.__send__([association_name, "dataset"].join("_"))
           end
 
           def record_wrapper(record)

data/lib/clowne/adapters/sequel/associations/many_to_many.rb

@@ -7,10 +7,10 @@ module Clowne
         class ManyToMany < Base
           def call(record)
             clones = with_scope
-                     .lazy
-                     .map(&method(:clone_one))
-                     .map(&method(:record_wrapper))
-                     .to_a
+              .lazy
+              .map(&method(:clone_one))
+              .map(&method(:record_wrapper))
+              .to_a
 
             record_wrapper(record).remember_assoc(:"#{association_name}_attributes", clones)
 

data/lib/clowne/adapters/sequel/associations/one_to_many.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'clowne/ext/yield_self_then'
+require "clowne/ext/yield_self_then"
 
 using Clowne::Ext::YieldSelfThen
 

data/lib/clowne/adapters/sequel/associations/one_to_one.rb

@@ -9,7 +9,7 @@ module Clowne
             child = association
             return record unless child
 
-            warn '[Clowne] Has one association does not support scope' unless declaration.scope.nil?
+            warn "[Clowne] Has one association does not support scope" unless declaration.scope.nil?
 
             child_clone = clone_one(child)
             child_clone[:"#{reflection[:key]}"] = nil

data/lib/clowne/adapters/sequel/associations.rb

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
-require 'clowne/adapters/sequel/associations/base'
-require 'clowne/adapters/sequel/associations/noop'
-require 'clowne/adapters/sequel/associations/one_to_one'
-require 'clowne/adapters/sequel/associations/one_to_many'
-require 'clowne/adapters/sequel/associations/many_to_many'
+require "clowne/adapters/sequel/associations/base"
+require "clowne/adapters/sequel/associations/noop"
+require "clowne/adapters/sequel/associations/one_to_one"
+require "clowne/adapters/sequel/associations/one_to_many"
+require "clowne/adapters/sequel/associations/many_to_many"
 
 module Clowne
   module Adapters # :nodoc: all

data/lib/clowne/adapters/sequel/operation.rb

@@ -1,13 +1,12 @@
 # frozen_string_literal: true
 
-require 'clowne/ext/record_key'
+require "clowne/ext/record_key"
 
 module Clowne
   module Adapters
     class Sequel # :nodoc: all
       class Operation < Clowne::Utils::Operation
         include Clowne::Ext::RecordKey
-
         def initialize(mapper)
           super
           @records = {}
@@ -24,7 +23,11 @@ module Clowne
         def to_record
           return @_record if defined?(@_record)
 
-          @_record = @records[key(@clone)].to_model
+          record_wrapper(@clone)
+
+          @_record = @records[key(@clone)].to_model.tap do
+            run_after_clone
+          end
         end
       end
     end

data/lib/clowne/adapters/sequel/resolvers/after_persist.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'clowne/adapters/sequel/specifications/after_persist_does_not_support'
+require "clowne/adapters/sequel/specifications/after_persist_does_not_support"
 
 module Clowne
   module Adapters # :nodoc: all

data/lib/clowne/adapters/sequel/resolvers/association.rb

@@ -6,7 +6,6 @@ module Clowne
       module Resolvers
         class Association
           class << self
-            # rubocop: disable Metrics/ParameterLists
             def call(source, record, declaration, adapter:, params:, **_options)
               with_clonable(source, record, declaration) do
                 reflection = source.class.association_reflections[declaration.name.to_sym]
@@ -36,7 +35,7 @@ module Clowne
 
             def clonable_assoc?(source, declaration)
               source.class.plugins.include?(::Sequel::Plugins::NestedAttributes) &&
-                source.respond_to?(:"#{declaration.name.to_s}_attributes=")
+                source.respond_to?(:"#{declaration.name}_attributes=")
             end
           end
         end

data/lib/clowne/adapters/sequel/specifications/after_persist_does_not_support.rb

@@ -6,7 +6,7 @@ module Clowne
       module Specifications # :nodoc: all
         class AfterPersistDoesNotSupportException < StandardError
           def message
-            'Sequel adapter does not support after_persist callback'
+            "Sequel adapter does not support after_persist callback"
           end
         end
       end

data/lib/clowne/adapters/sequel.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'clowne/ext/orm_ext'
+require "clowne/ext/orm_ext"
 
 module Clowne
   module Adapters
@@ -21,9 +21,9 @@ end
 
 ::Sequel::Model.extend Clowne::Ext::ORMExt
 
-require 'clowne/adapters/sequel/operation'
-require 'clowne/adapters/sequel/associations'
-require 'clowne/adapters/sequel/copier'
-require 'clowne/adapters/sequel/record_wrapper'
-require 'clowne/adapters/sequel/resolvers/association'
-require 'clowne/adapters/sequel/resolvers/after_persist'
+require "clowne/adapters/sequel/operation"
+require "clowne/adapters/sequel/associations"
+require "clowne/adapters/sequel/copier"
+require "clowne/adapters/sequel/record_wrapper"
+require "clowne/adapters/sequel/resolvers/association"
+require "clowne/adapters/sequel/resolvers/after_persist"

data/lib/clowne/cloner.rb

@@ -1,13 +1,14 @@
 # frozen_string_literal: true
 
-require 'clowne/planner'
-require 'clowne/dsl'
-require 'clowne/utils/options'
-require 'clowne/utils/params'
-require 'clowne/utils/operation'
+require "clowne/planner"
+require "clowne/dsl"
+require "clowne/utils/options"
+require "clowne/utils/params"
+require "clowne/utils/operation"
 
 module Clowne # :nodoc: all
   class UnprocessableSourceError < StandardError; end
+
   class ConfigurationError < StandardError; end
 
   class Cloner
@@ -43,14 +44,13 @@ module Clowne # :nodoc: all
         @traits[name].extend_with(block)
       end
 
-      # rubocop: disable Metrics/AbcSize, Metrics/MethodLength
-      def call(object, **options)
-        raise(UnprocessableSourceError, 'Nil is not cloneable object') if object.nil?
+      def call(object, **options, &block)
+        raise(UnprocessableSourceError, "Nil is not cloneable object") if object.nil?
 
         options = Clowne::Utils::Options.new(options)
         current_adapter = current_adapter(options.adapter)
 
-        raise(ConfigurationError, 'Adapter is not defined') if current_adapter.nil?
+        raise(ConfigurationError, "Adapter is not defined") if current_adapter.nil?
 
         plan =
           if options.traits.empty?
@@ -59,7 +59,7 @@ module Clowne # :nodoc: all
             plan_with_traits(options.traits, current_adapter: current_adapter)
           end
 
-        plan = Clowne::Planner.enhance(plan, Proc.new) if block_given?
+        plan = Clowne::Planner.enhance(plan, block) if block
 
         plan = Clowne::Planner.filter_declarations(plan, options.only)
 
@@ -79,7 +79,7 @@ module Clowne # :nodoc: all
 
       def plan_with_traits(ids, current_adapter: adapter)
         # Cache plans for combinations of traits
-        traits_id = ids.map(&:to_s).join(':')
+        traits_id = ids.map(&:to_s).join(":")
         return traits_plans[traits_id] if traits_plans.key?(traits_id)
 
         traits_plans[traits_id] = Clowne::Planner.compile(

data/lib/clowne/declarations/after_clone.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+module Clowne
+  module Declarations
+    class AfterClone < Base # :nodoc: all
+      attr_reader :block
+
+      def initialize(*, &block)
+        raise ArgumentError, "Block is required for after_clone" unless block
+
+        @block = block
+      end
+
+      def compile(plan)
+        plan.add(:after_clone, self)
+      end
+    end
+  end
+end
+
+Clowne::Declarations.add :after_clone, Clowne::Declarations::AfterClone

data/lib/clowne/declarations/after_persist.rb

@@ -5,10 +5,10 @@ module Clowne
     class AfterPersist < Base # :nodoc: all
       attr_reader :block
 
-      def initialize
-        raise ArgumentError, 'Block is required for after_persist' unless block_given?
+      def initialize(*, &block)
+        raise ArgumentError, "Block is required for after_persist" unless block
 
-        @block = Proc.new
+        @block = block
       end
 
       def compile(plan)

data/lib/clowne/declarations/exclude_association.rb

@@ -5,7 +5,7 @@ module Clowne
     class ExcludeAssociation < Base # :nodoc: all
       attr_accessor :name
 
-      def initialize(name)
+      def initialize(name, **)
         @name = name.to_sym
       end
 

data/lib/clowne/declarations/finalize.rb

@@ -5,10 +5,10 @@ module Clowne
     class Finalize < Base # :nodoc: all
       attr_reader :block
 
-      def initialize
-        raise ArgumentError, 'Block is required for finalize' unless block_given?
+      def initialize(*, &block)
+        raise ArgumentError, "Block is required for finalize" unless block
 
-        @block = Proc.new
+        @block = block
       end
 
       def compile(plan)

data/lib/clowne/declarations/include_association.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'clowne/ext/string_constantize'
+require "clowne/ext/string_constantize"
 
 module Clowne
   module Declarations

data/lib/clowne/declarations/init_as.rb

@@ -5,10 +5,10 @@ module Clowne
     class InitAs < Base # :nodoc: all
       attr_reader :block
 
-      def initialize
-        raise ArgumentError, 'Block is required for init_as' unless block_given?
+      def initialize(*, &block)
+        raise ArgumentError, "Block is required for init_as" unless block
 
-        @block = Proc.new
+        @block = block
       end
 
       def compile(plan)

data/lib/clowne/declarations/nullify.rb

@@ -5,8 +5,8 @@ module Clowne
     class Nullify < Base # :nodoc: all
       attr_reader :attributes
 
-      def initialize(*attributes)
-        raise ArgumentError, 'At least one attribute required' if attributes.empty?
+      def initialize(*attributes, **)
+        raise ArgumentError, "At least one attribute required" if attributes.empty?
 
         @attributes = attributes
       end

data/lib/clowne/declarations/trait.rb

@@ -17,7 +17,7 @@ module Clowne
         @compiled = compile
       end
 
-      alias declarations compiled
+      alias_method :declarations, :compiled
 
       def dup
         self.class.new.tap do |duped|

data/lib/clowne/declarations.rb

@@ -1,18 +1,18 @@
 # frozen_string_literal: true
 
-require 'clowne/dsl'
-require 'clowne/utils/plan'
+require "clowne/dsl"
+require "clowne/utils/plan"
 
 module Clowne
   module Declarations # :nodoc:
     module_function
 
-    def add(id, declaration = nil)
-      declaration = Proc.new if block_given?
+    def add(id, declaration = nil, &block)
+      declaration = block if block
 
       if declaration.is_a?(Class)
-        DSL.send(:define_method, id) do |*args, &block|
-          declarations.push declaration.new(*args, &block)
+        DSL.send(:define_method, id) do |*args, **hargs, &inner_block|
+          declarations.push declaration.new(*args, **hargs, &inner_block)
         end
       elsif declaration.is_a?(Proc)
         DSL.send(:define_method, id, &declaration)
@@ -23,11 +23,12 @@ module Clowne
   end
 end
 
-require 'clowne/declarations/base'
-require 'clowne/declarations/init_as'
-require 'clowne/declarations/exclude_association'
-require 'clowne/declarations/finalize'
-require 'clowne/declarations/include_association'
-require 'clowne/declarations/nullify'
-require 'clowne/declarations/trait'
-require 'clowne/declarations/after_persist'
+require "clowne/declarations/base"
+require "clowne/declarations/init_as"
+require "clowne/declarations/exclude_association"
+require "clowne/declarations/finalize"
+require "clowne/declarations/include_association"
+require "clowne/declarations/nullify"
+require "clowne/declarations/trait"
+require "clowne/declarations/after_persist"
+require "clowne/declarations/after_clone"

data/lib/clowne/ext/orm_ext.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'clowne/ext/string_constantize'
+require "clowne/ext/string_constantize"
 
 module Clowne
   module Ext

data/lib/clowne/ext/record_key.rb

@@ -5,7 +5,7 @@ module Clowne
     module RecordKey # :nodoc: all
       def key(record)
         id = record.respond_to?(:id) && record.id ? record.id : record.__id__
-        [record.class.name, id].join('#')
+        [record.class.name, id].join("#")
       end
     end
   end

data/lib/clowne/ext/string_constantize.rb

@@ -6,7 +6,7 @@ module Clowne
     module StringConstantize
       refine String do
         def constantize
-          names = split('::')
+          names = split("::")
 
           return nil if names.empty?
 

data/lib/clowne/ext/yield_self_then.rb

@@ -11,11 +11,11 @@ module Clowne
           end
         end
 
-        alias then yield_self
+        alias_method :then, :yield_self
       end
 
       # See https://github.com/jruby/jruby/issues/5220
-      ::Object.include(Ext) if RUBY_PLATFORM =~ /java/i
+      ::Object.include(Ext) if RUBY_PLATFORM.match?(/java/i)
 
       refine Object do
         include Ext

data/lib/clowne/planner.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'clowne/utils/plan'
+require "clowne/utils/plan"
 
 module Clowne
   class Planner # :nodoc: all

data/lib/clowne/resolvers/after_clone.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+module Clowne
+  class Resolvers
+    module AfterClone # :nodoc: all
+      def self.call(source, record, declaration, params:, **_options)
+        operation = Clowne::Utils::Operation.current
+        params ||= {}
+        operation.add_after_clone(
+          proc do
+            declaration.block.call(source, record, **params.merge(mapper: operation.mapper))
+          end
+        )
+        record
+      end
+    end
+  end
+end

data/lib/clowne/resolvers/after_persist.rb

@@ -8,7 +8,7 @@ module Clowne
         params ||= {}
         operation.add_after_persist(
           proc do
-            declaration.block.call(source, record, params.merge(mapper: operation.mapper))
+            declaration.block.call(source, record, **params.merge(mapper: operation.mapper))
           end
         )
         record

data/lib/clowne/resolvers/finalize.rb

@@ -4,7 +4,7 @@ module Clowne
   class Resolvers
     module Finalize # :nodoc: all
       def self.call(source, record, declaration, params:, **_options)
-        declaration.block.call(source, record, params)
+        declaration.block.call(source, record, **params)
         record
       end
     end

data/lib/clowne/resolvers/init_as.rb

@@ -3,7 +3,6 @@
 module Clowne
   class Resolvers
     module InitAs # :nodoc: all
-      # rubocop: disable Metrics/ParameterLists
       def self.call(source, _record, declaration, params:, adapter:, **_options)
         adapter.init_record(declaration.block.call(source, **params))
       end