Merge pull request #6 from Hivebrite/master

Add possibility to remove the 'var' keyword in order to use a global variable for locales

lib/po_to_json.rb

@@ -50,7 +50,8 @@ def generate_for_jed(language, overwrite = {})
     generated = build_json_for(build_jed_for(@parsed))
 
     [
-      "var #{@options[:variable]} = #{@options[:variable]} || {};",
+      @options[:variable_locale_scope] ? 'var' : '',
+      "#{@options[:variable]} = #{@options[:variable]} || {};",
       "#{@options[:variable]}['#{@options[:language]}'] = #{generated};"
     ].join(" ")
   end
@@ -191,7 +192,8 @@ def parse_options(options)
     defaults = {
       pretty: false,
       domain: "app",
-      variable: "locales"
+      variable: "locales",
+      variable_locale_scope: true
     }
 
     defaults.merge(options)

spec/po_to_json_spec.rb

@@ -188,6 +188,17 @@
         ).to be > 0
       end
     end
+    context "without locale scope variable" do
+      subject do
+        po_to_json.generate_for_jed("de", variable_locale_scope: false)
+      end
+
+      it "should output the var definition" do
+        expect(
+          subject.include?("locales = locales || {}; locales['de'] = ")
+        ).to be_truthy
+      end
+    end
   end
 
   # describe "generate simple hashes" do