From 823b623fe2de8846c37aa13250010809ac940b57 Mon Sep 17 00:00:00 2001 From: Eloy Duran Date: Fri, 21 Nov 2008 09:47:55 +0100 Subject: [PATCH] Allow optional arguments and/or block for Object#try like Object#send does. [#1425 state:resolved] Original suggestion by Pat Nakajima. Signed-off-by: Pratik Naik --- activesupport/lib/active_support/core_ext/object/misc.rb | 9 +++++++-- activesupport/test/core_ext/object_and_class_ext_test.rb | 7 +++++++ 2 files changed, 14 insertions(+), 2 deletions(-) diff --git a/activesupport/lib/active_support/core_ext/object/misc.rb b/activesupport/lib/active_support/core_ext/object/misc.rb index 50f824c358a0f..46f9c7d676a74 100644 --- a/activesupport/lib/active_support/core_ext/object/misc.rb +++ b/activesupport/lib/active_support/core_ext/object/misc.rb @@ -73,6 +73,7 @@ def acts_like?(duck) end # Tries to send the method only if object responds to it. Return +nil+ otherwise. + # It will also forward any arguments and/or block like Object#send does. # # ==== Example : # @@ -81,7 +82,11 @@ def acts_like?(duck) # # With try # @person.try(:name) - def try(method) - send(method) if respond_to?(method, true) + # + # # try also accepts arguments/blocks for the method it is trying + # Person.try(:find, 1) + # @people.try(:map) {|p| p.name} + def try(method, *args, &block) + send(method, *args, &block) if respond_to?(method, true) end end diff --git a/activesupport/test/core_ext/object_and_class_ext_test.rb b/activesupport/test/core_ext/object_and_class_ext_test.rb index fbdce56ac265a..2f79b6f67f344 100644 --- a/activesupport/test/core_ext/object_and_class_ext_test.rb +++ b/activesupport/test/core_ext/object_and_class_ext_test.rb @@ -271,4 +271,11 @@ class << @string assert_equal 5, @string.try(:size) end + def test_argument_forwarding + assert_equal 'Hey', @string.try(:sub, 'llo', 'y') + end + + def test_block_forwarding + assert_equal 'Hey', @string.try(:sub, 'llo') { |match| 'y' } + end end