diff options
| author | Steve Klabnik <steve@steveklabnik.com> | 2012-06-14 18:54:22 +0200 |
|---|---|---|
| committer | Steve Klabnik <steve@steveklabnik.com> | 2012-06-14 18:54:24 +0200 |
| commit | a7f4b0a1231bf3c65db2ad4066da78c3da5ffb01 (patch) | |
| tree | 29534415b0675dcadbdd28f840aa90be079befed /activerecord/lib/active_record/migration.rb | |
| parent | 99c9d18601539c7e7e87f26bb047add1f93072af (diff) | |
| download | rails-a7f4b0a1231bf3c65db2ad4066da78c3da5ffb01.tar.gz rails-a7f4b0a1231bf3c65db2ad4066da78c3da5ffb01.tar.bz2 rails-a7f4b0a1231bf3c65db2ad4066da78c3da5ffb01.zip | |
Remove update_attribute.
Historically, update_attribute and update_attributes are similar, but
with one big difference: update_attribute does not run validations.
These two methods are really easy to confuse given their similar
names. Therefore, update_attribute is being removed in favor of
update_column.
See the thread on rails-core here:
https://groups.google.com/forum/?fromgroups#!topic/rubyonrails-core/BWPUTK7WvYA
Diffstat (limited to 'activerecord/lib/active_record/migration.rb')
| -rw-r--r-- | activerecord/lib/active_record/migration.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/activerecord/lib/active_record/migration.rb b/activerecord/lib/active_record/migration.rb index acec65991d..d58176bc62 100644 --- a/activerecord/lib/active_record/migration.rb +++ b/activerecord/lib/active_record/migration.rb @@ -238,7 +238,7 @@ module ActiveRecord # add_column :people, :salary, :integer # Person.reset_column_information # Person.all.each do |p| - # p.update_attribute :salary, SalaryCalculator.compute(p) + # p.update_column :salary, SalaryCalculator.compute(p) # end # end # end @@ -258,7 +258,7 @@ module ActiveRecord # ... # say_with_time "Updating salaries..." do # Person.all.each do |p| - # p.update_attribute :salary, SalaryCalculator.compute(p) + # p.update_column :salary, SalaryCalculator.compute(p) # end # end # ... |