Permalink
Browse files

Refactored code

  • Loading branch information...
1 parent e6c98b8 commit fa464681128b379ca224e01dcefbe683999f8438 @raghunadhd raghunadhd committed Feb 9, 2012
@@ -542,7 +542,7 @@ namespace :railties do
# desc "Copies missing migrations from Railties (e.g. engines). You can specify Railties to use with FROM=railtie1,railtie2"
task :migrations => :'db:load_config' do
to_load = ENV['FROM'].blank? ? :all : ENV['FROM'].split(",").map {|n| n.strip }
- railties = ActiveSupport::OrderedHash.new
+ railties = {}
Rails.application.railties.all do |railtie|
next unless to_load == :all || to_load.include?(railtie.railtie_name)
@@ -286,7 +286,7 @@ def execute_grouped_calculation(operation, column_name, distinct) #:nodoc:
key_records = Hash[key_records.map { |r| [r.id, r] }]
end
- ActiveSupport::OrderedHash[calculated_data.map do |row|
+ Hash[calculated_data.map do |row|
key = group_columns.map { |aliaz, column|
type_cast_calculated_value(row[aliaz], column)
}
@@ -663,7 +663,7 @@ def test_should_ignore_new_associated_records_if_a_reject_if_proc_returns_false
end
def test_should_sort_the_hash_by_the_keys_before_building_new_associated_models
- attributes = ActiveSupport::OrderedHash.new
+ attributes = {}
attributes['123726353'] = { :name => 'Grace OMalley' }
attributes['2'] = { :name => 'Privateers Greed' } # 2 is lower then 123726353
@pirate.send(association_setter, attributes)

0 comments on commit fa46468

Please sign in to comment.