Skip to content

Commit b594d14

Browse files
committed
Merge pull request rails-api#1132 from beauby/fix-flatten-json
Get rid of unnecessary instance variables, and implied dependencies.
2 parents 885d611 + a9d07cd commit b594d14

File tree

2 files changed

+8
-9
lines changed

2 files changed

+8
-9
lines changed

lib/active_model/serializer/adapter/flatten_json.rb

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
class ActiveModel::Serializer::Adapter::FlattenJson < ActiveModel::Serializer::Adapter::Json
22
def serializable_hash(options = {})
3-
super
4-
@result
3+
super.each_value.first
54
end
65

76
private

lib/active_model/serializer/adapter/json.rb

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,11 @@ class ActiveModel::Serializer::Adapter::Json < ActiveModel::Serializer::Adapter
55
def serializable_hash(options = nil)
66
options ||= {}
77
if serializer.respond_to?(:each)
8-
@result = serializer.map { |s| FlattenJson.new(s).serializable_hash(options) }
8+
result = serializer.map { |s| FlattenJson.new(s).serializable_hash(options) }
99
else
10-
@hash = {}
10+
hash = {}
1111

12-
@core = cache_check(serializer) do
12+
core = cache_check(serializer) do
1313
serializer.attributes(options)
1414
end
1515

@@ -19,13 +19,13 @@ def serializable_hash(options = nil)
1919

2020
if serializer.respond_to?(:each)
2121
array_serializer = serializer
22-
@hash[association.key] = array_serializer.map do |item|
22+
hash[association.key] = array_serializer.map do |item|
2323
cache_check(item) do
2424
item.attributes(opts)
2525
end
2626
end
2727
else
28-
@hash[association.key] =
28+
hash[association.key] =
2929
if serializer && serializer.object
3030
cache_check(serializer) do
3131
serializer.attributes(options)
@@ -35,10 +35,10 @@ def serializable_hash(options = nil)
3535
end
3636
end
3737
end
38-
@result = @core.merge @hash
38+
result = core.merge hash
3939
end
4040

41-
{ root => @result }
41+
{ root => result }
4242
end
4343

4444
def fragment_cache(cached_hash, non_cached_hash)

0 commit comments

Comments
 (0)