Ticket #7356: add_argument_to_activerecord_to_param.patch

activerecord/test/base_test.rb

@@ -1585 +1585 @@
-  def test_to_param_should_return_string
-    assert_kind_of String, Client.find(:first).to_param
-  end
+  
+  def test_to_param_can_return_arbitrary_method
+    assert_equal topics(:first).title.to_s, topics(:first).to_param(:title)
+  end
+  
+  def test_to_param_returns_id_on_unknown_method
+    assert_equal topics(:first).id.to_s, topics(:first).to_param(:foo)
+  end
-
+  def test_to_param_returns_id_by_default
+    assert_equal topics(:first).id.to_s, topics(:first).to_param
+  end
+
-  # FIXME: this test ought to run, but it needs to run sandboxed so that it
-  # doesn't b0rk the current test environment by undefing everything.
-  #

activerecord/lib/active_record/base.rb

@@ -1522 +1522 @@
-      end
-
-      # Enables Active Record objects to be used as URL parameters in Action Pack automatically.
-
-
-
+
+
+
+
+
+
+
-      end
-
-      def id_before_type_cast #:nodoc:

actionpack/lib/action_controller/routing.rb

@@ -1179 +1179 @@
-
-        options_as_params = options[:controller] ? { :action => "index" } : {}
-        options.each do |k, value|
-
+
+
+
-        end
-        options_as_params
-      end