This project is archived and is in readonly mode.
Better style in NumberHelper
#3976
NumberHelper uses blank rescues heap, this is pretty bad form and makes the code harder to understand. I refactored out the obvious abuses, though left in the ones on the i18n translate lines since I figure that pattern (which I don't like) is probably used elsewhere.
This doesn't change tested behaviour.
-
JohnI just gave the patch a good readthrough, much neater.
+1
-
Santiago Pastorino- Tag changed from actionpack numberhelper style to actionpack numberhelper style, 3.0
- Assigned user set to Yehuda Katz (wycats)
- Milestone cleared.
Applies cleanly on master
-
Santiago Pastorino
- State changed from new to resolved
-
