ditkrg/active_model_serializers
4
0
Fork 0
mirror of https://github.com/ditkrg/active_model_serializers.git synced 2026-01-24 14:56:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

Passing options[:hash] is not public API of include!

Browse Source
This commit is contained in:
Santiago Pastorino 2013-05-13 17:37:36 -07:00
parent ea3566955c
commit a41de0286f
1 changed files with 2 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
lib/active_model/serializer.rb
View File

@ -352,23 +352,8 @@ module ActiveModel
end end
def include!(name, options={}) def include!(name, options={})
# Make sure that if a special options[:hash] was passed in, we generate hash = @options[:hash]
# a new unique values hash and don't clobber the original. If the hash unique_values = @options[:unique_values] ||= {}
# passed in is the same as the current options hash, use the current
# unique values.
#
# TODO: Should passing in a Hash even be public API here?
unique_values =
if hash = options[:hash]
if @options[:hash] == hash
@options[:unique_values] ||= {}
else
{}
end
else
hash = @options[:hash]
@options[:unique_values] ||= {}
end
node = options[:node] ||= @node node = options[:node] ||= @node
value = options[:value] value = options[:value]