mongoid 1.0.4 → 1.0.5

data/VERSION

@@ -1 +1 @@
-1.0.4
+1.0.5

data/lib/mongoid/associations/has_many.rb

@@ -38,8 +38,11 @@ module Mongoid #:nodoc:
       #
       # Returns the newly created object.
       def build(attrs = {}, type = nil)
-        object = type ? type.instantiate(attrs) : @klass.instantiate(attrs)
-        push(object)
+        object = type ? type.instantiate : @klass.instantiate
+        object.parentize(@parent, @association_name)
+        object.write_attributes(attrs)
+        object.generate_key
+        @documents << object
         object
       end
 

data/lib/mongoid/associations/has_one.rb

@@ -6,19 +6,6 @@ module Mongoid #:nodoc:
 
       attr_reader :association_name, :document, :parent, :options
 
-      # Build a new object for the association.
-      def build(attrs = {}, type = nil)
-        @document = attrs.assimilate(@parent, @options, type)
-        self
-      end
-
-      # Create a new object for the association and save it.
-      def create(attrs = {}, type = nil)
-        build(attrs, type)
-        @document.save
-        self
-      end
-
       # Creates the new association by finding the attributes in
       # the parent document with its name, and instantiating a
       # new document for it.
@@ -58,6 +45,13 @@ module Mongoid #:nodoc:
         @document.valid?
       end
 
+      protected
+      # Build a new object for the association.
+      def build(attrs = {}, type = nil)
+        @document = attrs.assimilate(@parent, @options, type)
+        self
+      end
+
       class << self
         # Preferred method of instantiating a new +HasOne+, since nil values
         # will be handled properly.

data/lib/mongoid/document.rb

@@ -137,6 +137,15 @@ module Mongoid #:nodoc:
         self.class.instantiate(@attributes.except(:_id).except(:versions).dup, true)
       end
 
+      # Generate an id for this +Document+.
+      def generate_key
+        if primary_key
+          @attributes[:_id] = extract_keys.join(" ").identify
+        else
+          @attributes[:_id] = Mongo::ObjectID.new.to_s unless id
+        end
+      end
+
       # Instantiate a new +Document+, setting the Document's attributes if
       # given. If no attributes are provided, they will be initialized with
       # an empty +Hash+.
@@ -269,14 +278,6 @@ module Mongoid #:nodoc:
       end
 
       protected
-      def generate_key
-        if primary_key
-          @attributes[:_id] = extract_keys.join(" ").identify
-        else
-          @attributes[:_id] = Mongo::ObjectID.new.to_s unless id
-        end
-      end
-
       def generate_type
         @attributes[:_type] ||= self.class.name
       end

data/lib/mongoid/extensions/hash/assimilation.rb

@@ -18,7 +18,10 @@ module Mongoid #:nodoc:
         #
         # Returns: The child +Document+.
         def assimilate(parent, options, type = nil)
-          child = type ? type.instantiate(self.merge(:_type => type.name)) : options.klass.instantiate(self)
+          klass = type ? type : options.klass
+          child = klass.instantiate(:_parent => parent)
+          child.write_attributes(self.merge(:_type => klass.name))
+          child.generate_key
           child.assimilate(parent, options)
         end
       end

data/mongoid.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{mongoid}
-  s.version = "1.0.4"
+  s.version = "1.0.5"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Durran Jordan"]
-  s.date = %q{2010-01-12}
+  s.date = %q{2010-01-13}
   s.email = %q{durran@gmail.com}
   s.extra_rdoc_files = [
     "README.rdoc"

data/spec/spec_helper.rb

@@ -149,9 +149,14 @@ class Address
   field :city
   field :state
   field :post_code
+  field :parent_title
   key :street
   has_many :locations
   belongs_to :addressable, :inverse_of => :addresses
+
+  def set_parent=(set = false)
+    self.parent_title = addressable.title if set
+  end
 end
 
 class Location
@@ -164,8 +169,13 @@ class Name
   include Mongoid::Document
   field :first_name
   field :last_name
+  field :parent_title
   key :first_name, :last_name
   belongs_to :person, :inverse_of => :name
+
+  def set_parent=(set = false)
+    self.parent_title = person.title if set
+  end
 end
 
 class Comment

data/spec/unit/mongoid/associations/has_many_spec.rb

@@ -60,6 +60,20 @@ describe Mongoid::Associations::HasMany do
 
   describe "#build" do
 
+    context "setting the parent relationship" do
+
+      before do
+        @person = Person.new
+      end
+
+      it "happens before any other operation" do
+        address = @person.addresses.build(:set_parent => true, :street => "Madison Ave")
+        address._parent.should == @person
+        @person.addresses.first.should == address
+      end
+
+    end
+
     context "when a type is not provided" do
 
       before do

data/spec/unit/mongoid/associations/has_one_spec.rb

@@ -10,7 +10,7 @@ describe Mongoid::Associations::HasOne do
     @document = stub(:attributes => @attributes, :update => true)
   end
 
-  describe "#build" do
+  describe "#build_*" do
 
     context "when attributes provided" do
 
@@ -23,7 +23,7 @@ describe Mongoid::Associations::HasOne do
       end
 
       it "replaces the existing has_one" do
-        drink = @association.build({ :name => "Sapphire and Tonic" })
+        drink = @association.send(:build, { :name => "Sapphire and Tonic" })
         drink.name.should == "Sapphire and Tonic"
       end
 
@@ -40,51 +40,23 @@ describe Mongoid::Associations::HasOne do
       end
 
       it "instantiates a class of that type" do
-        writer = @association.build({ :speed => 500 }, HtmlWriter)
+        writer = @association.send(:build, { :speed => 500 }, HtmlWriter)
         writer.should be_a_kind_of(HtmlWriter)
         writer.speed.should == 500
       end
 
     end
 
-  end
-
-  describe "#create" do
-
-    context "when attributes provided" do
+    context "setting the parent relationship" do
 
       before do
-        @association = Mongoid::Associations::HasOne.new(
-          @document,
-          @attributes[:mixed_drink],
-          Mongoid::Associations::Options.new(:name => :mixed_drink)
-        )
-        @drink = MixedDrink.new(:name => "Sapphire and Tonic")
+        @person = Person.new
       end
 
-      it "replaces and saves the existing has_one" do
-        Mongoid::Commands::Save.expects(:execute).returns(true)
-        drink = @association.create({ :name => "Sapphire and Tonic" })
-        drink.name.should == "Sapphire and Tonic"
-      end
-
-    end
-
-    context "when a type is supplied" do
-
-      before do
-        @association = Mongoid::Associations::HasOne.new(
-          @document,
-          @attributes[:writer],
-          Mongoid::Associations::Options.new(:name => :writer)
-        )
-      end
-
-      it "instantiates a class of that type" do
-        Mongoid::Commands::Save.expects(:execute).returns(true)
-        writer = @association.create({ :speed => 500 }, HtmlWriter)
-        writer.should be_a_kind_of(HtmlWriter)
-        writer.speed.should == 500
+      it "happens before any other operation" do
+        name = @person.build_name(:set_parent => true, :street => "Madison Ave")
+        name._parent.should == @person
+        @person.name.should == name
       end
 
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: mongoid
 version: !ruby/object:Gem::Version 
-  version: 1.0.4
+  version: 1.0.5
 platform: ruby
 authors: 
 - Durran Jordan
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-01-12 00:00:00 -05:00
+date: 2010-01-13 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency