Merge pull request #662 from amatsuda/default_empty_array

Simplify the "Translation missing" message when default is an empty Array

lib/i18n/exceptions.rb

@@ -63,8 +63,8 @@ def keys
       end
 
       def message
-        if options[:default].is_a?(Array)
-          other_options = ([key, *options[:default]]).map { |k| normalized_option(k).prepend('- ') }.join("\n")
+        if (default = options[:default]).is_a?(Array) && default.any?
+          other_options = ([key, *default]).map { |k| normalized_option(k).prepend('- ') }.join("\n")
           "Translation missing. Options considered were:\n#{other_options}"
         else
           "Translation missing: #{keys.join('.')}"

test/backend/fallbacks_test.rb

@@ -70,6 +70,10 @@ def setup
     assert_equal translation_missing_message.chomp, I18n.t(:missing_bar, :locale => :'de-DE', :default => [:missing_baz])
   end
 
+  test "returns the simple Translation missing: message when default is an empty Array" do
+    assert_equal "Translation missing: de-DE.missing_bar", I18n.t(:missing_bar, :locale => :'de-DE', :default => [])
+  end
+
   test "returns the :'de-DE' default :baz translation for a missing :'de-DE' when defaults contains Symbol" do
     assert_equal 'Baz in :de-DE', I18n.t(:missing_foo, :locale => :'de-DE', :default => [:baz, "Default Bar"])
   end