diff options
author | Xavier Noria <fxn@hashref.com> | 2011-12-28 23:47:10 +0100 |
---|---|---|
committer | Xavier Noria <fxn@hashref.com> | 2011-12-28 23:47:10 +0100 |
commit | e99987bc309a276bb6aea700ff184194768a6ee4 (patch) | |
tree | 48a5d41469fad71b07298d0b0ed814405bac820b /activerecord/lib/active_record/scoping | |
parent | 0b61e3f86a2f723582e3a9295d67b541f2bb2013 (diff) | |
download | rails-e99987bc309a276bb6aea700ff184194768a6ee4.tar.gz rails-e99987bc309a276bb6aea700ff184194768a6ee4.tar.bz2 rails-e99987bc309a276bb6aea700ff184194768a6ee4.zip |
app code in general wants Time.current, not Time.now
Diffstat (limited to 'activerecord/lib/active_record/scoping')
-rw-r--r-- | activerecord/lib/active_record/scoping/named.rb | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/activerecord/lib/active_record/scoping/named.rb b/activerecord/lib/active_record/scoping/named.rb index 17122740da..0edc3f1dcc 100644 --- a/activerecord/lib/active_record/scoping/named.rb +++ b/activerecord/lib/active_record/scoping/named.rb @@ -118,16 +118,16 @@ module ActiveRecord # when they are used. For example, the following would be incorrect: # # class Post < ActiveRecord::Base - # scope :recent, where('published_at >= ?', Time.now - 1.week) + # scope :recent, where('published_at >= ?', Time.current - 1.week) # end # - # The example above would be 'frozen' to the <tt>Time.now</tt> value when the <tt>Post</tt> + # The example above would be 'frozen' to the <tt>Time.current</tt> value when the <tt>Post</tt> # class was defined, and so the resultant SQL query would always be the same. The correct # way to do this would be via a lambda, which will re-evaluate the scope each time # it is called: # # class Post < ActiveRecord::Base - # scope :recent, lambda { where('published_at >= ?', Time.now - 1.week) } + # scope :recent, lambda { where('published_at >= ?', Time.current - 1.week) } # end # # Named \scopes can also have extensions, just as with <tt>has_many</tt> declarations: |