Handle false as a key correctly

[dmeremyanin]

Before this change, I18n.t(false) was always returning false.

[dmeremyanin]

Is there something I can do to work towards getting this merged?

[radar]

Sorry about the delay. I've been busy!

Pinging about it was the right thing to do here. I will now look at doing a new gem release.

[dmeremyanin]

Thanks, @radar!

[andrewyoo]

@radar I think this PR or somewhere >= 0.8.3 broke this:

Handle :default => false correctly
3312ed9

[dmeremyanin]

Hey @andrewyoo. Can you please give more details on how to reproduce the issue? It doesn't seem to be broken, all tests are passing for me.

[andrewyoo]

@dimko

Might be a rails(5.0.3) issue, but here's some detail:

v0.8.3:

[1] pry(main)> I18n.t('key', default: false)
=> false
[2] pry(main)> helper.t('key', default: false)
I18n::MissingTranslationData: translation missing: en.false
from /Users/ayoo/.rbenv/versions/2.4.1/lib/ruby/gems/2.4.0/bundler/gems/i18n-2a388d2d835a/lib/i18n.rb:311:in `handle_exception'

v0.8.2:

[1] pry(main)> I18n.t('key', default: false) 
=> false
[2] pry(main)> helper.t('key', default: false)
=> false