aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib/active_record/associations.rb
diff options
context:
space:
mode:
authorJon Leighton <j@jonathanleighton.com>2012-07-20 13:32:55 +0100
committerJon Leighton <j@jonathanleighton.com>2012-07-20 13:33:49 +0100
commit7a271a8e844a4a5c931652a4a33a0d27445137c0 (patch)
treeabd2f49e4c75e10dfd58cf566db51159f03d59d4 /activerecord/lib/active_record/associations.rb
parent3a07fdcf85672ac9727d9abe2a296a3b7c13c1c1 (diff)
downloadrails-7a271a8e844a4a5c931652a4a33a0d27445137c0.tar.gz
rails-7a271a8e844a4a5c931652a4a33a0d27445137c0.tar.bz2
rails-7a271a8e844a4a5c931652a4a33a0d27445137c0.zip
Avoid options ever being nil
This fixes active_record_deprecated_finders.
Diffstat (limited to 'activerecord/lib/active_record/associations.rb')
-rw-r--r--activerecord/lib/active_record/associations.rb8
1 files changed, 4 insertions, 4 deletions
diff --git a/activerecord/lib/active_record/associations.rb b/activerecord/lib/active_record/associations.rb
index a90c5a2a9a..e266cf3335 100644
--- a/activerecord/lib/active_record/associations.rb
+++ b/activerecord/lib/active_record/associations.rb
@@ -1192,7 +1192,7 @@ module ActiveRecord
# ORDER BY p.first_name
# }
# }
- def has_many(name, scope = {}, options = nil, &extension)
+ def has_many(name, scope = nil, options = {}, &extension)
Builder::HasMany.build(self, name, scope, options, &extension)
end
@@ -1308,7 +1308,7 @@ module ActiveRecord
# has_one :boss, :readonly => :true
# has_one :club, :through => :membership
# has_one :primary_address, :through => :addressables, :conditions => ["addressable.primary = ?", true], :source => :addressable
- def has_one(name, scope = {}, options = nil)
+ def has_one(name, scope = nil, options = {})
Builder::HasOne.build(self, name, scope, options)
end
@@ -1431,7 +1431,7 @@ module ActiveRecord
# belongs_to :post, :counter_cache => true
# belongs_to :company, :touch => true
# belongs_to :company, :touch => :employees_last_updated_at
- def belongs_to(name, scope = {}, options = nil)
+ def belongs_to(name, scope = nil, options = {})
Builder::BelongsTo.build(self, name, scope, options)
end
@@ -1605,7 +1605,7 @@ module ActiveRecord
# has_and_belongs_to_many :categories, :readonly => true
# has_and_belongs_to_many :active_projects, :join_table => 'developers_projects', :delete_sql =>
# proc { |record| "DELETE FROM developers_projects WHERE active=1 AND developer_id = #{id} AND project_id = #{record.id}" }
- def has_and_belongs_to_many(name, scope = {}, options = nil, &extension)
+ def has_and_belongs_to_many(name, scope = nil, options = {}, &extension)
Builder::HasAndBelongsToMany.build(self, name, scope, options, &extension)
end
end