1
0
Fork 0
mirror of https://github.com/rails/rails.git synced 2022-11-09 12:12:34 -05:00

Calling reset on a collection association should unload the assocation

Need to define #reset on CollectionProxy.
This commit is contained in:
Kelsey Schlarman 2014-01-21 18:19:51 -08:00
parent a4ce065943
commit 43675f014c
3 changed files with 36 additions and 0 deletions

View file

@ -1,3 +1,9 @@
* Calling reset on a collection association should unload the assocation.
Fixes #13777.
*Kelsey Schlarman*
* Make enum fields work as expected with the `ActiveModel::Dirty` API.
Before this change, using the dirty API would have surprising results:

View file

@ -1004,6 +1004,27 @@ module ActiveRecord
proxy_association.reload
self
end
# Unloads the association
#
# class Person < ActiveRecord::Base
# has_many :pets
# end
#
# person.pets # fetches pets from the database
# # => [#<Pet id: 1, name: "Snoop", group: "dogs", person_id: 1>]
#
# person.pets # uses the pets cache
# # => [#<Pet id: 1, name: "Snoop", group: "dogs", person_id: 1>]
#
# person.pets.reset # clears the pets cache
#
# person.pets # fetches pets from the database
# # => [#<Pet id: 1, name: "Snoop", group: "dogs", person_id: 1>]
def reset
proxy_association.reset
proxy_association.reset_scope
end
end
end
end

View file

@ -255,6 +255,15 @@ class AssociationProxyTest < ActiveRecord::TestCase
assert_equal man, man.interests.where("1=1").first.man
end
end
def test_reset_unloads_target
david = authors(:david)
david.posts.reload
assert david.posts.loaded?
david.posts.reset
assert !david.posts.loaded?
end
end
class OverridingAssociationsTest < ActiveRecord::TestCase