sjs/grape-active_model_serializers
1
0
Fork 0
mirror of https://github.com/samsonjs/grape-active_model_serializers.git synced 2026-03-25 08:45:55 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Use the same test for resource array-ness that ActiveModel::Serializers itself uses. This should be more reliable and it's probably good to conform to AM::Serializers behavior wherever possible.

Browse source
This commit is contained in:
Todd Eichel 2013-06-20 00:55:29 -07:00
parent 77e3b26251
commit 5f3f4d5904
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/grape-active_model_serializers/formatter.rb
View file

@ -13,7 +13,7 @@ module Grape
@endpoint = env["api.endpoint"] @endpoint = env["api.endpoint"]
options = endpoint.namespace_options.merge(endpoint.route_options) options = endpoint.namespace_options.merge(endpoint.route_options)
if resource.is_a?(Array) && !resource.empty? if resource.respond_to?(:to_ary) && !resource.empty?
# ensure we have an root to fallback on # ensure we have an root to fallback on
endpoint.controller_name = resource.first.class.name.underscore.pluralize endpoint.controller_name = resource.first.class.name.underscore.pluralize
end end