Added handlebars template scraping and localeapp reporting

Specified handlebars templates will be parsed for {{t some.key}}. Any
translations keys that are not found in the configured i18n YAML will
be reported to the configured localeapp backend via the localeapp api.

Gemfile

@@ -20,6 +20,7 @@ end
 group :development, :test do
   gem 'rake', '~> 0.9.2'
   gem 'localeapp'
+  gem 'handlebars'
 end
 
 group :development do

Gemfile.lock

@@ -25,6 +25,7 @@ GEM
       execjs
     coffee-script-source (1.3.3)
     columnize (0.3.6)
+    commonjs (0.2.6)
     compass (0.12.2)
       chunky_png (~> 1.2)
       fssm (>= 0.2.7)
@@ -47,8 +48,12 @@ GEM
     guard (1.4.0)
       listen (>= 0.4.2)
       thor (>= 0.14.6)
+    handlebars (0.3.1)
+      commonjs (~> 0.2.3)
+      therubyracer (~> 0.10.0)
     i18n (0.6.1)
     json (1.7.5)
+    libv8 (3.3.10.4)
     listen (0.5.3)
     localeapp (0.6.7)
       gli
@@ -70,7 +75,7 @@ GEM
     rack-test (0.6.2)
       rack (>= 1.0)
     rake (0.9.5)
-    rake-pipeline-i18n-filters (0.0.3)
+    rake-pipeline-i18n-filters (0.0.5)
     rb-fsevent (0.9.2)
     rerun (0.7.1)
       listen
@@ -96,6 +101,8 @@ GEM
       rack-test
       sinatra (~> 1.3.0)
       tilt (~> 1.3)
+    therubyracer (0.10.2)
+      libv8 (~> 3.3.10)
     thor (0.16.0)
     tilt (1.3.3)
     uglifier (1.3.0)
@@ -112,6 +119,7 @@ DEPENDENCIES
   debugger
   foreman
   guard
+  handlebars
   localeapp
   puma
   rack-cache

assets/scripts/app/templates/layouts/top.hbs

@@ -4,7 +4,7 @@
 
 <ul id="navigation">
   <li class="home">
-    <a {{action showRoot href=true}}>Home</a>
+  <a {{action showRoot href=true}}>Home</a>
   </li>
   <li class="stats">
     <a {{action showStats href=true}}>Stats</a>
@@ -17,7 +17,8 @@
   </li>
   <li {{bindAttr class="view.classProfile"}}>
     <p class="handle">
-      <a class="signed-out" href="#" {{action signIn target="Travis.app"}}>{{t layouts.top.github_login}}</a>
+      <a class="signed-out" href="#" {{action signIn
+        target="Travis.app"}}>{{t "layouts.top.github_login"}}</a>
       <a class="signed-in" {{action showProfile href=true}}><img {{bindAttr src="view.gravatarUrl"}}>{{view.userName}}</a>
       <span class="signing-in">Signing in</span>
     </p>

assets/scripts/app/templates/repos/show/tabs.hbs

@@ -3,7 +3,7 @@
     <h5>
       {{#if view.repo.slug}}
       <a {{action showRepo view.repo href=true}}>
-        {{t repositories.tabs.current}}
+        {{t "repositories.tabs.current"}}
       </a>
       {{/if}}
     </h5>

lib/localeapp-reporter.rb

@@ -0,0 +1,73 @@
+require 'handlebars'
+require 'localeapp'
+require 'localeapp/i18n_shim'
+require 'localeapp/exception_handler'
+
+module Localeapp
+  class Reporter
+    class << self
+
+      def hbs_load_path
+        @@load_path ||= []
+      end
+      def hbs_load_path=(dir)
+        @@load_path = dir
+      end
+    end
+    
+    def initialize(locale_file, *hbs_templates)
+      configure_localeapp locale_file
+      backend.load_translations locale_file
+      hbs_templates = Localeapp::Reporter.hbs_load_path if hbs_templates.empty?
+      hbs_templates.flatten.each { |file| extract_keys(file) }
+    end
+
+   
+    def send_missing_translations
+      register_missing_translations
+      Localeapp::Sender.new.post_missing_translations
+    end
+    
+    private
+    
+    def configure_localeapp(locale_file)
+      Localeapp.configure do |config|
+        config.translation_data_directory = Dir.new(File.expand_path('../',locale_file))
+      end
+    end
+
+    def matcher
+      @matcher ||= Regexp.new("{{t (.*?)}}")
+    end
+
+    def backend
+      @backend ||= I18n::Backend::Simple.new
+    end
+
+    def hbs_locale_keys
+      @hbs_locale_keys ||= []
+    end 
+
+    def extract_keys(template_file)
+      template = IO.read(template_file)
+      if matches = template.scan(matcher)
+        @hbs_locale_keys = hbs_locale_keys | matches.flatten
+      end
+    end
+ 
+    def register_missing_translations 
+      hbs_locale_keys.map do |locale_key|
+        begin
+          backend.translate(:en, locale_key)
+          nil
+        rescue Exception => e
+          Localeapp.missing_translations.add(:en, locale_key, nil, options || {})
+        end
+      end
+    end
+
+
+  end
+end
+
+

spec/localeapp_reporter_spec.rb

@@ -0,0 +1,34 @@
+require 'spec_helper'
+require 'localeapp-reporter'
+require 'localeapp'
+describe Localeapp::Reporter do
+  let(:hbs_load_path) { File.expand_path '../support/**.hbs', __FILE__ }
+  let(:locale_yaml) { File.expand_path '../support/en.yml', __FILE__ }
+  let(:reporter) { 
+    Localeapp::Reporter.hbs_load_path = Dir[hbs_load_path]
+    Localeapp::Reporter.new(locale_yaml) 
+  }
+ 
+  it "configures localeapp" do
+    reporter
+    Localeapp.configuration.translation_data_directory.path.should ==  Dir.new(File.expand_path('../', locale_yaml)).path
+  end
+
+  it "supports hbs_load_path at class level" do
+    assert_equal Dir[hbs_load_path], reporter.class.hbs_load_path
+  end
+  
+  it "adds missing translations to the Localeapp.missing_translations" do
+    reporter.send(:register_missing_translations)
+    Localeapp.missing_translations.instance_variable_get('@translations')[:en].keys.should include('missing.key')
+  end
+ 
+  it "sends missing translations to localeapp" do
+    # have to stub RestClient here as FakeWeb doesn't support looking at the post body yet
+    RestClient::Request.should_receive(:execute).with(hash_including(
+     :payload => { :translations => Localeapp.missing_translations.to_send }.to_json)).and_return(double('response', :code => 200))
+
+    reporter.send_missing_translations
+  end
+
+end

spec/support/en.yml

@@ -0,0 +1,3 @@
+en:
+  existing:
+    key: An existing key in locale YAML

spec/support/i18n-1.hbs

@@ -0,0 +1 @@
+{{t missing.key}}

spec/support/i18n.hbs

@@ -0,0 +1 @@
+{{t existing.key}}{{t missing.key}}