diff options
author | Andrew White <andyw@pixeltrix.co.uk> | 2013-01-21 12:19:10 +0000 |
---|---|---|
committer | Andrew White <andyw@pixeltrix.co.uk> | 2013-01-21 12:33:49 +0000 |
commit | b79adc4323ff289aed3f5787fdfbb9542aa4f89f (patch) | |
tree | 15dcc2a3165ef1170979586795bbc03976428eef /activesupport/CHANGELOG.md | |
parent | 68a6fb695346f1b6f84dcba21bb48114cde297a5 (diff) | |
download | rails-b79adc4323ff289aed3f5787fdfbb9542aa4f89f.tar.gz rails-b79adc4323ff289aed3f5787fdfbb9542aa4f89f.tar.bz2 rails-b79adc4323ff289aed3f5787fdfbb9542aa4f89f.zip |
Standardise the return value of `to_time`
This commit standardises the return value of `to_time` to an instance
of `Time` in the local system timezone, matching the Ruby core and
standard library behavior.
The default form for `String#to_time` has been changed from :utc to
:local but research seems to suggest the latter is the more common form.
Also fix an edge condition with `String#to_time` where the string has
a timezone offset in it and the mode is :local. e.g:
# Before:
>> "2000-01-01 00:00:00 -0500".to_time(:local)
=> 2000-01-01 05:00:00 -0500
# After:
>> "2000-01-01 00:00:00 -0500".to_time(:local)
=> 2000-01-01 00:00:00 -0500
Closes #2453
Diffstat (limited to 'activesupport/CHANGELOG.md')
-rw-r--r-- | activesupport/CHANGELOG.md | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/activesupport/CHANGELOG.md b/activesupport/CHANGELOG.md index 2c0d1de70f..72f28aefc7 100644 --- a/activesupport/CHANGELOG.md +++ b/activesupport/CHANGELOG.md @@ -1,5 +1,10 @@ ## Rails 4.0.0 (unreleased) ## +* Standardise on `to_time` returning an instance of `Time` in the local system timezone + across `String`, `Time`, `Date`, `DateTime` and `ActiveSupport::TimeWithZone`. + + *Andrew White* + * Extract `ActiveSupport::Testing::Performance` into https://github.com/rails/rails-perftest You can add the gem to your Gemfile to keep using performance tests. @@ -7,7 +12,6 @@ *Yves Senn* - * Hash.from_xml raises when it encounters type="symbol" or type="yaml". Use Hash.from_trusted_xml to parse this XML. |