kotovalexarian-likes-github/rails--rails
1
0
Fork 0
mirror of https://github.com/rails/rails.git synced 2022-11-09 12:12:34 -05:00
Code Releases Activity

Deprecating passing strings as class name in fixtures.

Browse source
This commit is contained in:
wangjohn 2013-08-23 12:31:29 -04:00
parent b77f25cb84
commit 92b0ce942f
1 changed files with 1 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
activerecord/lib/active_record/fixtures.rb
View file

@ -507,6 +507,7 @@ module ActiveRecord
if class_name.is_a?(Class) # TODO: Should be an AR::Base type class, or any?
@model_class = class_name
else
ActiveSupport::Deprecation.warn("The ability to pass in strings as a class name will be removed in Rails 4.1, consider using the class itself instead.")
@model_class = class_name.constantize rescue nil
end
@ -743,13 +744,6 @@ module ActiveRecord
# 'namespaced/fixture' => Another::Model
#
# The keys must be the fixture names, that coincide with the short paths to the fixture files.
#--
# It is also possible to pass the class name instead of the class:
# set_fixture_class 'some_fixture' => 'SomeModel'
# I think this option is redundant, i propose to deprecate it.
# Isn't it easier to always pass the class itself?
# (2011-12-20 alexeymuranov)
#++
def set_fixture_class(class_names = {})
self.fixture_class_names = self.fixture_class_names.merge(class_names.stringify_keys)
end