Add support for yml files with format locale.something.yml

Extract path transformation helper #replace_locale_in_path

Author: Christoph

lib/i18n_yaml_editor/store.rb

@@ -75,16 +75,7 @@ def create_missing_keys
         missing_locales = self.locales - key.translations.map(&:locale)
         missing_locales.each {|locale|
           translation = key.translations.first
-
-          # this just replaces the locale part of the file name. should
-          # be possible to do in a simpler way. gsub, baby.
-          path = Pathname.new(translation.file)
-          dirs, file = path.split
-          file = file.to_s.split(".")
-          file[-2] = locale
-          file = file.join(".")
-          path = dirs.join(file).to_s
-
+          path = replace_locale_in_path(translation.locale, locale, translation.file)
           new_translation = Translation.new(name: "#{locale}.#{key.name}", file: path)
           add_translation(new_translation)
         }

lib/i18n_yaml_editor/transformation.rb

@@ -37,5 +37,12 @@ def nest_hash hash
       result
     end
     module_function :nest_hash
+
+    def replace_locale_in_path(from_locale, to_locale, path)
+      parts = File.basename(path).split('.')
+      parts[parts.index(from_locale)] = to_locale
+      File.join(File.dirname(path), parts.join('.'))
+    end
+    module_function :replace_locale_in_path
   end
 end

test/unit/test_store.rb

@@ -102,6 +102,30 @@ def test_create_missing_translations_in_top_level_file
     assert_nil translation.text
   end
 
+  def test_create_missing_translations_in_suffix_named_file
+    @store.add_translation Translation.new(name: "da.app_name", text: "Oversætter", file: "/tmp/something.foo.da.yml")
+    @store.add_locale("en")
+
+    @store.create_missing_keys
+
+    assert(translation = @store.translations["en.app_name"])
+    assert_equal "en.app_name", translation.name
+    assert_equal "/tmp/something.foo.en.yml", translation.file
+    assert_nil translation.text
+  end
+
+  def test_create_missing_translations_in_prefix_named_file
+    @store.add_translation Translation.new(name: "da.app_name", text: "Oversætter", file: "/tmp/da.something.foo.yml")
+    @store.add_locale("en")
+
+    @store.create_missing_keys
+
+    assert(translation = @store.translations["en.app_name"])
+    assert_equal "en.app_name", translation.name
+    assert_equal "/tmp/en.something.foo.yml", translation.file
+    assert_nil translation.text
+  end
+
   def test_from_yaml
     input = {
       da: {