Fix #425 global root configuration not inherited

[mhenrixon]

This fixes that I could not turn off root elements in my json serialization.

[coveralls]

Coverage increased (+0.05%) when pulling 5d0add3 on mhenrixon:master into 74d19e7 on rails-api:master.

[spastorino]

This should be better moved to configs. I think @arthurnn was going to do something like that.

[mhenrixon]

You mean something like:

ActiveModel::Serializer.setup do |config|
  config.root = false
  config.array_root = false
end

Doesn't that mean the array serializer runs into some serious ugliness?

# lib/array_serializer.rb
module ActiveModel
  class ArraySerializer
    include Serializable

    def initialize(object, options={})
      @object          = object
      @root            = options[:root]
      # *Below gets real ugly*
      @root            = ActiveModel::Serializer::CONFIG.array_root if @root.nil?
      @root            = options[:resource_name] if @root.nil?
      @meta_key        = options[:meta_key] || :meta
      @meta            = options[@meta_key]
      @each_serializer = options[:each_serializer]
      @options         = options.merge(root: nil)
    end
end

The alternative isn't pretty either, having the following code makes the code base unnecessary complex imho.

ActiveModel::Serializer.setup do |config|
  config.root = false
end

ActiveModel::ArraySerializer.setup do |config|
  config.root = false
end

The current Config class is locked to Serializer so it can't be reused and I don't see any real benefit in having it dynamic as it is now. I can't fathom a scenario where AMS developers need to provide a way to set variables they don't know the name of up front.

I am always available on Skype as mhenrixon or send me an email mikael at zoolutions dot se feel like it might be a good idea to discuss some general design thoughts before trying to help further.

[aaronchi]

The documentation needs to be updated in the Readme. The intializer method doesn't seem to work at all.

Subclassing the serializer works but not as documented.

You have to use

def root
  false
end

instead of

self.root = false

[spastorino]

Fixed in 700e6b8