Accept the scope as an array.

piotrekbator · May 14, 2014 · 0b35ddd · 0b35ddd
1 parent 6b7d9fe
commit 0b35ddd
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/app/assets/javascripts/i18n.js b/app/assets/javascripts/i18n.js
@@ -235,6 +235,11 @@
       , translations
     ;
 
+    // Deal with the scope as an array.
+    if (scope.constructor === Array) {
+      scope = scope.join(this.defaultSeparator);
+    }
+
     // Deal with the scope option provided through the second argument.
     //
     //    I18n.t('hello', {scope: 'greetings'});

diff --git a/spec/js/translate.spec.js b/spec/js/translate.spec.js
@@ -129,4 +129,12 @@ describe("Translate", function(){
     var options = {scope: "greetings"};
     expect(I18n.t("stranger", options)).toEqual("Hello stranger!");
   });
+
+  it("accepts the scope as an array", function(){
+    expect(I18n.t(["greetings", "stranger"])).toEqual("Hello stranger!");
+  });
+
+  it("accepts the scope as an array using a base scope", function(){
+    expect(I18n.t(["stranger"], {scope: "greetings"})).toEqual("Hello stranger!");
+  });
 });