In has_many_through_association.rb:

      def <<(*records)
        ...
        load_target

        ...

            @target << associate
          end
        end
      end

In the interests of making the lazy loading lazier, wouldn't it be
better to *not* load the target, and instead:

            @target << associate if loaded?

When you have a lot of target records the load can be a significant
hit. Am I misunderstanding this? Is there any reason not to prefer it
this way?

Evan Weaver


--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups "Ruby 
on Rails: Core" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/rubyonrails-core?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to