TEST_CODEOWNERS/rails
3
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

no need to to_sym

Browse Source
This commit is contained in:
Aaron Patterson 2013-07-31 18:07:06 -07:00
parent c856d895bd
commit 946ac5eb43
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
activerecord/lib/active_record/associations/builder/collection_association.rb
View File

@ -55,7 +55,7 @@ def define_callback(callback_name)
full_callback_name = "#{callback_name}_for_#{name}"
# TODO : why do i need method_defined? I think its because of the inheritance chain
model.class_attribute full_callback_name.to_sym unless model.method_defined?(full_callback_name)
model.class_attribute full_callback_name unless model.method_defined?(full_callback_name)
model.send("#{full_callback_name}=", Array(options[callback_name.to_sym]))
end

2
activerecord/lib/active_record/associations/collection_association.rb
View File

@ -523,7 +523,7 @@ def callback(method, record)
def callbacks_for(callback_name)
full_callback_name = "#{callback_name}_for_#{reflection.name}"
owner.class.send(full_callback_name.to_sym) || []
owner.class.send(full_callback_name) || []
end
# Should we deal with assoc.first or assoc.last by issuing an independent query to