Bug #1131

[PATCH] Six small, tested, behavior-neutral refactorings

Added by stuart-halloway - 504 days ago. Updated 497 days ago.

Status :Closed Start :
Priority :Normal Due date :
Assigned to :scott - % Done :

0%
Category :frontend
Target version :-
Resolution :

fixed

Description

These six refactorings are the source for a "This Week in Refactoring" presentation, which is attached along with the diff. Most are behavior neutral, but the RouteCache change fixes a bug.

All the refactorings have tests that run successfully. I cannot say for sure about the entire test suite because it had broken tests when I checked it out from svn.

Refactoring-Week-2007-07-16.pdf (140.5 KB) stuart-halloway -, 07/16/2007 07:54 PM

this_week_in_refactoring.diff (11.1 KB) stuart-halloway -, 07/16/2007 07:51 PM

History

07/16/2007 08:55 PM - neuro -

Hey, don't know who you are, but interesting work, thank you very much.

Typo was started long ago, with an early rails version and all code is not as clean as it should be.

07/23/2007 08:19 PM - neuro -

  • Status changed from New to Closed
  • Resolution set to fixed

Integrated in rev 1494

Also available in: Atom PDF