Merge pull request mongodb#874 from nickhoffman/issue_836

Deleting from a M-M reference is one-sided
brajadorable · Jun 5, 2011 · 0b34731 · 0b34731
2 parents 5fbca70 + 6b88653
commit 0b34731
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/lib/mongoid/relations/referenced/many_to_many.rb b/lib/mongoid/relations/referenced/many_to_many.rb
@@ -97,6 +97,7 @@ def create!(attributes = nil, type = nil, &block)
         def delete(document, options = {})
           target.delete(document).tap do |doc|
             binding.unbind_one(doc, default_options.merge!(options)) if doc
+            doc.save if doc && base.persisted? && !options[:binding]
           end
         end
 

diff --git a/spec/functional/mongoid/relations/referenced/many_to_many_spec.rb b/spec/functional/mongoid/relations/referenced/many_to_many_spec.rb
@@ -1012,11 +1012,11 @@
       end
 
       it "removes the inverse reference" do
-        deleted.people.should be_empty
+        deleted.reload.people.should be_empty
       end
 
       it "removes the base id from the inverse keys" do
-        deleted.person_ids.should be_empty
+        deleted.reload.person_ids.should be_empty
       end