Refactor code to increase CodeClimate score.

lib/route_translator/extensions/route_set.rb

@@ -4,7 +4,9 @@ module ActionDispatch
   module Routing
     class RouteSet
       def add_localized_route(mapping, path_ast, name, anchor, scope, path, controller, default_action, to, via, formatted, options_constraints, options)
-        RouteTranslator::Translator.translations_for(self, path, name, options_constraints, options, mapping) do |translated_name, translated_path, translated_options_constraints, translated_options|
+        route = RouteTranslator::Route.new(self, path, name, options_constraints, options, mapping)
+
+        RouteTranslator::Translator.translations_for(route) do |translated_name, translated_path, translated_options_constraints, translated_options|
           translated_path_ast = ::ActionDispatch::Journey::Parser.parse(translated_path)
           translated_mapping  = ::ActionDispatch::Routing::Mapper::Mapping.build(scope, self, translated_path_ast, controller, default_action, to, via, formatted, translated_options_constraints, anchor, translated_options)
 

lib/route_translator/route.rb

@@ -0,0 +1,18 @@
+module RouteTranslator
+  class Route
+    attr_reader :route_set, :path, :name, :options_constraints, :options, :mapping
+
+    def initialize(route_set, path, name, options_constraints, options, mapping)
+      @route_set           = route_set
+      @path                = path
+      @name                = name
+      @options_constraints = options_constraints
+      @options             = options
+      @mapping             = mapping
+    end
+
+    def scope
+      @scope ||= [:routes, :controllers].concat mapping.defaults[:controller].split('/').map(&:to_sym)
+    end
+  end
+end

lib/route_translator/translator.rb

@@ -2,6 +2,7 @@
 
 require File.expand_path('../translator/route_helpers', __FILE__)
 require File.expand_path('../translator/path', __FILE__)
+require File.expand_path('../route', __FILE__)
 
 module RouteTranslator
   module Translator
@@ -35,28 +36,31 @@ def translate_name(name, locale, named_routes_names)
 
     module_function
 
-    def translations_for(route_set, path, name, options_constraints, options, mapping)
-      RouteTranslator::Translator::RouteHelpers.add name, route_set.named_routes
+    def translations_for(route)
+      RouteTranslator::Translator::RouteHelpers.add route.name, route.route_set.named_routes
 
-      scope = [:routes, :controllers].concat mapping.defaults[:controller].split('/').map(&:to_sym)
       available_locales.each do |locale|
         begin
-          translated_path = RouteTranslator::Translator::Path.translate(path, locale, scope)
+          translated_path = RouteTranslator::Translator::Path.translate(route.path, locale, route.scope)
         rescue I18n::MissingTranslationData => e
           raise e unless RouteTranslator.config.disable_fallback
           next
         end
 
-        translated_options_constraints = options_constraints.dup
-        translated_options             = options.dup
+        yield translated_attributes(locale, translated_path, route)
+      end
+    end
 
-        translated_options_constraints[RouteTranslator.locale_param_key] = locale.to_s
-        translated_options[RouteTranslator.locale_param_key]             = locale.to_s.gsub('native_', '') unless translated_options.include?(RouteTranslator.locale_param_key)
+    def translated_attributes(locale, translated_path, route)
+      translated_options_constraints = route.options_constraints.dup
+      translated_options             = route.options.dup
 
-        translated_name = translate_name(name, locale, route_set.named_routes.names)
+      translated_options_constraints[RouteTranslator.locale_param_key] = locale.to_s
+      translated_options[RouteTranslator.locale_param_key]             = locale.to_s.gsub('native_', '') unless translated_options.include?(RouteTranslator.locale_param_key)
 
-        yield translated_name, translated_path, translated_options_constraints, translated_options
-      end
+      translated_name = translate_name(route.name, locale, route.route_set.named_routes.names)
+
+      [translated_name, translated_path, translated_options_constraints, translated_options]
     end
 
     def route_name_for(args, old_name, suffix, kaller)