just use map and case / when rather than modifying the iterating array

1 files changed, 8 insertions, 5 deletions

diff --git a/actionpack/lib/action_view/path_set.rb b/actionpack/lib/action_view/path_set.rb
index e9773120c7..21dc5617ad 100644
--- a/actionpack/lib/action_view/path_set.rb
+++ b/actionpack/lib/action_view/path_set.rb
@@ -69,13 +69,16 @@ module ActionView #:nodoc:
       find_all(path, prefixes, *args).any?
     end
 
-  protected
+    private
 
     def typecast(paths)
-      paths.each_with_index do |path, i|
-        path = path.to_s if path.is_a?(Pathname)
-        next unless path.is_a?(String)
-        paths[i] = OptimizedFileSystemResolver.new(path)
+      paths.map do |path|
+        case path
+        when Pathname, String
+          OptimizedFileSystemResolver.new path.to_s
+        else
+          path
+        end
       end
     end
   end