Merge pull request #15325 from tgxworld/no_need_to_call_to_s

Remove duplicated to_s method call.
rails · May 26, 2014 · 2ae6e11 · 2ae6e11
2 parents 42fe948 + 86396f8
commit 2ae6e11
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/actionpack/lib/abstract_controller/base.rb b/actionpack/lib/abstract_controller/base.rb
@@ -255,7 +255,7 @@ def method_for_action(action_name)
 
       # Checks if the action name is valid and returns false otherwise.
       def _valid_action_name?(action_name)
-        action_name.to_s !~ Regexp.new(File::SEPARATOR)
+        action_name !~ Regexp.new(File::SEPARATOR)
       end
   end
 end
diff --git a/actionpack/lib/action_controller/metal.rb b/actionpack/lib/action_controller/metal.rb
@@ -226,7 +226,7 @@ def self.call(env)
 
     # Returns a Rack endpoint for the given action name.
     def self.action(name, klass = ActionDispatch::Request)
-      middleware_stack.build(name.to_s) do |env|
+      middleware_stack.build(name) do |env|
         new.dispatch(name, klass.new(env))
       end
     end