mongo_mapper 0.15.3 → 0.15.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5d5192b00f4bce2f16b7ac329a9bdb059c888ebea3c62ba91e9017a42a43371c
-  data.tar.gz: 9e3f0dca756cb0e260266fa147800c07d02133dfad74376f3aaa169f56d67f36
+  metadata.gz: 290286ec91fda33bce766c8cacff110abd4027b200fa5279c2ca8afdf68d3ad3
+  data.tar.gz: b38548fb44300f730c70e0e43abe48a7a9785f884d8b28d3d7447afc78a77ec5
 SHA512:
-  metadata.gz: 052506c11e7921c9412fdbbdaeedbc05a199700867502e87c73bb00d4bb649007ed4f5c8f3f57020f96084d26128aaf5ba9973504018a76a1ecf4cd93b014186
-  data.tar.gz: fb775b4103f3eaa22ae76c5e4875d6c3117e44db8c2e6cd39189800d0fcafb63a8d8368a91a195ea8380cf9ce0744431e41e303bec0c478d61c5606d4ca8bd14
+  metadata.gz: eef0f965e3d2b23cafcbfe6d3c39581ef4556e2b606a1668e3c26a2817bd4a75a94635c5102795171f4deeef6a0b52c74ff3bcac6f81c83a69e69d746f369a0b
+  data.tar.gz: '058f0007e68c8e8d3bbe5ea9236615ba252a44b12c5e91042390a8f920a4127cbe2950e8748f088daab02e8a7b8c6eb0a10b226c378cf4330951b1f8e94db898'

data/README.md

@@ -6,7 +6,7 @@ A Ruby Object Mapper for Mongo.
 
 [<img src="https://github.com/mongomapper/mongomapper/workflows/Ruby/badge.svg?branch=master" alt="Build Status" />](https://github.com/mongomapper/mongomapper/actions?query=workflow%3ARuby+branch%3Amaster)
 
-[<img src="https://coveralls.io/repos/mongomapper/mongomapper/badge.svg" alt="Coverage Status" />](https://coveralls.io/r/mongomapper/mongomapper)
+<!-- [<img src="https://coveralls.io/repos/mongomapper/mongomapper/badge.svg" alt="Coverage Status" />](https://coveralls.io/r/mongomapper/mongomapper) -->
 
 ## Install
 
@@ -18,16 +18,27 @@ http://mongomapper.com/documentation/
 
 http://rdoc.info/github/mongomapper/mongomapper
 
+## Open Commit Policy
+
+Like Rubinius, we're trying out an "open commit policy".
+
+If you've committed one (code) patch that has been accepted and would like to
+work some more on the project, send an email to Scott Taylor
+<scott@railsnewbie.com> along with your commit sha1.
+
 ## Compatibility
 
 MongoMapper is tested against:
 
-* MRI 2.4 - 2.7
+* MRI 2.4 - 3.0.1
 * JRuby (Versions with 1.9 compatibility)
 
 Additionally, MongoMapper is tested against:
 
-* Rails 5.0+ - 6.0
+* Rails 5.0 - 5.2
+* Rails 6.0 - 6.1
+
+Note, if you are using Ruby 3.0+, you'll need Rails 6.
 
 ## Contributing & Development
 
@@ -37,8 +48,8 @@ Additionally, MongoMapper is tested against:
 
 * Fork the project.
 * Make your feature addition or bug fix. All specs should pass.
-* Add specs for your changes. This is important so I don't break it in a future version unintentionally.
-* Commit, do not mess with Rakefile, version, or history. If you want to have your own version, that is fine but bump version in a commit by itself in another branch so a maintainer ignore it when your pull request is merged.
+* Add specs for your changes. This is important so that it doesn't break in a future version.
+* Commit, do not mess with Rakefile, version, or history. If you want to have your own version, that is fine but bump version in a commit by itself in another branch so a maintainer can ignore it when your pull request is merged.
 * Send a pull request. Bonus points for topic branches.
 
 ## Problems or Questions?
@@ -47,7 +58,7 @@ Hit up the Google group: http://groups.google.com/group/mongomapper
 
 ## Copyright
 
-Copyright (c) 2009-2020 MongoMapper. See LICENSE for details.
+Copyright (c) 2009-2021 MongoMapper. See LICENSE for details.
 
 ## Contributors
 

data/lib/mongo_mapper.rb

@@ -70,25 +70,27 @@ module MongoMapper
 
     module Associations
       autoload :Base,                         'mongo_mapper/plugins/associations/base'
-      autoload :Collection,                   'mongo_mapper/plugins/associations/collection'
-      autoload :EmbeddedCollection,           'mongo_mapper/plugins/associations/embedded_collection'
       autoload :ManyAssociation,              'mongo_mapper/plugins/associations/many_association'
       autoload :SingleAssociation,            'mongo_mapper/plugins/associations/single_association'
       autoload :BelongsToAssociation,         'mongo_mapper/plugins/associations/belongs_to_association'
       autoload :OneAssociation,               'mongo_mapper/plugins/associations/one_association'
-      autoload :ManyDocumentsProxy,           'mongo_mapper/plugins/associations/many_documents_proxy'
-      autoload :BelongsToProxy,               'mongo_mapper/plugins/associations/belongs_to_proxy'
-      autoload :BelongsToPolymorphicProxy,    'mongo_mapper/plugins/associations/belongs_to_polymorphic_proxy'
-      autoload :ManyPolymorphicProxy,         'mongo_mapper/plugins/associations/many_polymorphic_proxy'
-      autoload :ManyEmbeddedProxy,            'mongo_mapper/plugins/associations/many_embedded_proxy'
-      autoload :ManyEmbeddedPolymorphicProxy, 'mongo_mapper/plugins/associations/many_embedded_polymorphic_proxy'
-      autoload :ManyDocumentsAsProxy,         'mongo_mapper/plugins/associations/many_documents_as_proxy'
-      autoload :OneProxy,                     'mongo_mapper/plugins/associations/one_proxy'
-      autoload :OneAsProxy,                   'mongo_mapper/plugins/associations/one_as_proxy'
-      autoload :OneEmbeddedProxy,             'mongo_mapper/plugins/associations/one_embedded_proxy'
-      autoload :OneEmbeddedPolymorphicProxy,  'mongo_mapper/plugins/associations/one_embedded_polymorphic_proxy'
-      autoload :InArrayProxy,                 'mongo_mapper/plugins/associations/in_array_proxy'
-      autoload :InForeignArrayProxy,          'mongo_mapper/plugins/associations/in_foreign_array_proxy'
+
+      autoload :Proxy,                        'mongo_mapper/plugins/associations/proxy/proxy'
+      autoload :Collection,                   'mongo_mapper/plugins/associations/proxy/collection'
+      autoload :EmbeddedCollection,           'mongo_mapper/plugins/associations/proxy/embedded_collection'
+      autoload :ManyDocumentsProxy,           'mongo_mapper/plugins/associations/proxy/many_documents_proxy'
+      autoload :BelongsToProxy,               'mongo_mapper/plugins/associations/proxy/belongs_to_proxy'
+      autoload :BelongsToPolymorphicProxy,    'mongo_mapper/plugins/associations/proxy/belongs_to_polymorphic_proxy'
+      autoload :ManyPolymorphicProxy,         'mongo_mapper/plugins/associations/proxy/many_polymorphic_proxy'
+      autoload :ManyEmbeddedProxy,            'mongo_mapper/plugins/associations/proxy/many_embedded_proxy'
+      autoload :ManyEmbeddedPolymorphicProxy, 'mongo_mapper/plugins/associations/proxy/many_embedded_polymorphic_proxy'
+      autoload :ManyDocumentsAsProxy,         'mongo_mapper/plugins/associations/proxy/many_documents_as_proxy'
+      autoload :OneProxy,                     'mongo_mapper/plugins/associations/proxy/one_proxy'
+      autoload :OneAsProxy,                   'mongo_mapper/plugins/associations/proxy/one_as_proxy'
+      autoload :OneEmbeddedProxy,             'mongo_mapper/plugins/associations/proxy/one_embedded_proxy'
+      autoload :OneEmbeddedPolymorphicProxy,  'mongo_mapper/plugins/associations/proxy/one_embedded_polymorphic_proxy'
+      autoload :InArrayProxy,                 'mongo_mapper/plugins/associations/proxy/in_array_proxy'
+      autoload :InForeignArrayProxy,          'mongo_mapper/plugins/associations/proxy/in_foreign_array_proxy'
     end
   end
 
@@ -99,7 +101,4 @@ Dir[File.join(File.dirname(__FILE__), 'mongo_mapper', 'extensions', '*.rb')].eac
   require extension
 end
 
-# FIXME: autoload with proxy is failing, need to investigate
-require 'mongo_mapper/plugins/associations/proxy'
-
 ActiveSupport.run_load_hooks(:mongo_mapper, MongoMapper)

data/lib/mongo_mapper/extensions/array.rb

@@ -24,4 +24,4 @@ end
 
 class Array
   include MongoMapper::Extensions::Array
-end
+end

data/lib/mongo_mapper/extensions/binary.rb

@@ -19,4 +19,4 @@ end
 
 class Binary
   extend MongoMapper::Extensions::Binary
-end
+end

data/lib/mongo_mapper/extensions/date.rb

@@ -22,4 +22,4 @@ end
 
 class Date
   extend MongoMapper::Extensions::Date
-end
+end

data/lib/mongo_mapper/extensions/float.rb

@@ -11,4 +11,4 @@ end
 
 class Float
   extend MongoMapper::Extensions::Float
-end
+end

data/lib/mongo_mapper/extensions/hash.rb

@@ -23,4 +23,4 @@ end
 
 class Hash
   include MongoMapper::Extensions::Hash
-end
+end

data/lib/mongo_mapper/extensions/nil_class.rb

@@ -14,5 +14,5 @@ module MongoMapper
 end
 
 class NilClass
-  include MongoMapper::Extensions::NilClass
-end
+  extend MongoMapper::Extensions::NilClass
+end

data/lib/mongo_mapper/extensions/object.rb

@@ -27,4 +27,4 @@ end
 
 class Object
   include MongoMapper::Extensions::Object
-end
+end

data/lib/mongo_mapper/extensions/object_id.rb

@@ -33,4 +33,4 @@ class BSON::ObjectId
   def original_to_json(*args)
     original_as_json.to_json(*args)
   end
-end
+end

data/lib/mongo_mapper/extensions/set.rb

@@ -17,4 +17,4 @@ end
 
 class Set
   extend MongoMapper::Extensions::Set
-end
+end

data/lib/mongo_mapper/extensions/string.rb

@@ -23,4 +23,4 @@ end
 
 class String
   include MongoMapper::Extensions::String
-end
+end

data/lib/mongo_mapper/plugins/associations/many_association.rb

@@ -30,12 +30,11 @@ module MongoMapper
         def setup(model)
           model.associations_module.module_eval(<<-end_eval, __FILE__, __LINE__ + 1)
             def #{name}
-              get_proxy(associations[#{name.inspect}])
+              get_proxy(associations[#{name.inspect}]).read
             end
 
             def #{name}=(value)
-              get_proxy(associations[#{name.inspect}]).replace(value)
-              value
+              get_proxy(associations[#{name.inspect}]).write(value)
             end
           end_eval
 

data/lib/mongo_mapper/plugins/associations/proxy/collection.rb

@@ -0,0 +1,55 @@
+# encoding: UTF-8
+module MongoMapper
+  module Plugins
+    module Associations
+      class Collection < Proxy
+        include Enumerable
+
+        def to_a
+          load_target
+
+          target.is_a?(Array) ?
+            target :
+            Array(target)
+        end
+
+        alias_method :to_ary, :to_a
+
+        def each(&block)
+          to_a.each(&block)
+        end
+
+        def [](val)
+          objs = to_a
+          objs ? objs[val] : nil
+        end
+
+        def empty?
+          to_a.empty?
+        end
+
+        def size
+          to_a.size
+        end
+
+        def length
+          to_a.length
+        end
+
+        def reset
+          super
+          target = []
+        end
+
+        def read
+          self
+        end
+
+        def write(value)
+          replace(value)
+          read
+        end
+      end
+    end
+  end
+end

data/lib/mongo_mapper/plugins/associations/{many_documents_proxy.rb → proxy/many_documents_proxy.rb}

@@ -76,10 +76,6 @@ module MongoMapper
           @target.each { |doc| doc.save(options) } if @target
         end
 
-        def each(&block)
-          load_target.each(&block)
-        end
-
       protected
 
         def query(options={})

data/lib/mongo_mapper/plugins/associations/{proxy.rb → proxy/proxy.rb}

@@ -1,19 +1,25 @@
 # encoding: UTF-8
 require 'forwardable'
+
 module MongoMapper
   module Plugins
     module Associations
       class Proxy
         extend Forwardable
 
-        alias :proxy_respond_to? :respond_to?
-        alias :proxy_extend :extend
-
-        instance_methods.each { |m| undef_method m unless m =~ /(^__|^nil\?$|^send$|proxy_|^respond_to_missing\?$|^object_id$)/ }
+        class << self
+          def define_proxy_method(method)
+            define_method(method) do |*args, &block|
+              proxy_method(method, *args, &block)
+            end
+          end
+        end
 
         attr_reader :proxy_owner, :association, :target
 
-        alias :proxy_association :association
+        alias_method :proxy_respond_to?, :respond_to?
+        alias_method :proxy_extend, :extend
+        alias_method :proxy_association, :association
 
         def_delegators :proxy_association, :klass, :options
         def_delegator  :klass, :collection
@@ -24,26 +30,30 @@ module MongoMapper
           reset
         end
 
-        # Active support in rails 3 beta 4 can override to_json after this is loaded,
-        # at least when run in mongomapper tests. The implementation was changed in master
-        # some time after this, so not sure whether this is still a problem.
-        #
-        # In rails 2, this isn't a problem however it also solves an issue where
-        # to_json isn't forwarded because it supports to_json itself
-        def to_json(*options)
-          load_target
-          target.to_json(*options)
-        end
-
-        # see comments to to_json
-        def as_json(*options)
-          load_target
-          target.as_json(*options)
+        [
+          :is_a?,
+          :to_mongo,
+          :==,
+          :!=,
+          :nil?,
+          :blank?,
+          :present?,
+          # Active support in rails 3 beta 4 can override to_json after this is loaded,
+          # at least when run in mongomapper tests. The implementation was changed in master
+          # some time after this, so not sure whether this is still a problem.
+          #
+          # In rails 2, this isn't a problem however it also solves an issue where
+          # to_json isn't forwarded because it supports to_json itself
+          :to_json,
+          # see comments to to_json
+          :as_json,
+        ].each do |m|
+          define_proxy_method(m)
         end
 
         def inspect
           load_target
-          target.inspect
+          "#<#{self.class.inspect}:#{object_id} #{@target.inspect}>"
         end
 
         def loaded?
@@ -54,21 +64,6 @@ module MongoMapper
           @loaded = true
         end
 
-        def nil?
-          load_target
-          target.nil?
-        end
-
-        def blank?
-          load_target
-          target.blank?
-        end
-
-        def present?
-          load_target
-          target.present?
-        end
-
         def reload
           reset
           load_target
@@ -87,16 +82,7 @@ module MongoMapper
         end
 
         def respond_to?(*args)
-          proxy_respond_to?(*args) || (load_target && target.respond_to?(*args))
-        end
-
-        def send(method, *args, &block)
-          if proxy_respond_to?(method, true)
-            super
-          else
-            load_target
-            target.send(method, *args, &block)
-          end
+          super || (load_target && target.respond_to?(*args))
         end
 
         def read
@@ -104,6 +90,16 @@ module MongoMapper
           @target
         end
 
+        def write(value)
+          replace(value)
+          read
+        end
+
+        def proxy_method(method, *args, &block)
+          load_target
+          target.public_send(method, *args, &block)
+        end
+
       protected
 
         def load_target
@@ -138,9 +134,28 @@ module MongoMapper
           false
         end
 
+        def define_proxy_method(method)
+          metaclass = class << self; self; end
+          metaclass.class_eval do
+            define_proxy_method(method)
+          end
+        end
+
+        def define_and_call_proxy_method(method, *args, &block)
+          define_proxy_method(method)
+          public_send(method, *args, &block)
+        end
+
         def method_missing(method, *args, &block)
-          if load_target
-            target.public_send(method, *args, &block)
+          # load the target just in case it isn't loaded
+          load_target
+
+          # only define the method if the target has the method
+          # NOTE: include private methods!
+          if target.respond_to?(method, true)
+            define_and_call_proxy_method(method, *args, &block)
+          else
+            super
           end
         end
       end