This project is archived and is in readonly mode.
Rewrote ActionController.filter_chain for better readability
Reported by Thomas E. Glasgow | March 24th, 2009 @ 03:12 PM | in 2.x
While on my journey to explore the Rails source code and have a better understanding of how things work under the hood, I ran into the filter_chain method. The code was too verbose and was rewritten to a nicer Ruby idiom using the || operator.
All original tests pass.
Best regards,
Comments and changes to this ticket
-
Repository May 17th, 2009 @ 05:56 PM
- State changed from new to resolved
(from [e41984c5f7f38b62c05dfcb54940bf51f6961aeb]) Simplify filter_chain method implementation [#2327 state:resolved]
Signed-off-by: Pratik Naik pratiknaik@gmail.com
http://github.com/rails/rails/commit/e41984c5f7f38b62c05dfcb54940bf...
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
<h2 style="font-size: 14px">Tickets have moved to Github</h2>
The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>
People watching this ticket
- Nobody is watching this ticket.
Attachments
Referenced by
- 2327 Rewrote ActionController.filter_chain for better readability (from [e41984c5f7f38b62c05dfcb54940bf51f6961aeb]) Simplif...