Removed assign_attributes. Don't like the api for that. Trying to thi…

…nk things through more. Also refactored attributes to be more clear and simple instead of using returning.

lib/mongomapper/embedded_document.rb

@@ -166,20 +166,12 @@ def attributes=(attrs)
       end
 
       def attributes
-        returning HashWithIndifferentAccess.new do |attributes|
-          self.class.keys.each_pair do |name, key|
-            value = value_for_key(key)
-            attributes[name] = value unless value.nil?
-          end
-
-          attributes.merge!(embedded_association_attributes)
-        end
-      end
-
-      def assign_attributes(white_list, values)
-        white_list.each do |key|
-          send("#{key}=", values[key]) if values.has_key?(key)
+        attrs = HashWithIndifferentAccess.new
+        self.class.keys.each_pair do |name, key|
+          value = value_for_key(key)
+          attrs[name] = value unless value.nil?
         end
+        attrs.merge!(embedded_association_attributes)
       end
 
       def [](name)

test/functional/test_document.rb

@@ -873,29 +873,6 @@ class ::Thing
     end
   end
 
-  context "Updating an existing document using assign attributes" do
-    setup do
-      @doc = @document.create(:first_name => 'John', :age => '27')
-      @doc.assign_attributes([:first_name], {:first_name => 'David', :age => '26'})
-      @doc.save
-    end
-
-    should "not insert document into collection" do
-      @document.count.should == 1
-    end
-
-    should "update attributes" do
-      @doc.first_name.should == 'David'
-      @doc.age.should_not == 26
-    end
-
-    should "only update the first name in the database" do
-      from_db = @document.find(@doc.id)
-      from_db.first_name.should == 'David'
-      from_db.age.should == 27
-    end
-  end
-
   context "Destroying a document that exists" do
     setup do
       @doc = @document.create(:first_name => 'John', :age => '27')