diff options
| author | Marcel Molina <marcel@vernix.org> | 2006-04-26 21:49:41 +0000 |
|---|---|---|
| committer | Marcel Molina <marcel@vernix.org> | 2006-04-26 21:49:41 +0000 |
| commit | 71ff705449aa0cc38baa746c07d2520fdb64096a (patch) | |
| tree | a0aa21a9b6245dadf271bc38a5237524100a26cb /activesupport | |
| parent | ca2eb16bf9a5844607787c49be38bd36cfbfd568 (diff) | |
| download | rails-71ff705449aa0cc38baa746c07d2520fdb64096a.tar.gz rails-71ff705449aa0cc38baa746c07d2520fdb64096a.tar.bz2 rails-71ff705449aa0cc38baa746c07d2520fdb64096a.zip | |
Allow default options in with_options to be overridden. Closes #4480. [murphy@cYcnus.de]
git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@4282 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
Diffstat (limited to 'activesupport')
| -rw-r--r-- | activesupport/CHANGELOG | 2 | ||||
| -rw-r--r-- | activesupport/lib/active_support/option_merger.rb | 4 | ||||
| -rw-r--r-- | activesupport/test/option_merger_test.rb | 16 |
3 files changed, 20 insertions, 2 deletions
diff --git a/activesupport/CHANGELOG b/activesupport/CHANGELOG index 3a4b4e477f..bdd1c4d9f8 100644 --- a/activesupport/CHANGELOG +++ b/activesupport/CHANGELOG @@ -1,5 +1,7 @@ *SVN* +* Allow default options in with_options to be overridden. Closes #4480. [murphy@cYcnus.de] + * Added Module#alias_method_chain [Jamis Buck] * Updated to Builder 2.0 [DHH] diff --git a/activesupport/lib/active_support/option_merger.rb b/activesupport/lib/active_support/option_merger.rb index 51a2ea1328..f944729bbc 100644 --- a/activesupport/lib/active_support/option_merger.rb +++ b/activesupport/lib/active_support/option_merger.rb @@ -15,8 +15,8 @@ module ActiveSupport end def merge_argument_options!(arguments) - arguments << if arguments.last.respond_to? :merge! - arguments.pop.dup.merge!(@options) + arguments << if arguments.last.respond_to? :to_hash + @options.merge(arguments.pop) else @options.dup end diff --git a/activesupport/test/option_merger_test.rb b/activesupport/test/option_merger_test.rb index f5287d5725..e97a4f918b 100644 --- a/activesupport/test/option_merger_test.rb +++ b/activesupport/test/option_merger_test.rb @@ -27,6 +27,22 @@ class OptionMergerTest < Test::Unit::TestCase end end + def test_method_with_options_allows_to_overwrite_options + local_options = {:hello => 'moon'} + assert_equal @options.keys, local_options.keys + + with_options(@options) do |o| + assert_equal local_options, method_with_options(local_options) + assert_equal @options.merge(local_options), + o.method_with_options(local_options) + assert_equal local_options, o.method_with_options(local_options) + end + with_options(local_options) do |o| + assert_equal local_options.merge(@options), + o.method_with_options(@options) + end + end + private def method_with_options(options = {}) options |