diff options
author | Steve Klabnik <steve@steveklabnik.com> | 2012-06-15 11:19:40 +0200 |
---|---|---|
committer | Steve Klabnik <steve@steveklabnik.com> | 2012-06-18 14:53:03 -0400 |
commit | 14fc8b34521f8354a17e50cd11fa3f809e423592 (patch) | |
tree | 1e8f618d53ae20f55720e5a29d64dfb5c21f811b /activerecord/lib/active_record/persistence.rb | |
parent | 568394659c3e56581c684df77c0cc0e6e264a99f (diff) | |
download | rails-14fc8b34521f8354a17e50cd11fa3f809e423592.tar.gz rails-14fc8b34521f8354a17e50cd11fa3f809e423592.tar.bz2 rails-14fc8b34521f8354a17e50cd11fa3f809e423592.zip |
Removing composed_of from ActiveRecord.
This feature adds a lot of complication to ActiveRecord for dubious
value. Let's talk about what it does currently:
class Customer < ActiveRecord::Base
composed_of :balance, :class_name => "Money", :mapping => %w(balance amount)
end
Instead, you can do something like this:
def balance
@balance ||= Money.new(value, currency)
end
def balance=(balance)
self[:value] = balance.value
self[:currency] = balance.currency
@balance = balance
end
Since that's fairly easy code to write, and doesn't need anything
extra from the framework, if you use composed_of today, you'll
have to add accessors/mutators like that.
Closes #1436
Closes #2084
Closes #3807
Diffstat (limited to 'activerecord/lib/active_record/persistence.rb')
-rw-r--r-- | activerecord/lib/active_record/persistence.rb | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/activerecord/lib/active_record/persistence.rb b/activerecord/lib/active_record/persistence.rb index 643c3f82ad..a23597be28 100644 --- a/activerecord/lib/active_record/persistence.rb +++ b/activerecord/lib/active_record/persistence.rb @@ -267,7 +267,6 @@ module ActiveRecord # may do e.g. record.reload(:lock => true) to reload the same record with # an exclusive row lock. def reload(options = nil) - clear_aggregation_cache clear_association_cache fresh_object = |