Permalink
Browse files

Update AR Changelog with correct example using includes

These queries don't seem to work without the includes clause. [ci skip]
  • Loading branch information...
1 parent 69163cc commit ad59260f70ece277f24f9a847d6cd3c2ed3bb9b2 @caike caike committed with carlosantoniodasilva Dec 7, 2012
Showing with 4 additions and 4 deletions.
  1. +4 −4 activerecord/CHANGELOG.md
@@ -1113,11 +1113,11 @@
Note that you do not need to explicitly specify references in the
following cases, as they can be automatically inferred:
- Post.where(comments: { name: 'foo' })
- Post.where('comments.name' => 'foo')
- Post.order('comments.name')
+ Post.includes(:comments).where(comments: { name: 'foo' })
+ Post.includes(:comments).where('comments.name' => 'foo')
+ Post.includes(:comments).order('comments.name')
- You also do not need to worry about this unless you are doing eager
+ You do not need to worry about this unless you are doing eager
loading. Basically, don't worry unless you see a deprecation warning
or (in future releases) an SQL error due to a missing JOIN.

0 comments on commit ad59260

Please sign in to comment.