mongo_mapper-unstable 2010.2.19 → 2010.2.22

data/README.rdoc

@@ -24,6 +24,8 @@ http://groups.google.com/group/mongomapper
 To see if the problem you are having is a verified issue, you can see the MM pivotal tracker project:
 http://www.pivotaltracker.com/projects/33576
 
+There is no need to request to join the Pivotal Tracker project as I am only granting access to a select few (easier to keep things organized). If you have a problem, please use the mailing list. If I confirm it to be a bug, I am happy to add it to PT. Thanks!
+
 == Copyright
 
 Copyright (c) 2009 John Nunemaker. See LICENSE for details.

data/VERSION

@@ -1 +1 @@
-2010.02.19
+2010.02.22

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

@@ -79,6 +79,7 @@ module MongoMapper
           unless doc.new?
             ids << doc.id
             owner.save
+            reset
           end
           doc
         end
@@ -88,6 +89,7 @@ module MongoMapper
           unless doc.new?
             ids << doc.id
             owner.save
+            reset
           end
           doc
         end

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

@@ -52,18 +52,21 @@ module MongoMapper
         def build(attrs={})
           doc = klass.new(attrs)
           apply_scope(doc)
+          reset
           doc
         end
 
         def create(attrs={})
           doc = klass.new(attrs)
           apply_scope(doc).save
+          reset
           doc
         end
 
         def create!(attrs={})
           doc = klass.new(attrs)
           apply_scope(doc).save!
+          reset
           doc
         end
 

data/lib/mongo_mapper/plugins/protected.rb

@@ -37,7 +37,7 @@ module MongoMapper
         protected
           def filter_protected_attrs(attrs)
             return attrs if protected_attributes.blank?
-            attrs.dup.delete_if { |key, val| protected_attributes.include?(key) }
+            attrs.dup.delete_if { |key, val| protected_attributes.include?(key.to_sym) }
           end
       end
     end

data/test/functional/associations/test_in_array_proxy.rb

@@ -80,6 +80,12 @@ class InArrayProxyTest < Test::Unit::TestCase
       should "save doc" do
         @list.should_not be_new
       end
+
+      should "reset cache" do
+        @user.lists.size.should == 1
+        @user.lists.create(:name => 'Moo!')
+        @user.lists.size.should == 2
+      end
     end
 
     context "create!" do
@@ -110,6 +116,12 @@ class InArrayProxyTest < Test::Unit::TestCase
           @user.lists.create!
         end
       end
+
+      should "reset cache" do
+        @user.lists.size.should == 1
+        @user.lists.create!(:name => 'Moo!')
+        @user.lists.size.should == 2
+      end
     end
 
     context "Finding scoped to association" do

data/test/functional/associations/test_many_documents_proxy.rb

@@ -54,6 +54,14 @@ class ManyDocumentsProxyTest < Test::Unit::TestCase
       status = project.statuses.build(:name => 'Foo')
       status.name.should == 'Foo'
     end
+
+    should "reset cache" do
+      project = Project.create
+      project.statuses.size.should == 0
+      status = project.statuses.build(:name => 'Foo')
+      status.save!
+      project.statuses.size.should == 1
+    end
   end
   
   context "create" do
@@ -75,6 +83,13 @@ class ManyDocumentsProxyTest < Test::Unit::TestCase
       status = project.statuses.create(:name => 'Foo!')
       status.name.should == 'Foo!'
     end
+
+    should "reset cache" do
+      project = Project.create
+      project.statuses.size.should == 0
+      project.statuses.create(:name => 'Foo!')
+      project.statuses.size.should == 1
+    end
   end
   
   context "create!" do
@@ -103,6 +118,13 @@ class ManyDocumentsProxyTest < Test::Unit::TestCase
         project.statuses.create!(:name => nil)
       }.should raise_error(MongoMapper::DocumentNotValid)
     end
+
+    should "reset cache" do
+      project = Project.create
+      project.statuses.size.should == 0
+      project.statuses.create!(:name => 'Foo!')
+      project.statuses.size.should == 1
+    end
   end
   
   context "count" do

data/test/functional/test_protected.rb

@@ -16,7 +16,7 @@ class ProtectedTest < Test::Unit::TestCase
     should 'have protected attributes class method' do
       @doc_class.protected_attributes.should == [:admin].to_set
     end
-    
+
     should "default protected attributes to nil" do
       Doc().protected_attributes.should be_nil
     end
@@ -46,7 +46,7 @@ class ProtectedTest < Test::Unit::TestCase
       doc = @doc_class.new(:name => 'John')
       doc.admin = true
       doc.save!
-      
+
       doc = @doc_class.first(:name => 'John')
       doc.admin.should be_true
       doc.name.should == 'John'
@@ -63,8 +63,14 @@ class ProtectedTest < Test::Unit::TestCase
       @doc.name.should == 'Stimpson J. Cat'
       @doc.admin.should be_false
     end
+
+    should 'be indifferent to whether the protected keys are strings or symbols' do
+      @doc.update_attributes!("name" => 'Stimpson J. Cat', "admin" => true)
+      @doc.name.should == 'Stimpson J. Cat'
+      @doc.admin.should be_false
+    end
   end
-  
+
   context "Single collection inherited protected attributes" do
     setup do
       class ::GrandParent
@@ -126,7 +132,7 @@ class ProtectedTest < Test::Unit::TestCase
     should 'have protected attributes class method' do
       @edoc_class.protected_attributes.should == [:admin].to_set
     end
-    
+
     should "default protected attributes to nil" do
       EDoc().protected_attributes.should be_nil
     end
@@ -152,4 +158,4 @@ class ProtectedTest < Test::Unit::TestCase
       @edoc.admin.should be_false
     end
   end
-end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: mongo_mapper-unstable
 version: !ruby/object:Gem::Version 
-  version: 2010.2.19
+  version: 2010.2.22
 platform: ruby
 authors: 
 - John Nunemaker
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-02-19 00:00:00 -05:00
+date: 2010-02-22 00:00:00 -05:00
 default_executable: mmconsole
 dependencies: 
 - !ruby/object:Gem::Dependency