i18n-tasks 0.9.31 → 0.9.35

data/lib/i18n/tasks/data/file_system_base.rb

@@ -12,11 +12,12 @@ module I18n::Tasks
       include ::I18n::Tasks::Data::FileFormats
       include ::I18n::Tasks::Logging
 
-      attr_reader :config, :base_locale, :locales
-      attr_writer :locales
+      attr_accessor :locales
+      attr_reader :config, :base_locale
+      attr_writer :router
 
       DEFAULTS = {
-        read:  ['config/locales/%{locale}.yml'],
+        read: ['config/locales/%{locale}.yml'],
         write: ['config/locales/%{locale}.yml']
       }.freeze
 
@@ -131,11 +132,12 @@ module I18n::Tasks
       def t(key, locale)
         tree = self[locale.to_s]
         return unless tree
+
         tree[locale][key].try(:value_or_children_hash)
       end
 
       def config=(config)
-        @config = DEFAULTS.deep_merge((config || {}).reject { |_k, v| v.nil? })
+        @config = DEFAULTS.deep_merge((config || {}).compact)
         reload
       end
 
@@ -159,7 +161,6 @@ module I18n::Tasks
           router_class.new(self, @config.merge(base_locale: base_locale, locales: locales))
         end
       end
-      attr_writer :router
 
       protected
 
@@ -188,7 +189,7 @@ module I18n::Tasks
               The following unquoted YAML keys result in a nil key:
                 #{%w[null Null NULL ~].join(', ')}
               See http://yaml.org/type/null.html
-TEXT
+            TEXT
           elsif value.is_a?(Hash)
             filter_nil_keys! path, value, suffix + [key]
           end

data/lib/i18n/tasks/data/router/conservative_router.rb

@@ -13,8 +13,9 @@ module I18n::Tasks
         super
       end
 
-      def route(locale, forest, &block)
+      def route(locale, forest, &block) # rubocop:disable Metrics/AbcSize
         return to_enum(:route, locale, forest) unless block
+
         out = Hash.new { |hash, key| hash[key] = Set.new }
         not_found = Set.new
         forest.keys do |key, _node|

data/lib/i18n/tasks/data/router/pattern_router.rb

@@ -10,6 +10,7 @@ module I18n::Tasks
       include ::I18n::Tasks::KeyPatternMatching
 
       attr_reader :routes
+
       # @option data_config write [Array] of routes
       # @example
       #   {write:
@@ -29,6 +30,7 @@ module I18n::Tasks
       # @return [Hash] mapping of destination => [ [key, value], ... ]
       def route(locale, forest, &block)
         return to_enum(:route, locale, forest) unless block
+
         locale = locale.to_s
         out = {}
         forest.keys do |key, _node|

data/lib/i18n/tasks/data/tree/node.rb

@@ -18,7 +18,7 @@ module I18n::Tasks::Data::Tree
       @data = data
       @parent = parent
       @warn_about_add_children_to_leaf = warn_about_add_children_to_leaf
-      self.children = (children if children)
+      self.children = children if children
     end
     # rubocop:enable Metrics/ParameterLists
 
@@ -48,6 +48,7 @@ module I18n::Tasks::Data::Tree
 
     def each(&block)
       return to_enum(:each) { 1 } unless block
+
       block.yield(self)
       self
     end
@@ -112,6 +113,7 @@ module I18n::Tasks::Data::Tree
 
     def walk_to_root(&visitor)
       return to_enum(:walk_to_root) unless visitor
+
       visitor.yield self
       parent.walk_to_root(&visitor) if parent?
     end
@@ -124,6 +126,7 @@ module I18n::Tasks::Data::Tree
 
     def walk_from_root(&visitor)
       return to_enum(:walk_from_root) unless visitor
+
       walk_to_root.reverse_each do |node|
         visitor.yield node
       end
@@ -169,7 +172,7 @@ module I18n::Tasks::Data::Tree
                  (": #{format_value_for_inspect(value)}" if leaf?),
                  (" #{data}" if data?)].compact.join
               end
-      ['  ' * level, label, ("\n" + children.map { |c| c.inspect(level + 1) }.join("\n") if children?)].compact.join
+      ['  ' * level, label, ("\n#{children.map { |c| c.inspect(level + 1) }.join("\n")}" if children?)].compact.join
     end
 
     def format_value_for_inspect(value)

data/lib/i18n/tasks/data/tree/nodes.rb

@@ -80,6 +80,7 @@ module I18n::Tasks::Data::Tree
 
     def children(&block)
       return to_enum(:children) { map { |c| c.children ? c.children.size : 0 }.reduce(:+) } unless block
+
       each do |node|
         node.children.each(&block) if node.children?
       end

data/lib/i18n/tasks/data/tree/siblings.rb

@@ -62,6 +62,7 @@ module I18n::Tasks::Data::Tree
       # TODO: support nested references better
       nodes do |node|
         next unless node.reference?
+
         old_target = [(node.root.key if root), node.value.to_s].compact.join('.')
         new_target = old_key_to_new_key[old_target]
         if new_target
@@ -92,6 +93,7 @@ module I18n::Tasks::Data::Tree
     # add or replace node by full key
     def set(full_key, node)
       fail 'value should be a I18n::Tasks::Data::Tree::Node' unless node.is_a?(Node)
+
       key_part, rest = split_key(full_key, 2)
       child          = key_to_node[key_part]
 
@@ -131,6 +133,7 @@ module I18n::Tasks::Data::Tree
     def append!(nodes)
       nodes = nodes.map do |node|
         fail "already has a child with key '#{node.key}'" if key_to_node.key?(node.key)
+
         key_to_node[node.key] = (node.parent == parent ? node : node.derive(parent: parent))
       end
       super(nodes)
@@ -172,16 +175,18 @@ module I18n::Tasks::Data::Tree
 
     def set_root_key!(new_key, data = nil)
       return self if empty?
+
       rename_key first.key, new_key
       leaves { |node| node.data.merge! data } if data
       self
     end
 
     # @param on_leaves_merge [Proc] invoked when a leaf is merged with another leaf
-    def merge_node!(node, on_leaves_merge: nil) # rubocop:disable Metrics/AbcSize,Metrics/PerceivedComplexity
+    def merge_node!(node, on_leaves_merge: nil) # rubocop:disable Metrics/AbcSize
       if key_to_node.key?(node.key)
         our = key_to_node[node.key]
         return if our == node
+
         our.value = node.value if node.leaf?
         our.data.merge!(node.data) if node.data?
         if node.children?
@@ -262,7 +267,7 @@ module I18n::Tasks::Data::Tree
         build_forest(warn_about_add_children_to_leaf: false) do |forest|
           key_occurrences.each do |key_occurrence|
             forest[key_occurrence.key] = ::I18n::Tasks::Data::Tree::Node.new(
-              key:  split_key(key_occurrence.key).last,
+              key: split_key(key_occurrence.key).last,
               data: { occurrences: key_occurrence.occurrences }
             )
           end
@@ -273,6 +278,7 @@ module I18n::Tasks::Data::Tree
         build_forest(opts) do |forest|
           key_attrs.each do |(full_key, attr)|
             fail "Invalid key #{full_key.inspect}" if full_key.end_with?('.')
+
             node = ::I18n::Tasks::Data::Tree::Node.new(**attr.merge(key: split_key(full_key).last))
             yield(full_key, node) if block
             forest[full_key] = node
@@ -295,6 +301,7 @@ module I18n::Tasks::Data::Tree
       def from_nested_hash(hash, opts = {})
         parse_parent_opt!(opts)
         fail I18n::Tasks::CommandError, "invalid tree #{hash.inspect}" unless hash.respond_to?(:map)
+
         opts[:nodes] = hash.map { |key, value| Node.from_key_value key, value }
         Siblings.new(opts)
       end

data/lib/i18n/tasks/data/tree/traversal.rb

@@ -12,6 +12,7 @@ module I18n::Tasks
 
       def leaves(&visitor)
         return to_enum(:leaves) unless visitor
+
         nodes do |node|
           visitor.yield(node) if node.leaf?
         end
@@ -20,6 +21,7 @@ module I18n::Tasks
 
       def levels(&block)
         return to_enum(:levels) unless block
+
         nodes = to_nodes
         unless nodes.empty?
           block.yield nodes
@@ -35,6 +37,7 @@ module I18n::Tasks
 
       def breadth_first(&visitor)
         return to_enum(:breadth_first) unless visitor
+
         levels do |nodes|
           nodes.each { |node| visitor.yield(node) }
         end
@@ -43,9 +46,11 @@ module I18n::Tasks
 
       def depth_first(&visitor)
         return to_enum(:depth_first) unless visitor
+
         each do |node|
           visitor.yield node
           next unless node.children?
+
           node.children.each do |child|
             child.depth_first(&visitor)
           end
@@ -56,6 +61,7 @@ module I18n::Tasks
       # @option root include root in full key
       def keys(root: false, &visitor)
         return to_enum(:keys, root: root) unless visitor
+
         leaves { |node| visitor.yield(node.full_key(root: root), node) }
         self
       end
@@ -70,13 +76,13 @@ module I18n::Tasks
 
       def root_key_values(sort = false)
         result = keys(root: false).map { |key, node| [node.root.key, key, node.value] }
-        result.sort! { |a, b| a[0] != b[0] ? a[0] <=> b[0] : a[1] <=> b[1] } if sort
+        result.sort! { |a, b| a[0] == b[0] ? a[1] <=> b[1] : a[0] <=> b[0] } if sort
         result
       end
 
       def root_key_value_data(sort = false)
         result = keys(root: false).map { |key, node| [node.root.key, key, node.value, node.data] }
-        result.sort! { |a, b| a[0] != b[0] ? a[0] <=> b[0] : a[1] <=> b[1] } if sort
+        result.sort! { |a, b| a[0] == b[0] ? a[1] <=> b[1] : a[0] <=> b[0] } if sort
         result
       end
 
@@ -88,6 +94,7 @@ module I18n::Tasks
         tree = Siblings.new
         each do |node|
           next unless block.yield(node)
+
           tree.append! node.derive(
             parent: tree.parent,
             children: (node.children.select_nodes(&block).to_a if node.children)
@@ -163,6 +170,7 @@ module I18n::Tasks
         value_proc ||= proc do |node|
           node_value = node.value
           next node_value if node.reference?
+
           human_key = ActiveSupport::Inflector.humanize(node.key.to_s)
           full_key = node.full_key
           default = (node.data[:occurrences] || []).detect { |o| o.default_arg.presence }.try(:default_arg)
@@ -179,6 +187,7 @@ module I18n::Tasks
         pattern_re = I18n::Tasks::KeyPatternMatching.compile_key_pattern(key_pattern) if key_pattern.present?
         keys.each do |key, node|
           next if pattern_re && key !~ pattern_re
+
           node.value = value_proc.call(node)
         end
         self

data/lib/i18n/tasks/html_keys.rb

@@ -2,15 +2,13 @@
 
 module I18n::Tasks
   module HtmlKeys
-    HTML_KEY_PATTERN = /[.\-_]html\z/
-    MAYBE_PLURAL_HTML_KEY_PATTERN = /[.\-_]html\.[^.]+\z/
+    HTML_KEY_PATTERN = /[.\-_]html\z/.freeze
+    MAYBE_PLURAL_HTML_KEY_PATTERN = /[.\-_]html\.[^.]+\z/.freeze
 
     def html_key?(full_key, locale)
-      # rubocop:disable Style/DoubleNegation
       !!(full_key =~ HTML_KEY_PATTERN ||
           full_key =~ MAYBE_PLURAL_HTML_KEY_PATTERN &&
               depluralize_key(split_key(full_key, 2)[1], locale) =~ HTML_KEY_PATTERN)
-      # rubocop:enable Style/DoubleNegation
     end
   end
 end

data/lib/i18n/tasks/ignore_keys.rb

@@ -17,12 +17,13 @@ module I18n::Tasks::IgnoreKeys
     @ignore_patterns[type][locale] ||= begin
       global = ignore_config.presence || []
       type_ignore = ignore_config(type).presence || []
-      patterns = if type_ignore.is_a?(Array)
+      patterns = case type_ignore
+                 when Array
                    global + type_ignore
-                 elsif type_ignore.is_a?(Hash)
+                 when Hash
                    # ignore per locale
                    global + (type_ignore['all'] || []) +
-                     type_ignore.select { |k, _v| k.to_s =~ /\b#{locale}\b/ }.values.flatten(1).compact
+                   type_ignore.select { |k, _v| k.to_s =~ /\b#{locale}\b/ }.values.flatten(1).compact
                  end
       compile_patterns_re patterns
     end

data/lib/i18n/tasks/interpolations.rb

@@ -2,7 +2,7 @@
 
 module I18n::Tasks
   module Interpolations
-    VARIABLE_REGEX = /%{[^}]+}/
+    VARIABLE_REGEX = /%{[^}]+}/.freeze
 
     def inconsistent_interpolations(locales: nil, base_locale: nil) # rubocop:disable Metrics/AbcSize
       locales ||= self.locales
@@ -11,10 +11,12 @@ module I18n::Tasks
 
       data[base_locale].key_values.each do |key, value|
         next if !value.is_a?(String) || ignore_key?(key, :inconsistent_interpolations)
+
         base_vars = Set.new(value.scan(VARIABLE_REGEX))
         (locales - [base_locale]).each do |current_locale|
           node = data[current_locale].first.children[key]
-          next unless node&.value&.is_a?(String)
+          next unless node&.value.is_a?(String)
+
           if base_vars != Set.new(node.value.scan(VARIABLE_REGEX))
             result.merge!(node.walk_to_root.reduce(nil) { |c, p| [p.derive(children: c)] })
           end

data/lib/i18n/tasks/key_pattern_matching.rb

@@ -5,7 +5,7 @@ require 'strscan'
 module I18n::Tasks::KeyPatternMatching
   extend self # rubocop:disable Style/ModuleFunction
 
-  MATCH_NOTHING = /\z\A/
+  MATCH_NOTHING = /\z\A/.freeze
 
   # one regex to match any
   def compile_patterns_re(key_patterns)
@@ -13,7 +13,7 @@ module I18n::Tasks::KeyPatternMatching
       # match nothing
       MATCH_NOTHING
     else
-      /(?:#{ key_patterns.map { |p| compile_key_pattern p } * '|' })/m
+      /(?:#{key_patterns.map { |p| compile_key_pattern p } * '|'})/m
     end
   end
 
@@ -24,6 +24,7 @@ module I18n::Tasks::KeyPatternMatching
   #   { a, b.c } match any in set, can use : and *, match is captured
   def compile_key_pattern(key_pattern)
     return key_pattern if key_pattern.is_a?(Regexp)
+
     /\A#{key_pattern_re_body(key_pattern)}\z/
   end
 

data/lib/i18n/tasks/missing_keys.rb

@@ -60,11 +60,14 @@ module I18n::Tasks
       locales.each_with_object(empty_forest) do |locale, forest|
         required_keys = required_plural_keys_for_locale(locale)
         next if required_keys.empty?
+
         tree = empty_forest
         plural_nodes data[locale] do |node|
           children = node.children
           present_keys = Set.new(children.map { |c| c.key.to_sym })
+          next if ignore_key?(node.full_key(root: false), :missing)
           next if present_keys.superset?(required_keys)
+
           tree[node.full_key] = node.derive(
             value: children.to_hash,
             children: nil,
@@ -79,12 +82,8 @@ module I18n::Tasks
     def required_plural_keys_for_locale(locale)
       @plural_keys_for_locale ||= {}
       return @plural_keys_for_locale[locale] if @plural_keys_for_locale.key?(locale)
-      @plural_keys_for_locale[locale] =
-        begin
-          Set.new(load_rails_i18n_pluralization!(locale)[locale.to_sym][:i18n][:plural][:keys])
-        rescue SystemCallError, IOError
-          Set.new
-        end
+
+      @plural_keys_for_locale[locale] = plural_keys_for_locale(locale)
     end
 
     # Loads rails-i18n pluralization config for the given locale.
@@ -123,7 +122,7 @@ module I18n::Tasks
     end
 
     def locale_key_missing?(locale, key)
-      !key_value?(key, locale) && !external_key?(key) && !ignore_key?(key, :missing)
+      !key_value?(key, locale) && !external_key?(key, locale) && !ignore_key?(key, :missing)
     end
 
     # @param [::I18n::Tasks::Data::Tree::Siblings] forest
@@ -136,6 +135,7 @@ module I18n::Tasks
         locale = root.key
         root.keys do |key, node|
           next unless yield node
+
           if locales_and_node_by_key.key?(key)
             locales_and_node_by_key[key][0] << locale
           else
@@ -154,5 +154,32 @@ module I18n::Tasks
       end
       forest
     end
+
+    private
+
+    def plural_keys_for_locale(locale)
+      configuration = load_rails_i18n_pluralization!(locale)
+      if configuration[locale.to_sym].nil?
+        alternate_locale = alternate_locale_from(locale)
+        return Set.new if configuration[alternate_locale.to_sym].nil?
+
+        return set_from_rails_i18n_pluralization(configuration, alternate_locale)
+      end
+      set_from_rails_i18n_pluralization(configuration, locale)
+    rescue SystemCallError, IOError
+      Set.new
+    end
+
+    def alternate_locale_from(locale)
+      re = /(\w{2})-*(\w{2,3})*/
+      match = locale.match(re)
+      language_code = match[1]
+      country_code = match[2]
+      "#{language_code}-#{country_code.upcase}"
+    end
+
+    def set_from_rails_i18n_pluralization(configuration, locale)
+      Set.new(configuration[locale.to_sym][:i18n][:plural][:keys])
+    end
   end
 end

data/lib/i18n/tasks/plural_keys.rb

@@ -5,12 +5,13 @@ module I18n::Tasks::PluralKeys
   # Ref: http://cldr.unicode.org/index/cldr-spec/plural-rules
   CLDR_CATEGORY_KEYS = %w[zero one two few many other].freeze
   PLURAL_KEY_SUFFIXES = Set.new CLDR_CATEGORY_KEYS
-  PLURAL_KEY_RE = /\.(?:#{CLDR_CATEGORY_KEYS * '|'})$/
+  PLURAL_KEY_RE = /\.(?:#{CLDR_CATEGORY_KEYS * '|'})$/.freeze
 
   def collapse_plural_nodes!(tree)
     tree.leaves.map(&:parent).compact.uniq.each do |node|
       children = node.children
       next unless plural_forms?(children)
+
       node.value    = children.to_hash
       node.children = nil
       node.data.merge! children.first.data
@@ -23,6 +24,7 @@ module I18n::Tasks::PluralKeys
   # @return [String] the base form if the key is a specific plural form (e.g. apple for apple.many), the key otherwise.
   def depluralize_key(key, locale = base_locale)
     return key if key !~ PLURAL_KEY_RE
+
     key_name = last_key_part(key)
     parent_key = key[0..- (key_name.length + 2)]
     nodes = tree("#{locale}.#{parent_key}").presence || (locale != base_locale && tree("#{base_locale}.#{parent_key}"))
@@ -37,10 +39,12 @@ module I18n::Tasks::PluralKeys
   # @yieldparam node [::I18n::Tasks::Data::Tree::Node] plural node
   def plural_nodes(tree)
     return to_enum(:plural_nodes, tree) unless block_given?
+
     visited = Set.new
     tree.leaves do |node|
       parent = node.parent
       next if !parent || visited.include?(parent)
+
       yield parent if plural_forms?(parent.children)
       visited.add(parent)
     end
@@ -49,6 +53,7 @@ module I18n::Tasks::PluralKeys
 
   def plural_forms?(s)
     return false if non_plural_other?(s)
+
     s.present? && s.all? { |node| node.leaf? && plural_suffix?(node.key) }
   end
 

data/lib/i18n/tasks/references.rb

@@ -10,12 +10,14 @@ module I18n::Tasks
                            data_refs = merge_reference_trees(data_forest.select_keys { |_, node| node.reference? }))
       fail ArgumentError, 'usages must be a Data::Tree::Instance' unless usages.is_a?(Data::Tree::Siblings)
       fail ArgumentError, 'all_references must be a Data::Tree::Instance' unless data_refs.is_a?(Data::Tree::Siblings)
+
       raw_refs = empty_forest
       resolved_refs = empty_forest
       refs = empty_forest
       data_refs.key_to_node.each do |ref_key_part, ref_node|
         usages.each do |usage_node|
           next unless usage_node.key == ref_key_part
+
           if ref_node.leaf?
             process_leaf!(ref_node, usage_node, raw_refs, resolved_refs, refs)
           else

data/lib/i18n/tasks/reports/base.rb

@@ -9,6 +9,7 @@ module I18n::Tasks::Reports
     end
 
     attr_reader :task
+
     delegate :base_locale, :locales, to: :task
 
     protected
@@ -58,6 +59,7 @@ module I18n::Tasks::Reports
 
     def format_locale(locale)
       return '' unless locale
+
       if locale.split('+') == task.locales.sort
         'all'
       else
@@ -67,8 +69,7 @@ module I18n::Tasks::Reports
 
     def collapse_missing_tree!(forest)
       forest = task.collapse_plural_nodes!(forest)
-      forest = task.collapse_same_key_in_locales!(forest) { |node| node.data[:type] == :missing_used }
-      forest
+      task.collapse_same_key_in_locales!(forest) { |node| node.data[:type] == :missing_used }
     end
   end
 end

data/lib/i18n/tasks/reports/terminal.rb

@@ -106,9 +106,10 @@ module I18n
         private
 
         def missing_key_info(leaf)
-          if leaf[:type] == :missing_used
+          case leaf[:type]
+          when :missing_used
             first_occurrence leaf
-          elsif leaf[:type] == :missing_plural
+          when :missing_plural
             leaf[:data][:missing_keys].join(', ')
           else
             "#{Rainbow(leaf[:data][:missing_diff_locale]).cyan} "\
@@ -134,6 +135,7 @@ module I18n
 
         def format_reference_desc(node_data)
           return nil unless node_data
+
           case node_data[:ref_type]
           when :reference_usage
             Rainbow('(ref)').yellow.bright
@@ -159,9 +161,9 @@ module I18n
             print_table headings: [Rainbow(I18n.t('i18n_tasks.common.locale')).cyan.bright,
                                    Rainbow(I18n.t('i18n_tasks.common.key')).cyan.bright,
                                    I18n.t('i18n_tasks.common.value')] do |t|
-              t.rows = locale_key_value_datas.map { |(locale, k, v, data)|
+              t.rows = locale_key_value_datas.map do |(locale, k, v, data)|
                 [{ value: Rainbow(locale).cyan, alignment: :center }, format_key(k, data), format_value(v)]
-              }
+              end
             end
           else
             puts 'ø'

data/lib/i18n/tasks/scanners/file_scanner.rb

@@ -11,10 +11,11 @@ module I18n::Tasks::Scanners
     attr_reader :config
 
     def initialize(
-        config: {},
-        file_finder_provider: Files::CachingFileFinderProvider.new,
-        file_reader: Files::CachingFileReader.new
+      config: {},
+      file_finder_provider: Files::CachingFileFinderProvider.new,
+      file_reader: Files::CachingFileReader.new
     )
+      super()
       @config      = config
       @file_reader = file_reader
       @file_finder = file_finder_provider.get(**config.slice(:paths, :only, :exclude))

data/lib/i18n/tasks/scanners/files/caching_file_finder.rb

@@ -22,9 +22,6 @@ module I18n::Tasks::Scanners::Files
     # @param (see FileFinder#traverse_files)
     # @yieldparam (see FileFinder#traverse_files)
     # @return (see FileFinder#traverse_files)
-    def traverse_files
-      super
-    end
 
     alias uncached_find_files find_files
     private :uncached_find_files

data/lib/i18n/tasks/scanners/files/file_finder.rb

@@ -15,6 +15,7 @@ module I18n::Tasks::Scanners::Files
     #     Files matching any of the exclusion patterns will be excluded even if they match an inclusion pattern.
     def initialize(paths: ['.'], only: nil, exclude: [])
       fail 'paths argument is required' if paths.nil?
+
       @paths   = paths
       @include = only
       @exclude = exclude || []
@@ -25,8 +26,8 @@ module I18n::Tasks::Scanners::Files
     # @yield [path]
     # @yieldparam path [String] the path of the found file.
     # @return [Array<of block results>]
-    def traverse_files
-      find_files.map { |path| yield path }
+    def traverse_files(&block)
+      find_files.map(&block)
     end
 
     # @return [Array<String>] found files

data/lib/i18n/tasks/scanners/occurrence_from_position.rb

@@ -15,11 +15,11 @@ module I18n
           line_begin = contents.rindex(/^/, position - 1)
           line_end   = contents.index(/.(?=\r?\n|$)/, position)
           Results::Occurrence.new(
-            path:     path,
-            pos:      position,
+            path: path,
+            pos: position,
             line_num: contents[0..position].count("\n") + 1,
             line_pos: position - line_begin + 1,
-            line:     contents[line_begin..line_end],
+            line: contents[line_begin..line_end],
             raw_key: raw_key
           )
         end

data/lib/i18n/tasks/scanners/pattern_scanner.rb

@@ -12,7 +12,7 @@ module I18n::Tasks::Scanners
     include OccurrenceFromPosition
     include RubyKeyLiterals
 
-    TRANSLATE_CALL_RE = /(?<=^|[^\w'\-.]|[^\w'\-]I18n\.|I18n\.)t(?:!|ranslate!?)?/
+    TRANSLATE_CALL_RE = /(?<=^|[^\w'\-.]|[^\w'\-]I18n\.|I18n\.)t(?:!|ranslate!?)?/.freeze
     IGNORE_LINES = {
       'coffee' => /^\s*#(?!\si18n-tasks-use)/,
       'erb' => /^\s*<%\s*#(?!\si18n-tasks-use)/,
@@ -43,10 +43,13 @@ module I18n::Tasks::Scanners
         src_pos = Regexp.last_match.offset(0).first
         location = occurrence_from_position(path, text, src_pos, raw_key: strip_literal(match[0]))
         next if exclude_line?(location.line, path)
+
         key = match_to_key(match, path, location)
         next unless key
+
         key += ':' if key.end_with?('.')
         next unless valid_key?(key)
+
         keys << [key, location]
       end
       keys
@@ -67,7 +70,7 @@ module I18n::Tasks::Scanners
       re && re =~ line
     end
 
-    VALID_KEY_RE_DYNAMIC = /^(#{VALID_KEY_CHARS}|[:\#{@}\[\]])+$/
+    VALID_KEY_RE_DYNAMIC = /^(#{VALID_KEY_CHARS}|[:\#{@}\[\]])+$/.freeze
 
     def valid_key?(key)
       if @config[:strict]
@@ -84,7 +87,7 @@ module I18n::Tasks::Scanners
     def closest_method(occurrence)
       method = File.readlines(occurrence.path, encoding: 'UTF-8')
                    .first(occurrence.line_num - 1).reverse_each.find { |x| x =~ /\bdef\b/ }
-      method && method.strip.sub(/^def\s*/, '').sub(/[\(\s;].*$/, '')
+      method && method.strip.sub(/^def\s*/, '').sub(/[(\s;].*$/, '')
     end
 
     # This method only exists for backwards compatibility with monkey-patches and plugins
@@ -93,7 +96,7 @@ module I18n::Tasks::Scanners
     def default_pattern
       # capture only the first argument
       /
-      #{translate_call_re} [\( ] \s* (?# fn call begin )
+      #{translate_call_re} [( ] \s* (?# fn call begin )
       (#{first_argument_re})         (?# capture the first argument)
       /x
     end