eager_record 0.1.0 → 0.1.1

data/README.rdoc

@@ -49,11 +49,24 @@ generate two SQL queries. A couple of caveats:
 
   EagerRecord::ScopedPreloading.install
 
+	As an alternative to using this functionality, consider defining extra scoped
+	associations with `:readonly => true` as needed.
+
 === Does it work with all association types?
 
+==== Mostly...
+
 The test suite covers just about all of them; if you find one that doesn't work,
 please file an issue.
 
+==== Except!
+
+There's a bug in ActiveRecord's preloading that causes it to build an invalid
+query when preloading a `has_many :through` association that defines a
+`:conditions` option that references the join table. For that reason,
+EagerRecord does not attempt to eagerly load `has_many :through` associations
+that have a `:conditions` option.
+
 === Do I need to make any changes to my code?
 
 EagerRecord doesn't add any new methods to ActiveRecord, so you don't need to

data/lib/eager_record/eager_preloading.rb

@@ -65,7 +65,7 @@ module EagerRecord
       end
   
       def load_target_with_eager_preloading
-        if !@owner.new_record? || foreign_key_present
+        if @reflection.options[:conditions].nil? && (!@owner.new_record? || foreign_key_present)
           if !loaded?
             if originating_collection = @owner.instance_variable_get(:@originating_collection)
               @owner.class.__send__(:preload_associations, originating_collection, @reflection.name)

data/lib/eager_record/version.rb

@@ -1,3 +1,3 @@
 module EagerRecord
-  VERSION = '0.1.0'
+  VERSION = '0.1.1'
 end

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 1
-  - 0
-  version: 0.1.0
+  - 1
+  version: 0.1.1
 platform: ruby
 authors: 
 - Mat Brown
@@ -14,7 +14,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-05-28 00:00:00 -04:00
+date: 2010-06-04 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency