github/rails
1
1
Fork 0
mirror of https://github.com/github/rails.git synced 2026-04-26 03:00:59 -04:00
Code Issues Packages Projects Releases Wiki Activity

don't raise NoMethodError the tried method doesn't exists

Browse Source
This commit is contained in:
dmathieu
2011-05-21 23:43:12 +02:00
parent f674aed150
commit 29a5aeaae9
2 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
activesupport/lib/active_support/core_ext/object/try.rb
View File

@@ -28,6 +28,8 @@ class Object
def try(*a, &b) def try(*a, &b)
if a.empty? && block_given? if a.empty? && block_given?
yield self yield self
elsif !a.empty? && !respond_to?(a.first)
nil
else else
__send__(*a, &b) __send__(*a, &b)
end end

2
activesupport/test/core_ext/object_and_class_ext_test.rb
View File

@@ -99,7 +99,7 @@ class ObjectTryTest < Test::Unit::TestCase
def test_nonexisting_method def test_nonexisting_method
method = :undefined_method method = :undefined_method
assert !@string.respond_to?(method) assert !@string.respond_to?(method)
assert_raise(NoMethodError) { @string.try(method) } assert_nil @string.try(method)
end end
def test_valid_method def test_valid_method