mongoid_monkey 0.2.4 → 0.2.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5ca29354de5f6ae28af0f6d5ae2f4eb72ed6fdc5
-  data.tar.gz: 8f1d232f62817858c7a6a6631eb9371cb15bc580
+  metadata.gz: 3993c5f5fbdf5f1f30b9a824ffbd3447772bf68d
+  data.tar.gz: 77026f83baf29fd6ebbcc63c7487818d3b0e081c
 SHA512:
-  metadata.gz: 6fd84e25a8a1bce569497e2cbef12225c6c4d0a86ce90f8a3cf2be81c0c20e6a99efea9dcb3abf7ff18c502da4f83eae57d5ab06a49640a5e866b9a688cf3dbf
-  data.tar.gz: cad54a66f5a126c469a26330d8415a5d6cf8b0da0b3ee13b6bbb4217ad43827a0e388256146e94ae147584875f3675b199928e003b0e264dd5517ec52b7eca3a
+  metadata.gz: a64eee10db5554f9f068522d8d571a7d42ffeabf1b14b84df8b8052eb1730311cc6f4964dd4bbfead35bbd24f078f68f8cd597a56d328993e26f11563a01bb6f
+  data.tar.gz: 935a41fc23448476f511e49fba9fea2fd7e546625758bb26e78c6f7f2c784fedaae1392158dd2417701e0afd7551fa535264ff7d84d425e478b8d0ac9cebd77a

data/README.md

@@ -37,7 +37,8 @@ If you would only like some of the patches, please copy and paste the code to yo
 | `instrument.rb` | Backport instrumentation change to Moped 1. | ● | ○ | ○ |
 | `reorder.rb` | Backport `Criteria#reorder` method from Mongoid 4. | ● | ○ | ○ |
 | `only_pluck_localized.rb` | Backport [PR #4299](https://github.com/mongodb/mongoid/pull/4299) from Mongoid 6 which fixes `#only`, `#without`, and `#pluck` with localized fields. | ● | × | × |
-| `embedded_touch.rb` | Backport [Issue #3310](https://github.com/mongodb/mongoid/commit/a94c2f43573e58f973913c881ad9d11d62bf857c) from Mongoid 4 to add `:touch` option to `embedded_in`. | ● | ○ | ○ |
+| `embedded_touch.rb` (1) | Backport [Issue #3310](https://github.com/mongodb/mongoid/commit/a94c2f43573e58f973913c881ad9d11d62bf857c) from Mongoid 4 to add `:touch` option to `embedded_in`. | ● | ○ | ○ |
+| `embedded_touch.rb` (2) | Backport [PR #4392](https://github.com/mongodb/mongoid/pull/4392) from Mongoid 6 to fix an infinite loop issue related to `:touch` callbacks. | ● | ● | ● |
 | `index_options.rb` | Backport latest index valid index options from Mongoid 6. | ● | ● | ○ |
 
 ### License

data/lib/patches/embedded_touch.rb

@@ -1,37 +1,96 @@
-# Backport Mongoid 4 :touch option for #embedded_in to Mongoid 3.
+# Backport support #embedded_in :touch option from Mongoid 4 to Mongoid 3.
+# Also support touch callback on update, and fix infinite loop issue.
 
 if Mongoid::VERSION =~ /\A3\./
 
 module Mongoid
-module Relations
-  module Embedded
-    class In < Relations::One
-      class << self
-        def valid_options
-          [ :autobuild, :cyclic, :polymorphic, :touch ]
+  module Relations
+    module Embedded
+      class In < Relations::One
+        class << self
+          def valid_options
+            [ :autobuild, :cyclic, :polymorphic, :touch ]
+          end
+        end
+      end
+    end
+
+    module Macros
+      module ClassMethods
+
+        def embedded_in(name, options = {}, &block)
+          if ancestors.include?(Mongoid::Versioning)
+            raise Errors::VersioningNotOnRoot.new(self)
+          end
+          meta = characterize(name, Embedded::In, options, &block)
+          self.embedded = true
+          relate(name, meta)
+          builder(name, meta).creator(name, meta)
+          touchable(meta)
+          add_counter_cache_callbacks(meta) if meta.counter_cached?
+          meta
         end
       end
     end
-  end
 
-  module Macros
-    module ClassMethods
+    module Touchable
+      module ClassMethods
 
-      def embedded_in(name, options = {}, &block)
-        if ancestors.include?(Mongoid::Versioning)
-          raise Errors::VersioningNotOnRoot.new(self)
+        def touchable(metadata)
+          if metadata.touchable?
+            name = metadata.name
+            method_name = define_relation_touch_method(name)
+            after_save method_name
+            after_destroy method_name
+            after_touch method_name
+          end
+          self
         end
-        meta = characterize(name, Embedded::In, options, &block)
-        self.embedded = true
-        relate(name, meta)
-        builder(name, meta).creator(name, meta)
-        touchable(meta)
-        add_counter_cache_callbacks(meta) if meta.counter_cached?
-        meta
       end
     end
   end
+
+  module Callbacks
+    def cascadable_children(kind, children = Set.new)
+      embedded_relations.each_pair do |name, metadata|
+        next unless metadata.cascading_callbacks?
+        without_autobuild do
+          delayed_pulls = delayed_atomic_pulls[name]
+          delayed_unsets = delayed_atomic_unsets[name]
+          children.merge(delayed_pulls) if delayed_pulls
+          children.merge(delayed_unsets) if delayed_unsets
+          relation = send(name)
+          Array.wrap(relation).each do |child|
+            next if children.include?(child)
+            children.add(child) if cascadable_child?(kind, child, metadata)
+            child.send(:cascadable_children, kind, children)
+          end
+        end
+      end
+      children.to_a
+    end
+
+    def cascadable_child?(kind, child, metadata)
+      return false if kind == :initialize || kind == :find || kind == :touch
+      return false if kind == :validate && metadata.validate?
+      child.callback_executable?(kind) ? child.in_callback_state?(kind) : false
+    end
+  end
 end
+
+end
+
+if Mongoid::VERSION =~ /\A[45]\./
+
+module Mongoid
+
+  module Interceptable
+    def cascadable_child?(kind, child, metadata)
+      return false if kind == :initialize || kind == :find || kind == :touch
+      return false if kind == :validate && metadata.validate?
+      child.callback_executable?(kind) ? child.in_callback_state?(kind) : false
+    end
+  end
 end
 
 end

data/lib/version.rb

@@ -1,3 +1,3 @@
 module MongoidMonkey
-  VERSION = '0.2.4'
+  VERSION = '0.2.5'
 end

data/spec/app/models/book.rb

@@ -0,0 +1,6 @@
+class Book
+  include Mongoid::Document
+  include Mongoid::Timestamps
+
+  embeds_many :pages, cascade_callbacks: true
+end

data/spec/app/models/edit.rb

@@ -0,0 +1,5 @@
+class Edit
+  include Mongoid::Document
+  include Mongoid::Timestamps::Updated
+  embedded_in :wiki_page, touch: true
+end

data/spec/app/models/page.rb

@@ -0,0 +1,6 @@
+class Page
+  include Mongoid::Document
+
+  embedded_in :book, touch: true
+  field :content, :type => String
+end

data/spec/app/models/wiki_page.rb

@@ -0,0 +1,8 @@
+class WikiPage
+  include Mongoid::Document
+  include Mongoid::Timestamps
+
+  field :title, type: String
+
+  embeds_many :edits, validate: false
+end

data/spec/unit/embedded_touch_spec.rb

@@ -1,21 +1,6 @@
 require "spec_helper"
 
-if Mongoid::VERSION =~ /\A3\./
-
-  class Edit
-    include Mongoid::Document
-    include Mongoid::Timestamps::Updated
-    embedded_in :wiki_page, touch: true
-  end
-
-  class WikiPage
-    include Mongoid::Document
-    include Mongoid::Timestamps
-
-    field :title, type: String
-
-    embeds_many :edits, validate: false
-  end
+if Mongoid::VERSION =~ /\A[345]\./
 
   describe Mongoid::Relations::Embedded::In do
 
@@ -48,5 +33,43 @@ if Mongoid::VERSION =~ /\A3\./
         expect(page.updated_at).to be_within(5).of(Time.now)
       end
     end
+
+    context "when the parent of embedded doc has cascade callbacks" do
+
+      let!(:book) do
+        Book.new
+      end
+
+      before do
+        book.pages.new
+        book.save
+        book.unset(:updated_at)
+        book.pages.first.touch
+      end
+
+      it "touches the parent document" do
+        expect(book.updated_at).to be_within(5).of(Time.now)
+      end
+    end
+
+    context "when multiple embedded docs with cascade callbacks" do
+
+      let!(:book) do
+        Book.new
+      end
+
+      before do
+        2.times { book.pages.new }
+        book.save
+        book.unset(:updated_at)
+        book.pages.first.content  = "foo"
+        book.pages.second.content = "bar"
+        book.pages.first.touch
+      end
+
+      it "touches the parent document" do
+        expect(book.updated_at).to be_within(5).of(Time.now)
+      end
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mongoid_monkey
 version: !ruby/object:Gem::Version
-  version: 0.2.4
+  version: 0.2.5
 platform: ruby
 authors:
 - johnnyshields
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-02-21 00:00:00.000000000 Z
+date: 2017-02-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -74,10 +74,14 @@ files:
 - lib/version.rb
 - spec/app/models/account.rb
 - spec/app/models/address.rb
+- spec/app/models/book.rb
 - spec/app/models/draft.rb
+- spec/app/models/edit.rb
 - spec/app/models/name.rb
+- spec/app/models/page.rb
 - spec/app/models/person.rb
 - spec/app/models/user.rb
+- spec/app/models/wiki_page.rb
 - spec/config/mongodb-mmapv1.conf
 - spec/config/mongodb-wiredtiger.conf
 - spec/gemfile/mongoid3.gemfile
@@ -144,10 +148,14 @@ summary: Monkey patches for Mongoid
 test_files:
 - spec/app/models/account.rb
 - spec/app/models/address.rb
+- spec/app/models/book.rb
 - spec/app/models/draft.rb
+- spec/app/models/edit.rb
 - spec/app/models/name.rb
+- spec/app/models/page.rb
 - spec/app/models/person.rb
 - spec/app/models/user.rb
+- spec/app/models/wiki_page.rb
 - spec/config/mongodb-mmapv1.conf
 - spec/config/mongodb-wiredtiger.conf
 - spec/gemfile/mongoid3.gemfile