aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test/controller/new_base/bare_metal_test.rb
diff options
context:
space:
mode:
authorMatt Bridges <mbridges.91@gmail.com>2013-06-12 11:53:29 -0500
committerMatt Bridges <mbridges.91@gmail.com>2013-07-01 12:31:36 -0500
commit2da9d67c278b0f37d2dea89ff557dc07cbb3a7b0 (patch)
tree8e657d583379a482f25719fed62a0960deb3e8a8 /actionpack/test/controller/new_base/bare_metal_test.rb
parente47b6dee858e62dceba867dd160b968d679c82e8 (diff)
downloadrails-2da9d67c278b0f37d2dea89ff557dc07cbb3a7b0.tar.gz
rails-2da9d67c278b0f37d2dea89ff557dc07cbb3a7b0.tar.bz2
rails-2da9d67c278b0f37d2dea89ff557dc07cbb3a7b0.zip
Extract ActiveSupport::NumberHelper methods to classes
Due to the overall complexity of each method individually as well as the global shared private module methods, this pulls each helper into it's own converter class inheriting from a generic `NumberBuilder` class. * The `NumberBuilder` class contains the private methods needed for each helper method an eliminates the need for special definition of specialized private module methods. * The `ActiveSupport::NumberHelper::DEFAULTS` constant has been moved into the `NumberBuilder` class because the `NumberBuilder` is the only class which needs access to it. * For each of the builders, the `#convert` method is broken down to smaller parts and extracted into private methods for clarity of purpose. * Most of the mutation that once was necessary has now been eliminated. * Several of the mathematical operations for percentage, delimited, and rounded have been moved into private methods to ease readability and clarity. * Internationalization is still a bit crufty, and definitely could be improved, but it is functional and a bit easier to follow. The following helpers were extracted into their respective classes. * `#number_to_percentage` -> `NumberToPercentageConverter` * `#number_to_delimited` -> `NumberToDelimitedConverter` * `#number_to_phone` -> `NumberToPhoneConverter` * `#number_to_currency` -> `NumberToCurrencyConverter` * `#number_to_rounded` -> `NumberToRoundedConverter` * `#number_to_human_size` -> `NumberToHumanSizeConverter` * `#number_to_human` -> `NumberToHumanConverter`
Diffstat (limited to 'actionpack/test/controller/new_base/bare_metal_test.rb')
0 files changed, 0 insertions, 0 deletions