active_any 0.0.6 → 0.0.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f55edfb43e9d81dba6c9b0614b34db3b2ffe2f6b
-  data.tar.gz: 7520decd1d49bf940da804fe42ca899f5644c73c
+  metadata.gz: 39acdd4989cf8260cd694f9c90c7e1dee0781e90
+  data.tar.gz: 8942a09c7523b615a19bc3853141b47fdbfe9d46
 SHA512:
-  metadata.gz: 4679db223ffd527f341392d69e4adff8ba0524c4c5441cc116864497e6c37c7f525d78660075c48db29e08e05dccfb7d4e60e11d719accb12ddb9a129830bd44
-  data.tar.gz: 672f685c5c891eb4b49f36a6335813dfa89c17e0a78471c6aace41a94f51b3adc08483d72497b1c27a2b418dd9d04de9ad0c3cb2cfc1378843fb38a122efabe3
+  metadata.gz: a83b242f2d8a25fe53d78b3fe842f01a701cd8bc6a3605dbe097e337484876eafb804f0da660ab2e6cbe37efae74b6242977dff8fb6a3bad1bb0bf3df07a09bf
+  data.tar.gz: e60bc3993222d390e5d2d0b779fe94075d6abfa43f52bd3892c5443a4147a25f475afcea7fd3e6ca30eb48f073ed47a1bc6b33c12cfaecb571f089a87f8f065a

data/.rubocop.yml

@@ -13,6 +13,15 @@ Style/Documentation:
 Style/PredicateName:
   Enabled: false
 
+Style/AccessorMethodName:
+  Enabled: false
+
+Metrics/ClassLength:
+  Max: 120
+
+Metrics/MethodLength:
+  Max: 15
+
 Metrics/LineLength:
   Max: 150
   Exclude:

data/lib/active_any.rb

@@ -13,6 +13,7 @@ require 'active_any/association_relation'
 require 'active_any/attribute_assignment'
 require 'active_any/finders'
 require 'active_any/core'
+require 'active_any/relation/delegation'
 require 'active_any/base'
 require 'active_any/subscriber'
 require 'active_any/adapters/abstract_adapter'

data/lib/active_any/associations.rb

@@ -45,7 +45,7 @@ module ActiveAny
       association = association_instance_get(name)
 
       if association.nil?
-        reflection = self.class.reflections[name.to_s]
+        reflection = self.class._reflect_on_association(name)
         raise AssociationNotFoundError.new(self, name) unless reflection
 
         association = reflection.association_class.new(self, reflection)

data/lib/active_any/associations/preloader.rb

@@ -87,7 +87,7 @@ module ActiveAny
         end
       end
 
-      def preloader_for(reflection, owners, rhs_klass) # rubocop:disable Metrics/MethodLength
+      def preloader_for(reflection, owners, rhs_klass)
         return NullPreloader unless rhs_klass
 
         if owners.first.association(reflection.name).loaded?

data/lib/active_any/associations/preloader/association.rb

@@ -105,9 +105,7 @@ module ActiveAny
           #   scope.joins!(reflection_scope.joins_values)
           # end
           order_values = preload_values[:order] || values[:order]
-          if order_values
-            scope.order!(order_values)
-          end
+          scope.order!(order_values) if order_values
 
           # if preload_values[:reordering] || values[:reordering]
           #   scope.reordering_value = true

data/lib/active_any/associations/preloader/has_many.rb

@@ -17,7 +17,7 @@ module ActiveAny
         end
 
         def owner_key_name
-          reflection.primary_key
+          reflection.record_class_primary_key
         end
       end
     end

data/lib/active_any/base.rb

@@ -5,6 +5,8 @@ require 'pathname'
 
 module ActiveAny
   class Base
+    extend Delegation::DelegateCache
+
     include Core
     include Attribute
     include Associations
@@ -16,10 +18,6 @@ module ActiveAny
   class Object < Base
     self.abstract_class = true
 
-    def self.inherited(child)
-      child.abstract_class = false
-    end
-
     class << self
       attr_accessor :data
 

data/lib/active_any/core.rb

@@ -5,9 +5,7 @@ module ActiveAny
     extend ActiveSupport::Concern
 
     included do
-      class_attribute :abstract_class, instance_writer: false
       class_attribute :primary_key, instance_writer: false
-      self.abstract_class = true
       self.primary_key = nil
     end
 
@@ -16,6 +14,8 @@ module ActiveAny
     end
 
     module ClassMethods
+      attr_accessor :abstract_class
+
       def unscoped
         # TODO: implement
         all
@@ -25,6 +25,18 @@ module ActiveAny
         # TODO: implement
         all
       end
+
+      def new(*args, &block)
+        if abstract_class? || self == Base
+          raise NotImplementedError, "#{self} is an abstract class and cannot be instantiated."
+        end
+
+        super
+      end
+
+      def abstract_class?
+        defined?(@abstract_class) && abstract_class == true
+      end
     end
 
     def initialize(*args)

data/lib/active_any/reflection.rb

@@ -45,11 +45,13 @@ module ActiveAny
     end
 
     def self.add_reflection(klass, name, reflection)
-      klass.reflections = klass.reflections.merge(name.to_s => reflection)
+      klass._reflections = klass.reflections.merge(name.to_s => reflection)
     end
 
-    def self._reflect_on_association(association)
-      _reflections[association.to_s]
+    module ClassMethods
+      def _reflect_on_association(association)
+        _reflections[association.to_s]
+      end
     end
   end
 end

data/lib/active_any/reflection/association_reflection.rb

@@ -50,10 +50,6 @@ module ActiveAny
         @foreign_key ||= options[:foreign_key] || derive_foreign_key.freeze
       end
 
-      def primary_key
-        @primary_key ||= options[:primary_key] || primary_key_for_record_class
-      end
-
       def scope_for(klass)
         scope ? klass.unscoped.instance_exec(nil, &scope) : klass.unscoped
       end
@@ -77,8 +73,16 @@ module ActiveAny
         @inverse_of ||= klass._reflect_on_association inverse_name
       end
 
+      def record_class_primary_key
+        @primary_key ||= options[:primary_key] || primary_key(record_class)
+      end
+
       private
 
+      def primary_key(record_class)
+        record_class.primary_key || (raise UnknownPrimaryKey.new, klass)
+      end
+
       def inverse_name
         options.fetch(:inverse_of) do
           @automatic_inverse_of ||= automatic_inverse_of
@@ -87,7 +91,7 @@ module ActiveAny
 
       def automatic_inverse_of
         if can_find_inverse_of_automatically?(self)
-          inverse_name = ActiveSupport::Inflector.underscore(options[:as] || active_record.name.demodulize).to_sym
+          inverse_name = ActiveSupport::Inflector.underscore(options[:as] || record_class.name.demodulize).to_sym
 
           begin
             reflection = klass._reflect_on_association(inverse_name)
@@ -97,9 +101,7 @@ module ActiveAny
             reflection = false
           end
 
-          if valid_inverse_reflection?(reflection)
-            return inverse_name
-          end
+          return inverse_name if valid_inverse_reflection?(reflection)
         end
 
         false
@@ -129,10 +131,6 @@ module ActiveAny
         raise NotImplementedError
       end
 
-      def primary_key_for_record_class
-        klass.primary_key || (raise UnknownPrimaryKey.new, klass)
-      end
-
       def derive_class_name
         class_name = name.to_s
         class_name = class_name.singularize if collection?

data/lib/active_any/relation.rb

@@ -2,6 +2,7 @@
 
 require 'active_any/relation/finder_methods'
 require 'active_any/relation/query_methods'
+require 'active_any/relation/delegation'
 require 'active_any/relation/merger'
 require 'active_any/relation/where_clause'
 require 'active_any/relation/order_clause'
@@ -15,11 +16,16 @@ module ActiveAny
     include Enumerable
     include FinderMethods
     include QueryMethods
+    include Delegation
 
     class ImmutableRelation < StandardError; end
 
     def self.create(klass, *args)
-      new(klass, *args)
+      relation_class_for(klass).new(klass, *args)
+    end
+
+    def self.relation_class_for(klass)
+      klass.relation_delegate_class(self)
     end
 
     def initialize(klass)
@@ -85,6 +91,22 @@ module ActiveAny
       false
     end
 
+    def scoping
+      # previous, klass.current_scope = klass.current_scope, self
+      yield
+      # ensure
+      # klass.current_scope = previous
+    end
+
+    def inspect
+      subject = loaded? ? records : self
+      entries = subject.take([limit_value, 11].compact.min).map!(&:inspect)
+
+      entries[10] = '...' if entries.size == 11
+
+      "#<#{self.class.name} [#{entries.join(', ')}]>"
+    end
+
     private
 
     def spawn

data/lib/active_any/relation/delegation.rb

@@ -0,0 +1,88 @@
+# frozen_string_literal: true
+
+module ActiveAny
+  module Delegation
+    extend ActiveSupport::Concern
+
+    module DelegateCache
+      def relation_delegate_class(klass)
+        @relation_delegate_cache[klass]
+      end
+
+      def initialize_relation_delegate_cache
+        @relation_delegate_cache = cache = {}
+        [
+          ActiveAny::Relation,
+          ActiveAny::Associations::CollectionProxy,
+          ActiveAny::AssociationRelation
+        ].each do |klass|
+          delegate = Class.new(klass) { include ClassSpecificRelation }
+          mangled_name = klass.name.gsub('::', '_')
+          const_set mangled_name, delegate
+          private_constant mangled_name
+
+          cache[klass] = delegate
+        end
+      end
+
+      def inherited(child_class)
+        child_class.initialize_relation_delegate_cache
+        super
+      end
+    end
+
+    module ClassSpecificRelation
+      extend ActiveSupport::Concern
+
+      included do
+        @delegation_mutex = Mutex.new
+      end
+
+      module ClassMethods
+        def name
+          superclass.name
+        end
+
+        def delegate_to_scoped_klass(method)
+          @delegation_mutex.synchronize do
+            return if method_defined?(method)
+
+            if /\A[a-zA-Z_]\w*[!?]?\z/.match?(method)
+              module_eval <<-RUBY, __FILE__, __LINE__ + 1
+                def #{method}(*args, &block)
+                  scoping { @klass.#{method}(*args, &block) }
+                end
+              RUBY
+            else
+              define_method method do |*args, &block|
+                scoping { @klass.public_send(method, *args, &block) }
+              end
+            end
+          end
+        end
+
+        def delegate(method, opts = {})
+          @delegation_mutex.synchronize do
+            return if method_defined?(method)
+            super
+          end
+        end
+      end
+
+      private
+
+      def method_missing(method, *args, &block)
+        if @klass.respond_to?(method)
+          self.class.delegate_to_scoped_klass(method)
+          scoping { @klass.public_send(method, *args, &block) }
+        else
+          super
+        end
+      end
+
+      def respond_to_missing?(name, include_private)
+        super
+      end
+    end
+  end
+end

data/lib/active_any/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ActiveAny
-  VERSION = '0.0.6'
+  VERSION = '0.0.7'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: active_any
 version: !ruby/object:Gem::Version
-  version: 0.0.6
+  version: 0.0.7
 platform: ruby
 authors:
 - yuemori
@@ -226,6 +226,7 @@ files:
 - lib/active_any/reflection/belongs_to_reflection.rb
 - lib/active_any/reflection/has_many_reflection.rb
 - lib/active_any/relation.rb
+- lib/active_any/relation/delegation.rb
 - lib/active_any/relation/finder_methods.rb
 - lib/active_any/relation/merger.rb
 - lib/active_any/relation/order_clause.rb