zendesk_apps_tools 1.21.1 → 1.21.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 31889800a9bc18cfd1543f5b05279bcef4c6d684
-  data.tar.gz: 32336ce53277c686109a718d016837b3af016f4d
+  metadata.gz: 3f8e6b79aced2f157e01d6a0d133fbee07f1f82a
+  data.tar.gz: a003cfef2eb60656a75289b8086925ac9e6dbe98
 SHA512:
-  metadata.gz: 2e690d1a3f280ebec4ab9b2405640596ce13a70e29569d2f38fcc4b212fa5d9b61b1dab8dd60724338a6f53448ee5425ef9a40bc1c394c5e385b31546748fc17
-  data.tar.gz: 1e3334a995f7e081b288a6ac28ae9410dd5c51128b66a4d6f32cceadbe28b61501a404847448592828b58b74314961b8f23317922e17936318c921426e531789
+  metadata.gz: 0bb36b96d341ed0a0e58cb0654f8842607d213a8d418bc53a3c10e24407b29939488a5772a888f4f57062869e189cb1614b79d2539aae2a07b6717691977849b
+  data.tar.gz: 4d848d14c6c2f16e2c139d76eaa7a0751fc8269f80f998a617e3ecdcf88a2f1044307b3bc0ceb8460a6045c8ef3c4baed89b021543dfe7bd5a6b1ad1fe18eb89

data/features/step_definitions/app_steps.rb

@@ -1,5 +1,6 @@
 require 'fileutils'
 require 'zip/zip'
+require 'English'
 
 When /^I move to the app directory$/ do
   @previous_dir = Dir.pwd
@@ -17,7 +18,7 @@ Given /^an app directory "(.*?)" exists$/ do |app_dir|
 end
 
 Given /^an app is created in directory "(.*?)"$/ do |app_dir|
-  steps %Q{
+  steps %(
     Given an app directory "#{app_dir}" exists
     And I run "zat new" command with the following details:
       | author name  | John Citizen      |
@@ -25,33 +26,33 @@ Given /^an app is created in directory "(.*?)"$/ do |app_dir|
       | author url   | http://myapp.com  |
       | app name     | John Test App     |
       | app dir      | #{app_dir}        |
-  }
+    )
 end
 
 When /^I run "(.*?)" command with the following details:$/ do |cmd, table|
-  IO.popen(cmd, "w+") do |pipe|
+  IO.popen(cmd, 'w+') do |pipe|
     # [ ['parameter name', 'value'] ]
     table.raw.each do |row|
       pipe.puts row.last
     end
     pipe.close_write
     @output = pipe.readlines
-    @output.each {|line| puts line}
+    @output.each { |line| puts line }
   end
 end
 
-When  /^I create a symlink from "(.*?)" to "(.*?)"$/ do |src, dest|
+When /^I create a symlink from "(.*?)" to "(.*?)"$/ do |src, dest|
   @link_destname = File.basename(dest)
   # create a symlink
   FileUtils.ln_s(src, dest)
 end
 
-When /^I run the command "(.*?)" to (validate|package|clean) the app$/ do |cmd, action|
-  IO.popen(cmd, "w+") do |pipe|
+When /^I run the command "(.*?)" to (validate|package|clean) the app$/ do |cmd, _action|
+  IO.popen(cmd, 'w+') do |pipe|
     pipe.puts "\n"
     pipe.close_write
     @output = pipe.readlines
-    @output.each {|line| puts line}
+    @output.each { |line| puts line }
   end
 end
 
@@ -60,7 +61,7 @@ Then /^the app file "(.*?)" is created with:$/ do |file, content|
 end
 
 Then /^the app file "(.*?)" is created$/ do |filename|
-  File.exists?(filename).should be_truthy
+  File.exist?(filename).should be_truthy
 end
 
 Then /^the fixture "(.*?)" is used for "(.*?)"$/ do |fixture, app_file|
@@ -70,7 +71,6 @@ Then /^the fixture "(.*?)" is used for "(.*?)"$/ do |fixture, app_file|
   FileUtils.cp(fixture_file, app_file_path)
 end
 
-
 Then /^the zip file should exist in directory "(.*?)"$/ do |path|
   Dir[path + '/app-*.zip'].size.should == 1
 end
@@ -85,7 +85,7 @@ end
 
 Then /^it should pass the validation$/ do
   @output.last.should =~ /OK/
-  $?.should == 0
+  $CHILD_STATUS.should == 0
 end
 
 Then /^the command output should contain "(.*?)"$/ do |output|
@@ -97,7 +97,7 @@ Then /^"(.*?)" should be a symlink$/ do |path|
 end
 
 Then /^the zip file in "(.*?)" should not contain any symlinks$/ do |path|
-  Zip::ZipFile.foreach Dir[path+'/app-*.zip'][0] do |p|
-   p.symlink?.should be_falsy
+  Zip::ZipFile.foreach Dir[path + '/app-*.zip'][0] do |p|
+    p.symlink?.should be_falsy
   end
 end

data/lib/zendesk_apps_tools/api_connection.rb

@@ -4,28 +4,30 @@ module ZendeskAppsTools
     URL_TEMPLATE = 'https://%s.zendesk.com/'
 
     def prepare_api_auth
-      @subdomain ||= get_cache('subdomain') || get_value_from_stdin('Enter your Zendesk subdomain or full Zendesk URL:')
-      @username  ||= get_cache('username') || get_value_from_stdin('Enter your username:')
-      @password  ||= get_cache('password') || get_password_from_stdin('Enter your password:')
+      @subdomain ||= fetch_cache('subdomain') || get_value_from_stdin('Enter your Zendesk subdomain or full Zendesk URL:')
+      @username  ||= fetch_cache('username') || get_value_from_stdin('Enter your username:')
+      @password  ||= fetch_cache('password') || get_password_from_stdin('Enter your password:')
 
-      set_cache 'subdomain' => @subdomain, 'username' => @username
-    end
-
-    def get_full_url
-      if FULL_URL =~ @subdomain
-        @subdomain
-      else
-        URL_TEMPLATE % @subdomain
-      end
+      save_cache 'subdomain' => @subdomain, 'username' => @username
     end
 
     def get_connection(encoding = :url_encoded)
       prepare_api_auth
-      Faraday.new get_full_url do |f|
+      Faraday.new full_url do |f|
         f.request encoding
         f.adapter :net_http
         f.basic_auth @username, @password
       end
     end
+
+    private
+
+    def full_url
+      if FULL_URL =~ @subdomain
+        @subdomain
+      else
+        URL_TEMPLATE % @subdomain
+      end
+    end
   end
 end

data/lib/zendesk_apps_tools/cache.rb

@@ -2,20 +2,20 @@ module ZendeskAppsTools
   module Cache
     CACHE_FILE_NAME = '.zat'
 
-    def set_cache(hash)
+    def save_cache(hash)
       return if options[:zipfile]
 
-      @cache = File.exists?(cache_path) ? JSON.parse(File.read(@cache_path)).update(hash) : hash
+      @cache = File.exist?(cache_path) ? JSON.parse(File.read(@cache_path)).update(hash) : hash
       File.open(@cache_path, 'w') { |f| f.write JSON.pretty_generate(@cache) }
     end
 
-    def get_cache(key)
-      @cache ||= File.exists?(cache_path) ? JSON.parse(File.read(@cache_path)) : {}
+    def fetch_cache(key)
+      @cache ||= File.exist?(cache_path) ? JSON.parse(File.read(@cache_path)) : {}
       @cache[key] if @cache
     end
 
     def clear_cache
-      File.delete cache_path if options[:clean] && File.exists?(cache_path)
+      File.delete cache_path if options[:clean] && File.exist?(cache_path)
     end
 
     def cache_path

data/lib/zendesk_apps_tools/command.rb

@@ -1,4 +1,4 @@
-require "thor"
+require 'thor'
 require 'zip/zip'
 require 'pathname'
 require 'net/http'
@@ -9,11 +9,9 @@ require 'io/console'
 require 'zendesk_apps_tools/command_helpers'
 
 module ZendeskAppsTools
-
   require 'zendesk_apps_support'
 
   class Command < Thor
-
     SHARED_OPTIONS = {
       ['path', '-p'] => './',
       clean: false
@@ -23,24 +21,24 @@ module ZendeskAppsTools
     include ZendeskAppsSupport
     include ZendeskAppsTools::CommandHelpers
 
-    source_root File.expand_path(File.join(File.dirname(__FILE__), "../.."))
+    source_root File.expand_path(File.join(File.dirname(__FILE__), '../..'))
 
     desc 'translate SUBCOMMAND', 'Manage translation files', hide: true
     subcommand 'translate', Translate
 
-    desc "new", "Generate a new app"
+    desc 'new', 'Generate a new app'
     def new
-      @author_name  = get_value_from_stdin("Enter this app author's name:\n", error_msg: "Invalid name, try again:")
-      @author_email = get_value_from_stdin("Enter this app author's email:\n", valid_regex: /^.+@.+\..+$/, error_msg: "Invalid email, try again:")
-      @author_url   = get_value_from_stdin("Enter this app author's url:\n", valid_regex: /^https?:\/\/.+$/, error_msg: "Invalid url, try again:", allow_empty: true)
-      @app_name     = get_value_from_stdin("Enter a name for this new app:\n", error_msg: "Invalid app name, try again:")
+      @author_name  = get_value_from_stdin("Enter this app author's name:\n", error_msg: 'Invalid name, try again:')
+      @author_email = get_value_from_stdin("Enter this app author's email:\n", valid_regex: /^.+@.+\..+$/, error_msg: 'Invalid email, try again:')
+      @author_url   = get_value_from_stdin("Enter this app author's url:\n", valid_regex: /^https?:\/\/.+$/, error_msg: 'Invalid url, try again:', allow_empty: true)
+      @app_name     = get_value_from_stdin("Enter a name for this new app:\n", error_msg: 'Invalid app name, try again:')
 
-      get_new_app_directory
+      prompt_new_app_dir
 
       directory('app_template', @app_dir)
     end
 
-    desc "validate", "Validate your app"
+    desc 'validate', 'Validate your app'
     method_options SHARED_OPTIONS
     def validate
       setup_path(options[:path])
@@ -61,7 +59,7 @@ module ZendeskAppsTools
       true
     end
 
-    desc "package", "Package your app"
+    desc 'package', 'Package your app'
     method_options SHARED_OPTIONS
     def package
       return false unless invoke(:validate, [])
@@ -73,27 +71,27 @@ module ZendeskAppsTools
 
       zip archive_path
 
-      say_status "package", "created at #{archive_rel_path}"
+      say_status 'package', "created at #{archive_rel_path}"
       true
     end
 
-    desc "clean", "Remove app packages in temp folder"
-    method_option :path, default: './', required: false, aliases: "-p"
+    desc 'clean', 'Remove app packages in temp folder'
+    method_option :path, default: './', required: false, aliases: '-p'
     def clean
       setup_path(options[:path])
 
-      return unless File.exists?(Pathname.new(File.join(app_dir, "tmp")).to_s)
+      return unless File.exist?(Pathname.new(File.join(app_dir, 'tmp')).to_s)
 
       FileUtils.rm(Dir["#{tmp_dir}/app-*.zip"])
     end
 
-    DEFAULT_SERVER_PATH = "./"
-    DEFAULT_CONFIG_PATH = "./settings.yml"
+    DEFAULT_SERVER_PATH = './'
+    DEFAULT_CONFIG_PATH = './settings.yml'
     DEFAULT_SERVER_PORT = 4567
 
-    desc "server", "Run a http server to serve the local app"
-    method_option :path, default: DEFAULT_SERVER_PATH, required: false, aliases: "-p"
-    method_option :config, default: DEFAULT_CONFIG_PATH, required: false, aliases: "-c"
+    desc 'server', 'Run a http server to serve the local app'
+    method_option :path, default: DEFAULT_SERVER_PATH, required: false, aliases: '-p'
+    method_option :config, default: DEFAULT_CONFIG_PATH, required: false, aliases: '-c'
     method_option :port, default: DEFAULT_SERVER_PORT, required: false
     def server
       setup_path(options[:path])
@@ -116,7 +114,7 @@ module ZendeskAppsTools
       end
     end
 
-    desc "create", "Create app on your account"
+    desc 'create', 'Create app on your account'
     method_options SHARED_OPTIONS
     method_option :zipfile, default: nil, required: false, type: :string
     def create
@@ -127,17 +125,17 @@ module ZendeskAppsTools
         app_name = JSON.parse(File.read(File.join options[:path], 'manifest.json'))['name']
       end
       app_name ||= get_value_from_stdin('Enter app name:')
-      deploy_app(:post, '/api/v2/apps.json', { name: app_name })
+      deploy_app(:post, '/api/v2/apps.json',  name: app_name)
     end
 
-    desc "update", "Update app on the server"
+    desc 'update', 'Update app on the server'
     method_options SHARED_OPTIONS
     method_option :zipfile, default: nil, required: false, type: :string
     def update
       clear_cache
       @command = 'Update'
 
-      app_id = get_cache('app_id') || find_app_id
+      app_id = fetch_cache('app_id') || find_app_id
       unless /\d+/ =~ app_id.to_s
         say_error_and_exit "App id not found\nPlease try running command with --clean or check your internet connection"
       end
@@ -149,6 +147,5 @@ module ZendeskAppsTools
     def setup_path(path)
       @destination_stack << relative_to_original_destination_root(path) unless @destination_stack.last == path
     end
-
   end
 end

data/lib/zendesk_apps_tools/common.rb

@@ -19,18 +19,18 @@ module ZendeskAppsTools
       }.merge(opts)
 
       while input = ask(prompt)
-        return "" if input.empty? && options[:allow_empty]
-        unless input =~ options[:valid_regex]
-          say(options[:error_msg], :red)
-        else
+        return '' if input.empty? && options[:allow_empty]
+        if input =~ options[:valid_regex]
           break
+        else
+          say(options[:error_msg], :red)
         end
       end
 
-      return input
+      input
     end
 
-    def get_password_from_stdin(prompt, opts = {})
+    def get_password_from_stdin(prompt)
       print "#{prompt} "
       password = STDIN.noecho(&:gets).chomp
       puts

data/lib/zendesk_apps_tools/deploy.rb

@@ -48,10 +48,10 @@ module ZendeskAppsTools
 
       all_apps = connection.get('/api/v2/apps.json').body
 
-      app = JSON.parse(all_apps)['apps'].find { |app| app['name'] == name }
+      app_id = JSON.parse(all_apps)['apps'].find { |app| app['name'] == name }['id']
 
-      set_cache 'app_id' => app['id']
-      app['id']
+      save_cache 'app_id' => app_id
+      app_id
     rescue Faraday::Error::ClientError => e
       say_error_and_exit e.message
     end
@@ -59,9 +59,7 @@ module ZendeskAppsTools
     def check_status(response)
       job = response.body
       job_response = JSON.parse(job)
-      if job_response['error']
-        say_error_and_exit job_response['error']
-      end
+      say_error_and_exit job_response['error'] if job_response['error']
 
       job_id = job_response['job_id']
       check_job job_id
@@ -77,10 +75,10 @@ module ZendeskAppsTools
         message  = info['message']
         app_id   = info['app_id']
 
-        if ['completed', 'failed'].include? status
+        if %w(completed failed).include? status
           case status
           when 'completed'
-            set_cache 'app_id' => app_id
+            save_cache 'app_id' => app_id
             say_status @command, 'OK'
           when 'failed'
             say_status @command, message
@@ -94,6 +92,5 @@ module ZendeskAppsTools
     rescue Faraday::Error::ClientError => e
       say_error_and_exit e.message
     end
-
   end
 end

data/lib/zendesk_apps_tools/directory.rb

@@ -1,30 +1,28 @@
 module ZendeskAppsTools
   module Directory
-
     def app_dir
       @app_dir ||= Pathname.new(destination_root)
     end
 
     def tmp_dir
-      @tmp_dir ||= Pathname.new(File.join(app_dir, "tmp")).tap do |dir|
+      @tmp_dir ||= Pathname.new(File.join(app_dir, 'tmp')).tap do |dir|
         FileUtils.mkdir_p(dir)
       end
     end
 
-    def get_new_app_directory
+    def prompt_new_app_dir
       prompt = "Enter a directory name to save the new app (will create the dir if it does not exist, default to current dir):\n"
       opts = { valid_regex: /^(\w|\/|\\)*$/, allow_empty: true }
-      while @app_dir = get_value_from_stdin(prompt, opts) do
+      while @app_dir = get_value_from_stdin(prompt, opts)
         @app_dir = './' and break if @app_dir.empty?
-        if !File.exists?(@app_dir)
+        if !File.exist?(@app_dir)
           break
         elsif !File.directory?(@app_dir)
-          puts "Invalid dir, try again:"
+          puts 'Invalid dir, try again:'
         else
           break
         end
       end
     end
-
   end
 end

data/lib/zendesk_apps_tools/locale_identifier.rb

@@ -4,11 +4,7 @@ module ZendeskAppsTools
 
     # Convert :"en-US-x-12" to 'en-US'
     def initialize(code)
-      @locale_id = if code.start_with?('en-US') #default locale
-        'en-US'
-      else
-        code.sub(/-x-.*/, '').downcase
-      end
+      @locale_id = code.start_with?('en-US') ? 'en-US' : code.sub(/-x-.*/, '').downcase
     end
   end
 end

data/lib/zendesk_apps_tools/package_helper.rb

@@ -1,20 +1,18 @@
 module ZendeskAppsTools
-
   require 'zendesk_apps_support'
 
   module PackageHelper
-
     include ZendeskAppsSupport
 
     def app_package
-      @app_package ||= Package.new(self.app_dir.to_s)
+      @app_package ||= Package.new(app_dir.to_s)
     end
 
     def zip(archive_path)
       Zip::ZipFile.open(archive_path, 'w') do |zipfile|
         app_package.files.each do |file|
           path = file.relative_path
-          say_status "package", "adding #{path}"
+          say_status 'package', "adding #{path}"
 
           # resolve symlink to source path
           if File.symlink? file.absolute_path
@@ -24,6 +22,5 @@ module ZendeskAppsTools
         end
       end
     end
-
   end
 end

data/lib/zendesk_apps_tools/server.rb

@@ -3,12 +3,11 @@ require 'zendesk_apps_support/package'
 
 module ZendeskAppsTools
   class Server < Sinatra::Base
-    set :public_folder, Proc.new {"#{settings.root}/assets"}
+    set :public_folder, proc { "#{settings.root}/assets" }
 
     get '/app.js' do
       content_type 'text/javascript'
       ZendeskAppsSupport::Package.new(settings.root).readified_js(nil, 0, "http://localhost:#{settings.port}/", settings.parameters)
     end
-
   end
-end
+end

data/lib/zendesk_apps_tools/settings.rb

@@ -3,7 +3,6 @@ require 'yaml'
 
 module ZendeskAppsTools
   class Settings
-
     def get_settings_from_user_input(user_input, parameters)
       return {} if parameters.nil?
 
@@ -28,7 +27,7 @@ module ZendeskAppsTools
 
     def get_settings_from_file(filepath, parameters)
       return {} if parameters.nil?
-      return nil unless File.exists? filepath
+      return nil unless File.exist? filepath
 
       begin
         settings_file = File.read(filepath)
@@ -36,15 +35,17 @@ module ZendeskAppsTools
         if filepath =~ /\.json$/ || settings_file =~ /\A\s*{/
           settings_data = JSON.load(settings_file)
         else
-          settings_data = YAML::load(settings_file)
+          settings_data = YAML.load(settings_file)
         end
 
         settings_data.each do |index, setting|
-          if (setting.is_a?(Hash) || setting.is_a?(Array))
+          if setting.is_a?(Hash) || setting.is_a?(Array)
             settings_data[index] = JSON.dump(setting)
           end
         end
       rescue => err
+        puts "Failed to load #{filepath}"
+        puts err.message
         return nil
       end
 
@@ -72,11 +73,10 @@ module ZendeskAppsTools
     private
 
     def convert_to_boolean_for_checkbox(input)
-      if ![TrueClass, FalseClass].include?(input.class)
+      unless [TrueClass, FalseClass].include?(input.class)
         return (input =~ /^(true|t|yes|y|1)$/i) ? true : false
       end
       input
     end
-
   end
 end

data/lib/zendesk_apps_tools/translate.rb

@@ -12,7 +12,7 @@ module ZendeskAppsTools
     include ZendeskAppsTools::Common
     include ZendeskAppsSupport::BuildTranslation
 
-    LOCALE_ENDPOINT = "https://support.zendesk.com/api/v2/locales/agent.json"
+    LOCALE_ENDPOINT = 'https://support.zendesk.com/api/v2/locales/agent.json'
 
     desc 'to_yml', 'Create Zendesk translation file from en.json'
     method_option :path, default: './', required: false
@@ -22,14 +22,14 @@ module ZendeskAppsTools
       app_name = manifest['name']
 
       unless app_name
-        app_name = get_value_from_stdin('What is the name of this app?', error_msg: "Invalid name, try again:")
+        app_name = get_value_from_stdin('What is the name of this app?', error_msg: 'Invalid name, try again:')
       end
 
       en_json = JSON.parse(File.open("#{destination_root}/translations/en.json").read)
 
-      package = en_json["app"]["package"]
+      package = en_json['app']['package']
       say('No package defined inside en.json! Abort.', :red) and exit 1 unless package
-      en_json["app"].delete("package")
+      en_json['app'].delete('package')
 
       write_yml(en_json, app_name, package)
     end
@@ -39,50 +39,48 @@ module ZendeskAppsTools
     def to_json
       setup_path(options[:path]) if options[:path]
       en_yml = YAML.load_file("#{destination_root}/translations/en.yml")
-      en_yml['parts'][0]['translation']['key'] =~ /^txt.apps.([^\.]+)/
-      package = $1
+      package = /^txt.apps.([^\.]+)/.match(en_yml['parts'][0]['translation']['key'])[1]
       translations = en_yml['parts'].map { |part| part['translation'] }
       en_json = array_to_nested_hash(translations)['txt']['apps'][package]
       en_json['app']['package'] = package
 
-      write_json("translations/en.json", en_json)
+      write_json('translations/en.json', en_json)
     end
 
     desc 'update', 'Update translation files from Zendesk'
     method_option :path, default: './', required: false
     def update(request_builder = Faraday.new)
       setup_path(options[:path]) if options[:path]
-      app_package = get_value_from_stdin("What is the package name for this app? (without app_)", valid_regex: /^[a-z_]+$/, error_msg: "Invalid package name, try again:")
+      app_package = get_value_from_stdin('What is the package name for this app? (without app_)', valid_regex: /^[a-z_]+$/, error_msg: 'Invalid package name, try again:')
 
       key_prefix = "txt.apps.#{app_package}."
 
-      say("Fetching translations...")
+      say('Fetching translations...')
       locale_response = api_request(LOCALE_ENDPOINT, request_builder)
 
       if locale_response.status == 200
-        locales = JSON.parse(locale_response.body)["locales"]
+        locales = JSON.parse(locale_response.body)['locales']
 
         locales.each do |locale|
-          locale_url      = "#{locale["url"]}?include=translations&packages=app_#{app_package}"
+          locale_url      = "#{locale['url']}?include=translations&packages=app_#{app_package}"
           locale_response = api_request(locale_url, request_builder).body
           translations    = JSON.parse(locale_response)['locale']['translations']
 
           locale_name = ZendeskAppsTools::LocaleIdentifier.new(locale['locale']).locale_id
           write_json("#{destination_root}/translations/#{locale_name}.json", nest_translations_hash(translations, key_prefix))
         end
-        say("Translations updated", :green)
+        say('Translations updated', :green)
 
       elsif locale_response.status == 401
-        say("Authentication failed", :red)
+        say('Authentication failed', :red)
       end
     end
 
     def self.source_root
-      File.expand_path(File.join(File.dirname(__FILE__), "../.."))
+      File.expand_path(File.join(File.dirname(__FILE__), '../..'))
     end
 
     no_commands do
-
       def setup_path(path)
         @destination_stack << relative_to_original_destination_root(path) unless @destination_stack.last == path
       end
@@ -100,13 +98,11 @@ module ZendeskAppsTools
           context     = result
 
           parts.each_with_index do |part, i|
-
             if parts_count == i
               context[part] = value
             else
               context = context[part] ||= {}
             end
-
           end
         end
 
@@ -116,10 +112,10 @@ module ZendeskAppsTools
       def write_yml(en_json, app_name, package_name)
         titles        = to_flattened_namespaced_hash(en_json, I18N_TITLE_KEY)
         values        = to_flattened_namespaced_hash(en_json, I18N_VALUE_KEY)
-        @translations = titles.each { |k, v| titles[k] = {"title" => v, "value" => escape_special_characters(values[k]) }}
+        @translations = titles.each { |k, v| titles[k] = { 'title' => v, 'value' => escape_special_characters(values[k]) } }
         @app_name     = app_name
         @package_name = package_name
-        template(File.join(Translate.source_root, 'templates/translation.erb.tt'), "translations/en.yml")
+        template(File.join(Translate.source_root, 'templates/translation.erb.tt'), 'translations/en.yml')
       end
 
       def escape_special_characters(v)
@@ -133,7 +129,7 @@ module ZendeskAppsTools
           keys[0..-2].each do |key|
             current = (current[key] ||= {})
           end
-          current[keys[-1]] = {'title' => item['title'], 'value' => item['value']}
+          current[keys[-1]] = { 'title' => item['title'], 'value' => item['value'] }
           result
         end
       end

data/lib/zendesk_apps_tools.rb

@@ -3,4 +3,3 @@ module ZendeskAppsTools
   autoload :Translate,    'zendesk_apps_tools/translate'
   autoload :LocaleIdentifier, 'zendesk_apps_tools/locale_identifier'
 end
-

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: zendesk_apps_tools
 version: !ruby/object:Gem::Version
-  version: 1.21.1
+  version: 1.21.2
 platform: ruby
 authors:
 - James A. Rosen
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-11-05 00:00:00.000000000 Z
+date: 2015-02-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: thor