glenux-contrib/metamaps--metamaps
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

fix weird double-embed issue

Browse source
This commit is contained in:
Devin Howard 2017-03-07 21:14:15 -08:00
parent e3b4dac1e1
commit 725c02bf22
1 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
app/serializers/api/v2/application_serializer.rb
View file

@ -36,13 +36,15 @@ module Api
end
has_many(attr, opts.merge(if: -> { embeds.include?(key) })) do
list = Pundit.policy_scope(scope[:current_user], object.send(attr)) || []
child_serializer = "Api::V2::#{attr.to_s.singularize.camelize}Serializer".constantize
child_serializer = opts[:serializer] || "Api::V2::#{attr.to_s.singularize.camelize}Serializer".constantize
resource = ActiveModelSerializers::SerializableResource.new(
list,
each_serializer: child_serializer,
scope: scope.merge(embeds: [])
)
resource.as_json
# resource.as_json will return e.g. { users: [ ... ] } for collaborators
# since we can't get the :users key, convert to an array and use .first.second to get the needed values
resource.as_json.to_a.first.second
end
else
id_opts = opts.merge(key: "#{key}_id")
@ -56,7 +58,7 @@ module Api
serializer: child_serializer,
scope: scope.merge(embeds: [])
)
resource.as_json
resource.as_json.to_a.first.second
end
end
end