mirror of
https://github.com/ditkrg/active_model_serializers.git
synced 2026-01-22 22:06:50 +00:00
Address rubocop warnings
This commit is contained in:
parent
a26d3e4425
commit
68d5233b31
@ -14,7 +14,7 @@ module ActiveModel
|
|||||||
serializer_context_class = options.fetch(:serializer_context_class, ActiveModel::Serializer)
|
serializer_context_class = options.fetch(:serializer_context_class, ActiveModel::Serializer)
|
||||||
serializer_class = options.fetch(:serializer) { serializer_context_class.serializer_for(resource) }
|
serializer_class = options.fetch(:serializer) { serializer_context_class.serializer_for(resource) }
|
||||||
|
|
||||||
if serializer_class.nil?
|
if serializer_class.nil? # rubocop:disable Style/GuardClause
|
||||||
fail NoSerializerError, "No serializer found for resource: #{resource.inspect}"
|
fail NoSerializerError, "No serializer found for resource: #{resource.inspect}"
|
||||||
else
|
else
|
||||||
serializer_class.new(resource, options.except(:serializer))
|
serializer_class.new(resource, options.except(:serializer))
|
||||||
|
|||||||
@ -1,7 +1,7 @@
|
|||||||
module ActiveModelSerializers
|
module ActiveModelSerializers
|
||||||
module Adapter
|
module Adapter
|
||||||
UnknownAdapterError = Class.new(ArgumentError)
|
UnknownAdapterError = Class.new(ArgumentError)
|
||||||
ADAPTER_MAP = {}.freeze
|
ADAPTER_MAP = {} # rubocop:disable Style/MutableConstant
|
||||||
private_constant :ADAPTER_MAP if defined?(private_constant)
|
private_constant :ADAPTER_MAP if defined?(private_constant)
|
||||||
require 'active_model_serializers/adapter/fragment_cache'
|
require 'active_model_serializers/adapter/fragment_cache'
|
||||||
require 'active_model_serializers/adapter/cached_serializer'
|
require 'active_model_serializers/adapter/cached_serializer'
|
||||||
|
|||||||
@ -38,12 +38,10 @@ module ActiveModelSerializers
|
|||||||
def data_for(serializer, options)
|
def data_for(serializer, options)
|
||||||
if serializer.respond_to?(:each)
|
if serializer.respond_to?(:each)
|
||||||
serializer.map { |s| ResourceIdentifier.new(s).as_json }
|
serializer.map { |s| ResourceIdentifier.new(s).as_json }
|
||||||
else
|
elsif options[:virtual_value]
|
||||||
if options[:virtual_value]
|
options[:virtual_value]
|
||||||
options[:virtual_value]
|
elsif serializer && serializer.object
|
||||||
elsif serializer && serializer.object
|
ResourceIdentifier.new(serializer).as_json
|
||||||
ResourceIdentifier.new(serializer).as_json
|
|
||||||
end
|
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user