This project is archived and is in readonly mode.
Move capture to Kernel
#5641
Currently capture method appears twice:
http://github.com/rails/rails/blob/master/railties/lib/rails/genera...
http://github.com/rails/rails/blob/master/railties/test/railties/en...
I think its place is in Kernel module extension in reporting. I've attached patch for this.
-
capture_refactoring.patch
3.96 KB
-
-
Lake+1 for staying DRY. The tests pass for me and of course the code looks better.
-
Lake
- Title changed from [PATCH] Move capture to Kernel to Move capture to Kernel
-
Repository
- State changed from open to resolved
(from [d4fa120671e989eecb71c14647dd2051d28de4a5]) Move capture to Kernel. [#5641 Move capture to Kernel state:resolved] http://github.com/rails/rails/commit/d4fa120671e989eecb71c14647dd20...