Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

lib/active_support/time_with_zone.rb does not require used things #5854

Closed
betelgeuse opened this Issue · 0 comments

2 participants

@betelgeuse
1.9.3p125 :001 > require 'active_support/core_ext/time/zones'
 => true 
1.9.3p125 :003 > Time.zone = 'Europe/Helsinki'
 => "Europe/Helsinki" 
1.9.3p125 :004 > Time.zone.parse '2011-01-01'
NoMethodError: undefined method `utc_time' for Time:Class
    from /Users/betelgeuse/.rvm/gems/ruby-1.9.3-p125/gems/activesupport-3.2.3/lib/active_support/time_with_zone.rb:347:in `transfer_time_values_to_utc_constructor'
    from /Users/betelgeuse/.rvm/gems/ruby-1.9.3-p125/gems/activesupport-3.2.3/lib/active_support/time_with_zone.rb:336:in `get_period_and_ensure_valid_local_time'
    from /Users/betelgeuse/.rvm/gems/ruby-1.9.3-p125/gems/activesupport-3.2.3/lib/active_support/time_with_zone.rb:47:in `initialize'
    from /Users/betelgeuse/.rvm/gems/ruby-1.9.3-p125/gems/activesupport-3.2.3/lib/active_support/values/time_zone.rb:273:in `new'
    from /Users/betelgeuse/.rvm/gems/ruby-1.9.3-p125/gems/activesupport-3.2.3/lib/active_support/values/time_zone.rb:273:in `parse'
    from (irb):4
    from /Users/betelgeuse/.rvm/rubies/ruby-1.9.3-p125/bin/irb:16:in `<main>'
@drogus drogus closed this issue from a commit
@drogus drogus Add missing require in Active Support time zones (fixes #5854)
I also removed the other require as it's already present in
`activesupport/core_ext/time/calculations`
6a6b799
@drogus drogus closed this in 6a6b799
@arunagw arunagw referenced this issue from a commit in arunagw/rails
@drogus drogus Add missing require in Active Support time zones (fixes #5854)
I also removed the other require as it's already present in
`activesupport/core_ext/time/calculations`
a748eca
@romanvbabenko romanvbabenko referenced this issue from a commit in romanvbabenko/rails
@drogus drogus Add missing require in Active Support time zones (fixes #5854)
I also removed the other require as it's already present in
`activesupport/core_ext/time/calculations`
8d3a70a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.