diff options
author | Jeremy Kemper <jeremy@bitsweat.net> | 2013-10-11 13:05:29 -0700 |
---|---|---|
committer | Jeremy Kemper <jeremy@bitsweat.net> | 2013-10-11 13:05:29 -0700 |
commit | 365110196afcf952bc22729d4467d579b708328f (patch) | |
tree | 7525a43f663b790929004b504ad92f9cb4d5b63f | |
parent | cc362fe17b9a3e4be34a6a5a9f97f9c271bd7513 (diff) | |
download | rails-365110196afcf952bc22729d4467d579b708328f.tar.gz rails-365110196afcf952bc22729d4467d579b708328f.tar.bz2 rails-365110196afcf952bc22729d4467d579b708328f.zip |
Revert "Merge pull request #12480 from iwiznia/master"
This reverts commit e5f5a838b96a362534d9bb60d02334439ed9784c, reversing
changes made to d7567f3290a50952494e9213556a1f283a6cf3a0.
-rw-r--r-- | activesupport/CHANGELOG.md | 7 | ||||
-rw-r--r-- | activesupport/lib/active_support/duration.rb | 20 | ||||
-rw-r--r-- | activesupport/test/core_ext/duration_test.rb | 15 | ||||
-rw-r--r-- | guides/source/active_support_core_extensions.md | 15 |
4 files changed, 0 insertions, 57 deletions
diff --git a/activesupport/CHANGELOG.md b/activesupport/CHANGELOG.md index 478fef2baf..f1dd7c312d 100644 --- a/activesupport/CHANGELOG.md +++ b/activesupport/CHANGELOG.md @@ -1,10 +1,3 @@ -* Add `flatten` and `flatten!` methods to Duration objects. - - Example: - Date.today + (1.month + 1.month).flatten == Date.today + 2.months - - *Ionatan Wiznia* - * `require_dependency` accepts objects that respond to `to_path`, in particular `Pathname` instances. diff --git a/activesupport/lib/active_support/duration.rb b/activesupport/lib/active_support/duration.rb index 1db0ca51ae..87b6407038 100644 --- a/activesupport/lib/active_support/duration.rb +++ b/activesupport/lib/active_support/duration.rb @@ -81,18 +81,6 @@ module ActiveSupport to_i end - # Flattens all the +parts+ of the duration, and returns a - # new duration object. - def flatten - Duration.new(@value, flatten_parts) - end - - # Flattens all the +parts+ of this duration. - def flatten! - @parts = flatten_parts - self - end - protected def sum(sign, time = ::Time.current) #:nodoc: @@ -109,14 +97,6 @@ module ActiveSupport end end - def flatten_parts - @parts.inject({}) do |result, (name, value)| - result[name] ||= 0 - result[name] += value - result - end.to_a - end - private def method_missing(method, *args, &block) #:nodoc: diff --git a/activesupport/test/core_ext/duration_test.rb b/activesupport/test/core_ext/duration_test.rb index cc24d7c74c..ed267cf4b9 100644 --- a/activesupport/test/core_ext/duration_test.rb +++ b/activesupport/test/core_ext/duration_test.rb @@ -145,21 +145,6 @@ class DurationTest < ActiveSupport::TestCase assert_equal '172800', 2.days.to_json end - def test_flatten - a = 2.months - b = (1.month + 1.month).flatten - - assert_equal a.parts, b.parts - end - - def test_flatten! - a = (1.month + 1.month) - b = a.flatten - a.flatten! - - assert_equal a.parts, b.parts - end - protected def with_env_tz(new_tz = 'US/Eastern') old_tz, ENV['TZ'] = ENV['TZ'], new_tz diff --git a/guides/source/active_support_core_extensions.md b/guides/source/active_support_core_extensions.md index d11c17117e..d3f49b19fa 100644 --- a/guides/source/active_support_core_extensions.md +++ b/guides/source/active_support_core_extensions.md @@ -3699,21 +3699,6 @@ Time.utc(1582, 10, 3) + 5.days # => Mon Oct 18 00:00:00 UTC 1582 ``` -When addinng or substracting durations, the resulting duration will be equivalent to subsequent calls to since or advance, so: - -```ruby -Date.new(2013,1,28) + 1.month + 1.month -# => Thu, 28 Mar 2013 -``` - -If you want to add durations and then use them as just one call to since or advance, you can use the flatten or flatten! method: - -```ruby -Date.new(2013,1,31) + (1.month + 1.month).flatten -# => Sun, 31 Mar 2013 -``` - - Extensions to `File` -------------------- |