i18n 0.8.3 → 0.8.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1e5dfb6b01e690ee96c8fff66953f1ba5d9f5e8b
-  data.tar.gz: 92cdb82e1c6e4dcccc73c3321f0b32cfc07cdca9
+  metadata.gz: 7aea28fdffb1b83ffc558cca097cb9ddd692a5e5
+  data.tar.gz: 1143c46c544e1ca82f014f6995df8170f13d896d
 SHA512:
-  metadata.gz: ce6de9b2f93d878ad9b605400123a0d07882f37e2a46bf63ac010322b7d071ddfee00d4ca7f27648e5f00784d3dbfd334db3d4f6ccf01a4c5a42e7e469c76478
-  data.tar.gz: 514e89dd67ee76be070862f2381bd66411e5f82c75d72adcd458d345d5b1c740d214d5c32e99dc7f33d30cfc0bc87179d903c978dae683cdb96f80d6752132f4
+  metadata.gz: 835ea057465196d791bba7a9d1affdf4d80e6ef4321fc08ff704571106d112eb3387cc757c8be8e115d9ffcfff808f38b9d05740ce68b63c873a7bbe4eb21646
+  data.tar.gz: 4ccba735b7f85e8b755d8f9308c36d2c31d6aabb7051a162c917997e47cfd016b3ffddda27de5a4262165c6cb602be45f0e32c30686b8ee5f067f387faf36056

data/lib/i18n.rb

@@ -149,7 +149,7 @@ module I18n
       handling = options.delete(:throw) && :throw || options.delete(:raise) && :raise # TODO deprecate :raise
 
       enforce_available_locales!(locale)
-      raise I18n::ArgumentError if key.nil? || (key.is_a?(String) && key.empty?)
+      raise I18n::ArgumentError if key.is_a?(String) && key.empty?
 
       result = catch(:exception) do
         if key.is_a?(Array)
@@ -171,7 +171,7 @@ module I18n
 
     # Returns true if a translation exists for a given key, otherwise returns false.
     def exists?(key, locale = config.locale)
-      raise I18n::ArgumentError if key.nil? || (key.is_a?(String) && key.empty?)
+      raise I18n::ArgumentError if key.is_a?(String) && key.empty?
       config.backend.exists?(locale, key)
     end
 

data/lib/i18n/backend/fallbacks.rb

@@ -55,7 +55,7 @@ module I18n
           @fallback_locked = false
         end
 
-        return super(locale, key, options.merge(:default => default)) if default
+        return super(locale, nil, options.merge(:default => default)) if default
         throw(:exception, I18n::MissingTranslation.new(locale, key, options))
       end
 

data/lib/i18n/tests/defaults.rb

@@ -9,11 +9,11 @@ module I18n
       end
 
       test "defaults: given nil as a key it returns the given default" do
-        assert_equal 'default', I18n.t(:does_not_exist, :default => 'default')
+        assert_equal 'default', I18n.t(nil, :default => 'default')
       end
 
       test "defaults: given a symbol as a default it translates the symbol" do
-        assert_equal 'bar', I18n.t(:does_not_exist, :default => :'foo.bar')
+        assert_equal 'bar', I18n.t(nil, :default => :'foo.bar')
       end
 
       test "defaults: given a symbol as a default and a scope it stays inside the scope when looking up the symbol" do
@@ -41,7 +41,7 @@ module I18n
       test "defaults: using a custom scope separator" do
         # data must have been stored using the custom separator when using the ActiveRecord backend
         I18n.backend.store_translations(:en, { :foo => { :bar => 'bar' } }, { :separator => '|' })
-        assert_equal 'bar', I18n.t(:does_not_exist, :default => :'foo|bar', :separator => '|')
+        assert_equal 'bar', I18n.t(nil, :default => :'foo|bar', :separator => '|')
       end
     end
   end

data/lib/i18n/tests/pluralization.rb

@@ -4,31 +4,31 @@ module I18n
   module Tests
     module Pluralization
       test "pluralization: given 0 it returns the :zero translation if it is defined" do
-        assert_equal 'zero', I18n.t(:does_not_exist, :default => { :zero => 'zero' }, :count => 0)
+        assert_equal 'zero', I18n.t(:default => { :zero => 'zero' }, :count => 0)
       end
 
       test "pluralization: given 0 it returns the :other translation if :zero is not defined" do
-        assert_equal 'bars', I18n.t(:does_not_exist, :default => { :other => 'bars' }, :count => 0)
+        assert_equal 'bars', I18n.t(:default => { :other => 'bars' }, :count => 0)
       end
 
       test "pluralization: given 1 it returns the singular translation" do
-        assert_equal 'bar', I18n.t(:does_not_exist, :default => { :one => 'bar' }, :count => 1)
+        assert_equal 'bar', I18n.t(:default => { :one => 'bar' }, :count => 1)
       end
 
       test "pluralization: given 2 it returns the :other translation" do
-        assert_equal 'bars', I18n.t(:does_not_exist, :default => { :other => 'bars' }, :count => 2)
+        assert_equal 'bars', I18n.t(:default => { :other => 'bars' }, :count => 2)
       end
 
       test "pluralization: given 3 it returns the :other translation" do
-        assert_equal 'bars', I18n.t(:does_not_exist, :default => { :other => 'bars' }, :count => 3)
+        assert_equal 'bars', I18n.t(:default => { :other => 'bars' }, :count => 3)
       end
 
       test "pluralization: given nil it returns the whole entry" do
-        assert_equal({ :one => 'bar' }, I18n.t(:does_not_exist, :default => { :one => 'bar' }, :count => nil))
+        assert_equal({ :one => 'bar' }, I18n.t(:default => { :one => 'bar' }, :count => nil))
       end
 
       test "pluralization: given incomplete pluralization data it raises I18n::InvalidPluralizationData" do
-        assert_raise(I18n::InvalidPluralizationData) { I18n.t(:does_not_exist, :default => { :one => 'bar' }, :count => 2) }
+        assert_raise(I18n::InvalidPluralizationData) { I18n.t(:default => { :one => 'bar' }, :count => 2) }
       end
     end
   end

data/lib/i18n/tests/procs.rb

@@ -3,47 +3,47 @@
 module I18n
   module Tests
     module Procs
-      test "lookup: given a translation is a Proc it calls the Proc with the key and interpolation values" do
+      test "lookup: given a translation is a proc it calls the proc with the key and interpolation values" do
         I18n.backend.store_translations(:en, :a_lambda => lambda { |*args| I18n::Tests::Procs.filter_args(*args) })
         assert_equal '[:a_lambda, {:foo=>"foo"}]', I18n.t(:a_lambda, :foo => 'foo')
       end
 
       test "defaults: given a default is a Proc it calls it with the key and interpolation values" do
         proc = lambda { |*args| I18n::Tests::Procs.filter_args(*args) }
-        assert_equal '[:does_not_exist, {:foo=>"foo"}]', I18n.t(:does_not_exist, :default => proc, :foo => 'foo')
+        assert_equal '[nil, {:foo=>"foo"}]', I18n.t(nil, :default => proc, :foo => 'foo')
       end
 
       test "defaults: given a default is a key that resolves to a Proc it calls it with the key and interpolation values" do
         the_lambda = lambda { |*args| I18n::Tests::Procs.filter_args(*args) }
         I18n.backend.store_translations(:en, :a_lambda => the_lambda)
-        assert_equal '[:a_lambda, {:foo=>"foo"}]', I18n.t(:does_not_exist, :default => :a_lambda, :foo => 'foo')
-        assert_equal '[:a_lambda, {:foo=>"foo"}]', I18n.t(:does_not_exist, :default => [nil, :a_lambda], :foo => 'foo')
+        assert_equal '[:a_lambda, {:foo=>"foo"}]', I18n.t(nil, :default => :a_lambda, :foo => 'foo')
+        assert_equal '[:a_lambda, {:foo=>"foo"}]', I18n.t(nil, :default => [nil, :a_lambda], :foo => 'foo')
       end
 
       test "interpolation: given an interpolation value is a lambda it calls it with key and values before interpolating it" do
         proc = lambda { |*args| I18n::Tests::Procs.filter_args(*args) }
-        assert_match %r(\[\{:foo=>#<Proc.*>\}\]), I18n.t(:does_not_exist, :default => '%{foo}', :foo => proc)
+        assert_match %r(\[\{:foo=>#<Proc.*>\}\]), I18n.t(nil, :default => '%{foo}', :foo => proc)
       end
 
       test "interpolation: given a key resolves to a Proc that returns a string then interpolation still works" do
         proc = lambda { |*args| "%{foo}: " + I18n::Tests::Procs.filter_args(*args) }
-        assert_equal 'foo: [:does_not_exist, {:foo=>"foo"}]', I18n.t(:does_not_exist, :default => proc, :foo => 'foo')
+        assert_equal 'foo: [nil, {:foo=>"foo"}]', I18n.t(nil, :default => proc, :foo => 'foo')
       end
 
       test "pluralization: given a key resolves to a Proc that returns valid data then pluralization still works" do
         proc = lambda { |*args| { :zero => 'zero', :one => 'one', :other => 'other' } }
-        assert_equal 'zero',  I18n.t(:does_not_exist, :default => proc, :count => 0)
-        assert_equal 'one',   I18n.t(:does_not_exist, :default => proc, :count => 1)
-        assert_equal 'other', I18n.t(:does_not_exist, :default => proc, :count => 2)
+        assert_equal 'zero',  I18n.t(:default => proc, :count => 0)
+        assert_equal 'one',   I18n.t(:default => proc, :count => 1)
+        assert_equal 'other', I18n.t(:default => proc, :count => 2)
       end
 
-      test "lookup: given the option :resolve => false was passed it does not resolve Proc translations" do
+      test "lookup: given the option :resolve => false was passed it does not resolve proc translations" do
         I18n.backend.store_translations(:en, :a_lambda => lambda { |*args| I18n::Tests::Procs.filter_args(*args) })
         assert_equal Proc, I18n.t(:a_lambda, :resolve => false).class
       end
 
-      test "lookup: given the option :resolve => false was passed it does not resolve Proc default" do
-        assert_equal Proc, I18n.t(:does_not_exist, :default => lambda { |*args| I18n::Tests::Procs.filter_args(*args) }, :resolve => false).class
+      test "lookup: given the option :resolve => false was passed it does not resolve proc default" do
+        assert_equal Proc, I18n.t(nil, :default => lambda { |*args| I18n::Tests::Procs.filter_args(*args) }, :resolve => false).class
       end
 
 

data/lib/i18n/version.rb

@@ -1,3 +1,3 @@
 module I18n
-  VERSION = "0.8.3"
+  VERSION = "0.8.4"
 end

data/test/backend/cascade_test.rb

@@ -39,7 +39,7 @@ class I18nBackendCascadeTest < I18n::TestCase
   end
 
   test "cascades defaults, too" do
-    assert_equal 'foo', lookup(:does_not_exist, :default => [:'missing.missing', :'missing.foo'])
+    assert_equal 'foo', lookup(nil, :default => [:'missing.missing', :'missing.foo'])
   end
 
   test "works with :offset => 2 and a single key" do

data/test/backend/chain_test.rb

@@ -39,10 +39,10 @@ class I18nBackendChainTest < I18n::TestCase
   end
 
   test "default" do
-    assert_equal 'Fuh',  I18n.t(:does_not_exist, :default => 'Fuh')
-    assert_equal 'Zero', I18n.t(:does_not_exist, :default => { :zero => 'Zero' }, :count => 0)
-    assert_equal({ :zero => 'Zero' }, I18n.t(:does_not_exist, :default => { :zero => 'Zero' }))
-    assert_equal 'Foo', I18n.t(:does_not_exist, :default => :foo)
+    assert_equal 'Fuh',  I18n.t(:default => 'Fuh')
+    assert_equal 'Zero', I18n.t(:default => { :zero => 'Zero' }, :count => 0)
+    assert_equal({ :zero => 'Zero' }, I18n.t(:default => { :zero => 'Zero' }))
+    assert_equal 'Foo', I18n.t(:default => :foo)
   end
 
   test 'default is returned if translation is missing' do

data/test/backend/pluralization_test.rb

@@ -19,24 +19,24 @@ class I18nBackendPluralizationTest < I18n::TestCase
   end
 
   test "pluralization picks :one for 1" do
-    assert_equal 'one', I18n.t(:does_not_exist, :count => 1, :default => @entry, :locale => :xx)
+    assert_equal 'one', I18n.t(:count => 1, :default => @entry, :locale => :xx)
   end
 
   test "pluralization picks :few for 2" do
-    assert_equal 'few', I18n.t(:does_not_exist, :count => 2, :default => @entry, :locale => :xx)
+    assert_equal 'few', I18n.t(:count => 2, :default => @entry, :locale => :xx)
   end
 
   test "pluralization picks :many for 11" do
-    assert_equal 'many', I18n.t(:does_not_exist, :count => 11, :default => @entry, :locale => :xx)
+    assert_equal 'many', I18n.t(:count => 11, :default => @entry, :locale => :xx)
   end
 
   test "pluralization picks zero for 0 if the key is contained in the data" do
-    assert_equal 'zero', I18n.t(:does_not_exist, :count => 0, :default => @entry, :locale => :xx)
+    assert_equal 'zero', I18n.t(:count => 0, :default => @entry, :locale => :xx)
   end
 
   test "pluralization picks few for 0 if the key is not contained in the data" do
     @entry.delete(:zero)
-    assert_equal 'few', I18n.t(:does_not_exist, :count => 0, :default => @entry, :locale => :xx)
+    assert_equal 'few', I18n.t(:count => 0, :default => @entry, :locale => :xx)
   end
 
   test "Fallbacks can pick up rules from fallback locales, too" do

data/test/backend/simple_test.rb

@@ -8,8 +8,8 @@ class I18nBackendSimpleTest < I18n::TestCase
   end
 
   # useful because this way we can use the backend with no key for interpolation/pluralization
-  test "simple backend translate: given an invalid key it still interpolates the default value" do
-    assert_equal "Hi David", I18n.t(:does_not_exist, :default => "Hi %{name}", :name => "David")
+  test "simple backend translate: given nil as a key it still interpolations the default value" do
+    assert_equal "Hi David", I18n.t(nil, :default => "Hi %{name}", :name => "David")
   end
 
   # loading translations

data/test/i18n_test.rb

@@ -216,10 +216,6 @@ class I18nTest < I18n::TestCase
     assert_raise(I18n::ArgumentError) { I18n.t("") }
   end
 
-  test "translate given nil as a key raises an I18n::ArgumentError" do
-    assert_raise(I18n::ArgumentError) { I18n.t(nil) }
-  end
-
   test "translate given an unavailable locale rases an I18n::InvalidLocale" do
     begin
       I18n.config.enforce_available_locales = true
@@ -262,14 +258,6 @@ class I18nTest < I18n::TestCase
     assert_equal false, I18n.exists?(:bogus)
   end
 
-  test "exists? given an empty string will raise an error" do
-    assert_raise(I18n::ArgumentError) { I18n.exists?("") }
-  end
-
-  test "exists? given nil will raise an error" do
-    assert_raise(I18n::ArgumentError) { I18n.exists?(nil) }
-  end
-
   test "exists? given an existing dot-separated key will return true" do
     assert_equal true, I18n.exists?('currency.format.delimiter')
   end
@@ -417,7 +405,7 @@ class I18nTest < I18n::TestCase
       I18n.config.enforce_available_locales = false
     end
   end
-
+  
   test 'I18n.reload! reloads the set of locales that are enforced' do
     begin
       # Clear the backend that affects the available locales and somehow can remain
@@ -425,9 +413,9 @@ class I18nTest < I18n::TestCase
       # For instance, it contains enough translations to cause a false positive with
       # this test when ran with --seed=50992
       I18n.backend = I18n::Backend::Simple.new
-
+      
       assert !I18n.available_locales.include?(:de), "Available locales should not include :de at this point"
-
+      
       I18n.enforce_available_locales = true
 
       assert_raise(I18n::InvalidLocale) { I18n.default_locale = :de }
@@ -443,11 +431,11 @@ class I18nTest < I18n::TestCase
       store_translations(:en, :foo => 'Foo in :en')
       store_translations(:de, :foo => 'Foo in :de')
       store_translations(:pl, :foo => 'Foo in :pl')
-
+      
       assert I18n.available_locales.include?(:de), ":de should now be allowed"
       assert I18n.available_locales.include?(:en), ":en should now be allowed"
       assert I18n.available_locales.include?(:pl), ":pl should now be allowed"
-
+      
       assert_nothing_raised { I18n.default_locale = I18n.locale = :en }
       assert_nothing_raised { I18n.default_locale = I18n.locale = :de }
       assert_nothing_raised { I18n.default_locale = I18n.locale = :pl }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: i18n
 version: !ruby/object:Gem::Version
-  version: 0.8.3
+  version: 0.8.4
 platform: ruby
 authors:
 - Sven Fuchs
@@ -12,7 +12,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-05-30 00:00:00.000000000 Z
+date: 2017-05-31 00:00:00.000000000 Z
 dependencies: []
 description: New wave Internationalization support for Ruby.
 email: rails-i18n@googlegroups.com