diff options
author | Santiago Pastorino <santiago@wyeworks.com> | 2011-07-08 06:05:20 -0700 |
---|---|---|
committer | Santiago Pastorino <santiago@wyeworks.com> | 2011-07-08 06:05:20 -0700 |
commit | 3a1ec9b544c6056fb69b831cd3ed551a656a9b61 (patch) | |
tree | 6666da385707de5ffa5abb1a4a35e739a6f9ba43 /activerecord | |
parent | ad912c08a91b90f6a7b30ae75b4a3a5f6c513bd4 (diff) | |
parent | dab0de4a2058abe60c95ff7fdead43ad2b1f786e (diff) | |
download | rails-3a1ec9b544c6056fb69b831cd3ed551a656a9b61.tar.gz rails-3a1ec9b544c6056fb69b831cd3ed551a656a9b61.tar.bz2 rails-3a1ec9b544c6056fb69b831cd3ed551a656a9b61.zip |
Merge pull request #2011 from amatsuda/with_index
replaced some "i += 1" sort of codes with Enumerable#with_index
Diffstat (limited to 'activerecord')
-rw-r--r-- | activerecord/lib/active_record/relation.rb | 3 | ||||
-rw-r--r-- | activerecord/test/cases/fixtures_test.rb | 4 |
2 files changed, 2 insertions, 5 deletions
diff --git a/activerecord/lib/active_record/relation.rb b/activerecord/lib/active_record/relation.rb index aa7f7d6cea..6258a16d0f 100644 --- a/activerecord/lib/active_record/relation.rb +++ b/activerecord/lib/active_record/relation.rb @@ -250,8 +250,7 @@ module ActiveRecord # Person.update(people.keys, people.values) def update(id, attributes) if id.is_a?(Array) - idx = -1 - id.collect { |one_id| idx += 1; update(one_id, attributes[idx]) } + id.each.with_index.map {|one_id, idx| update(one_id, attributes[idx])} else object = find(id) object.update_attributes(attributes) diff --git a/activerecord/test/cases/fixtures_test.rb b/activerecord/test/cases/fixtures_test.rb index 854eb86c09..842e8a0049 100644 --- a/activerecord/test/cases/fixtures_test.rb +++ b/activerecord/test/cases/fixtures_test.rb @@ -178,11 +178,9 @@ class FixturesTest < ActiveRecord::TestCase assert_nothing_raised do fixtures = ActiveRecord::Fixtures.new(Account.connection, 'categories', 'Category', FIXTURES_ROOT + "/categories_ordered") - i = 0 - fixtures.each do |name, fixture| + fixtures.each.with_index do |(name, fixture), i| assert_equal "fixture_no_#{i}", name assert_equal "Category #{i}", fixture['name'] - i += 1 end end end |