diff options
| author | Jon Leighton <j@jonathanleighton.com> | 2010-10-17 23:29:56 +0100 |
|---|---|---|
| committer | Jon Leighton <j@jonathanleighton.com> | 2010-10-17 23:29:56 +0100 |
| commit | 78b8c51cb3b0c629152f3bbaf6d8bcf988cc936e (patch) | |
| tree | eae58b0877ee952c7a1809c02ad19ea7762e6440 /activerecord/lib/active_record/autosave_association.rb | |
| parent | edc176d33be9499f4c096779c5b4711b5daf0c06 (diff) | |
| download | rails-78b8c51cb3b0c629152f3bbaf6d8bcf988cc936e.tar.gz rails-78b8c51cb3b0c629152f3bbaf6d8bcf988cc936e.tar.bz2 rails-78b8c51cb3b0c629152f3bbaf6d8bcf988cc936e.zip | |
Refactoring: replace the mix of variables like @finder_sql, @counter_sql, etc with just a single scope hash (created on initialization of the proxy). This is now used consistently across all associations. Therefore, all you have to do to ensure finding/counting etc is done correctly is implement the scope correctly.
Diffstat (limited to 'activerecord/lib/active_record/autosave_association.rb')
| -rw-r--r-- | activerecord/lib/active_record/autosave_association.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/activerecord/lib/active_record/autosave_association.rb b/activerecord/lib/active_record/autosave_association.rb index 21a9a1f2cb..f3f89fe7c3 100644 --- a/activerecord/lib/active_record/autosave_association.rb +++ b/activerecord/lib/active_record/autosave_association.rb @@ -313,8 +313,8 @@ module ActiveRecord end end - # reconstruct the SQL queries now that we know the owner's id - association.send(:construct_sql) if association.respond_to?(:construct_sql) + # reconstruct the scope now that we know the owner's id + association.send(:construct_scope) if association.respond_to?(:construct_scope) end end |