missing_t 0.3.0 → 0.3.1

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    missing_t (0.3.0)
+    missing_t (0.3.1)
 
 GEM
   remote: http://rubygems.org/

data/bin/missing_t

@@ -4,16 +4,32 @@ $LOAD_PATH.unshift(File.expand_path(File.dirname(__FILE__) + "/../lib"))
 
 require 'missing_t'
 
-@missing_t = MissingT.new
+def hashify(strings)
+  strings.map { |s| s.split('.') }.each_with_object({}) do |segmented_string, h|
+    segmented_string.each do |segment|
+      h[segment] ||= {}
+      h = h[segment]
+    end
+  end
+end
 
-@missing_t = MissingT.new
-@missing_t.instance_eval do
-  parse_options(ARGV)
-  find_missing_translations(ARGV[0]).each do |file, queries|
-  puts
-  puts "#{file}:"
-  puts
-  queries.each { |q| puts "    #{red(q)}" }
+def print_hash(h, level)
+  h.each_pair do |k,v|
+    puts %(#{" " * (level*2)}#{k}:)
+    print_hash(v, level+1)
   end
 end
 
+missing_t = MissingT.new
+missing_t.parse_options(ARGV)
+missing_message_strings = missing_t.find_missing_translations(ARGV.first).values.map { |ms| hashify(ms) }
+
+missing = missing_message_strings.each_with_object({}) do |h, all_message_strings|
+  all_message_strings.deep_safe_merge!(h)
+end
+
+missing.each do |language, missing_for_language|
+  puts
+  puts "#{language}:"
+  print_hash(missing_for_language, 1)
+end

data/lib/missing_t.rb

@@ -52,7 +52,7 @@ end
 
 class MissingT
 
-  VERSION = "0.3.0"
+  VERSION = "0.3.1"
 
   include Helpers
   extend Forwardable
@@ -81,7 +81,7 @@ class MissingT
     end
 
     opts.on_tail("--version", "Show version") do
-      puts "0.1.2"
+      puts VERSION
       exit
     end
 
@@ -108,19 +108,6 @@ class MissingT
     end
   end
 
-  def hashify(strings)
-    h = Hash.new
-    strings.map { |s| s.split('.') }.
-      each do |segmented_string|
-        root = h
-        segmented_string.each do |segment|
-          root[segment] ||= {}
-          root = root[segment]
-        end
-      end
-    h
-  end
-
   def translations_in_file(yaml_file)
     open(yaml_file) { |f| YAML.load(f.read) }
   end
@@ -152,14 +139,12 @@ class MissingT
   end
 
   def collect_translation_queries
-    queries = {}
-    files_with_i18n_queries.each do |file|
+    files_with_i18n_queries.each_with_object({}) do |file, queries|
       queries_in_file = extract_i18n_queries(file)
-      unless queries_in_file.empty?
+      if queries_in_file.any?
         queries[file] = queries_in_file
       end
     end
-    queries
     #TODO: remove duplicate queries across files
   end
 
@@ -172,21 +157,18 @@ class MissingT
     true
   end
 
-  def get_missing_translations(queries, lang=nil)
-    missing = {}
-    languages = lang.nil? ? translations.keys : [lang]
-    languages.each do |l|
-      get_missing_translations_for_lang(queries, l).each do |file, qs|
+  def get_missing_translations(queries, languages)
+    languages.each_with_object({}) do |lang, missing|
+      get_missing_translations_for_lang(queries, lang).each do |file, queries|
         missing[file] ||= []
-        missing[file].concat(qs).uniq!
+        missing[file].concat(queries).uniq!
       end
     end
-    missing
   end
 
   def find_missing_translations(lang=nil)
     collect_translations
-    get_missing_translations(collect_translation_queries, lang)
+    get_missing_translations(collect_translation_queries, lang ? [lang] : translations.keys)
   end
 
   private

data/spec/missing_t_spec.rb

@@ -188,7 +188,7 @@ describe "MissingT" do
       @missing_t.has_translation?("fr", "zoo.bee").should == false
       @missing_t.has_translation?("es", "mother").should == false
     end
-    
+
     describe "of dynamic message strings" do
       it "should return true if it has a translation that matches the fix parts" do
         @missing_t.has_translation?("fr", %q(zoo.#{animal})).should == true
@@ -196,7 +196,7 @@ describe "MissingT" do
 
       it "should return false if it does not have a translation that matches all the fix parts" do
         @missing_t.has_translation?("fr", %q(household.#{animal})).should == false
-      end      
+      end
     end
 
     it "should correctly get missing translations for a spec. language" do
@@ -214,4 +214,4 @@ describe "MissingT" do
     end
   end
 
-end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: missing_t
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.3.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-02-07 00:00:00.000000000 Z
+date: 2013-02-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -148,7 +148,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1420340980301416439
+      hash: -311353985734395445
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -157,7 +157,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1420340980301416439
+      hash: -311353985734395445
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.23