Browse files

Axe outdated :ar_joins commentary

git-svn-id: 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
  • Loading branch information...
1 parent d0f192c commit 836f32cf7d5303e6582f4b89664755ab617827fd @jeremy jeremy committed Dec 10, 2007
Showing with 0 additions and 7 deletions.
  1. +0 −7 activerecord/lib/active_record/base.rb
7 activerecord/lib/active_record/base.rb
@@ -437,13 +437,6 @@ class << self # Class methods
# end
def find(*args)
options = args.extract_options!
- # Note: we extract any :joins option with a non-string value from the options, and turn it into
- # an internal option :ar_joins. This allows code called from here to find the ar_joins, and
- # it bypasses marking the result as read_only.
- # A normal string join marks the result as read-only because it contains attributes from joined tables
- # which are not in the base table and therefore prevent the result from being saved.
- # In the case of an ar_join, the JoinDependency created to instantiate the results eliminates these
- # bogus attributes. See JoinDependency#instantiate, and JoinBase#instantiate in associations.rb.

0 comments on commit 836f32c

Please sign in to comment.