ArgumentError: "+HH:MM" or "-HH:MM" expected for utc_offset

[dalizard]

Hello!

I am not 100% if this needs to be handled by JRuby or ActiveSupport, but here is the problem.

[1] pry(main)> `ruby -v`
=> "ruby 2.1.5p273 (2014-11-13 revision 48405) [x86_64-darwin14.0]\n"
[2] pry(main)> 1.hour.from_now.localtime

From: /Users/dimitar/Projects/List-It-Local/backend/.bundle/ruby/2.1.0/gems/activesupport-4.2.0.rc2/lib/active_support/time_with_zone.rb @ line 79 Time#localtime:

    78: def localtime(utc_offset = nil)
 => 79:   binding.pry
    80:   utc.respond_to?(:getlocal) ? utc.getlocal(utc_offset) : utc.to_time.getlocal(utc_offset)
    81: end

[1] pry(#<ActiveSupport::TimeWithZone>)> utc_offset
=> nil
[2] pry(#<ActiveSupport::TimeWithZone>)> utc.respond_to?(:getlocal)
=> true
[3] pry(#<ActiveSupport::TimeWithZone>)> utc.getlocal(utc_offset)
=> 2014-12-12 18:53:45 +0200

[5] pry(#<ActiveSupport::TimeWithZone>)> `ruby -v`
=> "jruby 1.7.16.2 (2.0.0p195) 2014-12-08 365d573 on Java HotSpot(TM) 64-Bit Server VM 1.7.0_71-b14 [darwin-x86_64]\n"
[6] pry(#<ActiveSupport::TimeWithZone>)> 1.hour.from_now.localtime

From: /Users/dimitar/Projects/Secret/app/user_service/vendor/jruby/1.9/gems/activesupport-4.2.0.rc2/lib/active_support/time_with_zone.rb @ line 79 Time#localtime:

    78: def localtime(utc_offset = nil)
 => 79:   binding.pry
    80:   utc.respond_to?(:getlocal) ? utc.getlocal(utc_offset) : utc.to_time.getlocal(utc_offset)
    81: end

[1] pry(#<ActiveSupport::TimeWithZone>)> utc_offset
=> nil
[2] pry(#<ActiveSupport::TimeWithZone>)> utc.respond_to?(:getlocal)
=> true
[3] pry(#<ActiveSupport::TimeWithZone>)> utc.getlocal(utc_offset)
ArgumentError: "+HH:MM" or "-HH:MM" expected for utc_offset
from org/jruby/RubyTime.java:442:in `getlocal'

The problem basically is coming from the fact that MRI handles nil cases of the passed utc_offset parameter, while JRuby does not. What do you think should be the proper behaviour here?

[enebo]

Confirmed on jruby-1_7 and master:

Time.now.getlocal(nil)

1.9+ compatibility

[headius]

Test added in http://mla.n-z.jp/~w3ml/w3ml.cgi/ruby-changes/msg/36771.

[dalizard]

Thank you!

[mjansing]

I got a similar issue with JRuby 1.7.18:

[1] pry(main)> `ruby -v`
=> "jruby 1.7.18 (1.9.3p551) 2014-12-22 625381c on Java HotSpot(TM) 64-Bit Server VM 1.7.0_51-b13 [darwin-x86_64]\n"
[2] pry(main)> t = Time.zone.now
=> Wed, 07 Jan 2015 14:49:56 UTC +00:00
[3] pry(main)> I18n.l(t, :format => :short)
ArgumentError: "+HH:MM" or "-HH:MM" expected for utc_offset
from org/jruby/RubyTime.java:443:in `getlocal'

On MRI everything works fine:

[1] pry(main)> `ruby -v`
=> "ruby 2.2.0p0 (2014-12-25 revision 49005) [x86_64-darwin14]\n"
[2] pry(main)> t = Time.zone.now
=> Wed, 07 Jan 2015 14:56:05 UTC +00:00
[3] pry(main)> I18n.l(t, :format => :short)
=> "07 Jan 14:56"
[4] pry(main)>

[enebo]

@mjansing This is showing the same error but it is a different issue (nil as argument was this issue and this must be something other than nil or a valid string). Can you open this as a fresh issue? You get bonus points if you can trace through I18n to the actual argument to the getlocal call...

[dalizard]

@enebo I debugged the problem a bit more, and opened a new issue since it's a different problem.

[enebo]

Curiousity got the better of me and I looked at MRI and then noticed we partially applied a fix to master. I commented on the issue you opened. Thanks @dalizard!

[mjansing]

@dalizard Thank you for digging into this issue.