Changeset 7764

Timestamp:

10/07/07 02:13:51 (1 year ago)

Author:

nzkoz

Message:

Add Base#to_param to Active Resource that quacks like ActiveRecord. Closes #9557 [bradediger]

Files:

• trunk/activeresource/lib/active_resource/base.rb (modified) (1 diff)
• trunk/activeresource/test/base_test.rb (modified) (1 diff)

trunk/activeresource/lib/active_resource/base.rb

@@ -549 +549 @@
-    def id=(id)
-      attributes[self.class.primary_key] = id
+    end
+
+    # Allows ActiveResource objects to be used as parameters in ActionPack URL generation.
+    def to_param
+      id && id.to_s
-    end
-

trunk/activeresource/test/base_test.rb

@@ -445 +445 @@
-    assert xml.include?('<id type="integer">1</id>')
-  end
+
+  def test_to_param_quacks_like_active_record
+    new_person = Person.new
+    assert_nil new_person.to_param
+    matz = Person.find(1)
+    assert_equal '1', matz.to_param
+  end
-end