Added to_param directly to embedded document and removed from rails c…

…ompat modules.

lib/mongo_mapper/embedded_document.rb

@@ -198,6 +198,10 @@ def initialize(attrs={})
       def new?
         !!@new_document
       end
+
+      def to_param
+        id
+      end
 
       def attributes=(attrs)
         return if attrs.blank?

lib/mongo_mapper/rails_compatibility/document.rb

@@ -6,10 +6,6 @@ def self.included(model)
           alias_method :new_record?, :new?
         end
       end
-
-      def to_param
-        id
-      end
     end
   end
 end

lib/mongo_mapper/rails_compatibility/embedded_document.rb

@@ -18,10 +18,6 @@ def column_names
           keys.keys
         end
       end
-
-      def to_param
-        raise "Missing to_param method in #{self.class}. You should implement it to return the unique identifier of this embedded document within a document."
-      end
     end
   end
 end

test/functional/test_rails_compatibility.rb

@@ -16,11 +16,6 @@ class Order
     setup do
       Order.collection.clear
     end
-
-    should "have to_param that returns id" do
-      instance = Order.create('_id' => 1234)
-      instance.to_param.should == '1234'
-    end
 
     should "alias new to new_record?" do
       instance = Order.new

test/unit/test_embedded_document.rb

@@ -289,6 +289,11 @@ class EmbeddedDocumentTest < Test::Unit::TestCase
       end
     end
 
+    should "have to_param that is id" do
+      doc = @document.new
+      doc.to_param.should == doc.id
+    end
+
     should "have access to class logger" do
       doc = @document.new
       doc.logger.should == @document.logger

test/unit/test_rails_compatibility.rb

@@ -15,11 +15,7 @@ class SecondItem < Item
     key :second_only, String
   end
 
-  context "EmbeddedDocument" do
-    should "raise error for to_param as embedded do not have id's" do
-      lambda { Item.new.to_param }.should raise_error
-    end
-
+  context "EmbeddedDocument" do    
     should "alias many to has_many" do
       FirstItem.should respond_to(:has_many)
       FirstItem.method(:has_many).should == FirstItem.method(:many)