This project is archived and is in readonly mode.
![Joseph Holsten Joseph Holsten](https://entp-lh-avatar-production.s3.amazonaws.com/avatars/4140/large.jpg?AWSAccessKeyId=AKIAJ4QBZRZBVMOUBNZA&Expires=2014466580&Signature=QX87B6Ftbh%2FQK7SnRYo8Gy2P8B4%3D)
Seperate CodeStatistics Rendering from Calculation
Reported by Joseph Holsten | January 11th, 2011 @ 03:59 PM
CodeStatistics is wonky. It's to_s actually just fires a bunch of puts, and there's no good way to get at the data if you want to present it differently. This patch separates CodeStatistics from the printing. I needed this to render a code stats in a web page for https://github.com/dhh/conductor, so I've merged this back in.
Also, this should make it easier to test #2299 and the spec bit in #4164.
Includes patch and test.
Comments and changes to this ticket
-
Dan Pickett March 12th, 2011 @ 04:11 PM
I dig the approach, and it applies cleanly to edge.
But, isn't the correlating test a bit brittle in the sense that any changes to the test application will cause your test to fail? I'm not sure I can suggest a better alternative, but I can see this as a potential annoyance when making significant changes to railties.
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>